Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-09 Thread John Snow
On Fri, Aug 9, 2024, 4:24 AM Mauro Carvalho Chehab < mchehab+hua...@kernel.org> wrote: > Em Thu, 8 Aug 2024 19:33:32 -0400 > John Snow escreveu: > > > > > Then here you'd use qmp.cmd (raises exception on QMPError) or > qmp.cmd_raw > > > > or qmp.cmd_obj (returns the QMP response as the return val

Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-09 Thread Mauro Carvalho Chehab
Em Thu, 8 Aug 2024 19:33:32 -0400 John Snow escreveu: > > > Then here you'd use qmp.cmd (raises exception on QMPError) or qmp.cmd_raw > > > or qmp.cmd_obj (returns the QMP response as the return value even if it > > was > > > an error.) > > > > Good to know, I'll try and see what fits best.

Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-09 Thread Mauro Carvalho Chehab
Em Fri, 9 Aug 2024 08:26:09 +0200 Mauro Carvalho Chehab escreveu: > Em Fri, 9 Aug 2024 00:41:37 +0200 > Mauro Carvalho Chehab escreveu: > > > > You should be able to use e.g. > > > > > > legacy.py's QEMUMonitorProtocol class for synchronous connections, e.g. > > > > > > from qemu.qmp.legacy i

Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-08 Thread Mauro Carvalho Chehab
Em Fri, 9 Aug 2024 00:41:37 +0200 Mauro Carvalho Chehab escreveu: > > You should be able to use e.g. > > > > legacy.py's QEMUMonitorProtocol class for synchronous connections, e.g. > > > > from qemu.qmp.legacy import QEMUMonitorProtocol > > > > qmp = QEMUMonitorProtocol((host, port)) > > qmp.c

Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-08 Thread John Snow
On Thu, Aug 8, 2024 at 6:41 PM Mauro Carvalho Chehab < mchehab+hua...@kernel.org> wrote: > Em Thu, 8 Aug 2024 17:21:33 -0400 > John Snow escreveu: > > > On Fri, Aug 2, 2024 at 5:44 PM Mauro Carvalho Chehab < > > mchehab+hua...@kernel.org> wrote: > > > > > > diff --git a/scripts/qmp_helper.py b/sc

Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-08 Thread Mauro Carvalho Chehab
Em Thu, 8 Aug 2024 17:21:33 -0400 John Snow escreveu: > On Fri, Aug 2, 2024 at 5:44 PM Mauro Carvalho Chehab < > mchehab+hua...@kernel.org> wrote: > > > diff --git a/scripts/qmp_helper.py b/scripts/qmp_helper.py > > new file mode 100644 > > index ..13fae7a7af0e > > --- /dev/null >

Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-08 Thread Mauro Carvalho Chehab
Em Thu, 8 Aug 2024 16:58:38 -0400 John Snow escreveu: > On Fri, Aug 2, 2024 at 5:44 PM Mauro Carvalho Chehab < > mchehab+hua...@kernel.org> wrote: > > > +#!/usr/bin/env python3 > > +# > > +# pylint: disable=C0301, C0114, R0912, R0913, R0914, R0915, W0511 > > > > Out of curiosity, what tool

Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-08 Thread John Snow
On Fri, Aug 2, 2024 at 5:44 PM Mauro Carvalho Chehab < mchehab+hua...@kernel.org> wrote: > Using the QMP GHESv2 API requires preparing a raw data array > containing a CPER record. > > Add a helper script with subcommands to prepare such data. > > Currently, only ARM Processor error CPER record is

Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-08 Thread John Snow
On Fri, Aug 2, 2024 at 5:44 PM Mauro Carvalho Chehab < mchehab+hua...@kernel.org> wrote: > Using the QMP GHESv2 API requires preparing a raw data array > containing a CPER record. > > Add a helper script with subcommands to prepare such data. > > Currently, only ARM Processor error CPER record is

Re: [PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-06 Thread Igor Mammedov
On Fri, 2 Aug 2024 23:44:02 +0200 Mauro Carvalho Chehab wrote: > Using the QMP GHESv2 API requires preparing a raw data array > containing a CPER record. > > Add a helper script with subcommands to prepare such data. > > Currently, only ARM Processor error CPER record is supported. > > Signed

[PATCH v5 7/7] scripts/ghes_inject: add a script to generate GHES error inject

2024-08-02 Thread Mauro Carvalho Chehab
Using the QMP GHESv2 API requires preparing a raw data array containing a CPER record. Add a helper script with subcommands to prepare such data. Currently, only ARM Processor error CPER record is supported. Signed-off-by: Mauro Carvalho Chehab --- MAINTAINERS| 3 + scrip