Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
From:   Joel Fernandes <joel@joelfernandes.org>
Mime-Version: 1.0 (1.0)
Subject: Re: [PATCH rcu 14/16] rxrpc: Use call_rcu_hurry() instead of call_rcu()
Date:   Wed, 30 Nov 2022 17:06:33 -0500
Message-Id: <5160D22D-1E2E-486C-BADF-CDEB752028C3@joelfernandes.org>
References: <20221130214327.GU4001@paulmck-ThinkPad-P17-Gen-1>
Cc:     David Howells <dhowells@redhat.com>, rcu@vger.kernel.org,
        linux-kernel@vger.kernel.org, kernel-team@meta.com,
        rostedt@goodmis.org, Marc Dionne <marc.dionne@auristor.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>, linux-afs@lists.infradead.org,
        netdev@vger.kernel.org
In-Reply-To: <20221130214327.GU4001@paulmck-ThinkPad-P17-Gen-1>
To:     paulmck@kernel.org
Precedence: bulk


> On Nov 30, 2022, at 4:43 PM, Paul E. McKenney <paulmck@kernel.org> wrote:
>=20
> =EF=BB=BFOn Wed, Nov 30, 2022 at 02:20:52PM -0500, Joel Fernandes wrote:
>>=20
>>=20
>>>> On Nov 30, 2022, at 2:09 PM, David Howells <dhowells@redhat.com> wrote:=

>>>=20
>>> =EF=BB=BFNote that this conflicts with my patch:
>>=20
>> Oh.  I don=E2=80=99t see any review or Ack tags on it. Is it still under r=
eview?
>=20
> So what I have done is to drop this patch from the series, but to also
> preserve it for posterity at -rcu branch lazy-obsolete.2022.11.30a.
>=20
> It looks like that wakeup is still delayed, but I could easily be
> missing something.
>=20
> Joel, could you please test the effects of having the current lazy branch,=

> but also David Howells's patch?  That way, if there is an issue, we can
> work it sooner rather than later, and if it all works fine, we can stop
> worrying about it.  ;-)

Sure, I will kick off the failing test and see if it passes with Davids patc=
h. Will let you know.

Thanks,

 - Joel


>                            Thanx, Paul
>=20
>> Thanks,
>>=20
>> - Joel
>>=20
>>=20
>>=20
>>>=20
>>>   rxrpc: Don't hold a ref for connection workqueue
>>>   https://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git/=
commit/?h=3Drxrpc-next&id=3D450b00011290660127c2d76f5c5ed264126eb229
>>>=20
>>> which should render it unnecessary.  It's a little ahead of yours in the=

>>> net-next queue, if that means anything.
>>>=20
>>> David
>>>=20