Search Linux Wireless

Re: [PATCH] libertas: before sleeping, check for a command result

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




----- Original Message ----- From: "Holger Schurig" <hs4233@xxxxxxxxxxxxxxxxxxxx>
To: "Johan Adolfsson" <johana@xxxxxxxx>
Cc: <libertas-dev@xxxxxxxxxxxxxxxxxxx>; "'Dan Williams'" <dcbw@xxxxxxxxxx>; <linux-wireless@xxxxxxxxxxxxxxx>; "'John W. Linville'" <linville@xxxxxxxxxxxxx>
Sent: Friday, May 23, 2008 4:57 PM
Subject: Re: [PATCH] libertas: before sleeping, check for a command result


I have experience a couple of timeout and resends and after
sending another commands I get the response from the previous
one, and the patch seems to solve a couple of those for me -
still se some problem fetching packet from  firmware with ret
= -22 though. And it could be dependant on how much debug I
have turned on..

Keep in mind that maybe the firmware actually really takes ages
to complete a command. David Woodhouse reported this.

In my case, because I had the proper debugging statements in my
interrupt handler, I knew for sure that I already received the
response. Can you verify that, for the comment timeouts you
noticed, if this did happen or not?

While trying to find out what went wrong I believe I stumbled
on a cache issue since our sdio driver didn't invalidate the cache
and the DMA is not cachecoherent.

Possible something for others to check in their SDIO drivers
as well.
Ran into some weird problems when trying to verify if it solved my problem though, so I will get back with more info when I get it up and running again.

Best regards
/Johan


--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux