Re: [PATCH] PM / wakeup: use seq_open() to show wakeup stats

From: Rafael J. Wysocki
Date: Fri Mar 02 2018 - 04:14:16 EST


On Fri, Mar 2, 2018 at 10:05 AM, Pavel Machek <pavel@xxxxxx> wrote:
> On Fri 2018-03-02 13:01:00, Ganesh Mahendran wrote:
>> single_open() interface requires that the whole output must
>> fit into a single buffer. This will lead to timeout when
>> system memory is not in a good situation.
>>
>> This patch use seq_open() to show wakeup stats. This method
>> need only one page, so timeout will not be observed.
>
> Can the rcu_read_unlock be delayed arbitrarily by userland code?
>
> Is that ok with RCU?

No, it isn't. :-)