lkml.org 
[lkml]   [2009]   [Sep]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRE: [patch] Staging: hv: Add new tasks to the TODO file
Date
That's a fair request.  I concur.

Thanks,

-- Tom

-----Original Message-----
From: Greg KH [mailto:gregkh@suse.de]
Sent: Thursday, September 10, 2009 11:01 PM
To: Haiyang Zhang
Cc: Hank Janssen; linux-kernel@vger.kernel.org; Tom Hanrahan
Subject: Re: [patch] Staging: hv: Add new tasks to the TODO file

On Thu, Sep 10, 2009 at 08:49:56PM +0000, Haiyang Zhang wrote:
> From: Haiyang Zhang <haiyangz@microsoft.com>
>
> Add new tasks to the TODO file.
>
> Cc: Hank Janssen <hjanssen@microsoft.com>
> Cc: Greg Kroah-Hartman <gregkh@suse.de>
> Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
>
> ---
> diff --git a/drivers/staging/hv/TODO b/drivers/staging/hv/TODO
> index fc754ab..345a59f 100644
> --- a/drivers/staging/hv/TODO
> +++ b/drivers/staging/hv/TODO
> @@ -11,6 +11,12 @@ TODO:
> - audit the network driver
> - audit the block driver
> - audit the scsi driver
> + - SMP Support
> + - Remote Shutdown
> + - Networking . Jumbo Frames
> + - Hot swappable storage devices
> + - Time Synchronization
> + - Key Value Pair Exchange

No, sorry, I'm not going to take this.

The TODO file is for what needs to be done to the code in order to get
it _out_ of the staging tree. Not new features. This should be a list
of things that people can help you out with, which none of these are,
given the lack of documentation on how adding this to the code is able
to be done.

So please, work on removing the items on the TODO list, that way the
code can then be merged into the main portion of the kernel tree. Then
you can go and add new features. There's lots that still needs to be
done with this code before you can think about adding anything new.

thanks,

greg k-h


\
 
 \ /
  Last update: 2009-09-10 23:21    [W:0.116 / U:0.160 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site