On Mon, Mar 01, 2021 at 09:04:20AM +0200, Leon Romanovsky wrote: > @@ -884,7 +884,7 @@ static void gid_table_reserve_default(struct ib_device *ib_dev, u8 port, > > static void gid_table_release_one(struct ib_device *ib_dev) > { > - unsigned int p; > + u32 p; > > rdma_for_each_port (ib_dev, p) { > release_gid_table(ib_dev, ib_dev->port_data[p].cache.gid); > @@ -895,7 +895,7 @@ static void gid_table_release_one(struct ib_device *ib_dev) > static int _gid_table_setup_one(struct ib_device *ib_dev) > { > struct ib_gid_table *table; > - unsigned int rdma_port; > + u32 rdma_port; > > rdma_for_each_port (ib_dev, rdma_port) { Why are we changing this? 'unsigned int' is the right type for port numbers Jason