Re: [PATCH 0/7] staging: lustre: last missing patches for lustre 2.6

From: Greg Kroah-Hartman
Date: Mon Aug 22 2016 - 16:37:57 EST


On Mon, Aug 22, 2016 at 10:55:42AM -0700, Joe Perches wrote:
> On Mon, 2016-08-22 at 18:41 +0100, James Simmons wrote:
> > On Fri, 2016-08-19 at 20:44 +0100, James Simmons wrote:
>
> Hi again James.
>
> > > I don't _need_ anything, but I think it'd be simpler to
> > > have just 2 directories, one for lustre kernel stuff
> > > and another for lustre uapi stuff.
> > >
> > > That applies for LNet and libcfs #includes as well.
> > >
> > > To me, ideally, there'd only be 2 #include directories
> > > so that the only used #include styles could become:
> > >
> > > #include
> > > and
> > > #include
> > >
> > > and that would work regardless of lustre's layout
> > > in staging or elsewhere.
> > I didn't expect this to be requested at this time. I thought this would be 
> > addressed just before we left staging.
>
> Sooner is better to me, but that's up to you all.

No, please, let's wait, there's still lots of "real" work to be done in
the lustre code before messing with the .h files like this.

James, please just continue the great work you are doing so far, all is
fine :)

thanks,

greg k-h