Re: [RESEND][PATCH] Bluetooth: Make request workqueue freezable

On 05/21/15 19:32, Takashi Iwai wrote:

At Thu, 21 May 2015 19:27:41 +0200,
Arend van Spriel wrote:

quoted

On 05/21/15 17:35, Takashi Iwai wrote:

quoted

At Thu, 21 May 2015 11:26:17 -0400 (EDT),
Alan Stern wrote:

quoted

On Thu, 21 May 2015, Takashi Iwai wrote:

quoted

At Thu, 21 May 2015 10:18:08 -0400 (EDT),
Alan Stern wrote:

quoted

On Thu, 21 May 2015, Takashi Iwai wrote:

quoted

Then avoiding the failed firmware is no solution, indeed.
If it's a new probe, it should be never executed during resume.

Can you expand this comment?  What's wrong with probing during resume?

Well, if the probe requires the access to a user-space file, it can't
be done during resume.  That's the very problem we're seeing now.
The firmware loader can't help much alone if it's a new device
object.

So you are saying each device driver should come up with some retry 
mechanism. Would make more sense to come up with something like that 
behind the scenes in the firmware loader so all device drivers can rely 
on one and the same solution.

Regards,
Arend

quoted

quoted

quoted

But the same thing happens during early boot, if the driver is built
into the kernel.  When the probe occurs, userspace isn't up and running
yet, so the firmware loader can't do anything.

Why should probe during resume be any worse than probe during early
boot?

The early boot has initrd, so the files can be there.  But the resume
has no way to fetch the file except for cached data.

but initrd is optional so without initrd it is pretty much the same.

User can build the firmware into the kernel.


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