From: Javier Pello <javier.pello@urjc.es>
To: Kay Sievers <kay.sievers@vrfy.org>
Cc: Cornelia Huck <cornelia.huck@de.ibm.com>,
linux-kernel@vger.kernel.org, Greg KH <greg@kroah.com>
Subject: Re: [PATCH] request_firmware: skip timeout if userspace was not notified
Date: Thu, 09 Aug 2007 11:13:12 +0200 [thread overview]
Message-ID: <46BADAA8.70404@urjc.es> (raw)
In-Reply-To: <1186495705.3611.39.camel@lov.localdomain>
On Tue, 07 Aug 2007, Kay Sievers wrote:
> Nope, you would just fulfill in a completely generic way all outstanding
> requests when you are ready. All requests are all nicely grouped and
> visible in sysfs. There would be no need of coding your own device
> specific rebind. No timeout is needed or wanted, all requests would stay
> until userspace has handled them successfully or canceled them.
If I'm not mistaken, as it is now, requests are not grouped in any way.
The only hint that a firmware loading request is in progress are a couple
of files in the device directory in sysfs. Should I run, at boot, through
all the device directories to check which firmware requests are pending?
Also, this could pose problems if two drivers can handle a device but
the first that gets to bind to it needs firmware and userspace doesn't
provide it or cancel the loading. The device will remain bound without
giving the other driver a chance to handle it.
Anyway, I think something must be done, either removing the timeout and
making all firmware requests asynchronous, or at least skipping the
delay when we're sure that it's useless.
Javier
next prev parent reply other threads:[~2007-08-09 9:13 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-03 19:07 [PATCH] request_firmware: skip timeout if userspace was not notified Javier Pello
2007-08-04 5:21 ` david
2007-08-04 8:50 ` Javier Pello
2007-08-04 17:09 ` david
2007-08-04 21:20 ` Javier Pello
2007-08-06 12:24 ` Cornelia Huck
2007-08-06 20:23 ` Javier Pello
[not found] ` <20070807125844.4d756b04@gondo lin.boeblingen.de.ibm.com>
2007-08-07 10:58 ` Cornelia Huck
2007-08-07 11:46 ` Kay Sievers
2007-08-07 12:10 ` Cornelia Huck
2007-08-07 12:31 ` Kay Sievers
2007-08-07 12:48 ` Cornelia Huck
2007-08-07 12:47 ` Javier Pello
2007-08-07 12:57 ` Kay Sievers
2007-08-07 13:15 ` Cornelia Huck
2007-08-07 13:59 ` Javier Pello
2007-08-07 14:08 ` Kay Sievers
2007-08-07 14:38 ` Cornelia Huck
2007-08-09 9:13 ` Javier Pello [this message]
2007-08-09 9:21 ` Kay Sievers
2007-08-09 9:26 ` Cornelia Huck
2007-08-07 14:26 ` Cornelia Huck
2007-08-09 9:36 ` Javier Pello
2007-08-09 11:58 ` Kay Sievers
2007-08-10 21:24 ` Javier Pello
2007-08-11 13:26 ` Kay Sievers
2007-08-07 20:05 ` Andrew Morton
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=46BADAA8.70404@urjc.es \
--to=javier.pello@urjc.es \
--cc=cornelia.huck@de.ibm.com \
--cc=greg@kroah.com \
--cc=kay.sievers@vrfy.org \
--cc=linux-kernel@vger.kernel.org \
/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