lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Sat, 13 Jan 2007 14:30:33 -0500
From:	Pavel Roskin <proski@....org>
To:	Michael Buesch <mb@...sch.de>
Cc:	bcm43xx-dev@...ts.berlios.de, netdev@...r.kernel.org
Subject: Re: What is in bcm43xx-wireless-dev.git?

Quoting Michael Buesch <mb@...sch.de>:

> I looked more closely at it, but I really can't see why it oopses.
> Any idea what's happening there?

I haven't looked at the code yet, but I tried to locate the bad commit.  I tried
commit a13f85d8a8eb40dfd157ab78c2fb91b5765b7b9d, which is your last merge, just
before the SSB changes.

The kernel doesn't oops anymore!  But it doesn't connect either.  I reconfigured
the AP to use WPA-PSK with TKIP.  MadWifi works with it, but bcm43xx fails in
the same configuration.  The wireless-dev tip has the same problem.

So, the oops was introduced in the transition to the new SSB. 
bcm43xx_setup_dmaring(), which appears in the stack dump, is affected by the
patch.

I'll try to look closely at the changes.  My immediate suspect is that we have
too many different fields called "dev".  All it takes is one cast to hide a
horrible mistake.  Although I think it would have affected you as well (unless
it's casts to/from integers, something that won't be a problem on 32-bit
kernels).

> Well, ok. So it should work in my tree, too, once we fixed the DMA oops.

You mean there is a reason for me to think that your changes would fix
accociation to the AP?

I have never had any success with any d80211 based driver, so I have a strong
suspicion that I'm doing something wrong.  I'll try to report a clear-cut case
with a less problematic driver in a separate thread.

--
Regards,
Pavel Roskin
-
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