From: Pete Zaitcev <zaitcev@redhat.com>
To: "Jouni Malinen" <jkm@devicescape.com>
Cc: johannes@sipsolutions.net, dcbw@redhat.com,
netdev@vger.kernel.org, linville@tuxdriver.com,
softmac-dev@sipsolutions.net, zaitcev@redhat.com
Subject: Re: [patch 1/3] softmac: return -EAGAIN from getscan while scanning
Date: Thu, 13 Apr 2006 16:35:25 -0700 [thread overview]
Message-ID: <20060413163525.6d866fee.zaitcev@redhat.com> (raw)
In-Reply-To: <20060413224522.GK15499@instant802.com>
On Thu, 13 Apr 2006 15:45:22 -0700, "Jouni Malinen" <jkm@devicescape.com> wrote:
> > [...] I can guess now what your concern is, even though
> > you failed to articulate it: a single-threaded GUI application,
> > which cannot respond to events when blocked in getting scan results.
> > If that's the case, we should be looking at having both blocking
> > and non-blocking calls to fetch scan results.
>
> No, my main concern was single-threaded design in wpa_suppliant.. If the
> ioctl call is blocking, I would need to create a new (well, the first
> additional) thread just for this use. Without that, the blocking call
> would also block all control interface commands (interaction with
> external programs) and controlling of other interfaces (if more than one
> is used).
I see... Thanks for explainig it.
> [...] Furthermore, blocking ioctl
> handlers is not really something I would like to see in the kernel..
> Aren't there some locks/semaphores/etc. kept for some cases?
I think you are taking this concept too far. Consider things like
an ioctl to seek a magnetic tape N files forward, or an ioctl to
flush characters to a terminal. They seem to work fine in blocking
mode. And no, we do not have a generic API to expose semaphores
(I suppose you did not mean SYSV IPC above :-).
-- Pete
next prev parent reply other threads:[~2006-04-13 23:37 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20060411085805.949313000@sipsolutions.net>
2006-04-13 0:42 ` [patch 4] softmac: fix event sending Johannes Berg
[not found] ` <20060411085841.252064000@sipsolutions.net>
2006-04-13 9:00 ` [patch 1/3] softmac: return -EAGAIN from getscan while scanning Pete Zaitcev
2006-04-13 9:06 ` Johannes Berg
2006-04-13 9:34 ` Johannes Berg
2006-04-13 12:14 ` Dan Williams
2006-04-13 16:13 ` Jean Tourrilhes
2006-04-15 19:24 ` Johannes Berg
2006-04-15 21:27 ` Dan Williams
2006-04-13 12:12 ` Dan Williams
2006-04-13 12:19 ` Johannes Berg
2006-04-13 16:00 ` Jouni Malinen
2006-04-13 22:28 ` Pete Zaitcev
2006-04-13 22:45 ` Jouni Malinen
2006-04-13 23:03 ` Stephen Hemminger
2006-04-13 23:35 ` Pete Zaitcev [this message]
2006-04-13 22:21 ` Pete Zaitcev
2006-04-13 9:41 ` [patch 5] softmac: report when scanning has finished Johannes Berg
2006-04-13 12:15 ` Dan Williams
2006-04-13 23:58 [patch 1/3] softmac: return -EAGAIN from getscan while scanning Jean Tourrilhes
-- strict thread matches above, loose matches on Subject: below --
2006-04-14 0:01 Jean Tourrilhes
2006-04-14 15:58 ` Herbert Xu
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=20060413163525.6d866fee.zaitcev@redhat.com \
--to=zaitcev@redhat.com \
--cc=dcbw@redhat.com \
--cc=jkm@devicescape.com \
--cc=johannes@sipsolutions.net \
--cc=linville@tuxdriver.com \
--cc=netdev@vger.kernel.org \
--cc=softmac-dev@sipsolutions.net \
/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).