[dm-devel] [2.6 patch] drivers/md/kcopyd.c: remove kcopyd_cancel()
Alasdair G Kergon
agk at redhat.com
Tue Nov 8 15:48:28 UTC 2005
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 at 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;
> -}
More information about the dm-devel
mailing list