Search Linux Wireless

Re: [PATCH] Add iwlwifi wireless drivers

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

 



James Ketrenos wrote:
Given the API mac80211 exposes, and how the stack operates, I don't see how using ifsta->work as the workqueue allows the driver to no longer evaluate the potential for deadlocks and races.

For the operations in the workqueue, they are obviously serialized.


The second argument you made:

"And then there is the Linus mantra: do what you must, and no more. No need for additional workqueues has been demonstrated."

To make the most use of the processing and power resources available, you must execute in parallel.

That's some impressive English parsing, but clearly wrong on the face of it. The Linus mantra cited is about MINIMALIZATION. When you say "to make the most use of..." you are MAXIMIZING something that need not be maximized.

As repeated many times, there is no NEED to execute this slow path stuff in parallel. Though yes, I agree parallel execution maximized power drain.

	Jeff


-
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