[PATCH 000/206] Staging: hv: Cleanup hv drivers

KY Srinivasan kys at microsoft.com
Wed May 11 15:06:14 UTC 2011



> -----Original Message-----
> From: Greg KH [mailto:greg at kroah.com]
> Sent: Wednesday, May 11, 2011 10:49 AM
> To: KY Srinivasan
> Cc: gregkh at suse.de; devel at linuxdriverproject.org
> Subject: Re: [PATCH 000/206] Staging: hv: Cleanup hv drivers
> 
> On Tue, May 10, 2011 at 07:56:34AM -0700, K. Y. Srinivasan wrote:
> > This is a resend of the patches sent on May 9, 2011. Per Greg's
> > suggestion, I have include additional context to the Subject line
> > where there was some ambiguity.
> >
> > This patch-set deals with some of the architectural isues in all of
> > the Hyper-V drivers. Specifically:
> 
> They finally all came through now, or at least I think they all did.
> 
> Please fix your email system to not swallow them for 8+ hours in the
> future, this really isn't an acceptable way of doing work.
> 
> > 	1) Fixes a number of locking issues in the block driver.
> >
> > 	2) Eliminates all the indirection through the hv_driver
> > 	   abstraction.
> >
> > 	3) Simplifies the code across the board eliminating
> > 	   unnecessary code and data structure layering. Prior to
> > 	   this restructuring there were about 11,000 lines of code
> > 	   in the hv directory. After this cleanup, there are about
> > 	   8500 lines of code.
> 
> That's great, remember how many lines were at the beginning?  I think it
> was over 40000 or so, but I could be wrong, I should go and dig up the
> original code dump from so long ago to be sure...
> 
> > 	4) I have dealt with style and naming issues across all the drivers.
> >
> > 	5) All the header files have been cleaned up and consolidated:
> > 		1) A single header file include/linux/hyperv.h. This file
> > 		   is to be included in all the drivers that attach to the
> > 		   vmbus.
> 
> Woah, no.  You can't move out of the drivers/staging/hv/ directory yet,
> sorry.  I'll go through and review and try to apply these patches but
> will probably stop when you do this file movement, sorry.  I'll let you
> know how far I get when I do this.

My mistake Greg; I was under the impression that this is what you wanted.
Do you want me to move this file back to the hv directory; I will have to re-spin 
all the patches after that. Let me know if I should go ahead and do that. 

Regards,

K. Y



More information about the devel mailing list