Re: [RFC PATCH 08/13] driver core: firmware loader: fix device lifetime

From: Ming Lei
Date: Wed Jul 25 2012 - 22:59:03 EST


On Thu, Jul 26, 2012 at 12:04 AM, Borislav Petkov <bp@xxxxxxxxx> wrote:
>> Also this patch holds the reference cound of @device before
>
> count

Good catch, will fix it in -v1.


>> - * it is not possible to sleep for long time. It can't be called
>> - * in atomic contexts.
>> + * it is not possible to sleep for long time.
>
> Let's state it explicitly:
>
> "it is not allowed to sleep for it is called in atomic context."

Looks you understand it incorrectly, the sentence is below

* Asynchronous variant of request_firmware() for user contexts where
* it is not possible to sleep for long time.

and maybe it should be changed to below:

* Asynchronous variant of request_firmware() for user contexts where
* it is not possible to sleep for long time or can't sleep at all.

Thanks,
--
Ming Lei
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/