2022-06-20 10:04:04

by Jürgen Groß

[permalink] [raw]
Subject: [PATCH v2] s390/kvm: avoid hypfs error message

When booting under KVM the following error messages are issued:

hypfs.7f5705: The hardware system does not support hypfs
hypfs.7a79f0: Initialization of hypfs failed with rc=-61

Demote the severity of first message from "error" to "info" and issue
the second message only in other error cases.

Signed-off-by: Juergen Gross <[email protected]>
---
V2:
- don't bail out if KVM, but avoid error messages instead (Christian
Borntraeger)
---
arch/s390/hypfs/hypfs_diag.c | 2 +-
arch/s390/hypfs/inode.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/s390/hypfs/hypfs_diag.c b/arch/s390/hypfs/hypfs_diag.c
index f0bc4dc3e9bf..6511d15ace45 100644
--- a/arch/s390/hypfs/hypfs_diag.c
+++ b/arch/s390/hypfs/hypfs_diag.c
@@ -437,7 +437,7 @@ __init int hypfs_diag_init(void)
int rc;

if (diag204_probe()) {
- pr_err("The hardware system does not support hypfs\n");
+ pr_info("The hardware system does not support hypfs\n");
return -ENODATA;
}

diff --git a/arch/s390/hypfs/inode.c b/arch/s390/hypfs/inode.c
index 5c97f48cea91..ee919bfc8186 100644
--- a/arch/s390/hypfs/inode.c
+++ b/arch/s390/hypfs/inode.c
@@ -496,9 +496,9 @@ static int __init hypfs_init(void)
hypfs_vm_exit();
fail_hypfs_diag_exit:
hypfs_diag_exit();
+ pr_err("Initialization of hypfs failed with rc=%i\n", rc);
fail_dbfs_exit:
hypfs_dbfs_exit();
- pr_err("Initialization of hypfs failed with rc=%i\n", rc);
return rc;
}
device_initcall(hypfs_init)
--
2.35.3


2022-06-20 10:05:11

by Heiko Carstens

[permalink] [raw]
Subject: Re: [PATCH v2] s390/kvm: avoid hypfs error message

On Mon, Jun 20, 2022 at 11:45:34AM +0200, Juergen Gross wrote:
> When booting under KVM the following error messages are issued:
>
> hypfs.7f5705: The hardware system does not support hypfs
> hypfs.7a79f0: Initialization of hypfs failed with rc=-61
>
> Demote the severity of first message from "error" to "info" and issue
> the second message only in other error cases.
>
> Signed-off-by: Juergen Gross <[email protected]>
> ---
> V2:
> - don't bail out if KVM, but avoid error messages instead (Christian
> Borntraeger)
> ---
> arch/s390/hypfs/hypfs_diag.c | 2 +-
> arch/s390/hypfs/inode.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)

Acked-by: Heiko Carstens <[email protected]>

2022-06-20 11:47:44

by Alexander Gordeev

[permalink] [raw]
Subject: Re: [PATCH v2] s390/kvm: avoid hypfs error message

On Mon, Jun 20, 2022 at 11:45:34AM +0200, Juergen Gross wrote:
> When booting under KVM the following error messages are issued:
>
> hypfs.7f5705: The hardware system does not support hypfs
> hypfs.7a79f0: Initialization of hypfs failed with rc=-61
>
> Demote the severity of first message from "error" to "info" and issue
> the second message only in other error cases.
>
> Signed-off-by: Juergen Gross <[email protected]>

Applied, thanks!

> ---
> V2:
> - don't bail out if KVM, but avoid error messages instead (Christian
> Borntraeger)
> ---
> arch/s390/hypfs/hypfs_diag.c | 2 +-
> arch/s390/hypfs/inode.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/s390/hypfs/hypfs_diag.c b/arch/s390/hypfs/hypfs_diag.c
> index f0bc4dc3e9bf..6511d15ace45 100644
> --- a/arch/s390/hypfs/hypfs_diag.c
> +++ b/arch/s390/hypfs/hypfs_diag.c
> @@ -437,7 +437,7 @@ __init int hypfs_diag_init(void)
> int rc;
>
> if (diag204_probe()) {
> - pr_err("The hardware system does not support hypfs\n");
> + pr_info("The hardware system does not support hypfs\n");
> return -ENODATA;
> }
>
> diff --git a/arch/s390/hypfs/inode.c b/arch/s390/hypfs/inode.c
> index 5c97f48cea91..ee919bfc8186 100644
> --- a/arch/s390/hypfs/inode.c
> +++ b/arch/s390/hypfs/inode.c
> @@ -496,9 +496,9 @@ static int __init hypfs_init(void)
> hypfs_vm_exit();
> fail_hypfs_diag_exit:
> hypfs_diag_exit();
> + pr_err("Initialization of hypfs failed with rc=%i\n", rc);
> fail_dbfs_exit:
> hypfs_dbfs_exit();
> - pr_err("Initialization of hypfs failed with rc=%i\n", rc);
> return rc;
> }
> device_initcall(hypfs_init)
> --
> 2.35.3
>

2022-08-11 09:09:36

by Jürgen Groß

[permalink] [raw]
Subject: Re: [PATCH v2] s390/kvm: avoid hypfs error message

On 20.06.22 13:31, Alexander Gordeev wrote:
> On Mon, Jun 20, 2022 at 11:45:34AM +0200, Juergen Gross wrote:
>> When booting under KVM the following error messages are issued:
>>
>> hypfs.7f5705: The hardware system does not support hypfs
>> hypfs.7a79f0: Initialization of hypfs failed with rc=-61
>>
>> Demote the severity of first message from "error" to "info" and issue
>> the second message only in other error cases.
>>
>> Signed-off-by: Juergen Gross <[email protected]>
>
> Applied, thanks!

Hmm, applied to which tree and branch?

I would have thought this patch should have reached upstream by now...


Juergen


Attachments:
OpenPGP_0xB0DE9DD628BF132F.asc (3.08 kB)
OpenPGP public key
OpenPGP_signature (505.00 B)
OpenPGP digital signature
Download all attachments

2022-08-11 15:25:28

by Alexander Gordeev

[permalink] [raw]
Subject: Re: [PATCH v2] s390/kvm: avoid hypfs error message

On Thu, Aug 11, 2022 at 10:14:55AM +0200, Juergen Gross wrote:
> On 20.06.22 13:31, Alexander Gordeev wrote:
> > Applied, thanks!
>
> Hmm, applied to which tree and branch?
>
> I would have thought this patch should have reached upstream by now...

Hi Juergen,

This one is going to land in -rc2 or -rc3 most likely.

Thanks!

> Juergen