On Tue, May 24, 2005 at 03:15:06PM +0200, Jiri Benc wrote:
> Patches to merge the ipw2100 and ipw2200 drivers to the kernel can be
> downloaded from:
> http://jikos.cz/~jbohac/wifi/import-ipw2100.patch
> http://jikos.cz/~jbohac/wifi/import-ipw2200.patch
> (they are too large to post them here)
>
> The ipw2100 patch is just a merge of ipw2100 driver with Pavel Machek's
> cleanup patch.
It contains a number of bogus uses of HZ.
+ IPW_DEBUG_INFO("Command completion failed out after %dms.\n",
+ HOST_COMPLETE_TIMEOUT / (HZ / 100));
..
+ schedule_timeout(HZ / 100);
..
What I don't understand, is that HZ in kernel-code should be 1000
according to <asm/param.h>, though for some reason, I've seen
divide by zeros, and really strange oopses with this code.
It's as if its for some reason, choosing to use the userspace HZ
definition. Hmm, does it need an explicit include of <asm/param.h>
perhaps ?
Dave
|