Hi Philippe,
On 04.04.19 21:24, Philippe Mathieu-Daudé wrote:
> Hi Helge,
>
> On 4/4/19 8:57 PM, Helge Deller wrote:
>> If a non-release architecture is found, and it's known that there is no
>> native TCG support for that CPU, automatically fall back to the TCI
>> implementation instead of
From: Markus Armbruster
Since the previous commit, find_machine() and find_default_machine()
don't have to deallocate on return. This permits further
simplifications.
Signed-off-by: Markus Armbruster
Reviewed-by: Wei Yang
---
vl.c | 25 -
1 file changed, 8
Here is two simple change related to select_machine()
[1]: make find_default_machine() local since no one outside file use it
[2]: allocate TYPE_MACHINE list only once to select machine type
[3]: cleanup after previous commit
[4]: Simplify machine_parse()
---
v2:
* adjust changelog in [1]
*
Function find_default_machine() is introduced by commit 2c8cffa599b7
"vl: make find_default_machine externally visible", and it was used
outside of vl.c until commit a904410af5f1 "pc_sysfw: remove the rom_only
property".
Commit a904410af5f1 "pc_sysfw: remove the rom_only property" removed the
On Fri, 5 Apr 2019 at 07:39, Palmer Dabbelt wrote:
>
> The following changes since commit 061b51e9195670e9d190cdec46fabcb3c77763fb:
>
> Update version for v4.0.0-rc2 release (2019-04-02 17:01:20 +0100)
>
> are available in the Git repository at:
>
> git://github.com/palmer-dabbelt/qemu.git
From: Markus Armbruster
Exploit that argument @name is nerver null. Check is_help_option()
first, because that's what we do elsewhere. If we (foolishly!)
defined a machine named "help", -machine help would now print help
instead of selecting the machine named "help".
Signed-off-by: Markus
On 05/04/2019 10:17, David Gibson wrote:
> On Thu, Apr 04, 2019 at 02:40:45PM +0530, Aravinda Prasad wrote:
>>
>>
>> On Monday 25 March 2019 11:52 AM, David Gibson wrote:
>>> On Fri, Mar 22, 2019 at 12:03:58PM +0530, Aravinda Prasad wrote:
Memory error such as bit flips that cannot be
On Thu 04 Apr 2019 05:07:46 PM CEST, Kevin Wolf wrote:
> When bdrv_temp_snapshot_options() is called for snapshot=on, the
> 'discard' option in the options QDict hasn't been parsed and merged into
> the flags yet. So copy the dict entry to make sure that the temporary
> overlay enables discard
I agree to Thomas, and in that case referring to your comment #2 - qemu
versions in Ubuntu are associated with the base Ubuntu version - so Ubuntu
16.04 will stick to qemu 2.5 + fixes.
Ubuntu 19.10 released next week has qemu 3.1 btw, see [1]
And vice versa for an upstream report (which is
On 04/04/2019 16.34, Jason J. Herne wrote:
> This is to support booting from vfio-ccw dasd devices. We basically implement
> the real hardware ipl procedure. This allows for booting Linux guests on
> vfio-ccw devices.
>
> vfio-ccw's channel program prefetch algorithm complicates ipl because most
On Thu, Apr 04, 2019 at 08:39:23PM +0200, Helge Deller wrote:
> All major distributions do support libseccomp version >= 2.3.0, so there
> is no need to special-case on various architectures any longer.
>
> Signed-off-by: Helge Deller
Reviewed-by: Daniel P. Berrangé
Regards,
Daniel
--
|:
Now all the functions used to select machine is local and the call flow
looks like below:
select_machine()
find_default_machine()
machine_parse()
find_machine()
All these related function will need a GSList for TYPE_MACHINE.
Currently we allocate this list each
Hi Marc,
I did more experiments.. It seems to be an issue in glib-2.44.1 which is quite
old. I updated and it seems I don't see the issue anymore.
Thanks for your input on this!
Cheers,
Fred
Le 4/4/19 à 5:49 PM, KONRAD Frederic a écrit :
Le 4/4/19 à 5:44 PM, Marc-André Lureau a écrit :
Hi
On 4/5/19 10:05 AM, Greg Kurz wrote:
> Signed-off-by: Greg Kurz
> ---
> target/ppc/trace-events |4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/target/ppc/trace-events b/target/ppc/trace-events
> index 3858f97dff54..ed4d57c6d9c1 100644
> ---
On Thu, Apr 04, 2019 at 10:21:26AM -0500, Mark Kanda wrote:
> From: Chris Kenna
>
> In situations where e1000 receives an undersized Ethernet frame,
> QEMU increments the emulated "Receive Undersize Count (RUC)"
> register when padding the frame.
>
> This is incorrect because this an expected
Am 05.04.2019 um 13:45 hat Markus Armbruster geschrieben:
> drive_new() returns null without setting an error when it provided
> help. add_init_drive() assumes null means failure, and crashes trying
> to report a null error.
>
> Fixes: c4f26c9f37ce511e5fe629c21c180dc6eb7c5a25
> Cc:
On 05/04/2019 08.58, Thomas Huth wrote:
> On 04/04/2019 16.34, Jason J. Herne wrote:
>> This is to support booting from vfio-ccw dasd devices. We basically implement
>> the real hardware ipl procedure. This allows for booting Linux guests on
>> vfio-ccw devices.
>>
>> vfio-ccw's channel program
Signed-off-by: Greg Kurz
---
target/ppc/trace-events |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/target/ppc/trace-events b/target/ppc/trace-events
index 3858f97dff54..ed4d57c6d9c1 100644
--- a/target/ppc/trace-events
+++ b/target/ppc/trace-events
@@ -1,5 +1,5 @@
#
Hi Helge,
On 4/5/19 9:56 AM, Helge Deller wrote:
> On 05.04.19 03:34, Peter Maydell wrote:
>> On Fri, 5 Apr 2019 at 01:59, Helge Deller wrote:
>>> If a non-release architecture is found, and it's known that there is no
>>> native TCG support for that CPU, automatically fall back to the TCI
>>>
* Jens Freimann (jfreim...@redhat.com) wrote:
> ping
>
> FYI: I'm also working on a few related tools to detect driver behaviour when
> assigning a MAC to the vf device. Code is at
> https://github.com/jensfr/netfailover_driver_detect
Hi Jens,
I've not been following this too uch, but:
>
On Fri, 5 Apr 2019 at 16:02, Helge Deller wrote:
> Sadly such special treatment by projects makes life for me
> as an architecture maintainer much harder :-(
It's kind of inevitable for programs that aren't straightforwardly
architecture agnostic. gcc doesn't work for architectures which
don't
On 4/4/19 8:39 PM, Helge Deller wrote:
> All major distributions do support libseccomp version >= 2.3.0, so there
> is no need to special-case on various architectures any longer.
>
> Signed-off-by: Helge Deller
>
> diff --git a/configure b/configure
> index 1c563a7027..8fe4fc84d8 100755
> ---
On 4/5/19 6:14 AM, Thomas Huth wrote:
> On 05/04/2019 00.12, Philippe Mathieu-Daudé wrote:
>> This patch is purely cosmetic. No functional change.
>> This will ease the next patch where we re-indent an if() statement.
>>
>> Signed-off-by: Philippe Mathieu-Daudé
>> ---
>> hw/isa/isa-superio.c |
On Thu, Mar 28, 2019 at 07:28:08PM +0100, Kevin Wolf wrote:
> auto-read-only=on changed its behaviour in file-posix for the 4.0
> release. This change cannot be detected through the usual mechanisms
> like schema introspection.
>
> I took Stefan's patch 'qmp: add query-qemu-capabilities', removed
Am Fri, 5 Apr 2019 12:49:15 +0200
schrieb Philippe Mathieu-Daudé :
> The EDK2 submodule was added for UEFI testing, you don't need to compile
> it to build/use QEMU.
>
> How did you end up compiling it?
The qemu.spec file has this since a very long time:
make -C roms efirom
Am Fri, 5 Apr 2019 13:29:44 +0200
schrieb Philippe Mathieu-Daudé :
> I'll submit that patch.
Can this actually work? It does not remove the naming conflict.
Olaf
pgpL9MyUiEtmy.pgp
Description: Digitale Signatur von OpenPGP
Since commit f590a812c210 we build the EfiRom utility
unconditionally. This has been tested on all the Linux
distribution providing continuous integration (namely
Debian and Fedora). Not all distributions are able to
build the EfiRom without specific patches (In particular
SUSE which enforces the
On 04/04/19 22:49, Philippe Mathieu-Daudé wrote:
> While I wonder if someone still use the r4k machine, I have more doubts
> about users running with more the 2 -serial options...
I sort of remember ISA extension cards that had jumpers or dip switches
to choose between COM1/COM2/COM3/COM4...
Signed-off-by: Greg Kurz
---
target/ppc/kvm.c| 68 +++
target/ppc/trace-events | 25 +
2 files changed, 52 insertions(+), 41 deletions(-)
diff --git a/target/ppc/kvm.c b/target/ppc/kvm.c
index 2427c8ee13ae..3a11d2e1060c
On Fri, Apr 05, 2019 at 09:56:46AM +0200, Helge Deller wrote:
> Hi Peter,
>
> On 05.04.19 03:34, Peter Maydell wrote:
> > On Fri, 5 Apr 2019 at 01:59, Helge Deller wrote:
> >> If a non-release architecture is found, and it's known that there is no
> >> native TCG support for that CPU,
Hi,
I am trying to understand the possible values for the MAX_FD variable
used by the floppy controller model (hw/block/fdc.c).
Looking at git history:
- 2004-01-05 7138fcfbf7dd + 8977f3c107ef (Jocelyn Mayer):
FDC introduced with "#define MAX_FD 2"
- 2008-04-29 78ae820cfeb0 (Hervé
On 4/5/19 1:14 PM, Philippe Mathieu-Daudé wrote:
> On 4/5/19 12:59 PM, Olaf Hering wrote:
>> Am Fri, 5 Apr 2019 12:49:15 +0200
>> schrieb Philippe Mathieu-Daudé :
>>
>>> The EDK2 submodule was added for UEFI testing, you don't need to compile
>>> it to build/use QEMU.
>>>
>>> How did you end up
Am Fri, 5 Apr 2019 13:14:35 +0200
schrieb Philippe Mathieu-Daudé :
> On 4/5/19 12:59 PM, Olaf Hering wrote:
> > Am Fri, 5 Apr 2019 12:49:15 +0200
> > schrieb Philippe Mathieu-Daudé :
> >
> >> The EDK2 submodule was added for UEFI testing, you don't need to compile
> >> it to build/use QEMU.
>
On 4/5/19 2:09 PM, Olaf Hering wrote:
> Am Fri, 5 Apr 2019 13:55:29 +0200
> schrieb Philippe Mathieu-Daudé :
>
>> +EFIROM ?= edk2/BaseTools/Source/C/bin/EfiRom
>
> This name is too generic and will conflict with ipxe.git if any of "bios
> seavgabios pxerom" is used for 'make -C roms'.
This is
Am Fri, 5 Apr 2019 14:59:16 +0200
schrieb Philippe Mathieu-Daudé :
> On 4/5/19 2:09 PM, Olaf Hering wrote:
> > Am Fri, 5 Apr 2019 13:55:29 +0200
> > schrieb Philippe Mathieu-Daudé :
> >
> >> +EFIROM ?= edk2/BaseTools/Source/C/bin/EfiRom
> >
> > This name is too generic and will conflict
On Fri, Apr 5, 2019 at 8:05 AM Christian Ehrhardt <
1823...@bugs.launchpad.net> wrote:
> I agree to Thomas, and in that case referring to your comment #2 - qemu
> versions in Ubuntu are associated with the base Ubuntu version - so Ubuntu
> 16.04 will stick to qemu 2.5 + fixes.
> Ubuntu 19.10
Hi Peter,
On 05.04.19 03:34, Peter Maydell wrote:
> On Fri, 5 Apr 2019 at 01:59, Helge Deller wrote:
>> If a non-release architecture is found, and it's known that there is no
>> native TCG support for that CPU, automatically fall back to the TCI
>> implementation instead of requesting the user
* Yury Kotov (yury-ko...@yandex-team.ru) wrote:
> 03.04.2019, 22:06, "Dr. David Alan Gilbert" :
> > * Yury Kotov (yury-ko...@yandex-team.ru) wrote:
> >> It fixes heap-use-after-free which was found by clang's ASAN.
> >>
> >> Control flow of this use-after-free:
> >> main_thread:
> >> * Got
On 04/04/2019 - 22:01:38, Thomas Huth wrote:
> On 04/04/2019 20.39, Helge Deller wrote:
> > All major distributions do support libseccomp version >= 2.3.0, so there
> > is no need to special-case on various architectures any longer.
> >
> > Signed-off-by: Helge Deller
> >
> > diff --git
On Fri, Apr 05, 2019 at 11:02:52AM +0200, Helge Deller wrote:
> On 05.04.19 10:26, Daniel P. Berrangé wrote:
> > On Fri, Apr 05, 2019 at 09:56:46AM +0200, Helge Deller wrote:
> >> Hi Peter,
> >>
> >> On 05.04.19 03:34, Peter Maydell wrote:
> >>> On Fri, 5 Apr 2019 at 01:59, Helge Deller wrote:
>
On 4/5/19 11:02 AM, Daniel P. Berrangé wrote:
> On Fri, Apr 05, 2019 at 10:47:54AM +0200, Philippe Mathieu-Daudé wrote:
>> Hi Helge,
>>
>> On 4/5/19 9:56 AM, Helge Deller wrote:
>>> On 05.04.19 03:34, Peter Maydell wrote:
On Fri, 5 Apr 2019 at 01:59, Helge Deller wrote:
> If a
* Philippe Mathieu-Daudé (phi...@redhat.com) wrote:
> Hi,
>
> I am trying to understand the possible values for the MAX_FD variable
> used by the floppy controller model (hw/block/fdc.c).
>
> Looking at git history:
>
> - 2004-01-05 7138fcfbf7dd + 8977f3c107ef (Jocelyn Mayer):
> FDC
* Greg Kurz (gr...@kaod.org) wrote:
> On Mon, 10 Dec 2018 17:31:44 +
> "Dr. David Alan Gilbert (git)" wrote:
>
> > From: "Dr. David Alan Gilbert"
> >
> > Hi,
> > This is the first RFC for the QEMU side of 'virtio-fs';
> > a new mechanism for mounting host directories into the guest
> >
On 05.04.19 10:26, Daniel P. Berrangé wrote:
> On Fri, Apr 05, 2019 at 09:56:46AM +0200, Helge Deller wrote:
>> Hi Peter,
>>
>> On 05.04.19 03:34, Peter Maydell wrote:
>>> On Fri, 5 Apr 2019 at 01:59, Helge Deller wrote:
If a non-release architecture is found, and it's known that there is no
On Fri, Apr 05, 2019 at 10:47:54AM +0200, Philippe Mathieu-Daudé wrote:
> Hi Helge,
>
> On 4/5/19 9:56 AM, Helge Deller wrote:
> > On 05.04.19 03:34, Peter Maydell wrote:
> >> On Fri, 5 Apr 2019 at 01:59, Helge Deller wrote:
> >>> If a non-release architecture is found, and it's known that there
On 4/5/19 12:59 PM, Olaf Hering wrote:
> Am Fri, 5 Apr 2019 12:49:15 +0200
> schrieb Philippe Mathieu-Daudé :
>
>> The EDK2 submodule was added for UEFI testing, you don't need to compile
>> it to build/use QEMU.
>>
>> How did you end up compiling it?
>
> The qemu.spec file has this since a very
On 4/5/19 6:51 AM, Thomas Huth wrote:
> On 05/04/2019 00.12, Philippe Mathieu-Daudé wrote:
>> ISA Super I/O are already modeled by the ISASuperIODevice abstract
>> device.
>> Since this board uses a generic ISA Super I/O chipset, refactor it
>> as the TYPE_R4K_SUPERIO device, child of
On 4/5/19 1:16 PM, Olaf Hering wrote:
> Am Fri, 5 Apr 2019 12:59:18 +0200
> schrieb Olaf Hering :
>
>> This used to work still in January with
>> c9d18c1c150c84e7a976df989ad04ddf01083f46.
>
> It is not possible to just override EFIROM=$(type -P EfiRom) because this
> variable is also used by
On Fri, Mar 29, 2019 at 03:18:25PM +0100, Vitaly Kuznetsov wrote:
> KVM now supports reporting supported Hyper-V features through CPUID
> (KVM_GET_SUPPORTED_HV_CPUID ioctl). Going forward, this is going to be
> the only way to announce new functionality and this has already happened
> with Direct
On Fri, 5 Apr 2019 16:04:27 +1100
Alexey Kardashevskiy wrote:
> On 05/04/2019 10:17, David Gibson wrote:
> > On Thu, Apr 04, 2019 at 02:40:45PM +0530, Aravinda Prasad wrote:
> >>
> >>
> >> On Monday 25 March 2019 11:52 AM, David Gibson wrote:
> >>> On Fri, Mar 22, 2019 at 12:03:58PM +0530,
On Tue, Apr 02, 2019 at 08:15:12AM +0200, Igor Mammedov wrote:
>On Tue, 2 Apr 2019 11:53:43 +0800
>Wei Yang wrote:
>
>
>> The migration infrastructure has several SaveStateEntry to help migrate
>> different elements. The one with name "ram" take charge of RAMBlock. So this
>> SaveStateEntry and
Cc'ing qemu-block.
On 4/5/19 11:02 AM, Helge Deller wrote:
[...]
> As another example, even if I only want to build "qemu-img", I still need
> to manually give the --enable-tcg-interpreter configure option.
You found a bug :)
On Fri, Apr 05, 2019 at 09:56:29AM +0100, Dr. David Alan Gilbert wrote:
* Jens Freimann (jfreim...@redhat.com) wrote:
[...]
> To summarize concerns/feedback from previous discussion:
> 1.- guest OS can reject or worse _delay_ unplug by any amount of time.
> Migration might get stuck for
It seems in qemu.git#master the edk2.git submodule is now mandatory.
For me it fails to compile. This is not a new error. It needs to be compiled
with -fPIC since essentially forever.
But I wonder, why does it fail to compile only for me?!
Example of failure:
$ grep -h CommonLib.o
On 04/04/2019 18.30, Markus Armbruster wrote:
> Thomas Huth writes:
>
>> On 04/04/2019 15.29, Philippe Mathieu-Daudé wrote:
>>> On 4/4/19 12:07 PM, Paolo Bonzini wrote:
On 04/04/19 09:14, Thomas Huth wrote:
> The i8042 PS/2 controller is part of the chipset on the motherboard.
> It
Am Fri, 5 Apr 2019 12:59:18 +0200
schrieb Olaf Hering :
> This used to work still in January with
> c9d18c1c150c84e7a976df989ad04ddf01083f46.
It is not possible to just override EFIROM=$(type -P EfiRom) because this
variable is also used by ipxe.git.
Olaf
pgp6kq871XR2n.pgp
Description:
Am 05.04.2019 um 12:32 hat Stefan Hajnoczi geschrieben:
> On Thu, Mar 28, 2019 at 07:28:08PM +0100, Kevin Wolf wrote:
> > auto-read-only=on changed its behaviour in file-posix for the 4.0
> > release. This change cannot be detected through the usual mechanisms
> > like schema introspection.
> >
>
We also hit this regression when testing CompCert for e5500 with qemu
3.1.0.
The minimal example
> #include
> #include
>
> union C { float f; unsigned long l; };
> int main (void) {
> union C c;
> c.f = NAN;
> printf("Float: %f\n Hex: 0x%x\n", c.f, c.l);
> printf("The above
Hi Olaf,
On 4/5/19 12:39 PM, Olaf Hering wrote:
> It seems in qemu.git#master the edk2.git submodule is now mandatory.
The EDK2 submodule was added for UEFI testing, you don't need to compile
it to build/use QEMU.
How did you end up compiling it?
> For me it fails to compile. This is not a new
ping!
On 26.03.2019 18:51, Denis Plotnikov wrote:
> ping ping ping ping ping!
>
> On 18.03.2019 10:53, Denis Plotnikov wrote:
>> ping ping ping ping!
>>
>> On 11.03.2019 11:20, Denis Plotnikov wrote:
>>> ping ping ping!
>>>
>>> On 04.03.2019 18:10, Denis Plotnikov wrote:
ping!
On
drive_new() returns null without setting an error when it provided
help. add_init_drive() assumes null means failure, and crashes trying
to report a null error.
Fixes: c4f26c9f37ce511e5fe629c21c180dc6eb7c5a25
Cc: qemu-sta...@nongnu.org
Signed-off-by: Markus Armbruster
---
device-hotplug.c | 2
ping!
On 26.03.2019 18:51, Denis Plotnikov wrote:
> ping ping ping ping ping!
>
> On 18.03.2019 10:53, Denis Plotnikov wrote:
>> ping ping ping ping!
>>
>> On 11.03.2019 11:20, Denis Plotnikov wrote:
>>> ping ping ping!
>>>
>>> On 04.03.2019 18:10, Denis Plotnikov wrote:
ping!
On
Am Fri, 5 Apr 2019 13:55:29 +0200
schrieb Philippe Mathieu-Daudé :
> +EFIROM ?= edk2/BaseTools/Source/C/bin/EfiRom
This name is too generic and will conflict with ipxe.git if any of "bios
seavgabios pxerom" is used for 'make -C roms'.
Olaf
pgpFSdFcL3tvf.pgp
Description: Digitale Signatur
On 4/5/19 9:26 AM, Thomas Huth wrote:
On 05/04/2019 15.11, Jason J. Herne wrote:
On 4/5/19 3:52 AM, Thomas Huth wrote:
On 05/04/2019 08.58, Thomas Huth wrote:
[...]
while running my s390-ccw bios tests, I noticed that network booting
seems to be broken now. This used to work before:
* Zhang Chen (chen.zh...@intel.com) wrote:
> From: Zhang Chen
>
> I found upstream codes conflict with COLO and lead to crash,
> and I located to this patch:
Queued.
>
> commit 386a907b37a9321bc5d699bc37104d6ffba1b34d
> Author: Wei Wang
> Date: Tue Dec 11 16:24:49 2018 +0800
>
>
This v3 improves on code alignments
Jules Irenge (4):
target/mips: realign comments to fix checkpatch warnings
target/mips: add or remove space to fix checkpatch errors
target/mips: wrap lines to fix checkpatch errors
target/mips: replace tab code indent with spaces to fix checkpatch
LSI mapping in spapr currently open-codes standard PCI swizzling. It thus
duplicates the code of pci_swizzle_map_irq_fn().
Expose the swizzling formula so that it can be used with a slot number
when building the device tree. Simply drop pci_spapr_map_irq() and call
pci_swizzle_map_irq_fn()
Am 05.04.2019 um 18:11 hat Dr. David Alan Gilbert geschrieben:
> * Kevin Wolf (kw...@redhat.com) wrote:
> > Am 05.04.2019 um 13:45 hat Markus Armbruster geschrieben:
> > > drive_new() returns null without setting an error when it provided
> > > help. add_init_drive() assumes null means failure,
03.04.2019 6:05, Eric Blake wrote:
> In commit 0c1d50bd, I added a couple of TODO comments about whether we
> consult bl.request_alignment when responding to NBD_OPT_INFO. At the
> time, qemu as server was hard-coding an advertised alignment of 512 to
> clients that promised to obey constraints,
On 4/5/19 3:04 PM, Olaf Hering wrote:
> Am Fri, 5 Apr 2019 14:59:16 +0200
> schrieb Philippe Mathieu-Daudé :
>> On 4/5/19 2:09 PM, Olaf Hering wrote:
>>> Am Fri, 5 Apr 2019 13:55:29 +0200
>>> schrieb Philippe Mathieu-Daudé :
>>>
+EFIROM ?= edk2/BaseTools/Source/C/bin/EfiRom
>>>
>>> This
Roman Kagan writes:
> On Fri, Mar 29, 2019 at 03:18:27PM +0100, Vitaly Kuznetsov wrote:
>> Currently, there is no doc describing hv-* CPU flags, people are
>> encouraged to get the information from Microsoft Hyper-V Top Level
>> Functional specification (TLFS). There is, however, a bit of QEMU
On Fri, Mar 29, 2019 at 03:18:28PM +0100, Vitaly Kuznetsov wrote:
> In many case we just want to give Windows guests all currently supported
> Hyper-V enlightenments and that's where this new mode may come handy. We
> pass through what was returned by KVM_GET_SUPPORTED_HV_CPUID.
The only one out
On 3/28/19 11:45 AM, 李强 wrote:
> Ping...
>
> What's your opinion, Philippe?
Eh sorry my email client tagged this series as reviewed since your
previous v1 was reviewed by Laszlo. I'll review your patches, but please
increase the version between series next time so I won't miss it that
easily ;)
According to the changelog of 298a971024534, SpaprPhbState::dtbusname was
introduced to "make it easier to relate the guest and qemu views of memory
to each other", hence its name.
Use it when creating the PHB node to avoid code duplication.
Signed-off-by: Greg Kurz
---
hw/ppc/spapr_pci.c |
On Fri, 5 Apr 2019 15:26:24 +0200
Thomas Huth wrote:
> On 05/04/2019 15.11, Jason J. Herne wrote:
> > Your analysis of the problem matches what I'm seeing as well. Here is
> > what I'm proposing to fix it. If you like it, let me know if you want me
> > to re-send just the final patch, or the
Hello,
Attached patch fixes "bad size" panic for 64-bit operands when "-accel hvf"
is enabled.
Max
fix-64bit-hvf.patch
Description: Binary data
On Fri, 05 Apr 2019 10:12:42 -0300
"Murilo Opsfelder Araújo" wrote:
> Hi, Greg.
>
Hi Murilo,
> Greg Kurz writes:
>
> > Signed-off-by: Greg Kurz
> > ---
> > target/ppc/kvm.c| 68
> > +++
> > target/ppc/trace-events | 25
* Juan Quintela (quint...@redhat.com) wrote:
> Otherwise we are setting err twice, what is wrong and causes an abort.
>
> Signed-off-by: Juan Quintela
Queued just this one.
> ---
> hmp.c | 6 --
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/hmp.c b/hmp.c
> index
Wrap lines to fix errors issued by checkpatch.pl tool
"ERROR: line over 90 characters"
within "target/mips/cpu.h" file.
Signed-off-by: Jules Irenge
---
target/mips/cpu.h | 25 +++--
1 file changed, 19 insertions(+), 6 deletions(-)
diff --git a/target/mips/cpu.h
* Kevin Wolf (kw...@redhat.com) wrote:
> Am 05.04.2019 um 13:45 hat Markus Armbruster geschrieben:
> > drive_new() returns null without setting an error when it provided
> > help. add_init_drive() assumes null means failure, and crashes trying
> > to report a null error.
> >
> > Fixes:
Le 05/04/2019 à 12:29, Philippe Mathieu-Daudé a écrit :
Hi,
I am trying to understand the possible values for the MAX_FD variable
used by the floppy controller model (hw/block/fdc.c).
Looking at git history:
- 2004-01-05 7138fcfbf7dd + 8977f3c107ef (Jocelyn Mayer):
FDC introduced with
Hi, Greg.
Greg Kurz writes:
> Signed-off-by: Greg Kurz
> ---
> target/ppc/kvm.c| 68
> +++
> target/ppc/trace-events | 25 +
> 2 files changed, 52 insertions(+), 41 deletions(-)
>
> diff --git a/target/ppc/kvm.c
On 4/5/19 3:52 AM, Thomas Huth wrote:
On 05/04/2019 08.58, Thomas Huth wrote:
On 04/04/2019 16.34, Jason J. Herne wrote:
This is to support booting from vfio-ccw dasd devices. We basically implement
the real hardware ipl procedure. This allows for booting Linux guests on
vfio-ccw devices.
03.04.2019 6:05, Eric Blake wrote:
> We've recently added traces for clients to flag server non-compliance;
> let's do the same for servers to flag client non-compliance. According
> to the spec, if the client requests NBD_INFO_BLOCK_SIZE, it is
> promising to send all requests aligned to those
The iPXE project already uses the EFIROM for a tool named 'efirom'
which is not the Intel EfiRom used by the EDK2 project. To make
the difference obvious, rename the variable.
This fixes a long standing issue which is now masked since commit
f590a812c21 "roms: build the EfiRom utility from the
Since commit f590a812c210 we build the EDK2 EfiRom utility
unconditionally. This has been tested on all the Linux
distribution providing continuous integration (namely Debian
and Fedora). Not all distributions are able to build the
EfiRom without specific patches (In particular SUSE which
enforces
Hi,
Two trivial fixes to avoid the latest EDK2 testing series to
cause trouble to downstream distributions (in particular if
they have PIE enforced).
Regards,
Phil.
Philippe Mathieu-Daudé (2):
roms: Rename the EFIROM variable to avoid clashing with iPXE
roms: Allow the EDK2_EFIROM variable
From: "Dr. David Alan Gilbert"
The following changes since commit 10546e09e174e0bb185b66a4c397aa845efcd36e:
Merge remote-tracking branch
'remotes/palmer/tags/riscv-for-master-4.0-rc3-v2' into staging (2019-04-05
04:50:30 +0100)
are available in the Git repository at:
From: Juan Quintela
Otherwise we are setting err twice, what is wrong and causes an abort.
Signed-off-by: Juan Quintela
Message-Id: <20190403114958.3705-2-quint...@redhat.com>
Reviewed-by: Dr. David Alan Gilbert
Signed-off-by: Dr. David Alan Gilbert
---
hmp.c | 6 --
1 file changed, 4
From: Zhang Chen
I found upstream codes conflict with COLO and lead to crash,
and I located to this patch:
commit 386a907b37a9321bc5d699bc37104d6ffba1b34d
Author: Wei Wang
Date: Tue Dec 11 16:24:49 2018 +0800
migration: use bitmap_mutex in migration_bitmap_clear_dirty
My colleague Wei's
Realign comments to fix warnings issued by checkpatch.pl tool
"WARNING: Block comments use a leading /* on a separate line"
within "target/mips/cpu.h" file.
Signed-off-by: Jules Irenge
---
target/mips/cpu.h | 34 ++
1 file changed, 22 insertions(+), 12
Add or remove space to fix errors issued by checkpatch.pl tool
"ERROR: spaces required around ..."
"ERROR: space required after that"
"ERROR: space required before the open parenthesis"
"ERROR: space prohibited between function name and open parenthesis"
within "target/mips/cpu.h" file.
Patchew URL:
https://patchew.org/QEMU/20190405160938.27494-1-jbi.oct...@gmail.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Message-id: 20190405160938.27494-1-jbi.oct...@gmail.com
Subject: [Qemu-devel] [PATCH v3 0/4] target/mips: errors
Paolo Bonzini writes:
> On 26/03/19 14:18, Stefan Hajnoczi wrote:
>> Hi Sergio,
>> Here are the forgotten event loop optimizations I mentioned:
>>
>> https://github.com/stefanha/qemu/commits/event-loop-optimizations
>>
>> The goal was to eliminate or reorder syscalls so that useful work
On Fri, Mar 29, 2019 at 03:18:27PM +0100, Vitaly Kuznetsov wrote:
> Currently, there is no doc describing hv-* CPU flags, people are
> encouraged to get the information from Microsoft Hyper-V Top Level
> Functional specification (TLFS). There is, however, a bit of QEMU
> specifics.
This is
On 05/04/2019 15.11, Jason J. Herne wrote:
> On 4/5/19 3:52 AM, Thomas Huth wrote:
>> On 05/04/2019 08.58, Thomas Huth wrote:
[...]
>>> while running my s390-ccw bios tests, I noticed that network booting
>>> seems to be broken now. This used to work before:
>>>
>>> s390x-softmmu/qemu-system-s390x
Signed-off-by: lichun <706701...@qq.com>
---
qemu-io-cmds.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/qemu-io-cmds.c b/qemu-io-cmds.c
index 09750a2..8d93dc6 100644
--- a/qemu-io-cmds.c
+++ b/qemu-io-cmds.c
@@ -357,7 +357,7 @@ static void dump_buffer(const void *buffer,
03.04.2019 6:05, Eric Blake wrote:
> Don't increment remaining_bytes until we know that we will actually be
> including the current block status extent in the reply; otherwise, the
> value traced will include a bytes value that is oversized by the
> length of the next block status extent which did
Replace tab code indent with spaces to fix errors issued by checkpatch.pl tool
"ERROR: code indent should never use tabs"
within "target/mips/cpu.h" file.
Signed-off-by: Jules Irenge
---
target/mips/cpu.h | 138 +++---
1 file changed, 69 insertions(+), 69
Stefan Hajnoczi writes:
> Hi Sergio,
> Here are the forgotten event loop optimizations I mentioned:
>
> https://github.com/stefanha/qemu/commits/event-loop-optimizations
>
> The goal was to eliminate or reorder syscalls so that useful work (like
> executing BHs) occurs as soon as possible
1 - 100 of 130 matches
Mail list logo