On Sun, Apr 18, 2021 at 09:02:12PM -0400, Valdis Klētnieks wrote: > While doing some code auditing for -Woverride_init, I spotted some > questionable code > > commit 502e84e2382d92654a2ecbc52cdbdb5a11cdcec7 > Author: Felix Fietkau <n...@nbd.name> > Date: Wed Mar 24 02:30:54 2021 +0100 > > net: ethernet: mtk_eth_soc: add flow offloading support > > In drivers/net/ethernet/mediatek/mtk_ppe_offload.c: > > +static const struct rhashtable_params mtk_flow_ht_params = { > + .head_offset = offsetof(struct mtk_flow_entry, node), > + .head_offset = offsetof(struct mtk_flow_entry, cookie), > + .key_len = sizeof(unsigned long), > + .automatic_shrinking = true, > +}; > > What's intended for head_offset here?
It's a bug, there's a fix here: https://www.spinics.net/lists/netdev/msg736368.html