[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <SJ1PR11MB6297802AB885096DB4103F819BA0A@SJ1PR11MB6297.namprd11.prod.outlook.com>
Date: Wed, 10 Dec 2025 21:28:09 +0000
From: "Salin, Samuel" <samuel.salin@...el.com>
To: "Loktionov, Aleksandr" <aleksandr.loktionov@...el.com>, Brian Vazquez
<brianvv@...gle.com>, Brian Vazquez <brianvv.kernel@...il.com>, "Nguyen,
Anthony L" <anthony.l.nguyen@...el.com>, "Kitszel, Przemyslaw"
<przemyslaw.kitszel@...el.com>, "David S. Miller" <davem@...emloft.net>, Eric
Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, "Paolo
Abeni" <pabeni@...hat.com>, "intel-wired-lan@...ts.osuosl.org"
<intel-wired-lan@...ts.osuosl.org>
CC: David Decotigny <decot@...gle.com>, "Singhai, Anjali"
<anjali.singhai@...el.com>, "Samudrala, Sridhar"
<sridhar.samudrala@...el.com>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>, "netdev@...r.kernel.org"
<netdev@...r.kernel.org>, "Tantilov, Emil S" <emil.s.tantilov@...el.com>
Subject: RE: [Intel-wired-lan] [iwl-net PATCH] idpf: reduce mbx_task schedule
delay to 300us
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@...osl.org> On Behalf Of
> Loktionov, Aleksandr
> Sent: Wednesday, November 12, 2025 5:06 AM
> To: Brian Vazquez <brianvv@...gle.com>; Brian Vazquez
> <brianvv.kernel@...il.com>; Nguyen, Anthony L
> <anthony.l.nguyen@...el.com>; Kitszel, Przemyslaw
> <przemyslaw.kitszel@...el.com>; David S. Miller <davem@...emloft.net>;
> Eric Dumazet <edumazet@...gle.com>; Jakub Kicinski <kuba@...nel.org>;
> Paolo Abeni <pabeni@...hat.com>; intel-wired-lan@...ts.osuosl.org
> Cc: David Decotigny <decot@...gle.com>; Singhai, Anjali
> <anjali.singhai@...el.com>; Samudrala, Sridhar
> <sridhar.samudrala@...el.com>; linux-kernel@...r.kernel.org;
> netdev@...r.kernel.org; Tantilov, Emil S <emil.s.tantilov@...el.com>
> Subject: Re: [Intel-wired-lan] [iwl-net PATCH] idpf: reduce mbx_task schedule
> delay to 300us
>
>
>
> > -----Original Message-----
> > From: Intel-wired-lan <intel-wired-lan-bounces@...osl.org> On Behalf
> > Of Brian Vazquez
> > Sent: Monday, November 10, 2025 9:59 PM
> > To: Brian Vazquez <brianvv.kernel@...il.com>; Nguyen, Anthony L
> > <anthony.l.nguyen@...el.com>; Kitszel, Przemyslaw
> > <przemyslaw.kitszel@...el.com>; David S. Miller <davem@...emloft.net>;
> > Eric Dumazet <edumazet@...gle.com>; Jakub Kicinski <kuba@...nel.org>;
> > Paolo Abeni <pabeni@...hat.com>; intel-wired-lan@...ts.osuosl.org
> > Cc: David Decotigny <decot@...gle.com>; Singhai, Anjali
> > <anjali.singhai@...el.com>; Samudrala, Sridhar
> > <sridhar.samudrala@...el.com>; linux-kernel@...r.kernel.org;
> > netdev@...r.kernel.org; Tantilov, Emil S <emil.s.tantilov@...el.com>;
> > Brian Vazquez <brianvv@...gle.com>
> > Subject: [Intel-wired-lan] [iwl-net PATCH] idpf: reduce mbx_task
> > schedule delay to 300us
> >
> > During the IDPF init phase, the mailbox runs in poll mode until it is
> > configured to properly handle interrupts. The previous delay of 300ms
> > is excessively long for the mailbox polling mechanism, which causes a
> > slow initialization of ~2s:
> >
> > echo 0000:06:12.4 > /sys/bus/pci/drivers/idpf/bind
> >
> > [ 52.444239] idpf 0000:06:12.4: enabling device (0000 -> 0002)
> > [ 52.485005] idpf 0000:06:12.4: Device HW Reset initiated
> > [ 54.177181] idpf 0000:06:12.4: PTP init failed, err=-EOPNOTSUPP
> > [ 54.206177] idpf 0000:06:12.4: Minimum RX descriptor support not
> > provided, using the default
> > [ 54.206182] idpf 0000:06:12.4: Minimum TX descriptor support not
> > provided, using the default
> >
> > Changing the delay to 300us avoids the delays during the initial
> > mailbox transactions, making the init phase much faster:
> >
> > [ 83.342590] idpf 0000:06:12.4: enabling device (0000 -> 0002)
> > [ 83.384402] idpf 0000:06:12.4: Device HW Reset initiated
> > [ 83.518323] idpf 0000:06:12.4: PTP init failed, err=-EOPNOTSUPP
> > [ 83.547430] idpf 0000:06:12.4: Minimum RX descriptor support not
> > provided, using the default
> > [ 83.547435] idpf 0000:06:12.4: Minimum TX descriptor support not
> > provided, using the default
> >
> > Signed-off-by: Brian Vazquez <brianvv@...gle.com>
> > ---
>
> Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@...el.com>
Tested-by: Samuel Salin <Samuel.salin@...el.com>
Powered by blists - more mailing lists