Hi Qi, > -----Original Message----- > From: Zhang, Qi Z <[email protected]> > Sent: Tuesday, January 19, 2021 11:45 AM > To: Ding, Xuan <[email protected]>; Wu, Jingjing <[email protected]>; > Xing, Beilei <[email protected]> > Cc: [email protected]; [email protected] > Subject: RE: [PATCH v2] net/iavf: fix VIRTHCNL_OP_SET_RSS_HENA handling > > > > > -----Original Message----- > > From: Ding, Xuan <[email protected]> > > Sent: Monday, January 18, 2021 4:28 PM > > To: Zhang, Qi Z <[email protected]>; Wu, Jingjing > > <[email protected]>; > > Xing, Beilei <[email protected]> > > Cc: [email protected]; Ding, Xuan <[email protected]>; [email protected] > > Subject: [PATCH v2] net/iavf: fix VIRTHCNL_OP_SET_RSS_HENA handling > > > > Allow error to be returned for VIRTCHNL_OP_SET_RSS_HENA when set hena = > 0. > > Add warning that PF does not support hena = 0 now. > > Actually this is not a fix, it’s a workaround to handle some defect in legacy > kernel driver. > So changed title and commit log as below: > > Title: workaround RSS cleanup failure handling > > Allow to ignore error returned from virtual channel command > VIRTCHNL_OP_SET_RSS_HENA with hena = 0 due to some legacy > kernel driver does not support this. Add warning message when > this happen. > > > > > Fixes: 95f2f0e9fc2a6("net/iavf: improve default RSS") > > The fix line should be removed. > > > Cc: [email protected]
Since fix line is removed, should CC to stable also be removed? Thanks for your correction. Regards, Xuan > > > > Signed-off-by: Xuan Ding <[email protected]> > > Acked by: Qi Zhang <[email protected]> > > Applied to dpdk-next-net-intel. > > Thanks > Qi

