Re: [Qemu-devel] [PATCH v7 3/3] i386: populate floppy drive information in DSDT

2016-02-07 Thread Michael S. Tsirkin
On Tue, Jan 26, 2016 at 02:50:25PM +0100, Igor Mammedov wrote: > From: Roman Kagan > > On x86-based systems Linux determines the presence > and the type of floppy drives via a query of a CMOS field. > So does SeaBIOS when populating the return data for > int 0x13 function 0x08. > > However Windo

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-07 Thread Michael S. Tsirkin
On Fri, Feb 05, 2016 at 09:55:08AM +0100, Markus Armbruster wrote: > "Michael S. Tsirkin" writes: > > > On Thu, Feb 04, 2016 at 03:16:17PM -0200, Eduardo Habkost wrote: > >> On Thu, Feb 04, 2016 at 06:01:50PM +0200, Michael S. Tsirkin wrote: > >> > On Sat, Jan 23, 2016 at 02:02:08PM -0200, Eduard

Re: [Qemu-devel] [PATCH RFC 0/3] linux-user: netlink support

2016-02-07 Thread Laurent Vivier
Le 30/01/2016 23:26, Laurent Vivier a écrit : > Since commit: > e36800c linux-user: add signalfd/signalfd4 syscalls > > It is now possible to register handlers to a file descriptor > to translate a data stream transiting by this file descriptor. > > We can now decode netlink information coming

[Qemu-devel] [PATCH 0/4] pc: drop AML files

2016-02-07 Thread Michael S. Tsirkin
This is a lightly tested patchset that shows how it's possible to clean out old bits without breaking any user configs. It's quite a lot of bits, too, since we are dealing with binaries here. Will test later if this is received favorably. As a bonus, seabios will be able to drop q35-acpi-dsdt.dsl

[Qemu-devel] [PATCH 2/4] pc/q35: drop compatiility with old bios types

2016-02-07 Thread Michael S. Tsirkin
We don't ship bios without support for loading ACPI anymore, and old q35 machine types don't allow migrating such bios. Drop it, as well as has_*mr flags that are there for migration compatibility. Signed-off-by: Michael S. Tsirkin --- hw/i386/pc_q35.c | 8 1 file changed, 8 deletions(

[Qemu-devel] [PATCH 1/4] pc/piix: drop acpi-dsdt.aml

2016-02-07 Thread Michael S. Tsirkin
commit 9fb7aaaf4c58c9108327f0ae4766087e3e496b47 ("pc: drop external DSDT loading") removes the only use of acpi-dsdt.aml. It's safe to drop it from QEMU source. Signed-off-by: Michael S. Tsirkin --- Makefile | 2 +- pc-bios/acpi-dsdt.aml | Bin 4405 -> 0 bytes 2 files changed, 1 i

[Qemu-devel] [PATCH 3/4] pc/q35: drop dsdt aml

2016-02-07 Thread Michael S. Tsirkin
DSDT AML binary is now unused, drop it. Signed-off-by: Michael S. Tsirkin --- Makefile | 1 - pc-bios/q35-acpi-dsdt.aml | Bin 7344 -> 0 bytes 2 files changed, 1 deletion(-) delete mode 100644 pc-bios/q35-acpi-dsdt.aml diff --git a/Makefile b/Makefile index fa36a48..bc9444a

[Qemu-devel] [PATCH 4/4] configure: don't install aml links

2016-02-07 Thread Michael S. Tsirkin
There aren't any left. Signed-off-by: Michael S. Tsirkin --- configure | 1 - 1 file changed, 1 deletion(-) diff --git a/configure b/configure index d4411a1..06cb815 100755 --- a/configure +++ b/configure @@ -5967,7 +5967,6 @@ FILES="$FILES roms/seabios/Makefile roms/vgabios/Makefile" FILES="$

[Qemu-devel] command line error handling broken?

2016-02-07 Thread Michael S. Tsirkin
./x86_64-softmmu/qemu-system-x86_64 -M q35-1.5 -redir tcp:8022::22 qemu-system-x86_64: -redir tcp:8022::22: unsupported machine type Use -machine help to list supported machines It should say q35-1.5 is unsupported, of course. Anyone knows what's going on? -- MST

Re: [Qemu-devel] [PATCH v12 1/2] mirror: Rewrite mirror_iteration

2016-02-07 Thread Fam Zheng
On Sat, 02/06 14:24, Max Reitz wrote: > On 05.02.2016 03:00, Fam Zheng wrote: > > The "pnum < nb_sectors" condition in deciding whether to actually copy > > data is unnecessarily strict, and the qiov initialization is > > unnecessarily for bdrv_aio_write_zeroes and bdrv_aio_discard. > > > > Rewrit

Re: [Qemu-devel] [PATCH] bcm2835_property: implement "get board revision" query

2016-02-07 Thread Peter Maydell
Hi; you forgot to cc qemu-devel (all patches must be cc'd there). thanks -- PMM On 7 February 2016 at 05:09, Stephen Warren wrote: > Return a valid value from the BCM2835 property mailbox query "get board > revision". This query is used by U-Boot. Implementing it fixes the first > obvious differ

Re: [Qemu-devel] [PATCH RFC 0/3] linux-user: netlink support

2016-02-07 Thread Peter Maydell
On 7 February 2016 at 10:24, Laurent Vivier wrote: > This is in fact a bug in TCG interpreter which not supports "Multiply > and add" instructions (in this case: "evmheumiaaw"). PPC instruction mnemonics never cease to amuse me. (On which note somebody just pointed me at https://twitter.com/ppcin

Re: [Qemu-devel] [PATCH v8 2/4] kobject: export kset_find_obj() for module use

2016-02-07 Thread Gabriel L. Somlo
On Sat, Feb 06, 2016 at 11:24:23PM -0800, Greg KH wrote: > On Thu, Jan 28, 2016 at 09:23:12AM -0500, Gabriel L. Somlo wrote: > > From: Gabriel Somlo > > > > Signed-off-by: Gabriel Somlo > > --- > > lib/kobject.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/lib/kobject.c b/lib

Re: [Qemu-devel] [PATCH v5] qom, qmp, hmp, qapi: create qom-type-prop-list for class properties

2016-02-07 Thread Valentin Rakush
Hi Eduardo, thank you for your explanations. > You don't have to, if you just do object_new() like > qmp_device_list_properties() does. Both ObjectClass::properties > and DeviceClas::props are translated to object instance > properties (Object::properties). I should foresee these. Qemu has the o

[Qemu-devel] [PATCH] vl.c: fixed regression of machine error message

2016-02-07 Thread Marcel Apfelbaum
Commit e1ce0c3cb(vl.c: fix regression when reading machine type from config file) fixed the error message when the machine type was supplied inside the config file. However now the option name is not displayed correctly if the error happens when the machine is specified at command line. Running

Re: [Qemu-devel] command line error handling broken?

2016-02-07 Thread Marcel Apfelbaum
On 02/07/2016 12:32 PM, Michael S. Tsirkin wrote: ./x86_64-softmmu/qemu-system-x86_64 -M q35-1.5 -redir tcp:8022::22 qemu-system-x86_64: -redir tcp:8022::22: unsupported machine type Use -machine help to list supported machines It should say q35-1.5 is unsupported, of course. Anyone knows what

[Qemu-devel] [PATCH] MAINTAINERS: add all-match entry for qemu-devel@

2016-02-07 Thread Stephen Warren
Add an entry to MAINTAINERS that matches every patch, and requests the user send patches to qemu-devel@nongnu.org. It's not 100% obvious to project newcomers that all patches should be sent there; checkpatch doesn't say so, and since it mentions other lists to CC, the wording "the list" from the S

[Qemu-devel] Serial telnet console drops incoming characters on git master

2016-02-07 Thread Mark Cave-Ayland
Hi Daniel, I found another issue where the latest set of console updates appear to break -serial telnet on qemu-system-sparc. What I see is that while incoming serial telnet appears fine, outgoing serial telnet traffic caused by typing over the serial connection results in every other character be

Re: [Qemu-devel] [PATCH 0/6] Add i.MX6 (Single/Dual/Quad) support

2016-02-07 Thread Jean-Christophe DUBOIS
Le 02/02/2016 18:01, Peter Maydell a écrit : On 28 January 2016 at 19:22, Jean-Christophe DUBOIS wrote: Sabrelite uses uart2 for console but on qemu, this is uart1 that is wired to the default stdout. Consequently, I changed stdout-path in "chosen" to uart1. This I find a bit awkward: we'd li

Re: [Qemu-devel] [PATCH 04/13] cuda: port SET_AUTO_RATE command to new framework

2016-02-07 Thread Mark Cave-Ayland
On 23/01/16 20:40, Hervé Poussineau wrote: > Take requested autopoll rate into account > > Signed-off-by: Hervé Poussineau > --- > hw/misc/macio/cuda.c | 31 +++ > hw/ppc/mac.h | 1 + > 2 files changed, 28 insertions(+), 4 deletions(-) > > diff --git a/hw/m

Re: [Qemu-devel] [PATCH 05/13] cuda: port SET_DEVICE_LIST command to new framework

2016-02-07 Thread Mark Cave-Ayland
On 23/01/16 20:40, Hervé Poussineau wrote: > Take device list mask into account when polling ADB devices. > > Signed-off-by: Hervé Poussineau > --- > hw/input/adb.c | 18 ++ > hw/misc/macio/cuda.c | 17 +++-- > hw/ppc/mac.h | 1 + > include/hw/inp

Re: [Qemu-devel] [PATCH 00/13] cuda: misc fixes and cleanups

2016-02-07 Thread Mark Cave-Ayland
On 06/02/16 14:30, Hervé Poussineau wrote: > Hi Mark, > > Le 06/02/2016 14:32, Mark Cave-Ayland a écrit : >> On 23/01/16 20:39, Hervé Poussineau wrote: >> >>> Hi, >>> >>> This patchset cleans up a little bit the Apple CUDA emulation: >>> - correctly reject unknown commands >>> - correctly reject

[Qemu-devel] [PATCH v2 07/13] cuda: port RESET_SYSTEM command to new framework

2016-02-07 Thread Hervé Poussineau
Reviewed-by: David Gibson Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index 9948e18..ca11fc8 100644 --- a/hw/misc/macio/cuda.c +++ b/hw/misc/macio/cuda.

[Qemu-devel] [PATCH v2 10/13] cuda: port GET_TIME command to new framework

2016-02-07 Thread Hervé Poussineau
Reviewed-by: David Gibson Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 29 + 1 file changed, 21 insertions(+), 8 deletions(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index 16f9ad6..8ee1414 100644 --- a/hw/misc/macio/cuda.c +++ b/hw/misc

[Qemu-devel] [PATCH v2 12/13] cuda: remove GET_6805_ADDR command

2016-02-07 Thread Hervé Poussineau
It doesn't seem to be used, and operating systems should accept a 'unknown command' answer. Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index 611b414..d65a36a 100644 --- a/hw/misc/maci

[Qemu-devel] [PATCH v2 08/13] cuda: port FILE_SERVER_FLAG command to new framework

2016-02-07 Thread Hervé Poussineau
This command tells if computer should automatically wake-up after a power loss. Reviewed-by: David Gibson Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c in

[Qemu-devel] [PATCH v2 09/13] cuda: port SET_POWER_MESSAGES command to new framework

2016-02-07 Thread Hervé Poussineau
Reviewed-by: David Gibson Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index 4ddec34..16f9ad6 100644 --- a/hw/misc/macio/cuda.c +++ b/hw/misc/macio/cud

[Qemu-devel] [PATCH v2 01/13] cuda: add a framework to handle commands

2016-02-07 Thread Hervé Poussineau
Next commits will port existing CUDA commands to this framework. Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 34 ++ 1 file changed, 34 insertions(+) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index 316c1ac..8659dc3 100644 --- a/hw/mis

[Qemu-devel] [PATCH v2 02/13] cuda: move unknown commands reject out of switch

2016-02-07 Thread Hervé Poussineau
Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 30 -- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index 8659dc3..81e34e7 100644 --- a/hw/misc/macio/cuda.c +++ b/hw/misc/macio/cuda.c @@ -590,15

[Qemu-devel] [PATCH v2 11/13] cuda: port SET_TIME command to new framework

2016-02-07 Thread Hervé Poussineau
Reviewed-by: David Gibson Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index 8ee1414..611b414 100644 --- a/hw/misc/macio/cuda.c +++ b/hw/misc/maci

[Qemu-devel] [PATCH v2 04/13] cuda: port SET_AUTO_RATE command to new framework

2016-02-07 Thread Hervé Poussineau
Also implement the command, by removing the hardcoded period of 20 ms/50 Hz and replacing it by the one requested by user. Update VMState version to store this new parameter. Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 36 ++-- hw/ppc/mac.h

[Qemu-devel] [PATCH v2 13/13] cuda: remove CUDA_GET_SET_IIC/CUDA_COMBINED_FORMAT_IIC commands

2016-02-07 Thread Hervé Poussineau
We currently don't emulate the I2C bus provided by CUDA. Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 23 --- 1 file changed, 23 deletions(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index d65a36a..481abdb 100644 --- a/hw/misc/macio/cuda.c +++ b

Re: [Qemu-devel] [PATCH 00/13] cuda: misc fixes and cleanups

2016-02-07 Thread Hervé Poussineau
Le 07/02/2016 20:50, Mark Cave-Ayland a écrit : On 06/02/16 14:30, Hervé Poussineau wrote: Hi Mark, Le 06/02/2016 14:32, Mark Cave-Ayland a écrit : On 23/01/16 20:39, Hervé Poussineau wrote: Hi, This patchset cleans up a little bit the Apple CUDA emulation: - correctly reject unknown comma

[Qemu-devel] [PATCH v2 06/13] cuda: port POWERDOWN command to new framework

2016-02-07 Thread Hervé Poussineau
Reviewed-by: David Gibson Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index a2e31d0..9948e18 100644 --- a/hw/misc/macio/cuda.c +++ b/hw/misc/macio/cuda.

[Qemu-devel] [PATCH v2 05/13] cuda: port SET_DEVICE_LIST command to new framework

2016-02-07 Thread Hervé Poussineau
Also implement the command, by taking device list mask into account when polling ADB devices. Signed-off-by: Hervé Poussineau --- hw/input/adb.c | 18 ++ hw/misc/macio/cuda.c | 18 -- hw/ppc/mac.h | 1 + include/hw/input/adb.h | 2 +- 4 files

[Qemu-devel] [PATCH v2 03/13] cuda: port AUTOPOLL command to new framework

2016-02-07 Thread Hervé Poussineau
Reviewed-by: David Gibson Signed-off-by: Hervé Poussineau --- hw/misc/macio/cuda.c | 40 +--- 1 file changed, 25 insertions(+), 15 deletions(-) diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c index 81e34e7..ea4237a 100644 --- a/hw/misc/macio/cuda.c +

[Qemu-devel] [PATCH v2 00/13] cuda: misc fixes and cleanups

2016-02-07 Thread Hervé Poussineau
Hi, This patchset cleans up a little bit the Apple CUDA emulation: - correctly reject commands with wrong parameters - support changing the frequency of auto-polling - support changing device list probed in auto-poll - add logs when using FILE_SERVER_FLAG/SET_POWER_MESSAGE - remove unused commands

Re: [Qemu-devel] Using directory as initrd

2016-02-07 Thread Kasper Dupont
On 18/01/16 10.45, KONRAD Frederic wrote: > Hi Kasper, > > The normal approach is to use git send email to send your patch on the list > instead of attaching the patch to the email so people can do inline comment. > > You can use scripts/checkpatch.pl to check the coding style before sending. > (

Re: [Qemu-devel] [PATCH v3 2/3] hw/sd: model a power-up delay, as a workaround for an EDK2 bug

2016-02-07 Thread Andrew Baumann
Hi Peter, > From: Peter Maydell [mailto:peter.mayd...@linaro.org] > Sent: Thursday, 4 February 2016 1:22 AM > > On 1 February 2016 at 22:15, Andrew Baumann > wrote: > > The SD spec for ACMD41 says that a zero argument is an "inquiry" > > ACMD41, which does not start initialisation and is used on

Re: [Qemu-devel] [PATCH 3/6] i.MX: Add i.MX6 CCM and ANALOG device.

2016-02-07 Thread Jean-Christophe DUBOIS
Le 02/02/2016 17:41, Peter Maydell a écrit : On 26 January 2016 at 21:45, Jean-Christophe Dubois wrote: Signed-off-by: Jean-Christophe Dubois +static uint32_t imx6_analog_get_pll2_clk(IMX6CCMState *dev) +{ +uint32_t freq = 2400; + +if (EXTRACT(dev->analog[CCM_ANALOG_PLL_SYS], DIV_

Re: [Qemu-devel] [PATCH 3/6] i.MX: Add i.MX6 CCM and ANALOG device.

2016-02-07 Thread Peter Maydell
On 7 February 2016 at 23:28, Jean-Christophe DUBOIS wrote: >> Consider doing this with a set of subregions inside a container, >> rather than an if-elseif chain. > > > Could you point me to an example device using such technique? hw/cpu/a9mpcore.c sets up a bunch of subregions. You don't need t

Re: [Qemu-devel] [PATCHv3 2/4] Split serial-isa into its own config option

2016-02-07 Thread David Gibson
On Wed, Feb 03, 2016 at 11:00:51AM +0100, Markus Armbruster wrote: > David Gibson writes: > > > At present, the core device model code for 8250-like serial ports > > (serial.c) and the code for serial ports attached to ISA-style legacy IO > > (serial-isa.c) are both controlled by the CONFIG_SERIA

Re: [Qemu-devel] [PATCHv3 3/4] Allow ISA bus to be configured out

2016-02-07 Thread David Gibson
On Wed, Feb 03, 2016 at 11:06:16AM +0100, Markus Armbruster wrote: > David Gibson writes: > > > Currently, the code to handle the legacy ISA bus is always included in > > qemu. However there are lots of platforms that don't include ISA legacy > > devies, and quite a few that have never used ISA

Re: [Qemu-devel] [PATCH] hw/intc: fix failure return for xics_alloc_block()

2016-02-07 Thread David Gibson
On Fri, Feb 05, 2016 at 09:43:40AM +0100, Greg Kurz wrote: > From: Brian W. Hart > > xics_alloc_block() does not return a clear error code when it > fails to allocate a block of interrupts. Instead it returns the > base interrupt number minus 1. This change updates it to return a > clear -1 in ca

[Qemu-devel] [PATCH v4 0/3] SD emulation fixes for Pi2 Tianocore EDK2 UEFI

2016-02-07 Thread Andrew Baumann
This series contains fixes to the SD card emulation that are needed to unblock Tianocore EDK2 UEFI (specifically, the bootloader for Windows on Raspberry Pi 2). Changes in v2, based on feedback from Peter Crosthwaite: * correct implementation of CMD23 to switch to transfer state on completion *

[Qemu-devel] [PATCH v4 1/3] hw/sd: implement CMD23 (SET_BLOCK_COUNT) for MMC compatibility

2016-02-07 Thread Andrew Baumann
CMD23 is optional for SD but required for MMC, and the UEFI bootloader used for Windows on Raspberry Pi 2 issues it. Reviewed-by: Peter Crosthwaite Signed-off-by: Andrew Baumann --- hw/sd/sd.c | 37 + 1 file changed, 37 insertions(+) diff --git a/hw/sd/sd.c

[Qemu-devel] [PATCH v4 3/3] hw/sd: use guest error logging rather than fprintf to stderr

2016-02-07 Thread Andrew Baumann
Some of these errors may be harmless (e.g. probing unimplemented commands, or issuing CMD12 in the wrong state), and may also be quite frequent. Spamming the standard error output isn't desirable in such cases. Reviewed-by: Peter Crosthwaite Signed-off-by: Andrew Baumann --- It might also be des

[Qemu-devel] [PATCH v4 2/3] hw/sd: model a power-up delay, as a workaround for an EDK2 bug

2016-02-07 Thread Andrew Baumann
The SD spec for ACMD41 says that a zero argument is an "inquiry" ACMD41, which does not start initialisation and is used only for retrieving the OCR. However, Tianocore EDK2 (UEFI) has a bug [1]: it first sends an inquiry (zero) ACMD41. If that first request returns an OCR value with the power up b

Re: [Qemu-devel] [Qemu-block] [PATCH v4 0/3] Use QCryptoSecret for block device passwords

2016-02-07 Thread Jeff Cody
On Thu, Jan 21, 2016 at 02:19:18PM +, Daniel P. Berrange wrote: > This series was previously posted: > > v1: https://lists.gnu.org/archive/html/qemu-devel/2015-10/msg04365.html > v2: https://lists.gnu.org/archive/html/qemu-devel/2015-12/msg03809.html > v3: https://lists.gnu.org/archive/h

Re: [Qemu-devel] [PATCH 4/6] pseries: Move hash page table allocation to reset time

2016-02-07 Thread Alexey Kardashevskiy
On 02/05/2016 01:13 PM, David Gibson wrote: At the moment the size of the hash page table (HPT) is fixed based on the maximum memory allowed to the guest. As such, we allocate the table during machine construction, and just clear it at reset. However, we're planning to implement a PAPR extensio

Re: [Qemu-devel] [PATCH 6/6] target-ppc: Add helpers for updating a CPU's SDR1 and external HPT

2016-02-07 Thread Alexey Kardashevskiy
On 02/05/2016 01:13 PM, David Gibson wrote: When a Power cpu with 64-bit hash MMU has it's hash page table (HPT) pointer updated by a write to the SDR1 register we need to update some derived variables. Likewise, when the cpu is configured for an external HPT (one not in the guest memory space)

Re: [Qemu-devel] [PATCH 6/6] target-ppc: Add helpers for updating a CPU's SDR1 and external HPT

2016-02-07 Thread Alexey Kardashevskiy
On 02/05/2016 01:13 PM, David Gibson wrote: When a Power cpu with 64-bit hash MMU has it's hash page table (HPT) pointer updated by a write to the SDR1 register we need to update some derived variables. Likewise, when the cpu is configured for an external HPT (one not in the guest memory space)

Re: [Qemu-devel] [PATCH 1/6] target-ppc: Remove unused kvmppc_update_sdr1() stub

2016-02-07 Thread Alexey Kardashevskiy
On 02/05/2016 01:13 PM, David Gibson wrote: This KVM stub implementation isn't used anywhere. Signed-off-by: David Gibson --- target-ppc/kvm_ppc.h | 5 - 1 file changed, 5 deletions(-) diff --git a/target-ppc/kvm_ppc.h b/target-ppc/kvm_ppc.h index 62406ce..aaa828c 100644 --- a/target-pp

Re: [Qemu-devel] [PATCH 2/6] target-ppc: Include missing MMU models for SDR1 in info registers

2016-02-07 Thread Alexey Kardashevskiy
On 02/05/2016 01:13 PM, David Gibson wrote: The HMP command "info registers" produces somewhat different information on different ppc cpu variants. For those with a hash MMU it's supposed to include the SDR1, DAR and DSISR registers related to the MMU. However, the switch is missing a couple of

Re: [Qemu-devel] [PATCH 3/6] pseries: Simplify handling of the hash page table fd

2016-02-07 Thread Alexey Kardashevskiy
On 02/05/2016 01:13 PM, David Gibson wrote: When migrating the 'pseries' machine type with KVM, we use a special fd to access the hash page table stored within KVM. Usually, this fd is opened at the beginning of migration, and kept open until the migration is complete. However, if there is a gu

Re: [Qemu-devel] [PATCH 5/6] target-ppc: Remove hack for ppc_hash64_load_hpte*() with HV KVM

2016-02-07 Thread Alexey Kardashevskiy
On 02/05/2016 01:13 PM, David Gibson wrote: With HV KVM, the guest's hash page table (HPT) is managed by the kernel and not directly accessible to QEMU. This means that spapr->htab is NULL and normally env->external_htab would also be NULL for each cpu. However, that would cause ppc_hash64_load