Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756211AbYGWWep (ORCPT ); Wed, 23 Jul 2008 18:34:45 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753789AbYGWWef (ORCPT ); Wed, 23 Jul 2008 18:34:35 -0400 Received: from stinky.trash.net ([213.144.137.162]:47862 "EHLO stinky.trash.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753720AbYGWWee (ORCPT ); Wed, 23 Jul 2008 18:34:34 -0400 Message-ID: <4887B1F0.3000407@trash.net> Date: Thu, 24 Jul 2008 00:34:24 +0200 From: Patrick McHardy User-Agent: Mozilla-Thunderbird 2.0.0.12 (X11/20080405) MIME-Version: 1.0 To: Andrew Morton CC: Ranjit Manomohan , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, lizf@cn.fujitsu.com, menage@google.com Subject: Re: [PATCH] Traffic control cgroups subsystem References: <20080723150536.ded38b22.akpm@linux-foundation.org> In-Reply-To: <20080723150536.ded38b22.akpm@linux-foundation.org> Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1356 Lines: 34 Andrew Morton wrote: > On Tue, 22 Jul 2008 10:44:18 -0700 (PDT) > Ranjit Manomohan wrote: > >> @@ -359,7 +370,12 @@ static int flow_classify(struct sk_buff *skb, struct tcf_proto *tp, >> classid %= f->divisor; >> >> res->class = 0; >> - res->classid = TC_H_MAKE(f->baseclass, f->baseclass + classid); >> + >> + if (key == FLOW_KEY_CGROUP_CLASSID) >> + res->classid = TC_H_MAKE(f->baseclass, classid); >> + else >> + res->classid = TC_H_MAKE(f->baseclass, >> + f->baseclass + classid); > > This causes a warning: > > net/sched/cls_flow.c: In function 'flow_classify': > net/sched/cls_flow.c:344: warning: 'key' may be used uninitialized in this function > > that warning is a non-issue if we happen to know that f->nkeys can > never be zero. I don't know if that is guaranteed at this code site? It is by the flow_change() function, but special casing the CGROUP_CLASSID is not acceptable anyway. There should be no need for that, a simple linear mapping to classids is done by default in mapping mode, the sk_cgroup_classid simply shouldn't include qdisc IDs. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/