On Thu, Jul 25, 2019 at 05:55:31PM +0800, wenxu@xxxxxxxxx wrote: > +struct flow_indr_block_dev { > + struct rhash_head ht_node; > + struct net_device *dev; > + unsigned int refcnt; > + struct list_head cb_list; > + flow_indr_block_ing_cmd_t *cmd_cb; > + void *block; There's now a flow_block object in order to avoid void * which is prone to bugs later on since the compiler cannot make type validation anymore.