Received-SPF: pass (google.com: best guess record for domain of linux-nfs-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <952928a350da64fd8de3e1a79deb8cc23552972f.1558362681.git.bcodding@redhat.com>
 <20190521155821.GD9499@fieldses.org> <A6640E39-5B5D-4CF3-8131-62F8106ED2E0@redhat.com>
In-Reply-To: <A6640E39-5B5D-4CF3-8131-62F8106ED2E0@redhat.com>
From:   Xuewei Zhang <xueweiz@google.com>
Date:   Tue, 21 May 2019 15:33:53 -0700
Message-ID: <CAPtwhKp=qezMw4DfWkettC37wvYd+-6e0=isVdMx51_gFbi8bw@mail.gmail.com>
Subject: Re: [PATCH] Revert "lockd: Show pid of lockd for remote locks"
To:     Benjamin Coddington <bcodding@redhat.com>
Cc:     "J . Bruce Fields" <bfields@fieldses.org>,
        linux-nfs@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Sender: linux-nfs-owner@vger.kernel.org
Precedence: bulk

On Tue, May 21, 2019 at 1:45 PM Benjamin Coddington <bcodding@redhat.com> wrote:
>
> On 21 May 2019, at 11:58, J . Bruce Fields wrote:
>
> > On Mon, May 20, 2019 at 10:33:07AM -0400, Benjamin Coddington wrote:
> >> This reverts most of commit b8eee0e90f97 ("lockd: Show pid of lockd for
> >> remote locks"), which caused remote locks to not be differentiated between
> >> remote processes for NLM.

I just tested this today to be sure. And yes, this revert fixes the
problem we observed.

Reviewed-by: Xuewei Zhang <xueweiz@google.com>

Best regards,
Xuewei