netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: jt@hpl.hp.com
Cc: netdev@oss.sgi.com, Gertjan van Wingerde <gwingerde@home.nl>,
	sfeldma@pobox.com, jkmaline@cc.hut.fi
Subject: Re: [RFC] Wireless extensions rethink
Date: Thu, 17 Jun 2004 16:06:25 -0400	[thread overview]
Message-ID: <40D1F9C1.1040907@pobox.com> (raw)
In-Reply-To: <20040617194448.GC31763@bougret.hpl.hp.com>

Jean Tourrilhes wrote:
> On Thu, Jun 17, 2004 at 03:34:22PM -0400, Jeff Garzik wrote:
> 
>>Your patch is half the job -- it allows development of a type-specific 
>>interface...
> 
> 
> 	Which is exactly what you want. Good.

The patch doesn't kill the type-opaque interface, so only 50% of what I 
want.


>>So while this patch may be useful in early development, it does not 
>>allow the direct exposure of core wireless code to the type-specific 
>>interfaces
> 
> 
> 	What is the core wireless code ?

At the moment, "the stuff that calls the driver-local iw_handlers", but 
hopefully more generic wireless core soon with the merge of HostAP.


>>and as such, it can paper over problems that would be 
>>immediately obviously if the type-specific interface were the only one 
>>to exist.
> 
> 
> 	Any new code in the kernel is free to use only the new
> API. That's a big enoug incentive to migrate drivers over to the new
> API.

With the type-opaque interface gone (key design goal), drivers not using 
the new API will not function...


>>>>Also there is a fourth -- WE doesn't work 100% when you have 
>>>>a 32-bit userland and a 64-bit kernel.
>>>
>>>
>>>	Since when ? What made you change your mind ?
>>>	Please check :
>>>http://marc.theaimsgroup.com/?l=linux-netdev&m=107894322418086&w=2
>>
>>The general API, yes.  But most driver-private interfaces will fail 
>>miserably through 32/64-bit translation.
> 
> 
> 	That's fixable, and easy to fix, if needed. You have all the
> data you need in the kernel.

Not really -- it's the same problem as SIOCDEVPRIVATE.  Driver-private 
interfaces by definition change for each driver.  Translation of the 
same ioctl differs on a per-driver basis.  Consider what happens when 
passing pointers from userland, for example...

It was for this reason that we created the MII ioctls, which were 
previously SIOCDEVPRIVATE.

	Jeff

  reply	other threads:[~2004-06-17 20:06 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-06-11 18:49 [RFC] Wireless extensions rethink Feldman, Scott
2004-06-15 16:39 ` Gertjan van Wingerde
2004-06-15 17:22   ` Vladimir Kondratiev
2004-06-16  9:13   ` Scott Feldman
2004-06-16 15:28     ` Gerald Britton
2004-06-16 17:40       ` Vladimir Kondratiev
2004-06-16 17:53       ` Scott Feldman
2004-06-16 19:06         ` Gerald Britton
2004-06-17  5:57         ` Luis R. Rodriguez
2004-06-16 17:46     ` Gertjan van Wingerde
2004-06-16 19:06       ` Scott Feldman
2004-06-16 19:49         ` Jeff Garzik
2004-06-16 22:25           ` Scott Feldman
2004-06-16 20:50         ` Jean Tourrilhes
2004-06-16 20:42       ` Jean Tourrilhes
2004-06-16 21:36         ` Jeff Garzik
2004-06-16 22:33           ` Jean Tourrilhes
2004-06-16 23:06             ` Jeff Garzik
2004-06-16 23:11               ` Jean Tourrilhes
2004-06-17 17:47               ` Jean Tourrilhes
2004-06-17 18:23                 ` Jeff Garzik
2004-06-17 18:26                   ` Jeff Garzik
2004-06-17 18:30                     ` Gertjan van Wingerde
2004-06-17 18:51                     ` Stephen Hemminger
2004-06-17 19:00                       ` Jean Tourrilhes
2004-06-17 19:10                         ` Jeff Garzik
2004-06-17 18:58                     ` Jean Tourrilhes
2004-06-17 19:02                       ` Jeff Garzik
2004-06-17 19:13                         ` Jean Tourrilhes
2004-06-17 19:34                           ` Jeff Garzik
2004-06-17 19:44                             ` Jean Tourrilhes
2004-06-17 20:06                               ` Jeff Garzik [this message]
2004-06-17 20:39                                 ` Jean Tourrilhes
2004-06-17 18:56                   ` Jean Tourrilhes
2004-06-17 19:09                     ` Jeff Garzik
2004-06-17 19:11                       ` Jeff Garzik
2004-06-17 19:31                       ` Jean Tourrilhes
2004-06-17 19:52                         ` Jeff Garzik
2004-06-17 20:46                           ` Jean Tourrilhes
2004-06-18 22:11                             ` Andrew Morton
2004-06-18 22:54                               ` Jeff Garzik
2004-06-16 22:48         ` Scott Feldman
  -- strict thread matches above, loose matches on Subject: below --
2004-06-07 19:51 Gertjan van Wingerde
2004-06-07 20:52 ` Ben Greear
2004-06-07 18:33 Feldman, Scott
2004-06-07 18:39 ` Stephen Hemminger
2004-06-08 11:19 ` Herbert Xu

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=40D1F9C1.1040907@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=gwingerde@home.nl \
    --cc=jkmaline@cc.hut.fi \
    --cc=jt@hpl.hp.com \
    --cc=netdev@oss.sgi.com \
    --cc=sfeldma@pobox.com \
    /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).