[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1258586065.2780.31.camel@achroite.uk.solarflarecom.com>
Date: Wed, 18 Nov 2009 23:14:25 +0000
From: Ben Hutchings <bhutchings@...arflare.com>
To: "Williams, Mitch A" <mitch.a.williams@...el.com>
Cc: "Kirsher, Jeffrey T" <jeffrey.t.kirsher@...el.com>,
"davem@...emloft.net" <davem@...emloft.net>,
"shemminger@...tta.com" <shemminger@...tta.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"gospo@...hat.com" <gospo@...hat.com>
Subject: RE: [RFC PATCH 1/4] net: Add support to netdev ops for changing
hardware queue MAC and VLAN filters
On Wed, 2009-11-18 at 14:37 -0700, Williams, Mitch A wrote:
>
> >From: Ben Hutchings [mailto:bhutchings@...arflare.com]
> >Sent: Wednesday, November 18, 2009 11:53 AM
>
> >How do you remove a filter?
> >
>
> You remove a filter by setting it to 0 on that queue. Works for either MAC or VLAN.
>
> >What about filtering on both MAC address and VLAN (our new controller
> >supports that)?
>
> Setting a MAC filter doesn't blow away the VLAN filter, or vice-versa. So just run 'ip' twice to set the filters. Our hardware does it too, and it works fine for me:
>
> $ ip link set eth1 queue 1 mac 00:11:22:33:44:55
> $ ip link set eth1 queue 1 vlan 10
Hmm, this is not what I would expect. I'm used to filters being defined
independently of queues in a hash table or CAM. In that case, setting
multiple filters pointing to one queue will result in the logical
disjunction of the filters. Do I understand correctly that you have
exactly one of each type of filter per queue, with multiple filters
interpreted as a logical conjunction?
Ben.
--
Ben Hutchings, Senior Software Engineer, Solarflare Communications
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists