On Mon, May 22, 2017 at 10:17 PM, Jiri Pirko <j...@resnulli.us> wrote: > Mon, May 22, 2017 at 11:04:58PM CEST, xiyou.wangc...@gmail.com wrote: >>On Mon, May 22, 2017 at 1:54 PM, Cong Wang <xiyou.wangc...@gmail.com> wrote: >>> On Sun, May 21, 2017 at 12:19 PM, Jiri Pirko <j...@resnulli.us> wrote: >>>>>You can't claim you really delete it as long as actions can still >>>>>see it and dump it. >>>> >>>> No, user just wants to delete all the filters. That is done. User does >>>> not care if the actual chain structure is there or not. >>>> >>> >>> Hmm, so users see a chain with no filters... Fair enough. >> >>But since you remove the chain from the chain_list, it means >>users could not add new filters to this chain after flushing? And > > No, in flush, I don't remove it from the list. That is not in the > patch. Why would you think so?
Oh, I missed the minus before the first list_del()... I blame my Web browser font. Your patch looks good.
