From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jamal Hadi Salim Subject: Re: [fw filter]: Broken! fw mark based tc class selection not working Date: Thu, 17 Sep 2015 08:16:31 -0400 Message-ID: <55FAAF1F.6000206@mojatatu.com> References: <55F6BD55.6080503@mojatatu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Akshat Kakkar , netdev , Cong Wang , Stephen Hemminger To: Cong Wang Return-path: Received: from mail-io0-f170.google.com ([209.85.223.170]:34878 "EHLO mail-io0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751260AbbIQMQe (ORCPT ); Thu, 17 Sep 2015 08:16:34 -0400 Received: by ioiz6 with SMTP id z6so19326096ioi.2 for ; Thu, 17 Sep 2015 05:16:34 -0700 (PDT) In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On 09/14/15 18:04, Cong Wang wrote: > > That is exactly the original code. But it is not readable at all, > at least I still missed it when I touched the tp->init() part. :( > Having a boolean doesn't harm anything. The default should really be no head alloced (given that is the main use case). The other part you can make more readable. cheers, jamal