Re: [Intel-wired-lan] [PATCH net v1] ice: Fix DSCP PFC TLV creation

[Tony Nguyen <anthony.l.nguyen@intel.com>]

On 1/27/2023 5:24 AM, Karen Ostrowska wrote:
> From: Dave Ertman <david.m.ertman@intel.com>
> 
> When creating the TLV to send to the FW for configuring DSCP mode PFC,the
> PFCENABLE field was being masked with a 4 bit mask (0xF), but this is an 8
> bit bitmask for enabled classes for PFC.  This means that traffic classes
> 4-7 could not be enabled for PFC.
> 
> Remove the mask completely, as it is not necessary, as we are assigning 8
> bits to an 8 bit field.
> 
> Fixes: 2a87bd73e50d ("ice: Add DSCP support")
> Signed-off-by: Dave Ertman <david.m.ertman@intel.com>
> Signed-off-by: Karen Ostrowska <karen.ostrowska@intel.com>
> ---

Applied, however, this should be a v2 (with changelog)

>   drivers/net/ethernet/intel/ice/ice_dcb.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/intel/ice/ice_dcb.c b/drivers/net/ethernet/intel/ice/ice_dcb.c
> index 6be02f9b0b8c..789b99963910 100644
> --- a/drivers/net/ethernet/intel/ice/ice_dcb.c
> +++ b/drivers/net/ethernet/intel/ice/ice_dcb.c
> @@ -1372,7 +1372,7 @@ ice_add_dscp_pfc_tlv(struct ice_lldp_org_tlv *tlv, struct ice_dcbx_cfg *dcbcfg)
>   	tlv->ouisubtype = htonl(ouisubtype);
>   
>   	buf[0] = dcbcfg->pfc.pfccap & 0xF;
> -	buf[1] = dcbcfg->pfc.pfcena & 0xF;
> +	buf[1] = dcbcfg->pfc.pfcena;
>   }
>   
>   /**
_______________________________________________
Intel-wired-lan mailing list
Intel-wired-lan@osuosl.org
https://lists.osuosl.org/mailman/listinfo/intel-wired-lan