* [Qemu-devel] [PATCH] spapr: Print out extra hints when CAS negotiation of interrupt mode fails
@ 2019-05-16 7:36 Greg Kurz
2019-05-16 7:45 ` Satheesh Rajendran
2019-05-20 6:21 ` David Gibson
0 siblings, 2 replies; 6+ messages in thread
From: Greg Kurz @ 2019-05-16 7:36 UTC (permalink / raw)
To: David Gibson, Cédric Le Goater, Greg Kurz
Cc: Satheesh Rajendran, qemu-ppc, qemu-devel
Let's suggest to the user how the machine should be configured to allow
the guest to boot successfully.
Suggested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
Signed-off-by: Greg Kurz <groug@kaod.org>
---
hw/ppc/spapr_hcall.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c
index 63a55614b83d..aae9fd2b3e6d 100644
--- a/hw/ppc/spapr_hcall.c
+++ b/hw/ppc/spapr_hcall.c
@@ -1646,12 +1646,12 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu,
*/
if (guest_xive) {
if (spapr->irq->ov5 == SPAPR_OV5_XIVE_LEGACY) {
- error_report("Guest requested unavailable interrupt mode (XIVE)");
+ error_report("Guest requested unavailable interrupt mode (XIVE), try the ic-mode=xive or ic-mode=dual machine property");
exit(EXIT_FAILURE);
}
} else {
if (spapr->irq->ov5 == SPAPR_OV5_XIVE_EXPLOIT) {
- error_report("Guest requested unavailable interrupt mode (XICS)");
+ error_report("Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual");
exit(EXIT_FAILURE);
}
}
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] spapr: Print out extra hints when CAS negotiation of interrupt mode fails
2019-05-16 7:36 [Qemu-devel] [PATCH] spapr: Print out extra hints when CAS negotiation of interrupt mode fails Greg Kurz
@ 2019-05-16 7:45 ` Satheesh Rajendran
2019-05-16 8:22 ` Cédric Le Goater
2019-05-20 6:21 ` David Gibson
1 sibling, 1 reply; 6+ messages in thread
From: Satheesh Rajendran @ 2019-05-16 7:45 UTC (permalink / raw)
To: Greg Kurz
Cc: qemu-devel, Satheesh Rajendran, qemu-ppc, Cédric Le Goater,
David Gibson
On Thu, May 16, 2019 at 09:36:57AM +0200, Greg Kurz wrote:
> Let's suggest to the user how the machine should be configured to allow
> the guest to boot successfully.
>
> Suggested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
> Signed-off-by: Greg Kurz <groug@kaod.org>
> ---
> hw/ppc/spapr_hcall.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-and-Tested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
2019-05-16T07:42:43.018771Z qemu-system-ppc64: Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual
2019-05-16 07:42:43.394+0000: shutting down, reason=crashed
>
> diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c
> index 63a55614b83d..aae9fd2b3e6d 100644
> --- a/hw/ppc/spapr_hcall.c
> +++ b/hw/ppc/spapr_hcall.c
> @@ -1646,12 +1646,12 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu,
> */
> if (guest_xive) {
> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_LEGACY) {
> - error_report("Guest requested unavailable interrupt mode (XIVE)");
> + error_report("Guest requested unavailable interrupt mode (XIVE), try the ic-mode=xive or ic-mode=dual machine property");
> exit(EXIT_FAILURE);
> }
> } else {
> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_EXPLOIT) {
> - error_report("Guest requested unavailable interrupt mode (XICS)");
> + error_report("Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual");
> exit(EXIT_FAILURE);
> }
> }
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] spapr: Print out extra hints when CAS negotiation of interrupt mode fails
2019-05-16 7:45 ` Satheesh Rajendran
@ 2019-05-16 8:22 ` Cédric Le Goater
2019-05-16 8:32 ` Greg Kurz
0 siblings, 1 reply; 6+ messages in thread
From: Cédric Le Goater @ 2019-05-16 8:22 UTC (permalink / raw)
To: Satheesh Rajendran, Greg Kurz; +Cc: qemu-ppc, qemu-devel, David Gibson
On 5/16/19 9:45 AM, Satheesh Rajendran wrote:
> On Thu, May 16, 2019 at 09:36:57AM +0200, Greg Kurz wrote:
>> Let's suggest to the user how the machine should be configured to allow
>> the guest to boot successfully.
>>
>> Suggested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
>> Signed-off-by: Greg Kurz <groug@kaod.org>
>> ---
>> hw/ppc/spapr_hcall.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Reviewed-and-Tested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
you should use two different tags. I don't think patchwork understand
compounds.
C.
> 2019-05-16T07:42:43.018771Z qemu-system-ppc64: Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual
> 2019-05-16 07:42:43.394+0000: shutting down, reason=crashed
>
>>
>> diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c
>> index 63a55614b83d..aae9fd2b3e6d 100644
>> --- a/hw/ppc/spapr_hcall.c
>> +++ b/hw/ppc/spapr_hcall.c
>> @@ -1646,12 +1646,12 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu,
>> */
>> if (guest_xive) {
>> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_LEGACY) {
>> - error_report("Guest requested unavailable interrupt mode (XIVE)");
>> + error_report("Guest requested unavailable interrupt mode (XIVE), try the ic-mode=xive or ic-mode=dual machine property");
>> exit(EXIT_FAILURE);
>> }
>> } else {
>> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_EXPLOIT) {
>> - error_report("Guest requested unavailable interrupt mode (XICS)");
>> + error_report("Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual");
>> exit(EXIT_FAILURE);
>> }
>> }
>>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] spapr: Print out extra hints when CAS negotiation of interrupt mode fails
2019-05-16 8:22 ` Cédric Le Goater
@ 2019-05-16 8:32 ` Greg Kurz
2019-05-16 8:44 ` Satheesh Rajendran
0 siblings, 1 reply; 6+ messages in thread
From: Greg Kurz @ 2019-05-16 8:32 UTC (permalink / raw)
To: Cédric Le Goater
Cc: qemu-ppc, Satheesh Rajendran, qemu-devel, David Gibson
On Thu, 16 May 2019 10:22:26 +0200
Cédric Le Goater <clg@kaod.org> wrote:
> On 5/16/19 9:45 AM, Satheesh Rajendran wrote:
> > On Thu, May 16, 2019 at 09:36:57AM +0200, Greg Kurz wrote:
> >> Let's suggest to the user how the machine should be configured to allow
> >> the guest to boot successfully.
> >>
> >> Suggested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
> >> Signed-off-by: Greg Kurz <groug@kaod.org>
> >> ---
> >> hw/ppc/spapr_hcall.c | 4 ++--
> >> 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > Reviewed-and-Tested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
>
> you should use two different tags. I don't think patchwork understand
> compounds.
>
> C.
>
It doesn't indeed.
$ pwclient view 1100396 | grep Reviewed
$
> > 2019-05-16T07:42:43.018771Z qemu-system-ppc64: Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual
> > 2019-05-16 07:42:43.394+0000: shutting down, reason=crashed
> >
> >>
> >> diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c
> >> index 63a55614b83d..aae9fd2b3e6d 100644
> >> --- a/hw/ppc/spapr_hcall.c
> >> +++ b/hw/ppc/spapr_hcall.c
> >> @@ -1646,12 +1646,12 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu,
> >> */
> >> if (guest_xive) {
> >> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_LEGACY) {
> >> - error_report("Guest requested unavailable interrupt mode (XIVE)");
> >> + error_report("Guest requested unavailable interrupt mode (XIVE), try the ic-mode=xive or ic-mode=dual machine property");
> >> exit(EXIT_FAILURE);
> >> }
> >> } else {
> >> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_EXPLOIT) {
> >> - error_report("Guest requested unavailable interrupt mode (XICS)");
> >> + error_report("Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual");
> >> exit(EXIT_FAILURE);
> >> }
> >> }
> >>
> >
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] spapr: Print out extra hints when CAS negotiation of interrupt mode fails
2019-05-16 8:32 ` Greg Kurz
@ 2019-05-16 8:44 ` Satheesh Rajendran
0 siblings, 0 replies; 6+ messages in thread
From: Satheesh Rajendran @ 2019-05-16 8:44 UTC (permalink / raw)
To: Greg Kurz
Cc: qemu-devel, qemu-ppc, Satheesh Rajendran, Cédric Le Goater,
David Gibson
On Thu, May 16, 2019 at 10:32:15AM +0200, Greg Kurz wrote:
> On Thu, 16 May 2019 10:22:26 +0200
> Cédric Le Goater <clg@kaod.org> wrote:
>
> > On 5/16/19 9:45 AM, Satheesh Rajendran wrote:
> > > On Thu, May 16, 2019 at 09:36:57AM +0200, Greg Kurz wrote:
> > >> Let's suggest to the user how the machine should be configured to allow
> > >> the guest to boot successfully.
> > >>
> > >> Suggested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
> > >> Signed-off-by: Greg Kurz <groug@kaod.org>
> > >> ---
> > >> hw/ppc/spapr_hcall.c | 4 ++--
> > >> 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > Reviewed-and-Tested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
Reviewed-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
Tested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
Done!
> >
> > you should use two different tags. I don't think patchwork understand
> > compounds.
> >
> > C.
> >
>
Thanks !!
> It doesn't indeed.
>
> $ pwclient view 1100396 | grep Reviewed
> $
>
Regards,
-Satheesh.
> > > 2019-05-16T07:42:43.018771Z qemu-system-ppc64: Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual
> > > 2019-05-16 07:42:43.394+0000: shutting down, reason=crashed
> > >
> > >>
> > >> diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c
> > >> index 63a55614b83d..aae9fd2b3e6d 100644
> > >> --- a/hw/ppc/spapr_hcall.c
> > >> +++ b/hw/ppc/spapr_hcall.c
> > >> @@ -1646,12 +1646,12 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu,
> > >> */
> > >> if (guest_xive) {
> > >> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_LEGACY) {
> > >> - error_report("Guest requested unavailable interrupt mode (XIVE)");
> > >> + error_report("Guest requested unavailable interrupt mode (XIVE), try the ic-mode=xive or ic-mode=dual machine property");
> > >> exit(EXIT_FAILURE);
> > >> }
> > >> } else {
> > >> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_EXPLOIT) {
> > >> - error_report("Guest requested unavailable interrupt mode (XICS)");
> > >> + error_report("Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual");
> > >> exit(EXIT_FAILURE);
> > >> }
> > >> }
> > >>
> > >
> >
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] spapr: Print out extra hints when CAS negotiation of interrupt mode fails
2019-05-16 7:36 [Qemu-devel] [PATCH] spapr: Print out extra hints when CAS negotiation of interrupt mode fails Greg Kurz
2019-05-16 7:45 ` Satheesh Rajendran
@ 2019-05-20 6:21 ` David Gibson
1 sibling, 0 replies; 6+ messages in thread
From: David Gibson @ 2019-05-20 6:21 UTC (permalink / raw)
To: Greg Kurz; +Cc: Satheesh Rajendran, qemu-ppc, Cédric Le Goater, qemu-devel
[-- Attachment #1: Type: text/plain, Size: 1620 bytes --]
On Thu, May 16, 2019 at 09:36:57AM +0200, Greg Kurz wrote:
> Let's suggest to the user how the machine should be configured to allow
> the guest to boot successfully.
>
> Suggested-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com>
> Signed-off-by: Greg Kurz <groug@kaod.org>
Applied, thanks.
> ---
> hw/ppc/spapr_hcall.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c
> index 63a55614b83d..aae9fd2b3e6d 100644
> --- a/hw/ppc/spapr_hcall.c
> +++ b/hw/ppc/spapr_hcall.c
> @@ -1646,12 +1646,12 @@ static target_ulong h_client_architecture_support(PowerPCCPU *cpu,
> */
> if (guest_xive) {
> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_LEGACY) {
> - error_report("Guest requested unavailable interrupt mode (XIVE)");
> + error_report("Guest requested unavailable interrupt mode (XIVE), try the ic-mode=xive or ic-mode=dual machine property");
> exit(EXIT_FAILURE);
> }
> } else {
> if (spapr->irq->ov5 == SPAPR_OV5_XIVE_EXPLOIT) {
> - error_report("Guest requested unavailable interrupt mode (XICS)");
> + error_report("Guest requested unavailable interrupt mode (XICS), either don't set the ic-mode machine property or try ic-mode=xics or ic-mode=dual");
> exit(EXIT_FAILURE);
> }
> }
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-05-20 6:34 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-05-16 7:36 [Qemu-devel] [PATCH] spapr: Print out extra hints when CAS negotiation of interrupt mode fails Greg Kurz
2019-05-16 7:45 ` Satheesh Rajendran
2019-05-16 8:22 ` Cédric Le Goater
2019-05-16 8:32 ` Greg Kurz
2019-05-16 8:44 ` Satheesh Rajendran
2019-05-20 6:21 ` David Gibson
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).