On 1/19/2022 12:31 AM, Wang, Haiyue wrote:
-----Original Message-----
From: Stephen Douthit <steph...@silicom-usa.com>
Sent: Wednesday, January 19, 2022 05:06
To: Wang, Haiyue <haiyue.w...@intel.com>; Morten Brørup 
<m...@smartsharesystems.com>; Lu, Wenzhuo
<wenzhuo...@intel.com>; Zhang, Helin <helin.zh...@intel.com>; Zhang, Qi Z 
<qi.z.zh...@intel.com>
Cc: dev@dpdk.org; Wang, Wen <w...@silicom-usa.com>; sta...@dpdk.org
Subject: Re: [PATCH v2 3/7] net/ixgbe: Check that SFF-8472 soft rate select is 
supported before write

On 12/22/21 19:55, Wang, Haiyue wrote:
OK, let's see in 2022, happy holidays. ;-)

A bit of a slower start to 2022 than I had hoped, but I have a v3 of the
hotplug fix in progress.  It's now working for the 82599 and C3000 ixgbe
devices under Linux, and I'm in the process of getting this built and
tested under FreeBSD.

Assuming the FreeBSD testing comes together smoothly my plan is to split
these patches into three series:

1) The SFP hotplug fix
2) Refactor SDP3 TX_DISABLE 82599 link check stuff
3) Support of additional SFP types under ixgbe

My focus right now is on the SFP hotplug fix, so please let me know if
there's any additional feedback on that portion of the original series.

Right now the feedback I have that impacts the hotplug fix is a cleanup
for patch 1, and the general comment that changes to files under
ixgbe/base are usually backported from Linux, and so I should refactor
my changes to live in ixgbe_ethdev.c

Yeah, just put the change in base code to *separate patch*, so that they are
easily to be reviewed, thanks.


Hi Steve, Wen, Haiyue,

Can you please clarify the above change request?

I though it is related to splitting base code updates into their own
patches, but that already seems the case in set (except from a few minor mix).

And what is the status, is there a new version worked on? Or is the
set waiting for more review?

Thanks,
ferruh

Reply via email to