netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jean Tourrilhes <jt@hpl.hp.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Michael Buesch <mb@bu3sch.de>,
	linux-wireless@vger.kernel.org, netdev <netdev@vger.kernel.org>,
	Jeff Garzik <jgarzik@pobox.com>, Dan Williams <dcbw@redhat.com>,
	Jouni Malinen <jkm@devicescape.com>
Subject: Re: wireless extensions vs. 64-bit architectures
Date: Thu, 8 Mar 2007 10:49:54 -0800	[thread overview]
Message-ID: <20070308184954.GA24485@bougret.hpl.hp.com> (raw)
In-Reply-To: <1173364747.14001.7.camel@johannes.berg>

On Thu, Mar 08, 2007 at 03:39:07PM +0100, Johannes Berg wrote:
> On Tue, 2007-03-06 at 18:03 -0800, Jean Tourrilhes wrote:
> 
> > 	Ok, please check the patch attached. I don't have a box to
> > test that on, and on my 32 bit kernel it is not even compiled, but I
> > believe I got everything all right.
> 
> Don't I wish it was that easy... Granted, that does seem to fix the
> issue with iw_point but that's not the only thing that's broken. Here's
> the next thing, I think this only happens after applying that patch,
> without it probably just craps out earlier. Same thing happens with
> iwevent (and maybe other things too.)
> 
> Enabling debugging in wireless tools yields (on my 64-bit kernel with
> 32-bit userland):
> # LD_LIBRARY_PATH=.  ./iwlist wlan0 scan
> Scan result 4096
> [00:18:8B:15:50:8A:35:E0:00:01:00:C0:02:E1:B8:0E:C0:00:00:00:50:8A:35:F0:00:19:8B:1B:50:8A:35:E0:00:09:00:01:50:8A:35:F0:47:6F:6C:6F:73:4E:65:74:7A:00:18:8B:01:50:8A:35:E0:49:45:45:45:20:38:30:32:2E:31:31:62:67:00:35:F0:00:0C:8B:07:50:8A:35:E0:00:00:00:03:00:10:8B:05:50:8A:35:E0:00:00:00:07:00:00:00:32:00:10:8B:2B:50:8A:35:E0:00:00:08:00:67:00:35:F0:00:70:8B:21:50:8A:35:E0:00:0F:42:40:00:00:00:32:00:1E:84:80:00:00:00:32:00:53:EC:60:00:00:00:32:00:5B:8D:80:00:00:00:32:00:89:54:40:00:00:00:32:00:A7:D8:C0:00:00:00:<lots more zeroes>
> wlan0     Scan completed : 
> DBG - stream->current = 0x10019008, stream->value = (nil), stream->end = 0x1001a008
> DBG - iwe->cmd = 0x8B15, iwe->len = 24
> 
>                          ^^^^^^^^^^^^^ that's already bogus

	Ok, now I get it. It's an alignement issue, the kernel align
struct to 64 bit boundary, whereas the userspace expect it to 32 bit
boundary. Pretty nasty.
	A proper fix would involve forcing the alignement in the
kernel. Unfortunately, that would break 64bit->64bit configs. I think
I can build a workaround for this in iwlib.

	Thanks for the detailed bug report !

	Jean

  parent reply	other threads:[~2007-03-08 18:51 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-06  1:27 wireless extensions vs. 64-bit architectures Johannes Berg
2007-03-06 14:31 ` Johannes Berg
2007-03-06 17:13 ` Jean Tourrilhes
     [not found]   ` <20070306171316.GA19669-yAE0UhLNZJawPNPzzlOzwdBPR1lH4CV8@public.gmane.org>
2007-03-06 18:43     ` Michael Buesch
     [not found]       ` <200703061943.07350.mb-fseUSCV1ubazQB+pC5nmwQ@public.gmane.org>
2007-03-07  1:42         ` Jean Tourrilhes
2007-03-07  2:03       ` Jean Tourrilhes
     [not found]         ` <20070307020310.GA20466-yAE0UhLNZJawPNPzzlOzwdBPR1lH4CV8@public.gmane.org>
2007-03-08 14:39           ` Johannes Berg
     [not found]             ` <1173364747.14001.7.camel-YfaajirXv214zXjbi5bjpg@public.gmane.org>
2007-03-08 16:51               ` Johannes Berg
2007-03-08 17:37               ` Johannes Berg
2007-03-08 18:49             ` Jean Tourrilhes [this message]
2007-03-08 19:08               ` Johannes Berg
     [not found]                 ` <1173380909.3248.52.camel-YfaajirXv214zXjbi5bjpg@public.gmane.org>
2007-03-08 19:13                   ` Jean Tourrilhes
2007-03-08 19:23                     ` Johannes Berg
     [not found]               ` <20070308184954.GA24485-yAE0UhLNZJawPNPzzlOzwdBPR1lH4CV8@public.gmane.org>
2007-03-08 19:27                 ` Johannes Berg
2007-03-08 19:34                   ` Jouni Malinen
     [not found]                     ` <20070308193412.GG23040-r/za7OOdgF0ztatW0fm/fQ@public.gmane.org>
2007-03-08 19:40                       ` Johannes Berg
     [not found]                         ` <1173382802.3248.68.camel-YfaajirXv214zXjbi5bjpg@public.gmane.org>
2007-03-08 22:11                           ` Jean Tourrilhes
     [not found]                             ` <20070308221128.GA24884-yAE0UhLNZJawPNPzzlOzwdBPR1lH4CV8@public.gmane.org>
2007-03-08 22:17                               ` Randy Dunlap
     [not found]                                 ` <20070308141756.efdfd6da.randy.dunlap-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
2007-03-08 22:30                                   ` Jean Tourrilhes
     [not found]                                     ` <20070308223058.GA24960-yAE0UhLNZJawPNPzzlOzwdBPR1lH4CV8@public.gmane.org>
2007-03-08 22:36                                       ` Johannes Berg
2007-03-08 22:34                                   ` David Miller
2007-03-08 22:49                                 ` Pavel Roskin
2007-03-08 22:22                               ` Johannes Berg
     [not found]                                 ` <1173392526.3831.12.camel-YfaajirXv214zXjbi5bjpg@public.gmane.org>
2007-03-08 22:36                                   ` Jean Tourrilhes
2007-03-08 22:35                               ` Johannes Berg
2007-03-09 21:35                                 ` Jean Tourrilhes
     [not found]                                   ` <20070309213531.GA28070-yAE0UhLNZJawPNPzzlOzwdBPR1lH4CV8@public.gmane.org>
2007-03-09 23:19                                     ` Jouni Malinen
     [not found]                                       ` <20070309231922.GF24143-r/za7OOdgF0ztatW0fm/fQ@public.gmane.org>
2007-03-10  1:01                                         ` Jean Tourrilhes
2007-03-11 17:40                                     ` Johannes Berg
     [not found]                                       ` <1173634801.3382.8.camel-YfaajirXv214zXjbi5bjpg@public.gmane.org>
2007-03-11 20:11                                         ` Ulrich Kunitz
     [not found]                                           ` <20070311201148.GA25938-WhJF3imHnk+bHbQv0o6mQ2hcgWyyV7dXYZdqe9AaVak@public.gmane.org>
2007-03-11 20:30                                             ` Michael Buesch
2007-03-12 17:56                                         ` Jean Tourrilhes
2007-03-12 18:21                                           ` Jouni Malinen
     [not found]                                             ` <20070312182149.GA1785-r/za7OOdgF0ztatW0fm/fQ@public.gmane.org>
2007-03-12 20:34                                               ` Jean Tourrilhes
     [not found]                                           ` <20070312175639.GA4048-yAE0UhLNZJawPNPzzlOzwdBPR1lH4CV8@public.gmane.org>
2007-03-13 19:42                                             ` Johannes Berg
     [not found]                                               ` <1173814925.12717.6.camel-YfaajirXv214zXjbi5bjpg@public.gmane.org>
2007-03-13 21:30                                                 ` Jean Tourrilhes

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=20070308184954.GA24485@bougret.hpl.hp.com \
    --to=jt@hpl.hp.com \
    --cc=dcbw@redhat.com \
    --cc=jgarzik@pobox.com \
    --cc=jkm@devicescape.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=mb@bu3sch.de \
    --cc=netdev@vger.kernel.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).