netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).