On Tue, Sep 15, 2026 at 5:11 AM Evgeny Sokolov <[email protected]> wrote:
> From: Sokolov Evgeny <[email protected]> > > Use 1ULL instead of 1 in the left shift operation when > calculating the burst size mantissa. > > This avoids undefined behavior when the shift count exceeds > the width of a 32-bit integer. > > Signed-off-by: Sokolov Evgeny <[email protected]> > Cc: [email protected] > --- > drivers/net/bnxt/tf_ulp/bnxt_ulp_meter.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp_meter.c > b/drivers/net/bnxt/tf_ulp/bnxt_ulp_meter.c > index 7f6a84e699..88a2f30521 100644 > --- a/drivers/net/bnxt/tf_ulp/bnxt_ulp_meter.c > +++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp_meter.c > @@ -155,7 +155,7 @@ bnxt_ulp_flow_mtr_xbs_calc(int64_t xbs, uint16_t *reg) > * = round(xbs*2^(8-e) - 2^7) > * > */ > - m = xbs / (1 << (e - 8)) - (1 << 7); > + m = xbs / (1ULL << (e - 8)) - (1ULL << 7); > *reg = ((m & 0x7F) << 5) | (e & 0x1F); > *reg = rte_cpu_to_be_16(*reg); > } > -- Acked-by: Kishore Padmanabha <[email protected]> > > 2.30.2 > >
smime.p7s
Description: S/MIME Cryptographic Signature
