On Mon, Jun 20, 2022 at 3:03 PM Devendra Singh Rawat <dsinghra...@marvell.com> wrote: > > > > > -----Original Message----- > > From: Ferruh Yigit <ferruh.yi...@xilinx.com> > > Sent: Thursday, June 16, 2022 10:32 PM > > To: Rasesh Mody <rm...@marvell.com>; Devendra Singh Rawat > > <dsinghra...@marvell.com>; Igor Russkikh <irussk...@marvell.com> > > Cc: dev@dpdk.org; Jerin Jacob Kollanukkaran <jer...@marvell.com>; > > sta...@dpdk.org > > Subject: [EXT] [PATCH] net/qede: fix build with GCC 13 > > > > External Email > > > > ---------------------------------------------------------------------- > > Reproduced with "gcc (GCC) 13.0.0 20220616 (experimental)" > > > > Build error: > > In file included from ../drivers/net/qede/qede_debug.c:9: > > ../drivers/net/qede/qede_debug.c: In function ‘qed_grc_dump_addr_range’: > > ../drivers/net/qede/base/ecore.h:95:17: > > warning: overflow in conversion from ‘int’ to ‘u8’ > > {aka ‘unsigned char’} changes value from ‘(int)vf_id << 8 | 128’ > > to ‘128’ [-Woverflow] > > 95 | ((_value & _name##_MASK) << _name##_SHIFT) > > | ^ > > ../drivers/net/qede/qede_debug.c:1907:31: > > note: in expansion of macro ‘FIELD_VALUE’ > > 1907 | fid = FIELD_VALUE(PXP_PRETEND_CONCRETE_FID_VFVALID, 1) > > | ^~~~~~~~~~~ > > > > To prevent overflow converting 'fib' to uint16_t, while updating it also > > updated 'vf_id' to 16 bit too. > > > > Fixes: ec55c118792b ("net/qede: add infrastructure for debug data > > collection") > > Cc: sta...@dpdk.org > > > > Signed-off-by: Ferruh Yigit <ferruh.yi...@xilinx.com> > > ---
> > 2.27.0 > > Acked. > Acked-by: Devendra Singh Rawat <dsinghra...@marvell.com> Applied to dpdk-next-net-mrvl/for-next-net. Thanks > > Thanks.