Re: [patch -mmotm 2/6] fix qla2xxx-use-memory_read_from_buffer.patch

[Andrew Morton <akpm@linux-foundation.org>]

On Thu, 17 Jul 2008 16:44:16 -0700
Andrew Vasquez <andrew.vasquez@qlogic.com> wrote:

> On Thu, 17 Jul 2008, Andrew Morton wrote:
> 
> > > On Mon, 14 Jul 2008, Akinobu Mita wrote:
> > > 
> > > > Now memory_read_from_buffer() belongs in string.h
> > > > 
> > > > Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
> > > > Cc: Andrew Vasquez <linux-driver@qlogic.com>
> > > > Cc: "James E.J. Bottomley" <James.Bottomley@HansenPartnership.com>
> > > > Cc: Andrew Vasquez <andrew.vasquez@qlogic.com>
> > > > Cc: Seokmann Ju <seokmann.ju@qlogic.com>
> > > 
> > > If needed:
> > > 
> > > Acked-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
> > 
> > Thanks. So we now have tested-by:you and acked-by:you.  That'll do ;)
> > 
> > > BTW: is the memory_read_from_buffer() work for qla2xxx going in
> > > through the mm tree submission path???
> > 
> > FSVO "through", yeah.  It's appeared on linux-scsi four times so far. 
> > Last time James complained about the fs.h inclusion so Akinobu raised a
> > patch "move memory_read_from_buffer() from fs.h to string.h" which I
> > just merged.
> > 
> > I'll just keep pressing send - it'll get there sometime.  Or you can
> > merge it if you like.
> 
> Would it too bothersome if I were to just collapse the two patches and
> submit the composite with the my next set of patches???  Seems silly
> to add the oneliner to the history...

Sure, no probs.