lkml.org 
[lkml]   [2018]   [Feb]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 2/2] xen: xenbus: WARN_ON XS_TRANSACTION_{START,END} misuse
From
Date


On 02/10/2018 08:27 PM, Simon Gaiser wrote:
> Boris Ostrovsky:
>> On 02/07/2018 05:22 PM, Simon Gaiser wrote:
>>> + users_old = xs_state_users;
>>> xs_state_users--;
>>> if ((req->type == XS_TRANSACTION_START && req->msg.type == XS_ERROR) ||
>>> req->type == XS_TRANSACTION_END)
>>> xs_state_users--;
>>> + if (WARN_ON(xs_state_users > users_old))
>>
>>
>> WARN_ON_ONCE()?
>
> Since we "fix" the wrong decrement by clamping at zero it should not
> happen immediately again. But if you prefer _ONCE I can change it.
>


If this error can happen once then someone at least theoretically can
construct a case when it is repeated. So let's switch to _ONCE() variant.

Thanks.
-boris

\
 
 \ /
  Last update: 2018-02-11 16:29    [W:0.092 / U:1.708 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site