[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CALs4sv3DGZJeDJKAcx3VXLjsLey=WzXer1xHGcX-xznOV_V8Ng@mail.gmail.com>
Date: Wed, 11 Feb 2026 23:20:09 +0530
From: Pavan Chebbi <pavan.chebbi@...adcom.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: Bobby Eshleman <bobbyeshleman@...il.com>, Michael Chan <michael.chan@...adcom.com>,
"David S . Miller" <davem@...emloft.net>, Linux Netdev List <netdev@...r.kernel.org>,
Eric Dumazet <edumazet@...gle.com>, Paolo Abeni <pabeni@...hat.com>,
Andrew Lunn <andrew+netdev@...n.ch>, Andrew Gospodarek <andrew.gospodarek@...adcom.com>
Subject: Re: [PATCH net v2 4/4] selftests: drv-net: rss_ctx: test RSS contexts
persist after ifdown/up
On Wed, Feb 11, 2026 at 10:53 PM Pavan Chebbi <pavan.chebbi@...adcom.com> wrote:
>
> On Wed, Feb 11, 2026 at 10:28 PM Jakub Kicinski <kuba@...nel.org> wrote:
> >
> > On Wed, 11 Feb 2026 22:08:39 +0530 Pavan Chebbi wrote:
> > > > > My experiments are pointing to 'self.cancel()' sometimes taking a lot
> > > > > of time as part of the exec().
> > > > > I don't know but I feel the new test code itself should be OK. I can
> > > > > send a v3 and you can run it once too?
> > > >
> > > > Could you reply to this thread with what the version of the patch where
> > > > you see defer() takes a long time? We use defer all over the place..
> > >
> > > Oh it is still in my sandbox. It's after I incorporate the comments I
> > > received on the v2.
> > > You should see it in v3. Let me ask Michael to send it.
> >
> > I'm asking for the version that doesn't work / causes trouble.
>
Ok, I now get what you are asking for. I will send the reply here with
the patch soon. thanks.
> It's not in the list yet. v2, which is the latest version, does not
> use exec/defer to bring the interface back up.
>
> > Presumably that's not the version you want Michael to submit?
>
> My submission is that the test code that will come in v3 is going to
> be correct, except that sometimes I see a delay in getting the
> interface back up, specifically when I use exec() or defer() for if
> up. And I am being certain that the delay is not caused because of any
> mistake I have made in the new tests..
Download attachment "smime.p7s" of type "application/pkcs7-signature" (5469 bytes)
Powered by blists - more mailing lists