All of lore.kernel.org
 help / color / mirror / Atom feed
From: Boaz Harrosh <bharrosh@panasas.com>
To: "J. Bruce Fields" <bfields@fieldses.org>
Cc: <stable@kernel.org>, <linux-nfs@vger.kernel.org>
Subject: Re: nfsd (and lock) changes for 3.2
Date: Fri, 4 Nov 2011 09:50:16 -0700	[thread overview]
Message-ID: <4EB417C8.4000502@panasas.com> (raw)
In-Reply-To: <20111104160439.GA32756@fieldses.org>

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 and not into the linux-next branch. And yes both branches should
be included in linux-next.

This way there is nothing to forget

Thanks
Boaz

  reply	other threads:[~2011-11-04 16:57 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-25 12:19 nfsd (and lock) changes for 3.2 J. Bruce Fields
2011-10-25 12:34 ` J. Bruce Fields
2011-10-26  4:41   ` Tom Tucker
2011-10-26  8:59     ` J. Bruce Fields
2011-10-26  3:16 ` Boaz Harrosh
2011-10-26  3:31   ` Boaz Harrosh
2011-10-27 20:34   ` J. Bruce Fields
2011-10-28  0:23     ` Boaz Harrosh
2011-10-28 14:42       ` J. Bruce Fields
2011-11-04 16:04         ` J. Bruce Fields
2011-11-04 16:50           ` Boaz Harrosh [this message]
2011-11-04 17:45             ` J. Bruce Fields
2011-11-07 16:21             ` Benny Halevy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EB417C8.4000502@panasas.com \
    --to=bharrosh@panasas.com \
    --cc=bfields@fieldses.org \
    --cc=linux-nfs@vger.kernel.org \
    --cc=stable@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.