> From: Intel-wired-lan <[email protected]> On Behalf Of > Stefan Assmann > Sent: Thursday, August 20, 2020 4:53 AM > To: [email protected] > Cc: [email protected]; Loktionov, Aleksandr > <[email protected]>; [email protected]; > [email protected] > Subject: [Intel-wired-lan] [PATCH] i40e: always propagate error value in > i40e_set_vsi_promisc() > > The for loop in i40e_set_vsi_promisc() reports errors via dev_err() but > does not propage the error up the call chain. Instead it continues the > loop and potentially overwrites the reported error value. > This results in the error being recorded in the log buffer, but the > caller might never know anything went the wrong way. > > To avoid this situation i40e_set_vsi_promisc() needs to temporary store > the error after reporting it. This is still not optimal as multiple > different errors may occur, so store the first error and hope that's > the main issue. > > Fixes: 37d318d7805f (i40e: Remove scheduling while atomic possibility) > Reported-by: Michal Schmidt <[email protected]> > Signed-off-by: Stefan Assmann <[email protected]> > --- > drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 12 +++++++++++- > 1 file changed, 11 insertions(+), 1 deletion(-)
In the summary text: %s/ propage/propagate/ %s/ temporary/temporarily/ I suspect Tony can fix that up before pushing. Aside from that, Tested-by: Aaron Brown <[email protected]>
