Re: [PATCH 00/10] target/i386/tcg: fixes for seg_helper.c

2024-07-10 Thread Robert Henry
I have only skimmed the diffs. Your knowledge of the deep semantics, gained by close differential reading of intel and amd docs, is truly amazing. Many thanks for pushing this through! I have 2 nits, perhaps stylistic only. For code like "sp -= 2" or "sp += 2" followed or preceded by a write to

Re: [PATCH 1/1] i386/tcg: Allow IRET from user mode to user mode for dotnet runtime

2024-06-16 Thread Robert Henry
e any of this chatter in this email thread on the bug report https://gitlab.com/qemu-project/qemu/-/issues/249 Robert Henry On Sat, Jun 15, 2024 at 4:25 PM Richard Henderson < richard.hender...@linaro.org> wrote: > On 6/11/24 09:20, Robert R. Henry wrote: > > This fixes a bug wher

[PATCH 0/1] i386/tcg fix for IRET as used in dotnet runtime

2024-06-11 Thread Robert R. Henry
/arch/amd64/context2.S#L241 shows how the dotnet runtime uses iret. I have booted BSD, solaris and macosX with this change, and await results for booting Windows from the Windows kernel team. I have not tested this with other modern JITers, such as Java, v8, or HHVM. Robert R. Henry (1): i386

[PATCH 1/1] i386/tcg: Allow IRET from user mode to user mode for dotnet runtime

2024-06-11 Thread Robert R. Henry
, IRET returns from user mode to user mode. This bug manifested itself as a page fault in the guest Linux kernel. This bug appears to have been in QEMU since the beginning. Resolves: https://gitlab.com/qemu-project/qemu/-/issues/249 Signed-off-by: Robert R. Henry --- target/i386/tcg/seg_helper.c

Re: [PATCH] msix: unset PCIDevice::msix_vector_poll_notifier in rollback

2023-11-13 Thread Robert Hoo
On 11/13/2023 6:05 PM, Philippe Mathieu-Daudé wrote: Hi Robert, On 13/11/23 09:13, Robert Hoo wrote: In the rollback in msix_set_vector_notifiers(), original patch forgot to undo msix_vector_poll_notifier pointer. Out of curiosity, nobody complained during 11 years, so in which use case did

[PATCH] msix: unset PCIDevice::msix_vector_poll_notifier in rollback

2023-11-13 Thread Robert Hoo
In the rollback in msix_set_vector_notifiers(), original patch forgot to undo msix_vector_poll_notifier pointer. Fixes: bbef882cc193 ("msi: add API to get notified about pending bit poll") Signed-off-by: Robert Hoo --- hw/pci/msix.c | 1 + 1 file changed, 1 insertion(+) diff --gi

Re: [QEMU PATCH 1/1] virtgpu: do not destroy resources when guest suspend

2023-06-21 Thread Robert Beckett
On 21/06/2023 09:39, Gerd Hoffmann wrote: On Tue, Jun 20, 2023 at 01:26:15PM +0100, Robert Beckett wrote: On 20/06/2023 10:41, Gerd Hoffmann wrote: Hi, The guest driver should be able to restore resources after resume. Thank you for your suggestion! As far as I know, resources are

Re: [QEMU PATCH 1/1] virtgpu: do not destroy resources when guest suspend

2023-06-20 Thread Robert Beckett
On 20/06/2023 10:41, Gerd Hoffmann wrote: Hi, The guest driver should be able to restore resources after resume. Thank you for your suggestion! As far as I know, resources are created on host side and guest has no backup, if resources are destroyed, guest can't restore them. Or do you me

Re: [QEMU PATCH 1/1] virtgpu: do not destroy resources when guest suspend

2023-06-08 Thread Robert Beckett
On 08/06/2023 03:56, Jiqian Chen wrote: After suspending and resuming guest VM, you will get a black screen, and the display can't come back. This is because when guest did suspending, it called into qemu to call virtio_gpu_gl_reset. In function virtio_gpu_gl_reset, it destroyed resources and

Re: [PATCH v3 1/7] target/i386: allow versioned CPUs to specify new cache_info

2023-04-24 Thread Robert Hoo
Babu Moger 于2023年4月25日周二 00:42写道: > > From: Michael Roth > > New EPYC CPUs versions require small changes to their cache_info's. Do you mean, for the real HW of EPYC CPU, each given model, e.g. Rome, has HW version updates periodically? > Because current QEMU x86 CPU definition does not support

[RESEND][QEMU PATCH] accel/kvm: Don't use KVM maximum support number to alloc user memslots

2023-04-20 Thread Robert Hoo
784949 30263 Signed-off-by: Robert Hoo --- Resend: Add stats about kvm_lookup_matching_slot() for example. CC kvm mail list per get_maintainer.pl suggests. I believe this benefits Live Migration, but not devices at hand to do the system level test. accel/

[PATCH] accel/kvm: Don't use KVM maximum support number to alloc user memslots

2023-04-09 Thread Robert Hoo
vm_lookup_matching_slot(), kvm_physical_memory_addr_from_host(), kvm_physical_log_clear(), kvm_log_sync_global(). Test: Temporarily set KVM_DEF_NR_SLOTS = 8, let it go through slot[] dynamic increase, VM launched and works well. Signed-off-by: Robert Hoo --- accel/kvm/kvm-all.c | 57 ++

Re: [PATCH v3 0/6] Support for new CPU model SapphireRapids

2023-03-02 Thread Robert Hoo
On Thu, 2023-02-02 at 12:05 +0100, Igor Mammedov wrote: > MultiBitFeatureInfo looks like an interesting > idea Yeah, we can feel how much effort Lei spent on this. > but among fixing whatever issues this has atm, > you'd probably need to introduce a new qdev_bitfield property > infrastructure s

[PATCH] i386: QEMU support LAM (Linear Address Masking)

2023-02-27 Thread Robert Hoo
getContent/671368 Signed-off-by: Robert Hoo --- target/i386/cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/i386/cpu.c b/target/i386/cpu.c index 4d2b8d0444..d6d573ca38 100644 --- a/target/i386/cpu.c +++ b/target/i386/cpu.c @@ -876,7 +876,7 @@ Featu

Re: [PATCH v3 2/3] KVM: keep track of running ioctls

2022-12-02 Thread Robert Hoo
On Fri, 2022-12-02 at 13:03 +0100, Emanuele Giuseppe Esposito wrote: ... > > > @@ -3032,7 +3035,9 @@ int kvm_vcpu_ioctl(CPUState *cpu, int type, > > > ...) > > > va_end(ap); > > > > > > trace_kvm_vcpu_ioctl(cpu->cpu_index, type, arg); > > > +accel_cpu_ioctl_begin(cpu); > > > > Does

Re: [PATCH v3 2/3] KVM: keep track of running ioctls

2022-12-01 Thread Robert Hoo
On Fri, 2022-11-11 at 10:47 -0500, Emanuele Giuseppe Esposito wrote: > Using the new accel-blocker API, mark where ioctls are being called > in KVM. Next, we will implement the critical section that will take > care of performing memslots modifications atomically, therefore > preventing any new ioc

Re: [PATCH v3 1/3] accel: introduce accelerator blocker API

2022-12-01 Thread Robert Hoo
On Fri, 2022-11-11 at 10:47 -0500, Emanuele Giuseppe Esposito wrote: > This API allows the accelerators to prevent vcpus from issuing > new ioctls while execting a critical section marked with the > accel_ioctl_inhibit_begin/end functions. > > Note that all functions submitting ioctls must mark wh

Re: [PATCH v4 5/5] test/acpi/bios-tables-test: SSDT: update golden master binaries

2022-10-26 Thread Robert Hoo
On Wed, 2022-10-26 at 10:45 -0400, Michael S. Tsirkin wrote: > On Thu, Sep 22, 2022 at 08:21:55PM +0800, Robert Hoo wrote: > > And empty bios-tables-test-allowed-diff.h. > > > > Diff of ASL form, from qtest testlog.txt: > > > > --- /tmp/asl-RFWZS1.dsl 20

Re: [PATCH v4 5/5] test/acpi/bios-tables-test: SSDT: update golden master binaries

2022-10-19 Thread Robert Hoo
Ping... On Fri, 2022-10-07 at 21:27 +0800, Robert Hoo wrote: > Ping... > On Tue, 2022-09-27 at 08:30 +0800, Robert Hoo wrote: > > On Mon, 2022-09-26 at 15:22 +0200, Igor Mammedov wrote: > > > > > 0800200c9a66"), One, 0x05, Local0, One) > > > > > +

Re: [PATCH v4 5/5] test/acpi/bios-tables-test: SSDT: update golden master binaries

2022-10-07 Thread Robert Hoo
Ping... On Tue, 2022-09-27 at 08:30 +0800, Robert Hoo wrote: > On Mon, 2022-09-26 at 15:22 +0200, Igor Mammedov wrote: > > > > 0800200c9a66"), One, 0x05, Local0, One) > > > > +CreateDWordField (Local3, Zero, STTS) > > > > +

Re: [PATCH v4 5/5] test/acpi/bios-tables-test: SSDT: update golden master binaries

2022-09-26 Thread Robert Hoo
On Mon, 2022-09-26 at 15:22 +0200, Igor Mammedov wrote: > > > 0800200c9a66"), One, 0x05, Local0, One) > > > +CreateDWordField (Local3, Zero, STTS) > > > +CreateField (Local3, 0x20, (LEN << 0x03), > > > LDAT) > > > +Name (LSA, Buffer (Zero)

[PATCH v4 3/5] acpi/nvdimm: define macro for NVDIMM Device _DSM

2022-09-22 Thread Robert Hoo
/documents/IntelOptanePMem_DSM_Interface-V2.0.pdf Signed-off-by: Robert Hoo Reviewed-by: Igor Mammedov --- hw/acpi/nvdimm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/acpi/nvdimm.c b/hw/acpi/nvdimm.c index 201317c611..afff911c1e 100644 --- a/hw/acpi/nvdimm.c +++ b/hw/acp

[PATCH v4 1/5] tests/acpi: allow SSDT changes

2022-09-22 Thread Robert Hoo
Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- tests/qtest/bios-tables-test-allowed-diff.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h index dfb8523c8b..eb8bae1407 100644 --- a/tests

[PATCH v4 5/5] test/acpi/bios-tables-test: SSDT: update golden master binaries

2022-09-22 Thread Robert Hoo
01) +{ +INPT +} +Local3 = NCAL (ToUUID ("4309ac30-0d11-11e4-9191-0800200c9a66"), One, 0x06, Local0, One) + CreateDWordField (Local3, Zero, STTS) +Retu

[PATCH v4 4/5] acpi/nvdimm: Implement ACPI NVDIMM Label Methods

2022-09-22 Thread Robert Hoo
/IntelOptanePMem_DSM_Interface-V2.0.pdf Signed-off-by: Robert Hoo --- hw/acpi/nvdimm.c | 95 1 file changed, 95 insertions(+) diff --git a/hw/acpi/nvdimm.c b/hw/acpi/nvdimm.c index afff911c1e..a3b25a92f3 100644 --- a/hw/acpi/nvdimm.c +++ b/hw/acpi/nvdimm.c @@ -1243,6

Re: [PATCH v4 5/5] test/acpi/bios-tables-test: SSDT: update golden master binaries

2022-09-22 Thread Robert Hoo
On Thu, 2022-09-22 at 20:21 +0800, Robert Hoo wrote: > And empty bios-tables-test-allowed-diff.h. > > Diff of ASL form, from qtest testlog.txt: > > --- /tmp/asl-RFWZS1.dsl 2022-09-22 18:25:06.191519589 +0800 > +++ /tmp/asl-B1ZZS1.dsl 2022-09-22 18:25:06.18751918

[PATCH v4 2/5] acpi/ssdt: Fix aml_or() and aml_and() in if clause

2022-09-22 Thread Robert Hoo
3) == 0x04) && (SizeOf (Arg3) == One))) Fixes: 90623ebf603 ("nvdimm acpi: check UUID") Fixes: 4568c948066 ("nvdimm acpi: save arg3 of _DSM method") Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu Reviewed-by: Igor Mammedov --- hw/acpi/nvdimm.c | 7 +++ 1 file changed,

[PATCH v4 0/5] Support ACPI NVDIMM Label Methods

2022-09-22 Thread Robert Hoo
ntation to the idea of simply wrapper _DSM. v1 --> v2: Almost rewritten Separate Patch 2 Dance with tests/qtest/bios-table-tests Add trace event Robert Hoo (5): tests/acpi: allow SSDT changes acpi/ssdt: Fix aml_or() and aml_and() in if clause acpi/nvdimm: define macro for NVDIMM Device _

Re: [PATCH v3 4/5] acpi/nvdimm: Implement ACPI NVDIMM Label Methods

2022-09-21 Thread Robert Hoo
On Wed, 2022-09-21 at 15:29 +0200, Igor Mammedov wrote: > On Tue, 20 Sep 2022 20:28:31 +0800 > Robert Hoo wrote: > > > On Tue, 2022-09-20 at 11:13 +0200, Igor Mammedov wrote: > > > On Fri, 16 Sep 2022 21:15:35 +0800 > > > Robert Hoo wrote: > > >

Re: [PATCH v3 4/5] acpi/nvdimm: Implement ACPI NVDIMM Label Methods

2022-09-20 Thread Robert Hoo
On Tue, 2022-09-20 at 11:13 +0200, Igor Mammedov wrote: > On Fri, 16 Sep 2022 21:15:35 +0800 > Robert Hoo wrote: > > > On Fri, 2022-09-16 at 09:37 +0200, Igor Mammedov wrote: > > > > > > Fine, get your point now. > > > > In ASL it will look

Re: [PATCH v3 4/5] acpi/nvdimm: Implement ACPI NVDIMM Label Methods

2022-09-16 Thread Robert Hoo
On Fri, 2022-09-16 at 09:37 +0200, Igor Mammedov wrote: > > Fine, get your point now. > > In ASL it will look like this: > > Local1 = Package (0x3) {STTS, SLSA, MAXT} > > Return (Local1) > > > > > > But as for > > CreateDWordField (Lo

Re: [PATCH v3 4/5] acpi/nvdimm: Implement ACPI NVDIMM Label Methods

2022-09-15 Thread Robert Hoo
On Fri, 2022-09-09 at 15:39 +0200, Igor Mammedov wrote: ... > looks more or less fine except of excessive use of named variables > which creates global scope variables. > > I'd suggest to store temporary buffers/packages in LocalX variales, > you should be able to do that for everything modulo > a

Re: [PATCH v3 4/5] acpi/nvdimm: Implement ACPI NVDIMM Label Methods

2022-09-09 Thread Robert Hoo
On Fri, 2022-09-09 at 15:39 +0200, Igor Mammedov wrote: > On Thu, 1 Sep 2022 11:27:20 +0800 > Robert Hoo wrote: > > > Recent ACPI spec [1] has defined NVDIMM Label Methods _LS{I,R,W}, > > which > > deprecates corresponding _DSM Functions defined by PMEM _DS

[PATCH v3 5/5] test/acpi/bios-tables-test: SSDT: update golden master binaries

2022-08-31 Thread Robert Hoo
(Local3, Zero, STTS) +Return (ToInteger (STTS)) +} + Method (_DSM, 4, NotSerialized) // _DSM: Device-Specific Method { Return (NCAL (Arg0, Arg1, Arg2, Arg3, One)) } } ... // iterates in each

[PATCH v3 3/5] acpi/nvdimm: define macro for NVDIMM Device _DSM

2022-08-31 Thread Robert Hoo
/documents/IntelOptanePMem_DSM_Interface-V2.0.pdf Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- hw/acpi/nvdimm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/acpi/nvdimm.c b/hw/acpi/nvdimm.c index 201317c611..afff911c1e 100644 --- a/hw/acpi/nvdimm.c +++ b/hw/acp

[PATCH v3 4/5] acpi/nvdimm: Implement ACPI NVDIMM Label Methods

2022-08-31 Thread Robert Hoo
/IntelOptanePMem_DSM_Interface-V2.0.pdf Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- hw/acpi/nvdimm.c | 91 1 file changed, 91 insertions(+) diff --git a/hw/acpi/nvdimm.c b/hw/acpi/nvdimm.c index afff911c1e..516acfe53b 100644 --- a/hw/acpi/nvdimm.c +++ b/hw/acpi

[PATCH v3 2/5] acpi/ssdt: Fix aml_or() and aml_and() in if clause

2022-08-31 Thread Robert Hoo
3) == 0x04) && (SizeOf (Arg3) == One))) Fixes: 90623ebf603 ("nvdimm acpi: check UUID") Fixes: 4568c948066 ("nvdimm acpi: save arg3 of _DSM method") Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu Reviewed-by: Igor Mammedov --- hw/acpi/nvdimm.c | 7 +++ 1 file changed,

[PATCH v3 1/5] tests/acpi: allow SSDT changes

2022-08-31 Thread Robert Hoo
Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- tests/qtest/bios-tables-test-allowed-diff.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h index dfb8523c8b..eb8bae1407 100644 --- a/tests

[PATCH v3 0/5] Support ACPI NVDIMM Label Methods

2022-08-31 Thread Robert Hoo
Patch 2 Dance with tests/qtest/bios-table-tests Add trace event Robert Hoo (5): tests/acpi: allow SSDT changes acpi/ssdt: Fix aml_or() and aml_and() in if clause acpi/nvdimm: define macro for NVDIMM Device _DSM acpi/nvdimm: Implement ACPI NVDIMM Label Methods test/acpi/bios-tables-test

Re: [QEMU PATCH v2 4/6] nvdimm: Implement ACPI NVDIMM Label Methods

2022-07-26 Thread Robert Hoo
On Thu, 2022-07-21 at 10:58 +0200, Igor Mammedov wrote: [...] Thanks Igor for review. > > > The patch it is too intrusive and my hunch is that it breaks > > > ABI and needs a bunch of compat knobs to work properly and > > > that I'd like to avoid unless there is not other way around > > > the probl

Re: [QEMU PATCH v2 4/6] nvdimm: Implement ACPI NVDIMM Label Methods

2022-07-18 Thread Robert Hoo
Ping... On Fri, 2022-07-01 at 17:23 +0800, Robert Hoo wrote: > On Thu, 2022-06-16 at 14:32 +0200, Igor Mammedov wrote: > > On Mon, 30 May 2022 11:40:45 +0800 > > Robert Hoo wrote: > > > > > Recent ACPI spec [1] has defined NVDIMM Label Methods _LS{I,R,W}

Re: [PATCH] acpi/nvdimm: Define trace events for NVDIMM and substitute nvdimm_debug()

2022-07-18 Thread Robert Hoo
On Mon, 2022-07-18 at 15:41 +0200, Igor Mammedov wrote: > On Mon, 18 Jul 2022 15:12:03 +0800 > Robert Hoo wrote: > [...] > > BTW, during the unit test, I met some bios-table test error, > > https://gitlab.com/qemu-project/qemu/-/issues/1098, perhaps related > >

Re: [PATCH] acpi/nvdimm: Define trace events for NVDIMM and substitute nvdimm_debug()

2022-07-18 Thread Robert Hoo
On Thu, 2022-07-07 at 11:21 +0200, Igor Mammedov wrote: > On Mon, 4 Jul 2022 16:58:52 +0800 > Robert Hoo wrote: > > > Signed-off-by: Robert Hoo > > Reviewed-by: Jingqi Liu > > Reviewed-by: Igor Mammedov Thanks for review Igor. BTW, during the unit test, I met

[PATCH] acpi/nvdimm: Define trace events for NVDIMM and substitute nvdimm_debug()

2022-07-04 Thread Robert Hoo
Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- This is separated from patch set https://lore.kernel.org/qemu-devel/20220616143542.3e049...@redhat.com/ hw/acpi/nvdimm.c| 35 --- hw/acpi/trace-events| 13 + include/hw/mem/nvdimm.h

Re: [QEMU PATCH v2 4/6] nvdimm: Implement ACPI NVDIMM Label Methods

2022-07-01 Thread Robert Hoo
On Thu, 2022-06-16 at 14:32 +0200, Igor Mammedov wrote: > On Mon, 30 May 2022 11:40:45 +0800 > Robert Hoo wrote: > > > Recent ACPI spec [1] has defined NVDIMM Label Methods _LS{I,R,W}, > > which > > depricates corresponding _DSM Functions defined by PMEM _DS

Re: [QEMU PATCH v2 6/6] acpi/nvdimm: Define trace events for NVDIMM and substitute nvdimm_debug()

2022-07-01 Thread Robert Hoo
On Thu, 2022-06-16 at 14:35 +0200, Igor Mammedov wrote: > On Mon, 30 May 2022 11:40:47 +0800 > Robert Hoo wrote: > > suggest to put this patch as the 1st in series > (well you can rebase it on current master and > post that right away for merging since it doesn't > real

Re: [QEMU PATCH v2 3/6] acpi/nvdimm: NVDIMM _DSM Spec supports revision 2

2022-07-01 Thread Robert Hoo
On Thu, 2022-06-16 at 13:38 +0200, Igor Mammedov wrote: > On Mon, 30 May 2022 11:40:44 +0800 > Robert Hoo wrote: > > > The Intel Optane PMem DSM Interface, Version 2.0 [1], is the up-to- > > date > > spec for NVDIMM _DSM definition, which supports revision_id == 2. &

RE: [QEMU PATCH v2 0/6] Support ACPI NVDIMM Label Methods

2022-06-05 Thread Hu, Robert
Ping... Best Regards, Robert Hoo > -Original Message- > From: Robert Hoo > Sent: Monday, May 30, 2022 11:41 > To: imamm...@redhat.com; m...@redhat.com; > xiaoguangrong.e...@gmail.com; a...@anisinha.ca; Williams, Dan J > ; Liu, Jingqi > Cc: qemu-devel@nongnu.org;

[QEMU PATCH v2 6/6] acpi/nvdimm: Define trace events for NVDIMM and substitute nvdimm_debug()

2022-05-29 Thread Robert Hoo
Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- hw/acpi/nvdimm.c| 38 ++ hw/acpi/trace-events| 14 ++ include/hw/mem/nvdimm.h | 8 3 files changed, 32 insertions(+), 28 deletions(-) diff --git a/hw/acpi/nvdimm.c b/hw

[QEMU PATCH v2 5/6] test/acpi/bios-tables-test: SSDT: update golden master binaries

2022-05-29 Thread Robert Hoo
Method (_DSM, 4, NotSerialized) // _DSM: Device-Specific Method { Return (NCAL (Arg0, Arg1, Arg2, Arg3, 0x03)) } } } } - -Name (MEMA, 0x07FFF000) } Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- tests/da

[QEMU PATCH v2 1/6] tests/acpi: allow SSDT changes

2022-05-29 Thread Robert Hoo
Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- tests/qtest/bios-tables-test-allowed-diff.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h index dfb8523c8b..eb8bae1407 100644 --- a/tests

[QEMU PATCH v2 4/6] nvdimm: Implement ACPI NVDIMM Label Methods

2022-05-29 Thread Robert Hoo
/sites/default/files/resources/ACPI_Spec_6_4_Jan22.pdf [2] Intel PMEM _DSM Interface Spec v2.0, 3.10 Deprecated Functions https://pmem.io/documents/IntelOptanePMem_DSM_Interface-V2.0.pdf Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- hw/acpi/nvdimm.c

[QEMU PATCH v2 2/6] acpi/ssdt: Fix aml_or() and aml_and() in if clause

2022-05-29 Thread Robert Hoo
3) == 0x04) && (SizeOf (Arg3) == One))) Fixes: 90623ebf603 ("nvdimm acpi: check UUID") Fixes: 4568c948066 ("nvdimm acpi: save arg3 of _DSM method") Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu Reviewed-by: Igor Mammedov --- hw/acpi/nvdimm.c | 7 +++ 1 file changed,

[QEMU PATCH v2 0/6] Support ACPI NVDIMM Label Methods

2022-05-29 Thread Robert Hoo
https://pmem.io/documents/IntelOptanePMem_DSM_Interface-V2.0.pdf [2] ACPI Spec v6.4, 6.5.10 NVDIMM Label Methods https://uefi.org/sites/default/files/resources/ACPI_Spec_6_4_Jan22.pdf --- Change Log: v2: Almost rewritten Separate Patch 2 Dance with tests/qtest/bios-table-tests Add trace events Robert Hoo (6)

[QEMU PATCH v2 3/6] acpi/nvdimm: NVDIMM _DSM Spec supports revision 2

2022-05-29 Thread Robert Hoo
change to support this revision_id == 2 case. [1] https://pmem.io/documents/IntelOptanePMem_DSM_Interface-V2.0.pdf Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- hw/acpi/nvdimm.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/hw/acpi/nvdimm.c b/hw/acpi

Re: [PATCH 1/2] acpi/nvdimm: Create _LS{I,R,W} method for NVDIMM device

2022-05-17 Thread Robert Hoo
On Fri, 2022-05-06 at 11:23 +0200, Igor Mammedov wrote: > > > > > No, sorry, I didn't explain it clear. > > No extra interface/ABI but these 3 must _LS{I,R,W} nvdimm-sub- > > device > > methods. Of course, I'm going to extract 'SystemIO' and > > 'SystemMemory' > > operation regions out of NACL to

Re: [PATCH 1/2] acpi/nvdimm: Create _LS{I,R,W} method for NVDIMM device

2022-05-05 Thread Robert Hoo
On Thu, 2022-05-05 at 10:50 +0200, Igor Mammedov wrote: ... > > > > > > @@ -1247,6 +1247,11 @@ static void nvdimm_build_fit(Aml > > > > > > *dev) > > > > > > static void nvdimm_build_nvdimm_devices(Aml *root_dev, > > > > > > uint32_t > > > > > > ram_slots) > > > > > > { > > > > > > uint32_t

Re: [PATCH 1/2] acpi/nvdimm: Create _LS{I,R,W} method for NVDIMM device

2022-05-04 Thread Robert Hoo
On Tue, 2022-05-03 at 10:27 +0200, Igor Mammedov wrote: > On Fri, 29 Apr 2022 17:01:47 +0800 > Robert Hoo wrote: > > > On Wed, 2022-04-27 at 16:34 +0200, Igor Mammedov wrote: > > > On Tue, 12 Apr 2022 14:57:52 +0800 > > > Robert Hoo wrote: > > > &g

Re: [PATCH 1/2] acpi/nvdimm: Create _LS{I,R,W} method for NVDIMM device

2022-04-29 Thread Robert Hoo
On Wed, 2022-04-27 at 16:34 +0200, Igor Mammedov wrote: > On Tue, 12 Apr 2022 14:57:52 +0800 > Robert Hoo wrote: > > > Since ACPI 6.2, previous NVDIMM/_DSM funcions "Get Namespace Label > > Data > > Size (function index 4)", "Get Namespace Label Data (

Re: [RESEND][PATCH 0/2] acpi/nvdimm: support NVDIMM _LS{I,R,W} methods

2022-04-29 Thread Robert Hoo
On Wed, 2022-04-27 at 16:39 +0200, Igor Mammedov wrote: > On Tue, 12 Apr 2022 14:57:51 +0800 > Robert Hoo wrote: > > > The original NVDIMM _DSM functions (index 4~6) for label operations > > have > > been deprecated by new ACPI methods _LS{I,R,W}[1][2]. > > >

Q: TX (THR) throttling for serial interface?

2022-04-21 Thread REITHER Robert
ry to switch to virtio_console? Thank you Robert

Re: [RESEND][PATCH 0/2] acpi/nvdimm: support NVDIMM _LS{I,R,W} methods

2022-04-19 Thread Robert Hoo
Ping... On Tue, 2022-04-12 at 14:57 +0800, Robert Hoo wrote: > The original NVDIMM _DSM functions (index 4~6) for label operations > have > been deprecated by new ACPI methods _LS{I,R,W}[1][2]. > > Patch 1 implements the new _LS{I,R,W} methods, on top of old _DSM > implementa

Re: [PATCH] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2022-04-15 Thread Robert Hoo
On Thu, 2022-03-24 at 09:22 +0100, Igor Mammedov wrote: > On Mon, 14 Mar 2022 16:50:59 +0800 > Robert Hoo wrote: > > > Icelake, is the codename for Intel 3rd generation Xeon Scalable > > server > > processors. There isn't ever client variants. This "Icelake

[PATCH 2/2] acpi/nvdimm: Fix aml_or() and aml_and() in if clause

2022-04-12 Thread Robert Hoo
== One))) ==> If (((ObjectType (Arg3) == 0x04) && (SizeOf (Arg3) == One))) Fixes: 90623ebf603 ("nvdimm acpi: check UUID") Fixes: 4568c948066 ("nvdimm acpi: save arg3 of _DSM method") Signed-off-by: Robert Hoo Reviewed-by: Jingqi Liu --- hw/acpi/nvdimm.c | 7 +++--

[PATCH 1/2] acpi/nvdimm: Create _LS{I,R,W} method for NVDIMM device

2022-04-12 Thread Robert Hoo
ID ("4309ac30-0d11-11e4-9191-0800200c9a66"), 0x02, 0x06, PKG1, One)) } Method (_DSM, 4, NotSerialized) // _DSM: Device-Specific Method { Return (NCAL (Arg0, Arg1, Arg2, Arg3, One)) } } Signed-off-by: Ro

[RESEND][PATCH 0/2] acpi/nvdimm: support NVDIMM _LS{I,R,W} methods

2022-04-11 Thread Robert Hoo
uot; due to 550-'Message headers fail syntax check'. Robert Hoo (2): acpi/nvdimm: Create _LS{I,R,W} method for NVDIMM device acpi/nvdimm: Fix aml_or() and aml_and() in if clause hw/acpi/nvdimm.c | 60 +++- 1 file changed, 54 insertions(+), 6 d

[PATCH 0/2] acpi/nvdimm: support NVDIMM _LS{I,R,W} methods

2022-04-11 Thread Robert Hoo
haven't causing trouble. [1] https://uefi.org/htmlspecs/ACPI_Spec_6_4_html/index.html, 6.5.10 NVDIMM Label Methods [2] https://pmem.io/documents/IntelOptanePMem_DSM_Interface-V2.0.pdf, 3.10 Deprecated Functions Robert Hoo (2): acpi/nvdimm: Create _LS{I,R,W} method for NVDIMM device

Re: [PATCH 2/2] NVDIMM: Init vNVDIMM's LSA index block if it hasn't been

2022-03-31 Thread Robert Hoo
On Thu, 2022-03-31 at 16:41 +0200, Igor Mammedov wrote: > On Thu, 31 Mar 2022 21:08:12 +0800 > Robert Hoo wrote: > > > > > > > Can user initialize/format LSA from guest using ndctl/some other > > > tool? > > > > > > > Yes, he c

Re: [PATCH 2/2] NVDIMM: Init vNVDIMM's LSA index block if it hasn't been

2022-03-31 Thread Robert Hoo
On Thu, 2022-03-31 at 14:09 +0200, Igor Mammedov wrote: > On Tue, 29 Mar 2022 15:07:43 +0800 > Robert Hoo wrote: > > > Since v2.7, QEMU has supported the emulation of NVDIMM's labels. > > With -device nvdimm,...,lsa-size=, the vNVDIMM to guest has this > > cap

Re: [PATCH 0/2] Init vNVDIMM LSA if applicable

2022-03-31 Thread Robert Hoo
On Thu, 2022-03-31 at 14:03 +0200, Igor Mammedov wrote: > On Tue, 29 Mar 2022 15:07:41 +0800 > Robert Hoo wrote: > > > QEMU option "-device nvdimm,...,label-size=" designates a vNVDIMM > > with > > Label Storage Area (LSA), where stores the namespace label

[PATCH 2/2] NVDIMM: Init vNVDIMM's LSA index block if it hasn't been

2022-03-29 Thread Robert Hoo
nitial Label Storage Area Configuration: "for Label Storage Areas of 128KB and 256KB, the corresponding Index Block size is 256 or 512 bytes." In driver and ndctl code, they refer to these 2 cases as v1.1 and v1.2. Signed-off-by: Robert Hoo Reviewed-by: Liu, Jingqi --- Note: most functions

[PATCH 0/2] Init vNVDIMM LSA if applicable

2022-03-29 Thread Robert Hoo
apable. So without initialization, guest Kernel will judge it label-less, though it actually support label. This patch set, is to init vNVDIMM's LSA, so that guest Kernel can correctly identify and use it. [1]: https://uefi.org/sites/default/files/resources/ACPI_Spec_6_4_Jan22.pdf, Section

[PATCH 1/2] NVDIMM: rename NVDIMM::label_size to NVDIMM::lsa_size

2022-03-29 Thread Robert Hoo
Per recent spec[1], change struct NVDIMMDevice::label_size semanteme to describe the label's size in LSA (Label Storage Area). Instead, use new 'lsa_size' for the total size of LSA. [1]: UEFI spec v2.9, "Label Storage Area Description" in section 13.19. Signed-off-by

Re: [EXTERNAL] Plugins Not Reporting AArch64 SVE Memory Operations

2022-03-24 Thread Robert Henry
I have not done anything on this front, alas. From: Aaron Lindsay Sent: Thursday, March 24, 2022 1:17 PM To: qemu-devel@nongnu.org ; qemu-...@nongnu.org Cc: Alex Bennée ; richard.hender...@linaro.org ; Robert Henry Subject: [EXTERNAL] Plugins Not Reporting

Re: [PATCH] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2022-03-20 Thread Robert Hoo
Hi Igor, Ping... Thanks. On Mon, 2022-03-14 at 16:50 +0800, Robert Hoo wrote: > Icelake, is the codename for Intel 3rd generation Xeon Scalable > server > processors. There isn't ever client variants. This "Icelake-Client" > CPU > model was added wrongly and

[PATCH] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2022-03-14 Thread Robert Hoo
Icelake, is the codename for Intel 3rd generation Xeon Scalable server processors. There isn't ever client variants. This "Icelake-Client" CPU model was added wrongly and imaginarily. It has been deprecated since v5.2, now it's time to remove it completely from code. Signe

Re: [PATCH v3] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2022-02-08 Thread Robert Hoo
Hi, Can we remove the deprecated 'Icelake-Client' CPU model now? if so, I can rebase patch to latest and resend. Thanks. On Sat, 2021-05-08 at 11:16 +0800, Robert Hoo wrote: > Hi, > > Ping... > > Thanks. > > On Thu, 2021-04-29 at 09:35 +0800, Robert Hoo

Re: [PATCH v3 kvm/queue 03/16] mm/memfd: Introduce MEMFD_OPS

2021-12-23 Thread Robert Hoo
On Thu, 2021-12-23 at 20:29 +0800, Chao Peng wrote: > From: "Kirill A. Shutemov" > > The patch introduces new MEMFD_OPS facility around file created by > memfd_create() to allow a third kernel component to make use of > memory > bookmarked in a memfd and gets notifier when the memory in the file

Re: [EXTERNAL] Re: Range of vcpu_index to plugin callbacks

2021-09-20 Thread Robert Henry
: Sunday, September 19, 2021 10:54 AM To: rrh.henry ; qemu-disc...@nongnu.org Cc: Alex Bennée ; qemu-devel Subject: [EXTERNAL] Re: Range of vcpu_index to plugin callbacks (Cc'ing qemu-devel@ mailing list since this is a development question). On 9/19/21 19:44, Robert Henry wrote: > What is t

Re: [PATCH v3] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2021-05-07 Thread Robert Hoo
Hi, Ping... Thanks. On Thu, 2021-04-29 at 09:35 +0800, Robert Hoo wrote: > As it's been marked deprecated since v5.2, now I think it's time > remove it > from code. > > Signed-off-by: Robert Hoo > --- > Changelog: > v3: > Update deprecated.rs

Re: [PATCH v5] i386/cpu_dump: support AVX512 ZMM regs dump

2021-05-06 Thread Robert Hoo
On Mon, 2021-04-19 at 17:59 -0400, Eduardo Habkost wrote: > On Mon, Apr 19, 2021 at 04:18:25PM -0400, Eduardo Habkost wrote: > > On Fri, Apr 16, 2021 at 10:08:24AM +0800, Robert Hoo wrote: > > > Since commit fa4518741e (target-i386: Rename struct XMMReg to > > > ZMMReg),

Re: [Trivial] docs: More precisely describe memory-backend-*::id's user

2021-04-30 Thread Robert Hoo
Ping for review ... Thanks. On Thu, 2021-04-22 at 16:42 +0800, Robert Hoo wrote: > 'id' of memory-backend-{file,ram} is not only for '-numa''s > reference, but > also other parameters like '-device nvdimm'. > More clearly call out this to avoid mi

Re: Let's remove some deprecated stuff

2021-04-29 Thread Robert Hoo
moval has come. > Extra points for telling us in a reply. "We should remove, but I > can't > do it myself right now" is a valid answer. Let's review the file: > > System emulator command line arguments > -- > [.

[PATCH v3] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2021-04-28 Thread Robert Hoo
As it's been marked deprecated since v5.2, now I think it's time remove it from code. Signed-off-by: Robert Hoo --- Changelog: v3: Update deprecated.rst. (Sorry for my carelessness in last search. I sware I did search.) v2: Update removed-features.rst. --- docs/system/depr

Re: [PATCH RESEND v2] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2021-04-28 Thread Robert Hoo
On Wed, 2021-04-28 at 11:24 -0400, Eduardo Habkost wrote: > On Wed, Apr 28, 2021 at 10:41:13AM +0800, Robert Hoo wrote: > > As it's been marked deprecated since v5.2, now I think it's time > > remove it > > from code. > > > > Signed-off-by: Rober

[PATCH RESEND v2] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2021-04-27 Thread Robert Hoo
As it's been marked deprecated since v5.2, now I think it's time remove it from code. Signed-off-by: Robert Hoo --- (Sorry, forgot to append changelog in last send.) Changelog: v2: Update removed-features.rst. Since previously no its deprecation info was recorded in d

[PATCH v2] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2021-04-27 Thread Robert Hoo
As it's been marked deprecated since v5.2, now I think it's time remove it from code. Signed-off-by: Robert Hoo --- P.S. Since previously no its deprecation info was recorded in docs/system/deprecated.rst, nothing to update it. --- docs/system/removed-features.rst | 5 ++ target/

Re: [PATCH] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2021-04-27 Thread Robert Hoo
On Tue, 2021-04-27 at 16:55 -0400, Eduardo Habkost wrote: > On Thu, Apr 22, 2021 at 05:42:16PM +0800, Robert Hoo wrote: > > As it's been marked deprecated since v5.2, now I think it's time > > remove it > > from code. > > > > Signed-off-by: Robert Ho

[PATCH] i386/cpu: Remove the deprecated cpu model 'Icelake-Client'

2021-04-22 Thread Robert Hoo
As it's been marked deprecated since v5.2, now I think it's time remove it from code. Signed-off-by: Robert Hoo --- target/i386/cpu.c | 118 -- 1 file changed, 118 deletions(-) diff --git a/target/i386/cpu.c b/target/i386/cpu.c ind

[Trivial] docs: More precisely describe memory-backend-*::id's user

2021-04-22 Thread Robert Hoo
'id' of memory-backend-{file,ram} is not only for '-numa''s reference, but also other parameters like '-device nvdimm'. More clearly call out this to avoid misinterpretation. Signed-off-by: Robert Hoo --- qemu-options.hx | 6 +++--- 1 file changed, 3 insertio

[PATCH v6] i386/cpu_dump: support AVX512 ZMM regs dump

2021-04-20 Thread Robert Hoo
/ZMM is enabled. This patch is to complement this, let it dump XMM/YMM/ZMM accordingly. Signed-off-by: Robert Hoo Reviewed-by: Richard Henderson --- Changelog: v6: use PRIx64 printf format to satisfy multiple target compiling. v5: fix a minor issue. rebase to latest master. v4: stringent AVX512 case

[PATCH v5] i386/cpu_dump: support AVX512 ZMM regs dump

2021-04-15 Thread Robert Hoo
/ZMM is enabled. This patch is to complement this, let it dump XMM/YMM/ZMM accordingly. Signed-off-by: Robert Hoo --- Changelog: v5: fix a minor issue. rebase to latest master. v4: stringent AVX512 case and AVX case judgement criteria v3: fix some coding style issue. v2: dump XMM/YMM/ZMM according to

Re: [PATCH v4] i386/cpu_dump: support AVX512 ZMM regs dump

2021-04-05 Thread Robert Hoo
Hi, Ping... Thanks On Fri, 2021-03-26 at 23:01 +0800, Robert Hoo wrote: > On Fri, 2021-03-26 at 22:54 +0800, Robert Hoo wrote: > > Since commit fa4518741e (target-i386: Rename struct XMMReg to > > ZMMReg), > > CPUX86State.xmm_regs[] has already been extended to 512bit to &

Re: [PATCH v4] i386/cpu_dump: support AVX512 ZMM regs dump

2021-03-26 Thread Robert Hoo
On Fri, 2021-03-26 at 22:54 +0800, Robert Hoo wrote: > Since commit fa4518741e (target-i386: Rename struct XMMReg to > ZMMReg), > CPUX86State.xmm_regs[] has already been extended to 512bit to support > AVX512. > Also, other qemu level supports for AVX512 registers are there for &g

[PATCH v4] i386/cpu_dump: support AVX512 ZMM regs dump

2021-03-26 Thread Robert Hoo
/ZMM is enabled. This patch is to complement this, let it dump XMM/YMM/ZMM accordingly. Signed-off-by: Robert Hoo --- Changelog: v4: stringent AVX512 case and AVX case judgement criteria v3: fix some coding style issue. v2: dump XMM/YMM/ZMM according to XSAVE state-components enablement. target

Re: [PATCH v3] i386/cpu_dump: support AVX512 ZMM regs dump

2021-03-26 Thread Robert Hoo
On Fri, 2021-03-26 at 07:11 -0600, Richard Henderson wrote: > On 3/25/21 7:47 PM, Robert Hoo wrote: > > On Thu, 2021-03-25 at 06:39 -0600, Richard Henderson wrote: > > > On 3/24/21 9:15 PM, Robert Hoo wrote: > > > > > > +} else if (env->xcr0 & X

Re: [PATCH v3] i386/cpu_dump: support AVX512 ZMM regs dump

2021-03-25 Thread Robert Hoo
On Thu, 2021-03-25 at 06:39 -0600, Richard Henderson wrote: > On 3/24/21 9:15 PM, Robert Hoo wrote: > > > > +} else if (env->xcr0 & XFEATURE_AVX) { > > > > > > This is normally a 2-bit test. > > > > I beg your pardon. What 2 bits? &g

Re: [PATCH v3] i386/cpu_dump: support AVX512 ZMM regs dump

2021-03-24 Thread Robert Hoo
On Wed, 2021-03-24 at 07:44 -0600, Richard Henderson wrote: > On 3/24/21 2:00 AM, Robert Hoo wrote: > > +if ((env->xcr0 & XFEATURE_AVX512) == XFEATURE_AVX512) { > > +/* XSAVE enabled AVX512 */ > > +nb = (env->hflags & HF_CS64_MASK

[PATCH v3] i386/cpu_dump: support AVX512 ZMM regs dump

2021-03-24 Thread Robert Hoo
/ZMM is enabled. This patch is to complement this, let it dump XMM/YMM/ZMM accordingly. Signed-off-by: Robert Hoo --- Changelog: v3: fix some coding style issue. v2: dump XMM/YMM/ZMM according to XSAVE state-components enablement. target/i386/cpu-dump.c | 55

[PATCH v2] i386/cpu_dump: support AVX512 ZMM regs dump

2021-03-23 Thread Robert Hoo
/ZMM is enabled. This patch is to complement this, let it dump XMM/YMM/ZMM accordingly. Signed-off-by: Robert Hoo --- Changelog: v2: dump XMM/YMM/ZMM according to XSAVE state-components enablement. target/i386/cpu-dump.c | 57 +- target/i386/cpu.h

  1   2   3   4   5   6   7   8   9   10   >