On Thu, May 30, 2013 at 7:34 PM, Kevin O'Connor wrote:
> On Tue, May 28, 2013 at 07:53:09PM -0400, Kevin O'Connor wrote:
>> There were discussions on potentially introducing a middle component
>> to generate the tables. Coreboot was raised as a possibility, and
>> David thought it would be okay t
Il 31/05/2013 08:56, Alexey Kardashevskiy ha scritto:
> On 05/31/2013 07:16 AM, Paolo Bonzini wrote:
>> So far, the size of all regions passed to listeners could fit in 64 bits,
>> because artificial regions (containers and aliases) are eliminated by
>> the memory core, leaving only device regions
On Thu, May 30, 2013 at 11:51 PM, Xiao Guangrong
wrote:
> On 05/31/2013 12:50 AM, Jordan Justen wrote:
>> On Thu, May 30, 2013 at 9:08 AM, Luiz Capitulino
>> wrote:
>>> On Thu, 30 May 2013 18:03:04 +0200
>>> Paolo Bonzini wrote:
>>>
Il 30/05/2013 17:46, Luiz Capitulino ha scritto:
> Th
On 31 May 2013 05:50, Peter Chubb wrote:
>
> The rfe instruction has been broken since patch
> 5a839c0d54fac9db0516904db873a4fe01f50f4b because of a typo.
>
> Signed-off-by: Peter Chubb
Oops. I thought I'd caught all those when I reread the
patches before submission but I evidently missed one :-
On 30/05/13 13:07, Eric Blake wrote:
Is there a way of escaping the
commas on the command line so that it is possible to list properties for
drivers named in this way?
Commas are escaped by doubling them. Try ./qemu-system-sparc -device
SUNW,,tcx,help
That's the generic quoting we've used
On Fri, May 31, 2013 at 12:14 AM, Jordan Justen wrote:
> On Thu, May 30, 2013 at 11:51 PM, Xiao Guangrong
> wrote:
>> On 05/31/2013 12:50 AM, Jordan Justen wrote:
>>> On Thu, May 30, 2013 at 9:08 AM, Luiz Capitulino
>>> wrote:
On Thu, 30 May 2013 18:03:04 +0200
Paolo Bonzini wrote:
>
Kevin O'Connor wrote:
> one possible way forward would be to split the current SeaBIOS rom
> into two roms: "qvmloader" and "seabios". The "qvmloader" would do
> the qemu specific platform init (pci init, smm init, mtrr init, bios
> tables) and then load and run the regular seabios rom.
qvmloader
Il 31/05/2013 07:58, Alexey Kardashevskiy ha scritto:
> On 05/27/2013 05:03 PM, Paolo Bonzini wrote:
>> Il 27/05/2013 08:48, Alexey Kardashevskiy ha scritto:
>
> This is only true when the rerror and werror options have the values
> "ignore" or "report". See virtio-scsi for an example
Il 31/05/2013 08:51, Xiao Guangrong ha scritto:
> On 05/31/2013 12:50 AM, Jordan Justen wrote:
>> On Thu, May 30, 2013 at 9:08 AM, Luiz Capitulino
>> wrote:
>>> On Thu, 30 May 2013 18:03:04 +0200
>>> Paolo Bonzini wrote:
>>>
Il 30/05/2013 17:46, Luiz Capitulino ha scritto:
> The culprit
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Il 31/05/2013 05:32, David Gibson ha scritto:
>>> Then again, although we certainly want to keep changes which
>>> require updates to libfdt_env.h rare, I'm not going to rule
>>> out extensions to libfdt which add new (minor) external
>>> dependencie
On 05/31/2013 02:22 AM, Eduardo Habkost wrote:
> On Thu, May 30, 2013 at 05:57:21PM +0800, Wanlong Gao wrote:
>>> Use mbind to pin guest numa node memory to host nodes manually.
>>>
>>> If we are not able to pin memory to host node, we may meet the
>>> cross node memory access performance regressio
Luiz Capitulino reported that guest refused to boot and qemu
complained with:
kvm_set_phys_mem: error unregistering overlapping slot: Invalid argument
It is caused by commit 235e8982ad that did double free for the memslot
so that the second one raises the -EINVAL error
Fix it by reset memory size
On 05/30/2013 07:26 PM, Peter Maydell wrote:
> On 30 May 2013 17:22, Fabien Chouteau wrote:
>> I'm not sure this was expected or not, but it looks like the "||" should
>> be a "&&". Otherwise it's not possible to disable interrupt.
>>
>> Signed-off-by: Fabien Chouteau
>
> We've had people trying
Fix the condition according to the requirement in the comment above.
Otherwise it's not possible to disable interrupt on M-profile.
Signed-off-by: Fabien Chouteau
---
V2: The expression in the previous patch was wrong.
cpu-exec.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
dif
Hi,
> I guess -bios would load coreboot. Coreboot would siphon the data
> necessary for ACPI table building through the current (same) fw_cfg
> bottleneck, build the tables,
Yes.
> load the boot firmware (SeaBIOS or OVMF or
> something else -- not sure how to configure that),
The coreboot rom
On 31 May 2013 10:04, Fabien Chouteau wrote:
> On 05/30/2013 07:26 PM, Peter Maydell wrote:
>> * why this patch fixes it (probably with reference to the
>>architecture manual and to what QEMU means when it sets
>>"CPSR_I" on M profile [hint: look at how we handle PRIMASK])
>
> I do not in
On 31 May 2013 09:25, Paolo Bonzini wrote:
> Please don't. Fedora is not going to use the bundled dtc because of a
> policy against bundling, and Fedora's dtc package doesn't include
> libfdt_env.h.
It sounds like Fedora's dtc package is broken then -- are you
going to fix it?
> If David says t
Il 31/05/2013 11:44, Peter Maydell ha scritto:
> On 31 May 2013 09:25, Paolo Bonzini wrote:
>> Please don't. Fedora is not going to use the bundled dtc because of a
>> policy against bundling, and Fedora's dtc package doesn't include
>> libfdt_env.h.
>
> It sounds like Fedora's dtc package is br
Gerd Hoffmann wrote:
> > and pass down the
> > tables to the firmware (through a now unspecified interface -- perhaps
> > the tables could even be installed at this point).
>
> As far I know coreboot can add more stuff such as acpi tables to cbfs at
> runtime and seabios able to access cbfs too an
On 05/31/13 10:13, Peter Stuge wrote:
> Kevin O'Connor wrote:
>> one possible way forward would be to split the current SeaBIOS rom
>> into two roms: "qvmloader" and "seabios". The "qvmloader" would do
>> the qemu specific platform init (pci init, smm init, mtrr init, bios
>> tables) and then load
On Fri, 2013-05-31 at 15:58 +1000, Alexey Kardashevskiy wrote:
>
> And another question (sorry I am not very familiar with terminology but
> cc:Ben is :) ) - what happens with indirect requests if migration happened
> in the middle of handling such a request? virtio-scsi does not seem to
> handle
Is there a git tag for 1.4.2?
On 05/31/2013 06:18 PM, Paolo Bonzini wrote:
> Il 31/05/2013 07:58, Alexey Kardashevskiy ha scritto:
>> On 05/27/2013 05:03 PM, Paolo Bonzini wrote:
>>> Il 27/05/2013 08:48, Alexey Kardashevskiy ha scritto:
>>
>> This is only true when the rerror and werror options have the values
>> "i
On 05/31/2013 08:07 PM, Benjamin Herrenschmidt wrote:
> On Fri, 2013-05-31 at 15:58 +1000, Alexey Kardashevskiy wrote:
>>
>> And another question (sorry I am not very familiar with terminology but
>> cc:Ben is :) ) - what happens with indirect requests if migration happened
>> in the middle of hand
Il 31/05/2013 12:12, Alexey Kardashevskiy ha scritto:
> On 05/31/2013 06:18 PM, Paolo Bonzini wrote:
>> Il 31/05/2013 07:58, Alexey Kardashevskiy ha scritto:
>>> On 05/27/2013 05:03 PM, Paolo Bonzini wrote:
Il 27/05/2013 08:48, Alexey Kardashevskiy ha scritto:
>>>
>>> This is only true
On 05/31/2013 08:26 PM, Paolo Bonzini wrote:
> Il 31/05/2013 12:12, Alexey Kardashevskiy ha scritto:
>> On 05/31/2013 06:18 PM, Paolo Bonzini wrote:
>>> Il 31/05/2013 07:58, Alexey Kardashevskiy ha scritto:
On 05/27/2013 05:03 PM, Paolo Bonzini wrote:
> Il 27/05/2013 08:48, Alexey Kardashe
Il 31/05/2013 12:33, Alexey Kardashevskiy ha scritto:
> How do you trigger the situation when there are inactive requests which
> have to be migrated?
You need to trigger an error. For example, you could use a sparse image
on an almost-full partition and let "dd" fill your d
Il 31/05/2013 12:25, Alexey Kardashevskiy ha scritto:
> On 05/31/2013 08:07 PM, Benjamin Herrenschmidt wrote:
>> On Fri, 2013-05-31 at 15:58 +1000, Alexey Kardashevskiy wrote:
>>>
>>> And another question (sorry I am not very familiar with terminology but
>>> cc:Ben is :) ) - what happens with indi
On Fri, May 31, 2013 at 08:57:00AM +0200, Stefan Priebe - Profihost AG wrote:
> Am 31.05.2013 00:51, schrieb Amos Kong:
> > On Thu, May 30, 2013 at 10:30:21PM +0200, Stefan Priebe wrote:
> >> Am 30.05.2013 15:13, schrieb Amos Kong:
> >>> On Thu, May 30, 2013 at 02:09:25PM +0200, Stefan Priebe - Pro
On Thu, 30 May 2013, Stefano Stabellini wrote:
> On Thu, 30 May 2013, Andreas Färber wrote:
> > Am 30.05.2013 15:20, schrieb Stefano Stabellini:
> > > We are currently setting the PCI hole to start at HVM_BELOW_4G_RAM_END,
> > > that is 0xf000.
> > > Start the PCI hole at 0xe000 instead, th
On Wed, 29 May 2013, Stefano Stabellini wrote:
> Hi all,
> I have few simple fixes for Xen for the development tree as well as QEMU
> 1.5 and older releases:
>
>
> - the first patch is just a cleanup (that is needed to simplify
> preprocessor dependencies);
> - the second patch avoids setting n
The copied code comes from vhost_scsi_start. Spotted by Coverity.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Paolo Bonzini
---
hw/scsi/vhost-scsi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/scsi/vhost-scsi.c b/hw/scsi/vhost-scsi.c
index d7a1c33..785e93f 100644
--- a/
Am 31.05.2013 13:02, schrieb Amos Kong:
>>
>> ...
>>
>> thanks for this great explanation. I've done what you sayd but it still
>> does not work.
>>
>> Here is the output of the seabis debug log where you see the loop:
>> http://pastebin.com/raw.php?i=e53rdW2b
>
> | found virtio-scsi at 0:5
> | Se
Hello,
since qemu's commit v1.4.0-736-gf17c90b [1]:
nbd: Keep hostname and port separate
* literal IPv6 addresses no longer work in nbd URIs, because getaddrinfo is
called with the surrounding brackets:
$ qemu-system-x86_64 -drive file=nbd://[::1]:1234/quack
qemu-system-x86_64: -drive file=nb
On 05/31/13 09:09, Jordan Justen wrote:
> Why is updating the ACPI tables in seabios viewed as such a burden?
> Either qemu does it, or seabios... (And, OVMF too, but I don't think
> you guys are concerned with that. :)
I am :)
> On the flip side, why is moving the ACPI tables to QEMU such an is
29.05.2013 17:30, Alexey Kardashevskiy wrote:
> The symlink to "asm" platform linux headers is made in the build tree by
> the configure script but gcc is not told to look for them there.
>
> The patch fixes this.
Thanks, applied to the trivial patches queue.
/mjt
29.05.2013 17:58, Ed Maste wrote:
> Based on the datasheet at
> http://www.micron.com/~/media/Documents/Products/Data%20Sheet/NOR%20Flash/Serial%20NOR/N25Q/n25q_32mb_1_8v_65nm.pdf
Thanks, applied to the trivial patches queue.
/mjt
Always free them in the iscsi_aio_*_acb functions and remove the
checks in their callers. Remove ifs when the task struct was
previously dereferenced (spotted by Coverity).
Cc: qemu-sta...@nongnu.org
Signed-off-by: Paolo Bonzini
---
block/iscsi.c | 35 ++-
1 file
Two instances, both spotted by Coverity. In one, two blocks were
swapped. In the other, the check is not needed anymore.
Cc: qemu-sta...@nongnu.org
Cc: qemu-triv...@nongnu.org
Signed-off-by: Paolo Bonzini
---
monitor.c | 2 +-
savevm.c | 10 +-
2 files changed, 6 insertions(+), 6 del
Avoid the goto, and use the same retry logic for the 10- and 16- byte versions.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Paolo Bonzini
---
block/iscsi.c | 94 +--
1 file changed, 46 insertions(+), 48 deletions(-)
diff --git a/block/iscsi.
31.05.2013 13:39, Thomas Schwinge wrote:
> Signed-off-by: Thomas Schwinge
> ---
> fpu/softfloat-specialize.h | 15 +++
> 1 file changed, 7 insertions(+), 8 deletions(-)
>
> diff --git fpu/softfloat-specialize.h fpu/softfloat-specialize.h
> index 518f694..83add1a 100644
> --- fpu/so
On Thu, 2013-05-30 at 09:20 -0700, Jordan Justen wrote:
> On Thu, May 30, 2013 at 5:19 AM, David Woodhouse
> wrote:
> > On Thu, 2013-05-30 at 13:13 +0200, Laszlo Ersek wrote:
> >> Where is CorebootPkg available from?
> >
> > https://github.com/pgeorgi/edk2/tree/coreboot-pkg
>
> Is the license on
Il 31/05/2013 12:58, Stefano Stabellini ha scritto:
> On Thu, 30 May 2013, Stefano Stabellini wrote:
>> On Thu, 30 May 2013, Andreas Färber wrote:
>>> Am 30.05.2013 15:20, schrieb Stefano Stabellini:
We are currently setting the PCI hole to start at HVM_BELOW_4G_RAM_END,
that is 0xf00
On Wed, 2013-05-29 at 21:12 -0400, Kevin O'Connor wrote:
>
> I remain doubtful that QOM has all the info needed to generate the
> BIOS tables. Does QOM describe how the 5th pci device uses global
> interrupt 11 when using global interrupts, legacy interrupt 5 when not
> using global interrupts, a
Il 31/05/2013 14:07, Michael Tokarev ha scritto:
> 31.05.2013 13:39, Thomas Schwinge wrote:
>> Signed-off-by: Thomas Schwinge
>> ---
>> fpu/softfloat-specialize.h | 15 +++
>> 1 file changed, 7 insertions(+), 8 deletions(-)
>>
>> diff --git fpu/softfloat-specialize.h fpu/softfloat-s
Il 31/05/2013 10:52, Xiao Guangrong ha scritto:
> Luiz Capitulino reported that guest refused to boot and qemu
> complained with:
> kvm_set_phys_mem: error unregistering overlapping slot: Invalid argument
>
> It is caused by commit 235e8982ad that did double free for the memslot
> so that the seco
On Thu, May 30, 2013 at 11:48:46AM -0500, Anthony Liguori wrote:
> Amos Kong writes:
>
> > Guest driver sets repeat rate and delay time by KBD_CMD_SET_RATE,
> > but ps2 backend doesn't process it and no auto-repeat implementation.
> > This patch adds support of auto-repeat feature. The repeated e
On 31 May 2013 13:07, Michael Tokarev wrote:
> Hmm. And where's the simplification? Here's context diff for the same:
>
> *** fpu/softfloat-specialize.h.orig 2013-05-31 16:02:51.614710351 +0400
> --- fpu/softfloat-specialize.h 2013-05-31 16:02:59.838820308 +0400
> ***
> *** 936,9
On 05/28/2013 12:34 AM, Gerd Hoffmann wrote:
> Fill unset CharDriverState->filename with the backend name, so
> 'info chardev' will return at least the chardev type. Don't
> touch it in case the chardev init function filled it already,
> like the socket+pty chardevs do for example.
>
> Signed-off
Hi,
On Fri, May 31, 2013 at 7:54 PM, Paolo Bonzini wrote:
> Il 31/05/2013 11:44, Peter Maydell ha scritto:
>> On 31 May 2013 09:25, Paolo Bonzini wrote:
>>> Please don't. Fedora is not going to use the bundled dtc because of a
>>> policy against bundling, and Fedora's dtc package doesn't includ
On 05/31/13 06:41, Jordan Justen wrote:
> On Thu, May 30, 2013 at 7:06 PM, Kevin O'Connor wrote:
>> On Wed, May 29, 2013 at 01:27:24AM -0700, Jordan Justen wrote:
>>> The isapc machine with seabios currently requires the BIOS region
>>> to be read/write memory rather than read-only memory.
>>>
>>>
On Fri, 31 May 2013 16:52:18 +0800
Xiao Guangrong wrote:
> Luiz Capitulino reported that guest refused to boot and qemu
> complained with:
> kvm_set_phys_mem: error unregistering overlapping slot: Invalid argument
>
> It is caused by commit 235e8982ad that did double free for the memslot
> so th
On 05/31/13 14:36, Eric Blake wrote:
> On 05/28/2013 12:34 AM, Gerd Hoffmann wrote:
>> Fill unset CharDriverState->filename with the backend name, so
>> 'info chardev' will return at least the chardev type. Don't
>> touch it in case the chardev init function filled it already,
>> like the socket+p
On 05/31/2013 06:00 AM, Paolo Bonzini wrote:
> Two instances, both spotted by Coverity. In one, two blocks were
> swapped. In the other, the check is not needed anymore.
>
> Cc: qemu-sta...@nongnu.org
> Cc: qemu-triv...@nongnu.org
> Signed-off-by: Paolo Bonzini
> ---
> monitor.c | 2 +-
> sav
Il 31/05/2013 04:06, Kevin O'Connor ha scritto:
> On Wed, May 29, 2013 at 01:27:24AM -0700, Jordan Justen wrote:
>> > The isapc machine with seabios currently requires the BIOS region
>> > to be read/write memory rather than read-only memory.
>> >
>> > KVM currently cannot support the BIOS as a RO
Il 31/05/2013 14:40, Laszlo Ersek ha scritto:
> I think we've been here before...
>
> - always resetting the PAM registers broke S3 resume:
> http://thread.gmane.org/gmane.comp.emulators.qemu/195931/focus=195932
> http://thread.gmane.org/gmane.comp.emulators.qemu/195931/focus=196081
>
> - there w
Kevin O'Connor writes:
> On Tue, May 28, 2013 at 07:53:09PM -0400, Kevin O'Connor wrote:
>> There were discussions on potentially introducing a middle component
>> to generate the tables. Coreboot was raised as a possibility, and
>> David thought it would be okay to use coreboot for both OVMF an
On 05/31/13 10:13, Peter Stuge wrote:
> ACPI bytes are obviously a function of QEMU configuration.
Precisely!
When we evaluate that (mathematical-sense) function in boot firmware, we
need to retrieve the function's arguments. Those arguments are bits of
QEMU configuration, as you say, and fw_cfg
On Fri, 2013-05-31 at 07:58 -0500, Anthony Liguori wrote:
> What about a small change to the SeaBIOS build system to allow ACPI
> table generation to be done via a "plugin".
SeaBIOS already accepts ACPI tables from Coreboot or UEFI, and queries
them to find things that it needs.
> This could be a
Laszlo Ersek writes:
> On 05/31/13 09:09, Jordan Justen wrote:
>
> Due to licensing differences I can't just port code from SeaBIOS to
> OVMF
Fork OVMF, drop the fat module, and just add GPL code. It's an easily
solvable problem.
Rewriting BSD implementations of everything is silly. Every o
于 2013-5-30 10:41, Wenchao Xia 写道:
于 2013-5-27 23:41, Kevin Wolf 写道:
Am 25.05.2013 um 05:09 hat Wenchao Xia geschrieben:
These patches are the common part of my hmp/qmp block query series
and Pavel's
qmp snapshot command converion series. It mainly does following things:
1 move snapshot related
Hi,
> Without this patch the returned message for "query-chardev" is:
>
> {
> "return": [
> {
> "filename": "pty:/dev/pts/8",
> "label": "charserial0"
> },
> {
> "filename": "unix:/var/lib/libvirt/qemu/qemu-git.monitor,server",
>
On 05/31/13 15:21, Gerd Hoffmann wrote:
Hi,
Hi,
Please double-check. Current master
(87d23f78aa79b72da022afda358bbc8a8509ca70 to be exact) works just fine
for me. libvirt works, including a serial line redirected to pty, and
'info chardev' looks sane too.
sorry for the fuzz. :/ Upstr
On Thu, 30 May 2013 17:07:56 +0200
Andreas Färber wrote:
> Signed-off-by: Andreas Färber
Nitpick alarm on.
> ---
> include/qom/cpu.h | 10 ++
> include/sysemu/memory_mapping.h | 1 -
> memory_mapping-stub.c | 6 --
> memory_mapping.c
On Thu, 30 May 2013 17:07:58 +0200
Andreas Färber wrote:
> Signed-off-by: Andreas Färber
> ---
> include/qom/cpu.h | 11 +++
> include/sysemu/memory_mapping.h | 2 --
> memory_mapping-stub.c | 6 --
> memory_mapping.c | 2 +-
> qom/c
On Fri, 2013-05-31 at 08:04 -0500, Anthony Liguori wrote:
>
>
>
> Fork OVMF, drop the fat module, and just add GPL code. It's an easily
> solvable problem.
Heh. Actually it doesn't need to be a fork. It's modular, and the FAT
driver is just a single module. Which is actually included in *binar
On Thu, 30 May 2013 17:08:01 +0200
Andreas Färber wrote:
> Previously it would search for the first CPU with paging enabled and
> retrieve memory mappings from this and any following CPUs or return an
> error if that fails.
>
> Instead walk all CPUs and if paging is enabled retrieve the memory
>
On Fri, 31 May 2013 21:04:10 +0800
Wenchao Xia wrote:
> 于 2013-5-30 10:41, Wenchao Xia 写道:
> > 于 2013-5-27 23:41, Kevin Wolf 写道:
> >> Am 25.05.2013 um 05:09 hat Wenchao Xia geschrieben:
> >>> These patches are the common part of my hmp/qmp block query series
> >>> and Pavel's
> >>> qmp snapshot c
On Thu, 30 May 2013 17:07:52 +0200
Andreas Färber wrote:
> Hello,
>
> This series is an alternative to patches previously queued or posted,
> based on virgin master.
>
> As requested by Paolo, this replaces Kate's previous memory_mapping split
> and my follow-ups and instead goes directly for m
Il 18/04/2013 20:47, Orr Dvory ha scritto:
> is this better?
Yes -- but I don't remember if this was fixed elsewhere. Can you
reproduce the problem in 1.5.0? If so, please send the patch according
to the guidelines at http://wiki.qemu.org/Contribute/SubmitAPatch.
Paolo
> diff -uprN qemu-1.4.1/
One qapi fix and two fixes that affect the dump-guest-memory QMP command.
The changes (since 87d23f78aa79b72da022afda358bbc8a8509ca70) are available
in the following repository:
git://repo.or.cz/qemu/qmp-unstable.git queue/qmp
Luiz Capitulino (1):
target-i386: fix abort on bad PML4E/PDPTE/
From: Michael Roth
With the introduction of native list types, we now have types such as
int64List where the 'value' field is not a pointer, but the actual
64-bit value.
On 32-bit architectures, this can lead to situations where 'next' field
offset in GenericList does not correspond to the 'next
The code used to walk IA-32e page-tables, and possibly PAE page-tables,
uses the bit mask ~0xfff to get the next PML4E/PDPTE/PDE/PTE address.
However, as we use a uint64_t to store the resulting address, that mask
gets expanded to 0xf000 which not only ends up selecting
reserved bits b
On 05/31/13 16:08, David Woodhouse wrote:
> On Fri, 2013-05-31 at 08:04 -0500, Anthony Liguori wrote:
>>
>>
>>
>> Fork OVMF, drop the fat module, and just add GPL code. It's an easily
>> solvable problem.
>
> Heh. Actually it doesn't need to be a fork. It's modular, and the FAT
> driver is just
From: Qiao Nuohan
Function walk_pte() needs pte index to calculate virtual address.
However, pte index of PAE paging or IA-32e paging is 9 bit, so the mask
should be 0x1ff.
Signed-off-by: Qiao Nuohan
Reviewed-by: Jesse Larrew
Signed-off-by: Andreas Färber
Signed-off-by: Luiz Capitulino
---
On 31 May 2013 14:01, Thomas Schwinge wrote:
> On Fri, 31 May 2013 13:34:12 +0100, Peter Maydell
> wrote:
>> That said, I think any new patches to fpu/ need to
>> come with an explicit statement that they can be
>> licensed under the softfloat-2a license or GPLv2
>> or BSD (etc etc) so they aren
Laszlo Ersek writes:
> On 05/31/13 15:04, Anthony Liguori wrote:
>> Laszlo Ersek writes:
>>
>>> On 05/31/13 09:09, Jordan Justen wrote:
>>>
>>> Due to licensing differences I can't just port code from SeaBIOS to
>>> OVMF
>>
>>
>
> :)
>
>> Fork OVMF, drop the fat module, and just add GPL code.
Ping.
On 05/29/2013 12:30 PM, Richard Henderson wrote:
> The code reorganization in commit 4a6fd938 broke handling of PREFIX_ADR.
> While fixing this, tidy and comment the code so that it's more obvious
> what's going on in setting both aflag and dflag.
>
> The TARGET_X86_64 ifdef can be eliminat
Il 31/05/2013 17:03, Richard Henderson ha scritto:
> Ping.
>
> On 05/29/2013 12:30 PM, Richard Henderson wrote:
>> The code reorganization in commit 4a6fd938 broke handling of PREFIX_ADR.
>> While fixing this, tidy and comment the code so that it's more obvious
>> what's going on in setting both a
Public bug reported:
Guest : windows Seven / XP
Qemu version : 1.5.0
cmd line :
-drive
file=fat:floppy:/mnt/vdisk/diskconf/TEST004/,if=none,id=drive-fdc0-0-0,readonly=on
generated by libvirt :
works with qemu <= 1.4.1
with qemu 1.5.0 , gues
31.05.2013 16:00, Paolo Bonzini wrote:
> Two instances, both spotted by Coverity. In one, two blocks were
> swapped. In the other, the check is not needed anymore.
Thanks, applied to the trivial patches queue.
/mjt
David Woodhouse writes:
> On Fri, 2013-05-31 at 08:04 -0500, Anthony Liguori wrote:
>>
>>
>>
>> Fork OVMF, drop the fat module, and just add GPL code. It's an easily
>> solvable problem.
>
> Heh. Actually it doesn't need to be a fork. It's modular, and the FAT
> driver is just a single module
Hi!
On Fri, 31 May 2013 13:34:12 +0100, Peter Maydell
wrote:
> On 31 May 2013 13:07, Michael Tokarev wrote:
> > Hmm. And where's the simplification? Here's context diff for the same:
> >
> > *** fpu/softfloat-specialize.h.orig 2013-05-31 16:02:51.614710351 +0400
> > --- fpu/softfloat-speci
Signed-off-by: Thomas Schwinge
---
fpu/softfloat-specialize.h | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git fpu/softfloat-specialize.h fpu/softfloat-specialize.h
index 518f694..83add1a 100644
--- fpu/softfloat-specialize.h
+++ fpu/softfloat-specialize.h
@@ -9
On Fri, 2013-05-31 at 10:43 -0500, Anthony Liguori wrote:
> It's even more fundamental. OVMF as a whole (at least in it's usable
> form) is not Open Source.
The FAT module is required to make EDK2 usable, and yes, that's not Open
Source. So in a sense you're right.
But we're talking here about
Am 23.05.2013 13:51, schrieb Christian Borntraeger:
> With the ccw ipl code sometimes an error message like
> "virtio: trying to map MMIO memory" or
> "Guest moved used index from %u to %u" appeared. Turns out
> that the ccw bios did not zero out the vring, which might
> cause stale values in avail
On 05/31/13 17:43, Anthony Liguori wrote:
> David Woodhouse writes:
>
>> On Fri, 2013-05-31 at 08:04 -0500, Anthony Liguori wrote:
>>>
>>>
>>>
>>> Fork OVMF, drop the fat module, and just add GPL code. It's an easily
>>> solvable problem.
>>
>> Heh. Actually it doesn't need to be a fork. It's m
On 05/31/13 16:38, Anthony Liguori wrote:
> It's either Open Source or it's not. It's currently not.
I disagree with this binary representation of Open Source or Not. If it
weren't (mostly) Open Source, how could we fork (most of) it as you're
suggesting (from the soapbox :))?
> I have a hard
>
This is a git script that will iterate through every commit in a
specified range, and perform a configure and make. The intention of
this script is not to act as a check of code correctness, but to see if
any commit breaks compilation of the tree.
The idea is that prior to submitting a patch or p
On 05/31/13 18:33, David Woodhouse wrote:
> On Fri, 2013-05-31 at 10:43 -0500, Anthony Liguori wrote:
>> It's even more fundamental. OVMF as a whole (at least in it's usable
>> form) is not Open Source.
>
> The FAT module is required to make EDK2 usable, and yes, that's not Open
> Source. So in
David Woodhouse writes:
> On Fri, 2013-05-31 at 10:43 -0500, Anthony Liguori wrote:
>> It's even more fundamental. OVMF as a whole (at least in it's usable
>> form) is not Open Source.
>
> The FAT module is required to make EDK2 usable, and yes, that's not Open
> Source. So in a sense you're ri
Laszlo Ersek writes:
> On 05/31/13 16:38, Anthony Liguori wrote:
>
>> It's either Open Source or it's not. It's currently not.
>
> I disagree with this binary representation of Open Source or Not. If it
> weren't (mostly) Open Source, how could we fork (most of) it as you're
> suggesting (from t
Spotted by Coverity.
Signed-off-by: Alon Levy
---
libcacard/vscclient.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/libcacard/vscclient.c b/libcacard/vscclient.c
index ac23647..9fcc548 100644
--- a/libcacard/vscclient.c
+++ b/libcacard/vscclient.c
@@ -618,18 +618,
> Spotted by Coverity.
>
Self NACK.
I'll send a more complete patch, and use closesocket.
> Signed-off-by: Alon Levy
> ---
> libcacard/vscclient.c | 8 ++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/libcacard/vscclient.c b/libcacard/vscclient.c
> index ac23647..9
Il 31/05/2013 19:06, Anthony Liguori ha scritto:
> David Woodhouse writes:
>
>> On Fri, 2013-05-31 at 10:43 -0500, Anthony Liguori wrote:
>>> It's even more fundamental. OVMF as a whole (at least in it's usable
>>> form) is not Open Source.
>>
>> The FAT module is required to make EDK2 usable,
'default_backend' isn't always set, but 'rng' is, so use that.
$ ./x86_64-softmmu/qemu-system-x86_64 -object
rng-random,id=rng0,filename=/dev/random -device virtio-rng-pci,rng=rng0
Segmentation fault (core dumped)
Regressed with virtio refactoring in 59ccd20a9ac719cff82180429458728f03ec612f
CC:
Fix RTC_CHANGE event description to match implementation.
Signed-off-by: Marcelo Tosatti
diff --git a/QMP/qmp-events.txt b/QMP/qmp-events.txt
index 92fe5fb..00b4087 100644
--- a/QMP/qmp-events.txt
+++ b/QMP/qmp-events.txt
@@ -203,7 +203,8 @@ Emitted when the guest changes the RTC time.
Data:
Paolo Bonzini writes:
> Il 31/05/2013 19:06, Anthony Liguori ha scritto:
>> David Woodhouse writes:
>>
>>> On Fri, 2013-05-31 at 10:43 -0500, Anthony Liguori wrote:
It's even more fundamental. OVMF as a whole (at least in it's usable
form) is not Open Source.
>>>
>>> The FAT module
Applied. Thanks.
Regards,
Anthony Liguori
Applied. Thanks.
Regards,
Anthony Liguori
1 - 100 of 147 matches
Mail list logo
