diff mbox

[4/4] xen: Drop un-informative message during boot

Message ID 20170720110402.15313-5-punit.agrawal@arm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Punit Agrawal July 20, 2017, 11:04 a.m. UTC
On systems that are not booted as a Xen domain, the xenfs driver prints
the following message during boot.

[    3.460595] xenfs: not registering filesystem on non-xen platform

As the user chose not to boot a Xen domain, this message does not
provide useful information. Drop this message.

Signed-off-by: Punit Agrawal <punit.agrawal@arm.com>
Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: Juergen Gross <jgross@suse.com>
---
 drivers/xen/xenfs/super.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Boris Ostrovsky July 20, 2017, 9:20 p.m. UTC | #1
On 07/20/2017 07:04 AM, Punit Agrawal wrote:
> On systems that are not booted as a Xen domain, the xenfs driver prints
> the following message during boot.
>
> [    3.460595] xenfs: not registering filesystem on non-xen platform
>
> As the user chose not to boot a Xen domain, this message does not
> provide useful information. Drop this message.
>
> Signed-off-by: Punit Agrawal <punit.agrawal@arm.com>
> Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> Cc: Juergen Gross <jgross@suse.com>

Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Punit Agrawal July 21, 2017, 10:12 a.m. UTC | #2
Boris Ostrovsky <boris.ostrovsky@oracle.com> writes:

> On 07/20/2017 07:04 AM, Punit Agrawal wrote:
>> On systems that are not booted as a Xen domain, the xenfs driver prints
>> the following message during boot.
>>
>> [    3.460595] xenfs: not registering filesystem on non-xen platform
>>
>> As the user chose not to boot a Xen domain, this message does not
>> provide useful information. Drop this message.
>>
>> Signed-off-by: Punit Agrawal <punit.agrawal@arm.com>
>> Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
>> Cc: Juergen Gross <jgross@suse.com>
>
> Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>

Thanks Boris.

Juergen, I assume you'll pick this up for the next round of xen
updates.

>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> https://lists.xen.org/xen-devel
Jürgen Groß July 21, 2017, 10:19 a.m. UTC | #3
On 21/07/17 12:12, Punit Agrawal wrote:
> Boris Ostrovsky <boris.ostrovsky@oracle.com> writes:
> 
>> On 07/20/2017 07:04 AM, Punit Agrawal wrote:
>>> On systems that are not booted as a Xen domain, the xenfs driver prints
>>> the following message during boot.
>>>
>>> [    3.460595] xenfs: not registering filesystem on non-xen platform
>>>
>>> As the user chose not to boot a Xen domain, this message does not
>>> provide useful information. Drop this message.
>>>
>>> Signed-off-by: Punit Agrawal <punit.agrawal@arm.com>
>>> Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
>>> Cc: Juergen Gross <jgross@suse.com>
>>
>> Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> 
> Thanks Boris.
> 
> Juergen, I assume you'll pick this up for the next round of xen
> updates.

Yes, probably for rc3 or rc4. For rc2 it was a liitle bit late and
there is no need to hurry.


Juergen
Punit Agrawal July 21, 2017, 10:21 a.m. UTC | #4
Juergen Gross <jgross@suse.com> writes:

> On 21/07/17 12:12, Punit Agrawal wrote:
>> Boris Ostrovsky <boris.ostrovsky@oracle.com> writes:
>> 
>>> On 07/20/2017 07:04 AM, Punit Agrawal wrote:
>>>> On systems that are not booted as a Xen domain, the xenfs driver prints
>>>> the following message during boot.
>>>>
>>>> [    3.460595] xenfs: not registering filesystem on non-xen platform
>>>>
>>>> As the user chose not to boot a Xen domain, this message does not
>>>> provide useful information. Drop this message.
>>>>
>>>> Signed-off-by: Punit Agrawal <punit.agrawal@arm.com>
>>>> Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
>>>> Cc: Juergen Gross <jgross@suse.com>
>>>
>>> Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
>> 
>> Thanks Boris.
>> 
>> Juergen, I assume you'll pick this up for the next round of xen
>> updates.
>
> Yes, probably for rc3 or rc4. For rc2 it was a liitle bit late and
> there is no need to hurry.

Agreed! Thanks for confirming.
Jürgen Groß July 28, 2017, 9:34 a.m. UTC | #5
On 20/07/17 13:04, Punit Agrawal wrote:
> On systems that are not booted as a Xen domain, the xenfs driver prints
> the following message during boot.
> 
> [    3.460595] xenfs: not registering filesystem on non-xen platform
> 
> As the user chose not to boot a Xen domain, this message does not
> provide useful information. Drop this message.
> 
> Signed-off-by: Punit Agrawal <punit.agrawal@arm.com>
> Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> Cc: Juergen Gross <jgross@suse.com>

Committed to: xen/tip.git for-linus-4.13b


Thanks,

Juergen
diff mbox

Patch

diff --git a/drivers/xen/xenfs/super.c b/drivers/xen/xenfs/super.c
index 967f069385d0..71ddfb4cf61c 100644
--- a/drivers/xen/xenfs/super.c
+++ b/drivers/xen/xenfs/super.c
@@ -87,7 +87,6 @@  static int __init xenfs_init(void)
 	if (xen_domain())
 		return register_filesystem(&xenfs_type);
 
-	pr_info("not registering filesystem on non-xen platform\n");
 	return 0;
 }