Re: [RFC PATCH] qos: Limit a filter's priority to a 16 bit value

[David Miller <davem@davemloft.net>]

From: "Love, Robert W" <robert.w.love@intel.com>
Date: Fri, 1 May 2009 19:10:36 -0700

> jamal wrote:
>> On Fri, 2009-05-01 at 16:30 -0700, Love, Robert W wrote:
>> 
>>> I should be more clear by saying that this should only
>>> fail with filter priorities assigned by the kernel.
>>> I think if the user passes down the priority when
>>> creating the filter it should always be 16bits and it's fine.
>>> 
>>> However, when the kernel is assigning priorities, the
>>> first assigned priority for a filter is 0xC0000000,
>>> the second is "the lowest priority - 1" so 0xBFFFFFFF.
>>> 
>>> It will assign this value in tcf_auto_prio() which will
>>> directly assign the 32bit value to to tp->prio with-
>>> 
>>> tp->prio = nprio ? : tcf_auto_prio(*back);
>> 
>> I think the above should read:
>> tp->prio = nprio ? : TC_H_MAJ(tcf_auto_prio(*back));
>> 
> I think you might be right, although, I don't see why
> we use a u32 when we're always storing a u16.

That's irrelevant.

You've essentially found what turned out to be a one-line bug, and
fixed it by changing and rearranging a lot of things.

So could I just get that one-liner fix patch from somebody?

Thanks :-)

Meanwhile, on the issue of how this value is stored, perhaps someone
intended to support "classful" filter priorities, just as we support
classful qdiscs.  Or perhaps someone had the idea to store the
protocol in there as well, who knows :-)