RE: [patch] d80211: use pfifo_qdisc_ops rather thand80211-specific qdisc

2006-11-02 Thread Simon Barber
There are many reasons to have the master device - and many reasons not
to get rid of it. The only excpetion is when using a limited fullmac
card.  Some fullmac cards only support a single ethernet interface, so
for these cards a single device may be OK, if we want to allow multiple
different appearences for different types of card (a bad thing in my
book). (Note - older prism 54g fullmac cards support WDS, hence need
multiple interfaces).

The master interface represents the physical networking device that
works using native 802.11 frames. The qdisc works on 802.11 frames - not
ethernet, and it can change how 802.11 management frames are queued. It
can only be got rid of in the case of a fullmac card with only a single
interface, where all management frames are processed in the card.

In addition as we move to make 802.11 devices native (i.e. create an
802.11 protocol) the 802.11 master interface will be essential.

I don't see the problem with 802.11 master devices - they are not
ethernet devices, and should not confuse any user space. Any user space
that gets confused by netdevices with arp-hdr types that they do not
recognise is broken, and needs to be fixed.

Simon


-Original Message-
From: Johannes Berg [mailto:[EMAIL PROTECTED] 
Sent: Thursday, November 02, 2006 6:23 AM
To: Christoph Hellwig
Cc: James Ketrenos; John W. Linville; Simon Barber; Jeff Garzik; Patrick
McHardy; David Kimdon; netdev@vger.kernel.org
Subject: Re: [patch] d80211: use pfifo_qdisc_ops rather
thand80211-specific qdisc

On Thu, 2006-11-02 at 15:05 +0100, Jiri Benc wrote:
 - Single device is not as easy as it sounds. It would require rather
   invasive changes in the networking core or ugly hacks in d80211. I'm
   afraid this is something not achievable in a near future with
current
   number of people working on d80211. But in general, it's doable and
   desirable, yes.

Actually, just as a note to all the others (talked to Jiri about it
already), I think it's easier than thought since the code is so
convoluted. The only purpose of the wmaster device appears to be to have
an outgoing frame queue through the qdisc. Since we require the 802.11
qdisc to be installed at the root for all netdevices anyway, we could
just as well hand the frame to it right away.

I'll look into that later today.

johannes
-
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [patch] d80211: use pfifo_qdisc_ops rather thand80211-specific qdisc

2006-11-02 Thread Johannes Berg
On Thu, 2006-11-02 at 08:33 -0800, Simon Barber wrote:
 There are many reasons to have the master device - and many reasons not
 to get rid of it. The only excpetion is when using a limited fullmac
 card.  Some fullmac cards only support a single ethernet interface, so
 for these cards a single device may be OK, if we want to allow multiple
 different appearences for different types of card (a bad thing in my
 book). (Note - older prism 54g fullmac cards support WDS, hence need
 multiple interfaces).
 
 The master interface represents the physical networking device that
 works using native 802.11 frames. The qdisc works on 802.11 frames - not
 ethernet, and it can change how 802.11 management frames are queued. It
 can only be got rid of in the case of a fullmac card with only a single
 interface, where all management frames are processed in the card.
 
 In addition as we move to make 802.11 devices native (i.e. create an
 802.11 protocol) the 802.11 master interface will be essential.

While these are good points, I don't believe having the master interface
as a *netdev* is essential. In fact, it now has no functionality as a
netdev, if you try routing packets over it they're simply dropped.

Of course we need some notion of the wireless PHY and cfg80211
explicitly represents that by assigning a wiphy index, but there's no
need for it to be a netdev.

johannes


signature.asc
Description: This is a digitally signed message part


Re: [patch] d80211: use pfifo_qdisc_ops rather thand80211-specific qdisc

2006-11-02 Thread Stephen Hemminger
On Thu, 02 Nov 2006 17:43:57 +0100
Johannes Berg [EMAIL PROTECTED] wrote:

 On Thu, 2006-11-02 at 08:33 -0800, Simon Barber wrote:
  There are many reasons to have the master device - and many reasons not
  to get rid of it. The only excpetion is when using a limited fullmac
  card.  Some fullmac cards only support a single ethernet interface, so
  for these cards a single device may be OK, if we want to allow multiple
  different appearences for different types of card (a bad thing in my
  book). (Note - older prism 54g fullmac cards support WDS, hence need
  multiple interfaces).
  
  The master interface represents the physical networking device that
  works using native 802.11 frames. The qdisc works on 802.11 frames - not
  ethernet, and it can change how 802.11 management frames are queued. It
  can only be got rid of in the case of a fullmac card with only a single
  interface, where all management frames are processed in the card.
  
  In addition as we move to make 802.11 devices native (i.e. create an
  802.11 protocol) the 802.11 master interface will be essential.
 
 While these are good points, I don't believe having the master interface
 as a *netdev* is essential. In fact, it now has no functionality as a
 netdev, if you try routing packets over it they're simply dropped.

It makes 802.11 packet capture easier as well.  Please don't invent yet
another network access object for the master device.

 Of course we need some notion of the wireless PHY and cfg80211
 explicitly represents that by assigning a wiphy index, but there's no
 need for it to be a netdev.
 
 johannes


-- 
Stephen Hemminger [EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe netdev in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [patch] d80211: use pfifo_qdisc_ops rather thand80211-specific qdisc

2006-11-02 Thread Johannes Berg
On Thu, 2006-11-02 at 14:34 -0800, Stephen Hemminger wrote:

 It makes 802.11 packet capture easier as well.  Please don't invent yet
 another network access object for the master device.

Oh, but the master device doesn't get any packets you could capture. For
that, you need to add a monitor interface.

johannes


signature.asc
Description: This is a digitally signed message part