* [PATCH] xen: don't print error message in case of missing Xenstore entry
@ 2017-05-29 9:13 Juergen Gross
2017-05-30 13:25 ` Boris Ostrovsky
0 siblings, 1 reply; 5+ messages in thread
From: Juergen Gross @ 2017-05-29 9:13 UTC (permalink / raw)
To: linux-kernel, xen-devel; +Cc: boris.ostrovsky, Juergen Gross
When registering for the Xenstore watch of the node control/sysrq the
handler will be called at once. Don't issue an error message if the
Xenstore node isn't there, as it will be created only when an event
is being triggered.
Signed-off-by: Juergen Gross <jgross@suse.com>
---
drivers/xen/manage.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/xen/manage.c b/drivers/xen/manage.c
index c1ec8ee80924..7ddd0803da23 100644
--- a/drivers/xen/manage.c
+++ b/drivers/xen/manage.c
@@ -277,8 +277,11 @@ static void sysrq_handler(struct xenbus_watch *watch, const char *path,
err = xenbus_transaction_start(&xbt);
if (err)
return;
- if (xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key) < 0) {
- pr_err("Unable to read sysrq code in control/sysrq\n");
+ err = xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key);
+ if (err < 0) {
+ if (err != -ENOENT)
+ pr_err("Error %d reading sysrq code in control/sysrq\n",
+ err);
xenbus_transaction_end(xbt, 1);
return;
}
--
2.12.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] xen: don't print error message in case of missing Xenstore entry
2017-05-29 9:13 [PATCH] xen: don't print error message in case of missing Xenstore entry Juergen Gross
@ 2017-05-30 13:25 ` Boris Ostrovsky
2017-05-30 15:03 ` Juergen Gross
0 siblings, 1 reply; 5+ messages in thread
From: Boris Ostrovsky @ 2017-05-30 13:25 UTC (permalink / raw)
To: Juergen Gross, linux-kernel, xen-devel
On 05/29/2017 05:13 AM, Juergen Gross wrote:
> When registering for the Xenstore watch of the node control/sysrq the
> handler will be called at once. Don't issue an error message if the
> Xenstore node isn't there, as it will be created only when an event
> is being triggered.
>
> Signed-off-by: Juergen Gross <jgross@suse.com>
> ---
> drivers/xen/manage.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/xen/manage.c b/drivers/xen/manage.c
> index c1ec8ee80924..7ddd0803da23 100644
> --- a/drivers/xen/manage.c
> +++ b/drivers/xen/manage.c
> @@ -277,8 +277,11 @@ static void sysrq_handler(struct xenbus_watch *watch, const char *path,
> err = xenbus_transaction_start(&xbt);
> if (err)
> return;
> - if (xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key) < 0) {
> - pr_err("Unable to read sysrq code in control/sysrq\n");
> + err = xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key);
> + if (err < 0) {
> + if (err != -ENOENT)
Can we distinguish initialization invocation from actual watch firing?
E.g. '|| (system_state >= SYSTEM_RUNNING)'?
-boris
> + pr_err("Error %d reading sysrq code in control/sysrq\n",
> + err);
> xenbus_transaction_end(xbt, 1);
> return;
> }
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] xen: don't print error message in case of missing Xenstore entry
2017-05-30 13:25 ` Boris Ostrovsky
@ 2017-05-30 15:03 ` Juergen Gross
2017-05-30 17:08 ` Boris Ostrovsky
0 siblings, 1 reply; 5+ messages in thread
From: Juergen Gross @ 2017-05-30 15:03 UTC (permalink / raw)
To: Boris Ostrovsky, linux-kernel, xen-devel
On 30/05/17 15:25, Boris Ostrovsky wrote:
> On 05/29/2017 05:13 AM, Juergen Gross wrote:
>> When registering for the Xenstore watch of the node control/sysrq the
>> handler will be called at once. Don't issue an error message if the
>> Xenstore node isn't there, as it will be created only when an event
>> is being triggered.
>>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>> ---
>> drivers/xen/manage.c | 7 +++++--
>> 1 file changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/xen/manage.c b/drivers/xen/manage.c
>> index c1ec8ee80924..7ddd0803da23 100644
>> --- a/drivers/xen/manage.c
>> +++ b/drivers/xen/manage.c
>> @@ -277,8 +277,11 @@ static void sysrq_handler(struct xenbus_watch *watch, const char *path,
>> err = xenbus_transaction_start(&xbt);
>> if (err)
>> return;
>> - if (xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key) < 0) {
>> - pr_err("Unable to read sysrq code in control/sysrq\n");
>> + err = xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key);
>> + if (err < 0) {
>> + if (err != -ENOENT)
>
> Can we distinguish initialization invocation from actual watch firing?
> E.g. '|| (system_state >= SYSTEM_RUNNING)'?
The watch will fire again after suspend/resume (e.g. live migration).
Juergen
>
> -boris
>
>> + pr_err("Error %d reading sysrq code in control/sysrq\n",
>> + err);
>> xenbus_transaction_end(xbt, 1);
>> return;
>> }
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] xen: don't print error message in case of missing Xenstore entry
2017-05-30 15:03 ` Juergen Gross
@ 2017-05-30 17:08 ` Boris Ostrovsky
2017-05-30 17:10 ` Juergen Gross
0 siblings, 1 reply; 5+ messages in thread
From: Boris Ostrovsky @ 2017-05-30 17:08 UTC (permalink / raw)
To: Juergen Gross, linux-kernel, xen-devel
On 05/30/2017 11:03 AM, Juergen Gross wrote:
> On 30/05/17 15:25, Boris Ostrovsky wrote:
>> On 05/29/2017 05:13 AM, Juergen Gross wrote:
>>> When registering for the Xenstore watch of the node control/sysrq the
>>> handler will be called at once. Don't issue an error message if the
>>> Xenstore node isn't there, as it will be created only when an event
>>> is being triggered.
>>>
>>> Signed-off-by: Juergen Gross <jgross@suse.com>
>>> ---
>>> drivers/xen/manage.c | 7 +++++--
>>> 1 file changed, 5 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/drivers/xen/manage.c b/drivers/xen/manage.c
>>> index c1ec8ee80924..7ddd0803da23 100644
>>> --- a/drivers/xen/manage.c
>>> +++ b/drivers/xen/manage.c
>>> @@ -277,8 +277,11 @@ static void sysrq_handler(struct xenbus_watch *watch, const char *path,
>>> err = xenbus_transaction_start(&xbt);
>>> if (err)
>>> return;
>>> - if (xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key) < 0) {
>>> - pr_err("Unable to read sysrq code in control/sysrq\n");
>>> + err = xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key);
>>> + if (err < 0) {
>>> + if (err != -ENOENT)
>> Can we distinguish initialization invocation from actual watch firing?
>> E.g. '|| (system_state >= SYSTEM_RUNNING)'?
> The watch will fire again after suspend/resume (e.g. live migration).
That's unfortunate. (And system_state check would also not be a good
solution btw since the watch might be processed by the watch thread
after we enter SYSTEM_RUNNING).
Can you add a comment explaining why we are ignoring ENOENT?
-boris
>
>
> Juergen
>
>> -boris
>>
>>> + pr_err("Error %d reading sysrq code in control/sysrq\n",
>>> + err);
>>> xenbus_transaction_end(xbt, 1);
>>> return;
>>> }
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] xen: don't print error message in case of missing Xenstore entry
2017-05-30 17:08 ` Boris Ostrovsky
@ 2017-05-30 17:10 ` Juergen Gross
0 siblings, 0 replies; 5+ messages in thread
From: Juergen Gross @ 2017-05-30 17:10 UTC (permalink / raw)
To: Boris Ostrovsky, linux-kernel, xen-devel
On 30/05/17 19:08, Boris Ostrovsky wrote:
> On 05/30/2017 11:03 AM, Juergen Gross wrote:
>> On 30/05/17 15:25, Boris Ostrovsky wrote:
>>> On 05/29/2017 05:13 AM, Juergen Gross wrote:
>>>> When registering for the Xenstore watch of the node control/sysrq the
>>>> handler will be called at once. Don't issue an error message if the
>>>> Xenstore node isn't there, as it will be created only when an event
>>>> is being triggered.
>>>>
>>>> Signed-off-by: Juergen Gross <jgross@suse.com>
>>>> ---
>>>> drivers/xen/manage.c | 7 +++++--
>>>> 1 file changed, 5 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/drivers/xen/manage.c b/drivers/xen/manage.c
>>>> index c1ec8ee80924..7ddd0803da23 100644
>>>> --- a/drivers/xen/manage.c
>>>> +++ b/drivers/xen/manage.c
>>>> @@ -277,8 +277,11 @@ static void sysrq_handler(struct xenbus_watch *watch, const char *path,
>>>> err = xenbus_transaction_start(&xbt);
>>>> if (err)
>>>> return;
>>>> - if (xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key) < 0) {
>>>> - pr_err("Unable to read sysrq code in control/sysrq\n");
>>>> + err = xenbus_scanf(xbt, "control", "sysrq", "%c", &sysrq_key);
>>>> + if (err < 0) {
>>>> + if (err != -ENOENT)
>>> Can we distinguish initialization invocation from actual watch firing?
>>> E.g. '|| (system_state >= SYSTEM_RUNNING)'?
>> The watch will fire again after suspend/resume (e.g. live migration).
>
>
> That's unfortunate. (And system_state check would also not be a good
> solution btw since the watch might be processed by the watch thread
> after we enter SYSTEM_RUNNING).
>
> Can you add a comment explaining why we are ignoring ENOENT?
Sure.
Juergen
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-05-30 17:10 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-29 9:13 [PATCH] xen: don't print error message in case of missing Xenstore entry Juergen Gross
2017-05-30 13:25 ` Boris Ostrovsky
2017-05-30 15:03 ` Juergen Gross
2017-05-30 17:08 ` Boris Ostrovsky
2017-05-30 17:10 ` Juergen Gross
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).