[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]
Re: [dm-devel] [2.6 patch] drivers/md/kcopyd.c: remove kcopyd_cancel()
- From: Alasdair G Kergon <agk redhat com>
- To: Adrian Bunk <bunk stusta de>
- Cc: Andrew Morton <akpm osdl org>, dm-devel redhat com, linux-kernel vger kernel org
- Subject: Re: [dm-devel] [2.6 patch] drivers/md/kcopyd.c: remove kcopyd_cancel()
- Date: Tue, 8 Nov 2005 15:48:28 +0000
On Tue, Nov 08, 2005 at 02:44:19PM +0100, Adrian Bunk wrote:
> A function that is
> - not used
> - empty
> - without a prototype in any header file
> should be removed.
I'd rather you submitted a patch to complete the function and invoke it
in the appropriate place:-) But please don't remove comments indicating
missing functionality: commenting it out till it's written would be better.
Alasdair
--
agk redhat com
> -/*
> - * Cancels a kcopyd job, eg. someone might be deactivating a
> - * mirror.
> - */
> -int kcopyd_cancel(struct kcopyd_job *job, int block)
> -{
> - /* FIXME: finish */
> - return -1;
> -}
[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]