Re: nfsd (and lock) changes for 3.2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 04, 2011 at 09:50:16AM -0700, Boaz Harrosh wrote:
> On 11/04/2011 09:04 AM, J. Bruce Fields wrote:
> > Boaz points out that I forgot to add stable cc's to two commits:
> > 
> > 	48483bf23a568f3ef4cc7ad2c8f1a082f10ad0e7 "nfsd4: simplify recovery dir setting"
> > 	6577aac01f00636c16cd583c30bd4dedf18475d5 "nfsd4: fix failure to
> > 	end nfsd4 grace period"
> > 
> > which fix a regression introduced by
> > ab1350b2b3c1dd2e465a6abdda608d8c44facfb8 "nfsd41: Deny new lock before
> > RECLAIM_COMPLETE done" (first included in v3.1-rc1).
> > 
> > Included below for reference, but I assume simplest for you will be to
> > cherry-pick the above two sha1's.
> > 
> > Thanks!
> > 
> > --b.
> > 
> 
> Bruce hi, thanks
> 
> Yesterday I hit this bug again. I rebased on Benny's tree which did not have them
> for some reason. It's hunting me. ;-)
> 
> Just for Future reference, these patches were suppose to be submitted very early around
> rc3 when we found them. This is what the -rcX are for, to test and fix what was broken
> in the merge window.
> 
> Please forgive me I should have followed it through, but was too busy and forgot.
> 
> But in general you should always have an rc-fixes branch and bug fixes patches
> should go there

More specifically (and especially later in the -rc's) Linus has ask that
we raise the bar somewhat above just "bug fixes"--but yes, this was a
regression with a pretty simple fix, so I should have gotten it in.

> and not into the linux-next branch. And yes both branches should
> be included in linux-next.

Agreed.  Feel free to yell when you see something I missed.  I'm feeling
spread a little thin....

--b.
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux