Re: Concerns regarding e17bebd049 ("dump: Set correct vaddr for ELF dump")

2023-09-20 Thread Jon Doron
lly do all the page table walks yourself. But I wanted to see if there was some justification for this behavior that I missed. Thanks, Stephen [1]: https://github.com/osandov/drgn [2]: https://lore.kernel.org/qemu-devel/20181225125344.4482-1-ari...@gmail.com/ --- commit e17bebd049d78f489c2cff75

[PATCH v3 1/4] hyperv: SControl is optional to enable SynIc

2022-03-15 Thread Jon Doron
with the pending flag. When the guest polls from the message slot, in case the pending flag is set it will write to the HV_X64_MSR_EOM indicating it has cleared the slot and we can try and push our message again. Signed-off-by: Jon Doron --- hw/hyperv/hyperv.c | 109

[PATCH v3 3/4] hyperv: Add support to process syndbg commands

2022-03-15 Thread Jon Doron
SynDbg commands can come from two different flows: 1. Hypercalls, in this mode the data being sent is fully encapsulated network packets. 2. SynDbg specific MSRs, in this mode only the data that needs to be transfered is passed. Signed-off-by: Jon Doron --- docs/hyperv.txt

[PATCH v3 0/4] HyperV: Synthetic Debugging device

2022-03-15 Thread Jon Doron
configured correctly. v3: Fixed review from Paolo changes from QLIST*RCU to non RCU Jon Doron (4): hyperv: SControl is optional to enable SynIc hyperv: Add definitions for syndbg hyperv: Add support to process syndbg commands hw: hyperv: Initial commit for Synthetic Debugging devi

[PATCH v3 4/4] hw: hyperv: Initial commit for Synthetic Debugging device

2022-03-15 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/hyperv/Kconfig | 5 + hw/hyperv/meson.build | 1 + hw/hyperv/syndbg.c| 402 ++ 3 files changed, 408 insertions(+) create mode 100644 hw/hyperv/syndbg.c diff --git a/hw/hyperv/Kconfig b/hw/hyperv/Kconfig index

[PATCH v3 2/4] hyperv: Add definitions for syndbg

2022-03-15 Thread Jon Doron
Add all required definitions for hyperv synthetic debugger interface. Signed-off-by: Jon Doron --- include/hw/hyperv/hyperv-proto.h | 52 target/i386/kvm/hyperv-proto.h | 37 +++ 2 files changed, 89 insertions(+) diff --git a/include/hw

Re: [PATCH v2 1/4] hyperv: SControl is optional to enable SynIc

2022-03-06 Thread Jon Doron
Thanks! is there an estimate when will this patchset be merged? On Thu, Feb 24, 2022, 18:36 Emanuele Giuseppe Esposito wrote: > > > On 16/02/2022 11:24, Jon Doron wrote: > > SynIc can be enabled regardless of the SControl mechanisim which can > > register a GSI

Re: [PATCH v2 4/4] hw: hyperv: Initial commit for Synthetic Debugging device

2022-02-23 Thread Jon Doron
ping On Wed, Feb 16, 2022, 12:25 Jon Doron wrote: > Signed-off-by: Jon Doron > --- > hw/hyperv/Kconfig | 5 + > hw/hyperv/meson.build | 1 + > hw/hyperv/syndbg.c| 402 ++ > 3 files changed, 408 insertions(+) > create

Re: [PATCH v1 2/4] hyperv: Add definitions for syndbg

2022-02-16 Thread Jon Doron
On 16/02/2022, Emanuele Giuseppe Esposito wrote: On 04/02/2022 11:07, Jon Doron wrote: Add all required definitions for hyperv synthetic debugger interface. Signed-off-by: Jon Doron --- include/hw/hyperv/hyperv-proto.h | 52 target/i386/kvm/hyperv-proto.h

Re: [PATCH v1 4/4] hw: hyperv: Initial commit for Synthetic Debugging device

2022-02-16 Thread Jon Doron
On 16/02/2022, Emanuele Giuseppe Esposito wrote: + +static uint16_t handle_recv_msg(HvSynDbg *syndbg, uint64_t outgpa, +uint32_t count, bool is_raw, uint32_t options, +uint64_t timeout, uint32_t *retrieved_count) +{ +uint16_t

Re: [PATCH v1 1/4] hyperv: SControl is optional to enable SynIc

2022-02-16 Thread Jon Doron
On 16/02/2022, Emanuele Giuseppe Esposito wrote: On 04/02/2022 11:07, Jon Doron wrote: SynIc can be enabled regardless of the SControl mechanisim which can register a GSI for a given SintRoute. This behaviour can achived by setting enabling SIMP and then the guest will poll on the message

Re: [PATCH v1 3/4] hyperv: Add support to process syndbg commands

2022-02-16 Thread Jon Doron
On 16/02/2022, Emanuele Giuseppe Esposito wrote: On 04/02/2022 11:07, Jon Doron wrote: SynDbg commands can come from two different flows: 1. Hypercalls, in this mode the data being sent is fully encapsulated network packets. 2. SynDbg specific MSRs, in this mode only the data that needs

[PATCH v2 3/4] hyperv: Add support to process syndbg commands

2022-02-16 Thread Jon Doron
SynDbg commands can come from two different flows: 1. Hypercalls, in this mode the data being sent is fully encapsulated network packets. 2. SynDbg specific MSRs, in this mode only the data that needs to be transfered is passed. Signed-off-by: Jon Doron --- docs/hyperv.txt

[PATCH v2 0/4] HyperV: Synthetic Debugging device

2022-02-16 Thread Jon Doron
configured correctly. Jon Doron (4): hyperv: SControl is optional to enable SynIc hyperv: Add definitions for syndbg hyperv: Add support to process syndbg commands hw: hyperv: Initial commit for Synthetic Debugging device docs/hyperv.txt | 15 ++ hw/hyperv/Kconfig

[PATCH v2 2/4] hyperv: Add definitions for syndbg

2022-02-16 Thread Jon Doron
Add all required definitions for hyperv synthetic debugger interface. Signed-off-by: Jon Doron --- include/hw/hyperv/hyperv-proto.h | 52 target/i386/kvm/hyperv-proto.h | 37 +++ 2 files changed, 89 insertions(+) diff --git a/include/hw

[PATCH v2 4/4] hw: hyperv: Initial commit for Synthetic Debugging device

2022-02-16 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/hyperv/Kconfig | 5 + hw/hyperv/meson.build | 1 + hw/hyperv/syndbg.c| 402 ++ 3 files changed, 408 insertions(+) create mode 100644 hw/hyperv/syndbg.c diff --git a/hw/hyperv/Kconfig b/hw/hyperv/Kconfig index

[PATCH v2 1/4] hyperv: SControl is optional to enable SynIc

2022-02-16 Thread Jon Doron
with the pending flag. When the guest polls from the message slot, in case the pending flag is set it will write to the HV_X64_MSR_EOM indicating it has cleared the slot and we can try and push our message again. Signed-off-by: Jon Doron --- hw/hyperv/hyperv.c | 109

Re: [PATCH v1 0/4] HyperV: Synthetic Debugging device

2022-02-12 Thread Jon Doron
On 04/02/2022, Jon Doron wrote: Ping This patchset adds support for the synthetic debugging device. HyperV supports a special transport layer for the kernel debugger when running in HyperV. This patchset add supports for this device so you could have a setup fast windows kernel debugging

[PATCH v1 4/4] hw: hyperv: Initial commit for Synthetic Debugging device

2022-02-04 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/hyperv/Kconfig | 5 + hw/hyperv/meson.build | 1 + hw/hyperv/syndbg.c| 407 ++ 3 files changed, 413 insertions(+) create mode 100644 hw/hyperv/syndbg.c diff --git a/hw/hyperv/Kconfig b/hw/hyperv/Kconfig index

[PATCH v1 2/4] hyperv: Add definitions for syndbg

2022-02-04 Thread Jon Doron
Add all required definitions for hyperv synthetic debugger interface. Signed-off-by: Jon Doron --- include/hw/hyperv/hyperv-proto.h | 52 target/i386/kvm/hyperv-proto.h | 37 +++ 2 files changed, 89 insertions(+) diff --git a/include/hw

[PATCH v1 1/4] hyperv: SControl is optional to enable SynIc

2022-02-04 Thread Jon Doron
with the pending flag. When the guest polls from the message slot, incase the pending flag is set it will write to the HV_X64_MSR_EOM indicating it has cleared the slow and we can try and push our message again. Signed-off-by: Jon Doron --- hw/hyperv/hyperv.c | 233

[PATCH v1 3/4] hyperv: Add support to process syndbg commands

2022-02-04 Thread Jon Doron
SynDbg commands can come from two different flows: 1. Hypercalls, in this mode the data being sent is fully encapsulated network packets. 2. SynDbg specific MSRs, in this mode only the data that needs to be transfered is passed. Signed-off-by: Jon Doron --- docs/hyperv.txt

[PATCH v1 0/4] HyperV: Synthetic Debugging device

2022-02-04 Thread Jon Doron
configured correctly. Jon Doron (4): hyperv: SControl is optional to enable SynIc hyperv: Add definitions for syndbg hyperv: Add support to process syndbg commands hw: hyperv: Initial commit for Synthetic Debugging device docs/hyperv.txt | 15 + hw/hyperv/Kconfig

Re: [PATCH v4 0/1] hw/hyperv/vmbus: Is it maintained?

2021-11-16 Thread Jon Doron
On 13/11/2021, Maciej S. Szmigiero wrote: On 12.11.2021 21:39, Roman Kagan wrote: On Fri, Nov 12, 2021 at 09:32:31PM +0300, Vladimir Sementsov-Ogievskiy wrote: Add Den and Roman (his new address) Thanks, I missed it on the list indeed. 06.11.2021 16:41, Philippe Mathieu-Daudé wrote: This

Re: Who uses TYPE_VMBUS_DEVICE?

2020-10-12 Thread Jon Doron
Hi Eduardo, Sorry for the late reply, in general there are also hv-net and hv-scsi implementations from Roman Kagan, which have not been merged in... I have a WIP component which does not use vmbus but Windows needs it to exist and function in order for it to work (it's the synthetic debugger

Re: [PATCH] hw/hyperv: Fix 32-bit build error for vmbus.

2020-09-06 Thread Jon Doron
> > Fixes: 0d71f7082d7 ("vmbus: vmbus implementation") > Signed-off-by: Richard Henderson > --- > Cc: Jon Doron > --- > hw/hyperv/vmbus.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/hw/hyperv/vmbus.c b/hw/hyperv/vmbus.c >

Re: [PATCH v1 1/1] hyperv: Enable SCONTROL if SYNIC is enabled

2020-08-10 Thread Jon Doron
I think both patches are On Mon, Aug 10, 2020, 20:27 Paolo Bonzini wrote: > On 17/07/20 14:56, Jon Doron wrote: > > Based on an analysis of the HyperV firmwares (Gen1 and Gen2) it seems > > like the SCONTROL is not being set to the ENABLED state as like we have > > thoug

[PATCH v1 1/1] hyperv: Enable SCONTROL if SYNIC is enabled

2020-07-17 Thread Jon Doron
with the value of 0x1, aka HV_SYNIC_CONTROL_ENABLE. It's important to note that this diverges from the value states in the HyperV TLFS of 0. Signed-off-by: Jon Doron --- target/i386/kvm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/target/i386/kvm.c b/target/i386/kvm.c index

[PATCH v1 0/1] hyperv: Enable SCONTROL if SYNIC is enabled

2020-07-17 Thread Jon Doron
with the value of 0x1, aka HV_SYNIC_CONTROL_ENABLE. It's important to note that this diverges from the value states in the HyperV TLFS of 0. Jon Doron (1): hyperv: Enable SCONTROL if SYNIC is enabled target/i386/kvm.c | 2 ++ 1 file changed, 2 insertions(+) -- 2.24.1

[PATCH v5 2/2] hw: hyperv: vmbus: Fix 32bit compilation

2020-07-15 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/hyperv/vmbus.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/hyperv/vmbus.c b/hw/hyperv/vmbus.c index 34392e892a..c28bb4201b 100644 --- a/hw/hyperv/vmbus.c +++ b/hw/hyperv/vmbus.c @@ -383,7 +383,8 @@ static ssize_t gpadl_iter_io

[PATCH v5 0/2] hyperv: vmbus: ACPI various corrections

2020-07-15 Thread Jon Doron
under the SB v3: Removed accidental change for the dct submodule head v2: Renamed irq0 to irq now that there is a single IRQ required Jon Doron (2): acpi: i386: Move VMBus DSDT entry to SB hw: hyperv: vmbus: Fix 32bit compilation hw/hyperv/vmbus.c| 3 ++- hw/i386/acpi-build.c | 12

[PATCH v5 1/2] acpi: i386: Move VMBus DSDT entry to SB

2020-07-15 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index b7bc2a..7708a27f70 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -975,7 +975,6 @@ static Aml

Re: [PATCH v4 0/2] hyperv: vmbus: ACPI various corrections

2020-07-13 Thread Jon Doron
Sure thing I'll try to take a look at it this weekend. Jon. On Mon, Jul 13, 2020, 11:44 Igor Mammedov wrote: > On Thu, 25 Jun 2020 07:50:09 +0300 > Jon Doron wrote: > > > After doing further tests and looking at the latest HyperV ACPI DSDT. > > Do minor fix to our VMBu

Re: [Bug 1885247] [NEW] Build error in Intel 32-bit hosts

2020-06-26 Thread Jon Doron
Is there a container I can download which has your build environment? On Fri, Jun 26, 2020 at 12:27 PM Philippe Mathieu-Daudé wrote: > > On 6/26/20 11:20 AM, Thomas Huth wrote: > > On 26/06/2020 11.13, Philippe Mathieu-Daudé wrote: > >> On 6/26/20 9:37 AM, Aleksandar Markovic wrote: > >>> пет,

[PATCH v4 2/2] acpi: i386: Move VMBus DSDT entry to SB

2020-06-24 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 91af0d2d0d..1f938a53b2 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -1091,7 +1091,6 @@ static

[PATCH v4 0/2] hyperv: vmbus: ACPI various corrections

2020-06-24 Thread Jon Doron
: Renamed irq0 to irq now that there is a single IRQ required Jon Doron (2): hyperv: vmbus: Remove the 2nd IRQ acpi: i386: Move VMBus DSDT entry to SB hw/hyperv/vmbus.c| 3 +-- hw/i386/acpi-build.c | 16 include/hw/hyperv/vmbus-bridge.h | 3 +-- 3 files

[PATCH v4 1/2] hyperv: vmbus: Remove the 2nd IRQ

2020-06-24 Thread Jon Doron
It seems like Windows does not really require 2 IRQs to have a functioning VMBus. Signed-off-by: Jon Doron Reviewed-by: Igor Mammedov --- hw/hyperv/vmbus.c| 3 +-- hw/i386/acpi-build.c | 4 +--- include/hw/hyperv/vmbus-bridge.h | 3 +-- 3 files changed, 3 insertions

Re: [PATCH v3 3/3] acpi: i386: Move VMBus DSDT entry to SB

2020-06-24 Thread Jon Doron
On 25/06/2020, Jon Doron wrote: On 23/06/2020, Igor Mammedov wrote: On Thu, 18 Jun 2020 06:00:27 +0300 Jon Doron wrote: Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi

Re: [PATCH v3 3/3] acpi: i386: Move VMBus DSDT entry to SB

2020-06-24 Thread Jon Doron
On 23/06/2020, Igor Mammedov wrote: On Thu, 18 Jun 2020 06:00:27 +0300 Jon Doron wrote: Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 6d9df38e31..38be9e5a58

Re: [PATCH v3 2/3] i386: acpi: vmbus: Add _ADR definition

2020-06-24 Thread Jon Doron
On 23/06/2020, Igor Mammedov wrote: On Thu, 18 Jun 2020 06:00:26 +0300 Jon Doron wrote: It seems like latest HyperV sets _ADR to 0 in the ACPI for the VMBS that's a hardly a good reason to add this. To me looks like a pointless addition, _ADR mostly is used when device resides on a bus

[PATCH v3 3/3] acpi: i386: Move VMBus DSDT entry to SB

2020-06-17 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 6d9df38e31..38be9e5a58 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -1092,7 +1092,6 @@ static

[PATCH v3 1/3] hyperv: vmbus: Remove the 2nd IRQ

2020-06-17 Thread Jon Doron
It seems like Windows does not really require 2 IRQs to have a functioning VMBus. Signed-off-by: Jon Doron --- hw/hyperv/vmbus.c| 3 +-- hw/i386/acpi-build.c | 4 +--- include/hw/hyperv/vmbus-bridge.h | 3 +-- 3 files changed, 3 insertions(+), 7 deletions(-) diff

[PATCH v3 2/3] i386: acpi: vmbus: Add _ADR definition

2020-06-17 Thread Jon Doron
It seems like latest HyperV sets _ADR to 0 in the ACPI for the VMBS Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 91af0d2d0d..6d9df38e31 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi

[PATCH v3 0/3] hyperv: vmbus: ACPI various corrections

2020-06-17 Thread Jon Doron
After doing further tests and looking at the latest HyperV ACPI DSDT. Do minor fix to our VMBus ACPI entry. v3: Removed accidental change for the dct submodule head v2: Renamed irq0 to irq now that there is a single IRQ required Jon Doron (3): hyperv: vmbus: Remove the 2nd IRQ i386: acpi

[PATCH v2 3/3] acpi: i386: Move VMBus DSDT entry to SB

2020-06-17 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 6d9df38e31..38be9e5a58 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -1092,7 +1092,6 @@ static

[PATCH v2 2/3] i386: acpi: vmbus: Add _ADR definition

2020-06-17 Thread Jon Doron
It seems like latest HyperV sets _ADR to 0 in the ACPI for the VMBS Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 91af0d2d0d..6d9df38e31 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi

[PATCH v2 1/3] hyperv: vmbus: Remove the 2nd IRQ

2020-06-17 Thread Jon Doron
It seems like Windows does not really require 2 IRQs to have a functioning VMBus. Signed-off-by: Jon Doron --- dtc | 2 +- hw/hyperv/vmbus.c| 3 +-- hw/i386/acpi-build.c | 4 +--- include/hw/hyperv/vmbus-bridge.h | 3 +-- 4 files changed

[PATCH v2 0/3] hyperv: vmbus: ACPI various corrections

2020-06-17 Thread Jon Doron
After doing further tests and looking at the latest HyperV ACPI DSDT. Do minor fix to our VMBus ACPI entry. v2: Renamed irq0 to irq now that there is a single IRQ required Jon Doron (3): hyperv: vmbus: Remove the 2nd IRQ i386: acpi: vmbus: Add _ADR definition acpi: i386: Move VMBus DSDT

[PATCH v1 3/3] acpi: i386: Move VMBus DSDT entry to SB

2020-06-17 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 508f5d9dd7..2c97358d5b 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi-build.c @@ -1092,7 +1092,6 @@ static

[PATCH v1 2/3] i386: acpi: vmbus: Add _ADR definition

2020-06-17 Thread Jon Doron
It seems like latest HyperV sets _ADR to 0 in the ACPI for the VMBS Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index dce680e97c..508f5d9dd7 100644 --- a/hw/i386/acpi-build.c +++ b/hw/i386/acpi

[PATCH v1 0/3] hyperv: vmbus: ACPI various corrections

2020-06-17 Thread Jon Doron
After doing further tests and looking at the latest HyperV ACPI DSDT. Do minor fix to our VMBus ACPI entry. Jon Doron (3): hyperv: vmbus: Remove the 2nd IRQ i386: acpi: vmbus: Add _ADR definition acpi: i386: Move VMBus DSDT entry to SB hw/hyperv/vmbus.c| 1 - hw/i386/acpi

[PATCH v1 1/3] hyperv: vmbus: Remove the 2nd IRQ

2020-06-17 Thread Jon Doron
It seems like Windows does not really require 2 IRQs to have a functioning VMBus. Signed-off-by: Jon Doron --- hw/hyperv/vmbus.c| 1 - hw/i386/acpi-build.c | 2 -- include/hw/hyperv/vmbus-bridge.h | 1 - 3 files changed, 4 deletions(-) diff --git a/hw/hyperv/vmbus.c

Re: [PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-06-14 Thread Jon Doron
On 14/06/2020, Maciej S. Szmigiero wrote: Hi Jon, On 14.06.2020 16:11, Jon Doron wrote: On 28/05/2020, Jon Doron wrote: On 28/05/2020, Igor Mammedov wrote: On Thu, 28 May 2020 08:26:42 +0300 Jon Doron wrote: On 22/05/2020, Igor Mammedow wrote: On Thu, 21 May 2020 18:02:07 +0200 Paolo

Re: [PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-06-14 Thread Jon Doron
On 14/06/2020, Jon Doron wrote: On 28/05/2020, Jon Doron wrote: On 28/05/2020, Igor Mammedov wrote: On Thu, 28 May 2020 08:26:42 +0300 Jon Doron wrote: On 22/05/2020, Igor Mammedow wrote: On Thu, 21 May 2020 18:02:07 +0200 Paolo Bonzini wrote: On 13/05/20 17:34, Igor Mammedov wrote

Re: [PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-06-14 Thread Jon Doron
On 28/05/2020, Jon Doron wrote: On 28/05/2020, Igor Mammedov wrote: On Thu, 28 May 2020 08:26:42 +0300 Jon Doron wrote: On 22/05/2020, Igor Mammedow wrote: On Thu, 21 May 2020 18:02:07 +0200 Paolo Bonzini wrote: On 13/05/20 17:34, Igor Mammedov wrote: > I'd rather avoid using random

[v1] docs: Add to gdbstub documentation the PhyMemMode

2020-06-01 Thread Jon Doron
The PhyMemMode gdb extension command was missing from the gdb.rst document. Signed-off-by: Jon Doron --- docs/system/gdb.rst | 20 1 file changed, 20 insertions(+) diff --git a/docs/system/gdb.rst b/docs/system/gdb.rst index a40145fcf8..abda961e2b 100644 --- a/docs/system

Re: [PATCH-for-5.0] gdbstub: Use correct address space with Qqemu.PhyMemMode packet

2020-06-01 Thread Jon Doron
On 01/06/2020, Alex Bennée wrote: Jon Doron writes: On 31/05/2020, Peter Maydell wrote: On Sun, 31 May 2020 at 17:42, Jon Doron wrote: On 31/05/2020, Philippe Mathieu-Daudé wrote: >On 3/30/20 6:41 PM, Peter Maydell wrote: >> PS: do we have any documentation of this ne

Re: [PATCH-for-5.0] gdbstub: Use correct address space with Qqemu.PhyMemMode packet

2020-06-01 Thread Jon Doron
On 31/05/2020, Peter Maydell wrote: On Sun, 31 May 2020 at 17:42, Jon Doron wrote: On 31/05/2020, Philippe Mathieu-Daudé wrote: >On 3/30/20 6:41 PM, Peter Maydell wrote: >> PS: do we have any documentation of this new command ? >> ab4752ec8d9 has the implementation but n

Re: [PATCH-for-5.0] gdbstub: Use correct address space with Qqemu.PhyMemMode packet

2020-05-31 Thread Jon Doron
On 31/05/2020, Philippe Mathieu-Daudé wrote: On 3/30/20 6:41 PM, Peter Maydell wrote: On Mon, 30 Mar 2020 at 17:21, Philippe Mathieu-Daudé wrote: On 3/30/20 6:08 PM, Peter Maydell wrote: On Mon, 30 Mar 2020 at 16:30, Philippe Mathieu-Daudé wrote: Since commit 3f940dc98, we added support

Re: [PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-05-28 Thread Jon Doron
On 28/05/2020, Igor Mammedov wrote: On Thu, 28 May 2020 08:26:42 +0300 Jon Doron wrote: On 22/05/2020, Igor Mammedow wrote: >On Thu, 21 May 2020 18:02:07 +0200 >Paolo Bonzini wrote: > >> On 13/05/20 17:34, Igor Mammedov wrote: >> > I'd rather avoid using random IRQ

Re: [PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-05-27 Thread Jon Doron
On 28/05/2020, Jon Doron wrote: On 22/05/2020, Igor Mammedow wrote: On Thu, 21 May 2020 18:02:07 +0200 Paolo Bonzini wrote: On 13/05/20 17:34, Igor Mammedov wrote: I'd rather avoid using random IRQ numbers (considering we are dealing with black-box here). So if it's really necessary to have

Re: [PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-05-27 Thread Jon Doron
On 22/05/2020, Igor Mammedow wrote: On Thu, 21 May 2020 18:02:07 +0200 Paolo Bonzini wrote: On 13/05/20 17:34, Igor Mammedov wrote: > I'd rather avoid using random IRQ numbers (considering we are > dealing with black-box here). So if it's really necessary to have > IRQ described here, I'd

Re: [PATCH v4 0/6] hyperv: VMBus implementation

2020-05-21 Thread Jon Doron
On 21/05/2020, Paolo Bonzini wrote: On 24/04/20 14:34, Jon Doron wrote: This is a rebase of the old patchset from Roman for HyperV VMBus implementation. How to use: -device vmbus-bridge Later on new paravirtualized devices can be implemented on top of it (Network/SCSI/etc.) VMBus

Re: [PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-05-15 Thread Jon Doron
? Do we need a full emulation setup (aka your hv-scsi / hv-net) patches in order to see Windows working this ACPI entry? Thanks, -- Jon. On Mon, 11 May 2020 23:11:23 +0300 Roman Kagan wrote: On Thu, May 07, 2020 at 06:14:25AM +0300, Jon Doron wrote: > Igor it seems like the IRQ being u

Re: [PATCH v1 5/5] i386: Hyper-V VMBus ACPI DSDT entry

2020-05-12 Thread Jon Doron
On 12/05/2020, Jon Doron wrote: On 12/05/2020, Roman Kagan wrote: On Thu, Apr 09, 2020 at 06:35:18AM +0300, Jon Doron wrote: On 08/04/2020, Roman Kagan wrote: On Wed, Apr 08, 2020 at 07:16:39AM +0300, Jon Doron wrote: > Well I have implemented the hyperv synthetic kernel debugger interf

Re: [PATCH v1 5/5] i386: Hyper-V VMBus ACPI DSDT entry

2020-05-12 Thread Jon Doron
On 12/05/2020, Roman Kagan wrote: On Thu, Apr 09, 2020 at 06:35:18AM +0300, Jon Doron wrote: On 08/04/2020, Roman Kagan wrote: > On Wed, Apr 08, 2020 at 07:16:39AM +0300, Jon Doron wrote: > > Well I have implemented the hyperv synthetic kernel debugger interface, but > >

Re: [PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-05-06 Thread Jon Doron
know if there is anything else. Thanks, -- Jon. On 06/05/2020, Maciej S. Szmigiero wrote: On 05.05.2020 17:38, Jon Doron wrote: On 05/05/2020, Igor Mammedov wrote: I dont know what were the original intentions of the original patch authors (at this point I simply rebased it, and to be honest I did not

Re: [PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-05-05 Thread Jon Doron
changes so we can keep going forward with this. On Fri, 24 Apr 2020 15:34:43 +0300 Jon Doron wrote: Guest OS uses ACPI to discover VMBus presence. Add a corresponding entry to DSDT in case VMBus has been enabled. Experimentally Windows guests were found to require this entry to include two

[PATCH v4 6/6] vmbus: add infrastructure to save/load vmbus requests

2020-04-24 Thread Jon Doron
This can be allow to include controller-specific data while saving/loading in-flight scsi requests of the vmbus scsi controller. Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- hw/hyperv/vmbus.c | 99

[PATCH v4 2/6] vmbus: add vmbus protocol definitions

2020-04-24 Thread Jon Doron
Add a header with data structures and constants used in Hyper-V VMBus hypervisor <-> guest interactions. Based on the respective stuff from Linux kernel. Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- include/hw/hyperv/vmbus-proto.h

[PATCH v4 3/6] vmbus: vmbus implementation

2020-04-24 Thread Jon Doron
Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- Makefile.objs|1 + hw/hyperv/Kconfig|5 + hw/hyperv/Makefile.objs |1 + hw/hyperv/trace-events | 18 + hw/hyperv/vmbus.c| 2672

[PATCH v4 4/6] i386:pc: whitelist dynamic vmbus-bridge

2020-04-24 Thread Jon Doron
As vmbus-bridge is derived from sysbus device, it has to be whitelisted to be allowed to be created with -device. Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- hw/i386/pc_piix.c | 2 ++ hw/i386/pc_q35.c | 2 ++ 2 files changed, 4 insertions

[PATCH v4 5/6] i386: Hyper-V VMBus ACPI DSDT entry

2020-04-24 Thread Jon Doron
-configurable via corresponding properties; use 7 and 13 by default. Signed-off-by: Evgeny Yakovlev Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- hw/hyperv/vmbus.c| 7 ++ hw/i386/acpi-build.c | 43

[PATCH v4 1/6] hyperv: expose API to determine if synic is enabled

2020-04-24 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/hyperv/hyperv.c | 8 include/hw/hyperv/hyperv.h | 1 + 2 files changed, 9 insertions(+) diff --git a/hw/hyperv/hyperv.c b/hw/hyperv/hyperv.c index 8ca3706f5b..ddf4f32c60 100644 --- a/hw/hyperv/hyperv.c +++ b/hw/hyperv/hyperv.c @@ -37,6 +37,13

[PATCH v4 0/6] hyperv: VMBus implementation

2020-04-24 Thread Jon Doron
of latest patchset from Roman and Maciej Jon Doron (6): hyperv: expose API to determine if synic is enabled vmbus: add vmbus protocol definitions vmbus: vmbus implementation i386:pc: whitelist dynamic vmbus-bridge i386: Hyper-V VMBus ACPI DSDT entry vmbus: add infrastructure to save/load

Re: [PATCH v1 5/5] i386: Hyper-V VMBus ACPI DSDT entry

2020-04-08 Thread Jon Doron
On 08/04/2020, Roman Kagan wrote: On Wed, Apr 08, 2020 at 07:16:39AM +0300, Jon Doron wrote: On 07/04/2020, Maciej S. Szmigiero wrote: > On 07.04.2020 20:56, Roman Kagan wrote: > > On Mon, Apr 06, 2020 at 11:20:39AM +0300, Jon Doron wrote: > > > Well I want it to be merged

Re: [PATCH v1 5/5] i386: Hyper-V VMBus ACPI DSDT entry

2020-04-07 Thread Jon Doron
On 07/04/2020, Maciej S. Szmigiero wrote: On 07.04.2020 20:56, Roman Kagan wrote: On Mon, Apr 06, 2020 at 11:20:39AM +0300, Jon Doron wrote: Well I want it to be merged in :-) Hmm I'm curious why, it has little to offer over virtio. Anyway the series you've posted seems to be based

[PATCH v3 4/7] vmbus: vmbus implementation

2020-04-06 Thread Jon Doron
Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- Makefile.objs|1 + hw/hyperv/Kconfig|5 + hw/hyperv/Makefile.objs |1 + hw/hyperv/trace-events | 18 + hw/hyperv/vmbus.c| 2672

[PATCH v3 6/7] i386: Hyper-V VMBus ACPI DSDT entry

2020-04-06 Thread Jon Doron
-configurable via corresponding properties; use 7 and 13 by default. Signed-off-by: Evgeny Yakovlev Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- hw/hyperv/vmbus.c| 7 ++ hw/i386/acpi-build.c | 43

[PATCH v3 5/7] i386:pc: whitelist dynamic vmbus-bridge

2020-04-06 Thread Jon Doron
As vmbus-bridge is derived from sysbus device, it has to be whitelisted to be allowed to be created with -device. Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- hw/i386/pc_piix.c | 2 ++ hw/i386/pc_q35.c | 2 ++ 2 files changed, 4 insertions

[PATCH v3 2/7] hyperv: SControl is optional to enable SynIc

2020-04-06 Thread Jon Doron
with the pending flag. When the guest polls from the message slot, incase the pending flag is set it will write to the HV_X64_MSR_EOM indicating it has cleared the slow and we can try and push our message again. Signed-off-by: Jon Doron --- hw/hyperv/hyperv.c | 242

[PATCH v3 7/7] vmbus: add infrastructure to save/load vmbus requests

2020-04-06 Thread Jon Doron
This can be allow to include controller-specific data while saving/loading in-flight scsi requests of the vmbus scsi controller. Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- hw/hyperv/vmbus.c | 99

[PATCH v3 3/7] vmbus: add vmbus protocol definitions

2020-04-06 Thread Jon Doron
Add a header with data structures and constants used in Hyper-V VMBus hypervisor <-> guest interactions. Based on the respective stuff from Linux kernel. Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- include/hw/hyperv/vmbus-proto.h

[PATCH v3 1/7] hyperv: expose API to determine if synic is enabled

2020-04-06 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/hyperv/hyperv.c | 8 include/hw/hyperv/hyperv.h | 1 + 2 files changed, 9 insertions(+) diff --git a/hw/hyperv/hyperv.c b/hw/hyperv/hyperv.c index 8ca3706f5b..ddf4f32c60 100644 --- a/hw/hyperv/hyperv.c +++ b/hw/hyperv/hyperv.c @@ -37,6 +37,13

[PATCH v3 0/7] hyperv: VMBus implementation

2020-04-06 Thread Jon Doron
this in the guest. More information about VMBus can be found in the file vmbuskernelmodeclientlibapi.h in Microsoft's WDK. v3: Fixed an error asan v2: Rebased on top of latest patchset from Roman and Maciej Jon Doron (7): hyperv: expose API to determine if synic is enabled hyperv: SControl

Re: [PATCH v1 5/5] i386: Hyper-V VMBus ACPI DSDT entry

2020-04-06 Thread Jon Doron
's "Signed-off-by:" > tags, > > > so I haven't applied mine, either. > > > > > > If you are able to establish a proper SoB chain then please also add: > > > "Signed-off-by: Maciej S. Szmigiero ". > > > > > > Tha

[PATCH v2 7/7] vmbus: add infrastructure to save/load vmbus requests

2020-04-04 Thread Jon Doron
This can be allow to include controller-specific data while saving/loading in-flight scsi requests of the vmbus scsi controller. Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- hw/hyperv/vmbus.c | 99

[PATCH v2 6/7] i386: Hyper-V VMBus ACPI DSDT entry

2020-04-04 Thread Jon Doron
-configurable via corresponding properties; use 7 and 13 by default. Signed-off-by: Evgeny Yakovlev Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- hw/hyperv/vmbus.c| 7 ++ hw/i386/acpi-build.c | 43

[PATCH v2 2/7] hyperv: SControl is optional to enable SynIc

2020-04-04 Thread Jon Doron
with the pending flag. When the guest polls from the message slot, incase the pending flag is set it will write to the HV_X64_MSR_EOM indicating it has cleared the slow and we can try and push our message again. Signed-off-by: Jon Doron --- hw/hyperv/hyperv.c | 242

[PATCH v2 5/7] i386:pc: whitelist dynamic vmbus-bridge

2020-04-04 Thread Jon Doron
As vmbus-bridge is derived from sysbus device, it has to be whitelisted to be allowed to be created with -device. Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- hw/i386/pc_piix.c | 2 ++ hw/i386/pc_q35.c | 2 ++ 2 files changed, 4 insertions

[PATCH v2 3/7] vmbus: add vmbus protocol definitions

2020-04-04 Thread Jon Doron
Add a header with data structures and constants used in Hyper-V VMBus hypervisor <-> guest interactions. Based on the respective stuff from Linux kernel. Signed-off-by: Roman Kagan Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- include/hw/hyperv/vmbus-proto.h

[PATCH v2 4/7] vmbus: vmbus implementation

2020-04-04 Thread Jon Doron
Signed-off-by: Maciej S. Szmigiero Signed-off-by: Jon Doron --- Makefile.objs|1 + hw/hyperv/Kconfig|5 + hw/hyperv/Makefile.objs |1 + hw/hyperv/trace-events | 18 + hw/hyperv/vmbus.c| 2672

[PATCH v2 0/7] hyperv: VMBus implementation

2020-04-04 Thread Jon Doron
this in the guest. More information about VMBus can be found in the file vmbuskernelmodeclientlibapi.h in Microsoft's WDK. Jon Doron (7): hyperv: expose API to determine if synic is enabled hyperv: SControl is optional to enable SynIc vmbus: add vmbus protocol definitions vmbus: vmbus

[PATCH v2 1/7] hyperv: expose API to determine if synic is enabled

2020-04-04 Thread Jon Doron
Signed-off-by: Jon Doron --- hw/hyperv/hyperv.c | 8 include/hw/hyperv/hyperv.h | 1 + 2 files changed, 9 insertions(+) diff --git a/hw/hyperv/hyperv.c b/hw/hyperv/hyperv.c index 8ca3706f5b..ddf4f32c60 100644 --- a/hw/hyperv/hyperv.c +++ b/hw/hyperv/hyperv.c @@ -37,6 +37,13

Re: [PATCH v1 5/5] i386: Hyper-V VMBus ACPI DSDT entry

2020-04-03 Thread Jon Doron
Thank you Maciej it seems like your version is really ahead I'll do the required work and merge it so i can submit a v2 with the latest patchset from Roman On Fri, Apr 3, 2020 at 6:06 PM Jon Doron wrote: > > Thank you Maciej, I based it on top of what Denis (d...@openvz.org) gave me &

Re: [PATCH v1 5/5] i386: Hyper-V VMBus ACPI DSDT entry

2020-04-03 Thread Jon Doron
version of the patchset? Thanks, -- Jon. On Fri, Apr 3, 2020 at 5:56 PM Maciej S. Szmigiero wrote: > > Hi Jon, > > On 03.04.2020 16:23, Jon Doron wrote: > > Guest OS uses ACPI to discover vmbus presence. Add a corresponding > > entry to DSDT in case vmbus has been enabled

[PATCH v1 4/5] vmbus: vmbus implementation

2020-04-03 Thread Jon Doron
Signed-off-by: Jon Doron --- Makefile.objs|1 + hw/Kconfig |1 + hw/Makefile.objs |1 + hw/i386/Kconfig |1 + hw/i386/pc_q35.c |2 + hw/vmbus/Kconfig |3 + hw/vmbus/Makefile.objs |1 + hw/vmbus/trace-events

[PATCH v1 5/5] i386: Hyper-V VMBus ACPI DSDT entry

2020-04-03 Thread Jon Doron
, in particular, in parallel port conflicting with vmbus. TODO: discover and use spare IRQs to avoid conflicts. Signed-off-by: Evgeny Yakovlev Signed-off-by: Roman Kagan Signed-off-by: Jon Doron --- hw/i386/acpi-build.c | 42 ++ 1 file changed, 42 insertions(+) diff

[PATCH v1 2/5] hyperv: SControl is optional to enable SynIc

2020-04-03 Thread Jon Doron
with the pending flag. When the guest polls from the message slot, incase the pending flag is set it will write to the HV_X64_MSR_EOM indicating it has cleared the slow and we can try and push our message again. Signed-off-by: Jon Doron --- hw/hyperv/hyperv.c | 242

[PATCH v1 0/5] hyperv: VMBus implementation

2020-04-03 Thread Jon Doron
This is a rebase of the old patchset from Roman for HyperV VMBus implementation. How to use: -device vmbus-bridge Later on new paravirtualized devices can be implemented on top of it (Network/SCSI/etc.) Jon Doron (5): hyperv: expose API to determine if synic is enabled hyperv: SControl

  1   2   3   4   >