netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Daniel Drake <dsd@gentoo.org>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: Stephen Hemminger <shemminger@osdl.org>,
	Arjan van de Ven <arjan@infradead.org>,
	linux-kernel@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: Network drivers that don't suspend on interface down
Date: Wed, 20 Dec 2006 21:29:35 -0500	[thread overview]
Message-ID: <4589F18F.1090703@gentoo.org> (raw)
In-Reply-To: <20061221011209.GA32625@srcf.ucam.org>

Matthew Garrett wrote:
> Veering off at something of a tangent - how much of this should be true 
> for wireless devices? Softmac seems to be unhappy about setting the 
> essid unless the card is up, which breaks various assumptions...

You might regard that as a bug - I agree it probably makes sense for you 
to be able to set certain configuration variables before the interface 
is up, within reason.

However, the mentality adopted by most wireless drivers is the SIWESSID 
wireless extension ioctl means *associate*, something which obviously 
shouldn't be possible when the interface is down (radio off, etc).

While you might blame drivers for this possible misinterpretation, it 
can also be viewed as a design flaw in WE: the drivers have to handle 
the ioctl's directly, meaning that if you want some kind of 
configuration management then you have to do it on the driver level, and 
this doesn't feel right.

The situation is also made worse due to WE generally being hard to 
implement, and also the lack of documentation (really the only source 
here is the iwconfig man page).

This screams out for an 802.11-centric configuration system, and it 
looks like we have one on the way: cfg80211
 From reading some mails, it looks like the drivers will simply have to 
provide functions for "associate", "scan", etc, and the configuration 
management will be offloaded to the upper layers.

For the time being, I suggest you bring the interface up before setting 
the configuration. Regardless of the inconsistency of the current 
situation, and lack documentation saying which way it should be done, 
you are at least playing it safe and guaranteeing it works on all drivers.

Daniel

  parent reply	other threads:[~2006-12-21  2:29 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20061219185223.GA13256@srcf.ucam.org>
     [not found] ` <200612191959.43019.david-b@pacbell.net>
     [not found]   ` <20061220042648.GA19814@srcf.ucam.org>
     [not found]     ` <200612192114.49920.david-b@pacbell.net>
     [not found]       ` <20061220053417.GA29877@suse.de>
     [not found]         ` <20061220055209.GA20483@srcf.ucam.org>
     [not found]           ` <1166601025.3365.1345.camel@laptopd505.fenrus.org>
2006-12-20 12:53             ` Network drivers that don't suspend on interface down Matthew Garrett
2006-12-20 13:38               ` Arjan van de Ven
2006-12-20 14:31                 ` Matthew Garrett
2006-12-20 15:51                   ` Arjan van de Ven
2006-12-20 22:49                     ` Stephen Hemminger
2006-12-20 23:37                       ` Rick Jones
2006-12-19 23:51                         ` Stephen Hemminger
2006-12-21  0:11                       ` Francois Romieu
2006-12-20  0:26                         ` Stephen Hemminger
2006-12-21 11:18                           ` Francois Romieu
2006-12-21  1:12                       ` Matthew Garrett
2006-12-21  2:05                         ` Michael Wu
2006-12-21  2:18                           ` Matthew Garrett
2006-12-21  2:38                             ` Daniel Drake
2006-12-21  2:45                               ` Matthew Garrett
2006-12-21  3:08                                 ` Daniel Drake
2006-12-21  3:25                                   ` Matthew Garrett
2006-12-21  3:37                                     ` Dan Williams
2006-12-21  3:29                                   ` Dan Williams
2006-12-21  3:14                             ` Dan Williams
2006-12-21 13:14                               ` jamal
2006-12-21  2:29                         ` Daniel Drake [this message]
2006-12-21  2:10                     ` Jesse Brandeburg
2006-12-21  8:54                       ` Arjan van de Ven
2006-12-22  1:03                   ` Herbert Xu
2006-12-23  8:54                   ` Pavel Machek
2006-12-20 15:27                 ` Olivier Galibert
2006-12-20 15:34                   ` Arjan van de Ven
2006-12-20 16:40                     ` Olivier Galibert
2006-12-20 17:21                       ` Arjan van de Ven
2006-12-20 20:40                         ` Benny Amorsen
2006-12-20 21:49                           ` Arjan van de Ven
2006-12-20 21:15                     ` Stefan Rompf
2006-12-20 14:00               ` Jiri Benc
2006-12-20 18:12                 ` Dan Williams
2006-12-21  1:15                   ` Matthew Garrett
2006-12-21  1:57                     ` Michael Wu
2006-12-21  2:20                       ` Matthew Garrett
2006-12-21  3:02                         ` Dan Williams
2006-12-21  3:06                     ` Dan Williams
2006-12-21  3:14                       ` Matthew Garrett
2006-12-21  3:32                         ` Dan Williams
2006-12-21 13:19                           ` Sven-Haegar Koch
2006-12-21 17:16                             ` Dan Williams
2006-12-21 18:27                       ` Valdis.Kletnieks
2006-12-22  1:25                         ` Matt Domsch
2006-12-20 16:04               ` Maciej W. Rozycki
2006-12-21  5:25 David Brownell
2006-12-21  7:08 ` Stephen Hemminger
2006-12-21  8:11   ` David Brownell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4589F18F.1090703@gentoo.org \
    --to=dsd@gentoo.org \
    --cc=arjan@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjg59@srcf.ucam.org \
    --cc=netdev@vger.kernel.org \
    --cc=shemminger@osdl.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).