SMB3 Multi-Channel (Re: What I'm working on)

Jeremy Allison jra at samba.org
Tue Aug 5 09:55:21 MDT 2014


On Tue, Aug 05, 2014 at 02:09:37PM +0200, Michael Adam wrote:
> On 2014-08-05 at 14:03 +0200, Stefan (metze) Metzmacher wrote:
> > Hi Volker,
> > 
> > >>>> Attached find a diff between the old and new patchset
> > >>>> together with the new patchset.
> > >>>>
> > >>>> If you're fine with the changes we can finally push them.
> > >>>
> > >>> Reviewing the 85 patches piece by piece a third time now. I
> > >>> wonder how many times this in the end will have taken.
> > >>
> > >> Why? Just look at cmp.diff and the commits which still have
> > >> the TODO/REVIEW.
> > >>
> > >> Thanks for looking at it!
> > > 
> > > This time it certainly went quicker. Sometimes the diff of a
> > > diff can be pretty confusing, so I hope I did not miss a
> > > patch in the attached patchset.
> > 
> > Yes, a diff of a diff is mostly useful to to verify that nothing changed.
> > Or to have an indication where something changed to reduce the further work.
> > 
> > If you have ideas how to make such a review easier I'd like to hear them:-)
> > 
> > > In the current version of this patchset I don't see any problems anymore,
> > > but I am not good at reviewing such a large patchset. So before pushing
> > > you might ask someone else to also take a look.
> > 
> > Jeremy, Michael: Do you want to have a look and push this?
> 
> Yes, I will start to do it this afternoon.

Sure, I'll try to take a look too !


More information about the samba-technical mailing list