[PATCH v2] net/i40e: don't check link status on device start

Zhang, Helin helin.zhang at intel.com
Fri Jan 13 14:39:37 CET 2023



> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Friday, January 13, 2023 9:33 PM
> To: Zhang, Yuying <yuying.zhang at intel.com>; Xing, Beilei
> <beilei.xing at intel.com>; Mcnamara, John <john.mcnamara at intel.com>
> Cc: dev at dpdk.org; stable at dpdk.org; Zhang, Qi Z <qi.z.zhang at intel.com>;
> Dapeng Yu <dapengx.yu at intel.com>; Wenxuan Wu
> <wenxuanx.wu at intel.com>
> Subject: Re: [PATCH v2] net/i40e: don't check link status on device start
> 
> Hello i40e maintainers, John,
> 
> On Mon, Jan 9, 2023 at 10:21 AM David Marchand
> <david.marchand at redhat.com> wrote:
> > On Tue, Jan 3, 2023 at 3:02 PM David Marchand
> <david.marchand at redhat.com> wrote:
> > > Hi i40e maintainers,
> > >
> > > On Tue, Dec 13, 2022 at 10:19 AM David Marchand
> > > <david.marchand at redhat.com> wrote:
> > > >
> > > > The mentioned changes broke existing applications when the link
> > > > status of i40e ports is down at the time the port is started.
> > > > Revert those changes, the original issue will need a different fix.
Hi David

Does it break all the application or just a specific application?
We may need to understand the issue you met, and try to fix it later.
Thank you very much for reporting it out!

Regards,
Helin

> > > >
> > > > Fixes: a4ba77367923 ("net/i40e: enable maximum frame size at port
> > > > level")
> > > > Fixes: 2184f7cdeeaa ("net/i40e: fix max frame size config at port
> > > > level")
> > > > Fixes: 719469f13b11 ("net/i40e: fix jumbo frame Rx with X722")
> > > > Cc: stable at dpdk.org
> > > >
> > > > Signed-off-by: David Marchand <david.marchand at redhat.com>
> > >
> > > This issue was reported only by our QE, but I suspect that real
> > > deployments will get affected.
> > > Please review.
> >
> > Ping.
> 
> Ping.
> 
> Note: I removed Jie Wang from Cc list since I get bounces.
> 
> --
> David Marchand



More information about the stable mailing list