Stefano Brivio <sbrivio@xxxxxxxxxx> wrote:
> > -struct nft_set_type nft_set_pipapo_type __read_mostly = {
> > - .owner = THIS_MODULE,
> > +const struct nft_set_type nft_set_pipapo_type __read_mostly = {
>
> const ... read_mostly should make no sense because const already forces
> the data to a read-only segment. It might actually cause some issues,
> see https://lore.kernel.org/patchwork/patch/439824/.
>
> It's not there for the other set types, so I'm assuming it's a typo :)
Grrr, thanks for noticing. This came from an old branch that
was before pipapo got merged.
I will wait some more and then send a v2.
