Actually, the PREROUTING lines are not needed in the above script,
sorry.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1078305
Title:
source IP address of broadcast packets gets rewritten when usi
On Tue, Nov 13, 2012 at 03:14:55PM +0100, Kevin Wolf wrote:
> @@ -691,12 +685,15 @@ static int bdrv_open_common(BlockDriverState *bs, const
> char *filename,
>
> /* Open the image, either directly or using a protocol */
> if (drv->bdrv_file_open) {
> +if (file != NULL) {
> +
On 11/14/12 06:23, David Gibson wrote:
> Elements in qemu SGLists can cross IOMMU page boundaries. So, in commit
> 39c138c8420f51a7da7b35233a8d7400a0b589ac "usb: Fix usb_packet_map() in the
> presence of IOMMUs", I changed usb_packet_map() to split up each SGList
> element on IOMMU page boundaries
Hi,
> An alternate solution (which works in my build environment) is passing
> --host=... (value derived from QEMU cross_prefix) to the pixman configure.
Sounds reasonable, I'll try.
>> +nm="${AR-${cross_prefix}nm}"
>
> NM instead of AR?
>
> RANLIB instead of LD?
Indeed. /me wonders why th
On Wed, Nov 14, 2012 at 11:50:06AM +0800, Wenchao Xia wrote:
> In order to resolve OOM issue, I am trying wrap all APIs using
> sunrpc, need some suggestion before coding.
Is the client/server approach really necessary or can you write a
library that invokes qemu-nbd/qemu-img?
If there is a sta
Il 14/11/2012 09:32, Stefan Hajnoczi ha scritto:
> On Tue, Nov 13, 2012 at 03:14:55PM +0100, Kevin Wolf wrote:
>> @@ -691,12 +685,15 @@ static int bdrv_open_common(BlockDriverState *bs,
>> const char *filename,
>>
>> /* Open the image, either directly or using a protocol */
>> if (drv-
Il 14/11/2012 04:50, Wenchao Xia ha scritto:
> There are some different way to implement, not sure which would be
> better:
> 1 keep client as thin as possible, client stores opaque pointer used
> in server side, for eg, QBlockContext *ctx, client only get a pointer
> pointing to the address wh
Am 14.11.2012 09:32, schrieb Stefan Hajnoczi:
> On Tue, Nov 13, 2012 at 03:14:55PM +0100, Kevin Wolf wrote:
>> @@ -691,12 +685,15 @@ static int bdrv_open_common(BlockDriverState *bs,
>> const char *filename,
>>
>> /* Open the image, either directly or using a protocol */
>> if (drv->bd
On 11/13/12 18:57, Stefan Weil wrote:
> Am 13.11.2012 09:42, schrieb Gerd Hoffmann:
>> Needed to make sure the (generated) pixman-version.h file is found.
>> Based on a patch from Blue Swirl.
>>
>> Signed-off-by: Gerd Hoffmann
>> ---
>> configure |6 +++---
>> 1 files changed, 3 insertions(
Ok, I srewed up the iptables rules a little, so this is a working
workaround script:
IFACE=$(route -n | grep '^0.0.0.0 ' | sed 's/.* //g')
main_broadcast_addr=$(ip -4 addr show "$IFACE" | grep 'inet .* brd ' | sed
's/.* brd //;s/\([0-9.]*\).*/\1/')
iptables -t nat -A OUTPUT -d 255.255.255.255 -p
On Tue, Nov 13, 2012 at 6:11 PM, Paolo Bonzini wrote:
>> > Il 05/11/2012 06:38, Liu Ping Fan ha scritto:
>> > > From: Liu Ping Fan
>> > >
>> > > If out of global lock, we will be challenged by SMP in low level,
>> > > so need atomic ops.
>> > >
>> > > This file is a wrapper of GCC atomic builtin.
Il 14/11/2012 10:38, liu ping fan ha scritto:
> On Tue, Nov 13, 2012 at 6:11 PM, Paolo Bonzini wrote:
Il 05/11/2012 06:38, Liu Ping Fan ha scritto:
> From: Liu Ping Fan
>
> If out of global lock, we will be challenged by SMP in low level,
> so need atomic ops.
>
> Thi
> On Wed, Nov 14, 2012 at 11:50:06AM +0800, Wenchao Xia wrote:
In order to resolve OOM issue, I am trying wrap all APIs using
sunrpc, need some suggestion before coding.
Is the client/server approach really necessary or can you write a
library that invokes qemu-nbd/qemu-img?
If there is a s
Inline
> -Original Message-
> From: Isaku Yamahata [mailto:yamah...@valinux.co.jp]
> Sent: 14 November 2012 02:23
> To: Hudzia, Benoit
> Cc: quint...@redhat.com; qemu-devel qemu-devel; Orit Wasserman;
> chegu_vi...@hp.com; Michael Roth
> Subject: Re: Migration To-do list
>
> On Tue, Nov
> Il 14/11/2012 04:50, Wenchao Xia ha scritto:
>>There are some different way to implement, not sure which would be
>> better:
>>1 keep client as thin as possible, client stores opaque pointer used
>> in server side, for eg, QBlockContext *ctx, client only get a pointer
>> pointing to the a
Il 14/11/2012 10:55, Wenchao Xia ha scritto:
>>In order to resolve OOM issue, I am trying wrap all APIs using
>> sunrpc, need some suggestion before coding.
>
> Is the client/server approach really necessary or can you write a
> library that invokes qemu-nbd/qemu-img?
>
> If there is a startu
Add a standard vga variant which doesn't occupy any legacy
ressources and thus can easily be used as secondary (or legacy-free)
graphics adapter. Programming must be done using the MMIO bar.
---
docs/specs/standard-vga.txt | 13 +++--
hw/vga-pci.c| 62 +
Add a standard vga variant which doesn't occupy any legacy
ressources and thus can easily be used as secondary (or legacy-free)
graphics adapter. Programming must be done using the MMIO bar.
---
docs/specs/standard-vga.txt | 13 +++--
hw/vga-pci.c| 62 +
Dear AllI am going to add another debugger server into QEMU, so i need to1)
create another socket server, similar than gdbserver, to receive the command
from debugger2) add another option, such as "--start-debugger-server" to start
the socket server.
After I work out the whole thing, how can
On 14 November 2012 10:45, Peter Cheung wrote:
> Dear All
> I am going to add another debugger server into QEMU, so i need to
> 1) create another socket server, similar than gdbserver, to receive the
> command from debugger
> 2) add another option, such as "--start-debugger-server" to start th
On 11/14/2012 04:23 AM, Isaku Yamahata wrote:
> On Tue, Nov 13, 2012 at 05:46:13PM +, Hudzia, Benoit wrote:
>> Hi,
>>
>> One concept we have been playing around in the context of and hybrid and
>> post copy and might make sense if you are orienting your effort toward RDMA
>> / Post copy is t
NAT networking is provided by libvirt, changing the package.
** Project changed: qemu => libvirt (Ubuntu)
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1078305
Title:
source IP address of broadcas
On Wed, Nov 14, 2012 at 01:03:46AM +0100, Andreas Färber wrote:
> Am 09.11.2012 15:56, schrieb Eduardo Habkost:
> > From: Igor Mammedov
> >
> > it's necessary for making CPU child of DEVICE without
> > causing circular header deps.
> >
> > Signed-off-by: Igor Mammedov
> > [ehabkost: re-added th
On 11/14/12 09:40, Gerd Hoffmann wrote:
>>> >> + pixman_cflags="-I${source_path}/pixman/pixman -I$(pwd)/pixman/pixman"
>>> >> + pixman_libs="-L$(pwd)/pixman/pixman/.libs -lpixman-1"
>> >
>> > I'd prefer \$(BUILD_DIR) instead of $(pwd) for cflags and libs
>> > because it allows moving the build
On 11/13/12 19:41, Johnson, Eric wrote:
> It's OK to add.
> Signed-off-by: Eric Johnson
Updated patch.
> I wasn't sure how to submit a patch to an unsubmitted patch.
As usual: "git commit -s" + "git format-patch" + "git send-email",
especially as this one stands on its own and has no dependency
Hi,
This patch series fixes the build issues with the internal pixman
submodule. It also adds licensing info to the qemu-pixman.[ch] files.
v2 improves things a bit based on comments from Stefan Weil, fixes some
tyops found by Peter Maydell and adds a patch which also cleans up the
pixman dire
gtk is only needed to build test cases.
Disable it to simplify the build.
Signed-off-by: Gerd Hoffmann
---
Makefile |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/Makefile b/Makefile
index ff2c16d..e68bb8f 100644
--- a/Makefile
+++ b/Makefile
@@ -122,7 +122,7 @@ subdir
Needed to make sure the (generated) pixman-version.h file is found.
Based on a patch from Blue Swirl.
Signed-off-by: Gerd Hoffmann
---
configure | 12 +++-
1 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/configure b/configure
index 7290f50..e6fe4f8 100755
--- a/configure
Set --host when calling pixman configure while doing cross builds so
pixman's autoconf picks up the cross build tools correctly.
Signed-off-by: Gerd Hoffmann
---
Makefile |2 +-
configure |5 +
2 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/Makefile b/Makefile
index
Signed-off-by: Eric Johnson
Signed-off-by: Gerd Hoffmann
---
configure |7 ---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/configure b/configure
index 0a241eb..f847ee2 100755
--- a/configure
+++ b/configure
@@ -3956,9 +3956,6 @@ if test "$target_softmmu" = "yes" ; then
Pass on CFLAGS to the pixman configure script.
Add -fPIC to the cflags, needed to make the final link succeed.
Signed-off-by: Gerd Hoffmann
---
Makefile |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/Makefile b/Makefile
index e68bb8f..4538b87 100644
--- a/Makefile
+++
Signed-off-by: Gerd Hoffmann
---
Makefile |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/Makefile b/Makefile
index 4538b87..81c660f 100644
--- a/Makefile
+++ b/Makefile
@@ -278,6 +278,7 @@ distclean: clean
for d in $(TARGET_DIRS) $(QEMULIBS); do \
rm -rf
Signed-off-by: Gerd Hoffmann
---
qemu-pixman.c |5 +
qemu-pixman.h |5 +
2 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/qemu-pixman.c b/qemu-pixman.c
index 7547ed7..063b52d 100644
--- a/qemu-pixman.c
+++ b/qemu-pixman.c
@@ -1,3 +1,8 @@
+/*
+ * This work is license
Add a new timer based on vm_clock for 1 ns in the future from post_load
to do the event send in case host_connected differs between migration
source and target.
RHBZ: 867366
Signed-off-by: Alon Levy
---
hw/virtio-serial-bus.c | 54 --
1 file chang
Hi,
I've tried to make 'make test' at least build on an s390 host
(not that it does anything interesting yet...) by changing the
compiler for a test that's supposed to be architecture agnostic
and doing some whitespace fixes in find-in-path.
Build tested on Linux/s390 and Linux/x86.
Cornelia Huc
Whitespace needs to be stripped before building the path and in
the end to ensure we really get an empty string.
Signed-off-by: Cornelia Huck
---
rules.mak | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/rules.mak b/rules.mak
index 1b173aa..66b1d79 100644
--- a/rules.mak
test_path is supposed to be run for all architectures, so it should
use the main compiler instead of the i386 compiler.
Signed-off-by: Cornelia Huck
---
tests/tcg/Makefile | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/tcg/Makefile b/tests/tcg/Makefile
index 24e315
Il 14/11/2012 14:09, Alon Levy ha scritto:
> Add a new timer based on vm_clock for 1 ns in the future from post_load
> to do the event send in case host_connected differs between migration
> source and target.
>
> RHBZ: 867366
>
> Signed-off-by: Alon Levy
> ---
> hw/virtio-serial-bus.c | 54
>
Am 09.11.2012 15:56, schrieb Eduardo Habkost:
> From: Anthony Liguori
>
> Header file dependency is a frickin' nightmare right now. cpu.h tends to get
> included in our 'include everything' header files but qdev also needs to
> include
> those headers mainly for qdev-properties since it knows a
Am 09.11.2012 15:56, schrieb Eduardo Habkost:
> module.h is where machine_init() is defined, but qemu-fsdev-dummy.c
> doesn't include it.
>
> The header is probably being included by accident because some other
> headers are including qemu-common.h, but those headers should eventually
> stop inclu
Signed-off-by: Olivia Yin
---
hw/loader.c | 39 +++
hw/loader.h |7 +++
2 files changed, 46 insertions(+), 0 deletions(-)
diff --git a/hw/loader.c b/hw/loader.c
index ba01ca6..a8a0a09 100644
--- a/hw/loader.c
+++ b/hw/loader.c
@@ -86,6 +86,39 @@ int
Signed-off-by: Olivia Yin
---
elf.h| 10 ++
hw/elf_ops.h | 101 ++
2 files changed, 111 insertions(+), 0 deletions(-)
diff --git a/elf.h b/elf.h
index a21ea53..335f1af 100644
--- a/elf.h
+++ b/elf.h
@@ -1078,6 +1078,16 @@ typ
The current model of loader copy "rom blobs" and kept in memory until
a reset occurs and waste host memory.
This serial of patches uses private reset handlers to load from hard
disk on reset, which could make loader framework more dynamic and
reduce the memory consumption of QEMU process.
Oliv
Can someone go ahead and follow up on comment #6? (Marking incomplete
in the meantime)
I'll mark as affecting the QEMU project since it has been confirmed
against 1.2.0 per comment #4.
** Changed in: qemu-kvm (Ubuntu)
Importance: Undecided => Medium
** Changed in: qemu-kvm (Ubuntu)
St
Am 09.11.2012 15:56, schrieb Eduardo Habkost:
> is needed for the 'bool' type, used in the header.
>
> The header is probably being included by accident because some other
> headers are including qemu-common.h, but those headers should eventually
> stop including qemu-common.h.
>
> Signed-off-by
Am 09.11.2012 15:56, schrieb Eduardo Habkost:
> Include:
> - for g_malloc0()
> - for memset()
>
> Some of those headers were probably being included by accident because
> some other headers were including qemu-common.h, but those headers
> should eventually stop including qemu-common.h.
>
> S
According to the MIPS Malta Developement Platform User's Manual, the
i8259 interrupt controller is supposed to be connected to the hardware
IRQ0, and the CBUS UART to the hardware interrupt 2.
In QEMU they are both connected to hardware interrupt 0, the CBUS UART
interrupt being wrong. This patch
Hi Cannot use gdb protocol, because it is slow.Also, the original gdb stub
can dump out GDT/IDT/CR0 directly.
Thanksfrom Peter
> Date: Wed, 14 Nov 2012 10:54:02 +
> From: peter.mayd...@linaro.org
> To: mcheun...@hotmail.com
> CC: qemu-devel@nongnu.org
> Subject: Re: [Qemu-devel] Adding ano
Am 09.11.2012 15:56, schrieb Eduardo Habkost:
> Include:
> - for FILE
> - qemu-option.h for QemuOptsList
>
> Some of those headers were probably being included by accident because
> some other headers were including qemu-common.h, but those headers
> should eventually stop including qemu-common.h
Signed-off-by: Olivia Yin
---
hw/loader.c | 64 ++
1 files changed, 46 insertions(+), 18 deletions(-)
diff --git a/hw/loader.c b/hw/loader.c
index a8a0a09..1a909d0 100644
--- a/hw/loader.c
+++ b/hw/loader.c
@@ -55,6 +55,8 @@
#include
On Wed, Nov 14, 2012 at 12:33:50AM +0900, Takuya Yoshikawa wrote:
> Ccing live migration developers who should be interested in this work,
>
> On Mon, 12 Nov 2012 21:10:32 -0200
> Marcelo Tosatti wrote:
>
> > On Mon, Nov 05, 2012 at 05:59:26PM +0800, Xiao Guangrong wrote:
> > > Do not drop large
Commit a844ed842d9a9d929645c09ae0f52f753d7a02e0 leads to usb-host
detecting devices not right after qemu startup because the guest
isn't running yet. Instead they are found on the first of the
regular usb device poll runs. Which is too late for seabios to see
them, so booting from usb sticks fail
On 2012-11-14 15:40, Peter Cheung wrote:
> Hi Cannot use gdb protocol, because it is slow.
For any practical purpose I came across so far while debugging guest
kernels, that is not true when running against QEMU. Sure, if you want
to dump gigabytes of RAM this way, it will be a bottleneck. But w
Il 12/11/2012 12:44, Paolo Bonzini ha scritto:
>> @@ -339,7 +351,11 @@ void migrate_fd_put_ready(MigrationState *s)
>> DPRINTF("done iterating\n");
>> start_time = qemu_get_clock_ms(rt_clock);
>> qemu_system_wakeup_request(QEMU_WAKEUP_REASON_OTHER);
>> -vm_stop_fo
They are implemented in osdep.c, so keep the prototypes in osdep.h.
Suggested-by: Igor Mammedov
Signed-off-by: Andreas Färber
Cc: Eduardo Habkost
---
osdep.h |3 +++
qemu-common.h |2 --
2 Dateien geändert, 3 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-)
diff --git a/osdep.h b/osd
Am 09.11.2012 15:56, schrieb Eduardo Habkost:
> Include:
> - for g_malloc0()
> - for strcmp()
>
> Some of those headers were probably being included by accident because
> some other headers were including qemu-common.h, but those headers
> should eventually stop including qemu-common.h.
>
> Sig
On Wed, Nov 14, 2012 at 04:46:07PM +0100, Andreas Färber wrote:
> They are implemented in osdep.c, so keep the prototypes in osdep.h.
>
> Suggested-by: Igor Mammedov
> Signed-off-by: Andreas Färber
> Cc: Eduardo Habkost
Acked-by: Eduardo Habkost
> ---
> osdep.h |3 +++
> qemu-com
Am 09.11.2012 15:56, schrieb Eduardo Habkost:
> Include:
> - for errno
> - & for fcntl()
> - "qemu-stdio.h" for qemu_open()
>
> Some of those headers were probably being included by accident because
> some other headers were including qemu-common.h, but those headers
> should eventually stop
While working on moving usb-redir and usb-hid over to using async packet
handling for their interrupt input endpoints. I've found and fixed quite
a few ehci bugs.
Unfortunately the moving to async for interrupt endpoints turns out to be a
bad idea, as it causes issues for migration, an async compl
Signed-off-by: Hans de Goede
---
hw/usb/hcd-ehci.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c
index 13e9fd3..6d23af1 100644
--- a/hw/usb/hcd-ehci.c
+++ b/hw/usb/hcd-ehci.c
@@ -191,6 +191,7 @@ static int ehci_state_executing(EHCIQueue
This is not allowed, except for clearing active on cancellation, so don't
warn when the new token does not have its active bit set.
This unifies the cancellation path for modified qtd-s, and prepares
ehci_verify_qtd to be used ad an extra check inside
ehci_writeback_async_complete_packet().
Signe
Windows-XP likes to play tricks with the next pointer for periodic qh-s, so
far we were not hit by this as we never called fill_queue for periodic qh-s,
but with the move to async packet handling for interrupt endpoints this
becomes an issue.
Signed-off-by: Hans de Goede
---
hw/usb/hcd-ehci.c |
No other changes.
Signed-off-by: Hans de Goede
---
hw/usb/hcd-ehci.c | 60 +++
1 file changed, 30 insertions(+), 30 deletions(-)
diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c
index e83bdde..13e9fd3 100644
--- a/hw/usb/hcd-ehci.c
+++ b/hw/
Gerd Hoffmann writes:
> Hi,
>
> Here comes the spice patch queue. It carries some qxl bugfixes,
> switches spice-display over to pixman, drops the obsolete pflib bits
> and fixes some fallout from the console cleanup patch series.
>
> please pull,
> Gerd
>
> The following changes since commi
Gerd Hoffmann writes:
> Hi,
>
> This is the usb patch queue. It cleans up the USBPacket error handling
> to have separate length and status fields which is needed to get some
> corner cases correct. While being at it it also makes status reporting
> identical for both sync and async USBPacket
Remove the short-circuiting of fetchqtd in fetchqh, so that the
qtd gets properly verified before completing the transaction.
Signed-off-by: Hans de Goede
---
hw/usb/hcd-ehci.c | 15 +--
1 file changed, 1 insertion(+), 14 deletions(-)
diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci
Gerd Hoffmann writes:
> Hi,
>
> This patch series fixes the build issues with the internal pixman
> submodule. It also adds licensing info to the qemu-pixman.[ch] files.
>
> please pull,
> Gerd
>
> The following changes since commit f5022a135e4309a54d433c69b2a056756b2d0d6b:
>
> aio: fix ai
Paolo Bonzini writes:
> Anthony,
>
> The following changes since commit 3c5645fab3c4b65d0cffbe1aaafc787e4be63d0f:
>
> tcg: properly check that op's output needs to be synced to memory
> (2012-11-11 16:06:46 +0100)
>
> are available in the git repository at:
>
> git://github.com/bonzini/qemu.
Alex Williamson writes:
> Hi Anthony,
>
> Please pull the tag below. I posted the linux-headers update
> separately on Oct-15; since it hasn't been applied and should be
> non-controversial, I include it again here. Thanks,
>
> Alex
>
Pulled. Thanks.
Regards,
Anthony Liguori
> The following
hi Jan, you are the maintainer of the gdb server of qemu?
I think if I can't create my debug protocol, it is not easy to adopt
peter-bochs debugger to qemu, in peter-bochs, there are some features I think
current gdb protocol doesn't care, such as profiling, kernel module monitoring,
call graph
ehci_state_writeback() will free the packet, so we should not access
the packet after calling ehci_state_writeback().
Signed-off-by: Hans de Goede
---
hw/usb/hcd-ehci.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c
index ee6c9a
Il 14/11/2012 17:25, Thanos Makatos ha scritto:
> We don't use qemu's VHD driver in XenServer. Instead, we use blktap2
> to create a block device in dom0 serving the VHD file in question,
> and have qemu open that block device instead of the VHD file itself.
Yes, the question is how you handle dis
On 2012-11-14 17:28, Peter Cheung wrote:
> hi Jan, you are the maintainer of the gdb server of qemu?
Not formally. I'm heavily using it for kernel debugging for a couple of
years. Therefore, I'm fixing and enhancing it from time to time.
> I think if I can't create my debug protocol, it is not ea
Windows links interrupt qtd-s in circles, which means that when interrupt
endpoints return USB_RET_ASYNC, combined with the recent
"ehci: Retry to fill the queue while waiting for td completion" patch,
we keep adding the tds to the queue over and over again, as we detect the
circle from fill_queue,
Signed-off-by: Hans de Goede
---
hw/usb/hcd-ehci.c | 45 -
1 file changed, 32 insertions(+), 13 deletions(-)
diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c
index 7772c33..a694346 100644
--- a/hw/usb/hcd-ehci.c
+++ b/hw/usb/hcd-ehci.c
@@ -437,6 +437
This separates the qdev properties code in two parts:
- qdev-properties.c, that contains most of the qdev properties code;
- qdev-properties-system.c for code specific for qemu-system-*,
containing:
- Property types: drive, chr, netdev, vlan, that depend on code that
won't be included
This avoids the q->qtdaddr == p->qtdaddr asserts we have triggering, when
a queue contains multiple completed packages when we cancel the queue.
I triggered this with windows7 + async interrupt endpoint handling (*)
+ not detecting circles in ehci_fill_queue() properly, which makes the qtd
validat
Also drop the warning printf, which was there mainly because this was an
untested code path (as the previous bug fixes to it show), but that no
longer is the case now :)
Signed-off-by: Hans de Goede
---
hw/usb/hcd-ehci.c | 28 +---
1 file changed, 17 insertions(+), 11 del
Thanks, Paul, I'll cherrypick commit
c8969eded252058e90e91f12f75f32aceae46ec9 into the ubuntu packages
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1077838
Title:
qemu-nbd -r -c taints device for
** Changed in: qemu-kvm (Ubuntu)
Assignee: (unassigned) => Serge Hallyn (serge-hallyn)
** Changed in: qemu-kvm (Ubuntu)
Status: Triaged => In Progress
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad
On Fri, Nov 02, 2012 at 09:21:26AM -0400, Boris Ostrovsky wrote:
> From: Andre Przywara
>
> Update QEMU's knowledge of CPUID bit names. This allows to
> enable/disable those new features on QEMU's command line when
> using KVM and prepares future feature enablement in QEMU.
>
> This adds F16C, R
On Fri, Nov 02, 2012 at 09:21:38AM -0400, Boris Ostrovsky wrote:
> From: Andre Przywara
>
> Add a new base CPU model called Opteron_G5 to model the latest
> Opteron CPUs. This increases the model value and model numbers and
> adds TBM, F16C and FMA over the latest G4 model.
>
> Signed-off-by: An
From: Andre Przywara
Add a new base CPU model called Opteron_G5 to model the latest
Opteron CPUs. This increases the model value and model numbers and
adds TBM, F16C and FMA over the latest G4 model.
Signed-off-by: Andre Przywara
Signed-off-by: Boris Ostrovsky
[ehabkost: edited commit message]
> -Original Message-
> From: Gerd Hoffmann [mailto:kra...@redhat.com]
> Sent: Wednesday, November 14, 2012 4:42 AM
[...]
> On 11/13/12 19:41, Johnson, Eric wrote:
[...]
> > I wasn't sure how to submit a patch to an unsubmitted patch.
>
> As usual: "git commit -s" + "git format-patch" + "gi
I'm resending the patches sent by Boris Ostrovsky previously, as it has to be
rebased against latest qemu.git, and to add a new Haswell CPU model too.
Andre Przywara (2):
target-i386: cpu: name new CPUID bits
target-i386: cpu: add new Opteron CPU model
Eduardo Habkost (1):
target-i386: add
Features added to the model, in relation to SandyBridge:
fma CPUID[1].ECX[12]
pcid CPUID[1].ECX[17]
movbe CPUID[1].ECX[22]
fsgsbase CPUID[EAX=7,ECX=0].EBX[0]
bmi1 CPUID[EAX=7,ECX=0].EBX[3]
hle CPUID[EAX=7,ECX=0].EBX[4]
avx2 CPUID[EAX=7,ECX=0].EBX[5]
smep CPUID[
From: Hervé Poussineau
Signed-off-by: Hervé Poussineau
Signed-off-by: Kevin Wolf
---
hw/fdc.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/fdc.c b/hw/fdc.c
index a197c48..de8778b 100644
--- a/hw/fdc.c
+++ b/hw/fdc.c
@@ -1244,7 +1244,7 @@ static void fdctrl_start
From: Stefan Hajnoczi
Kevin has requested co-maintainership to give him more time to write
code. We will alternate patch review duties on a weekly basis.
Signed-off-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
MAINTAINERS |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff
From: Hervé Poussineau
Signed-off-by: Hervé Poussineau
Signed-off-by: Kevin Wolf
---
hw/fdc.c | 27 ---
1 files changed, 16 insertions(+), 11 deletions(-)
diff --git a/hw/fdc.c b/hw/fdc.c
index 3585220..7e4b0ce 100644
--- a/hw/fdc.c
+++ b/hw/fdc.c
@@ -327,7 +327,7 @@
It decided whether an interrupt is triggered. Only one caller made use
of this functionality, so move the code there.
In this one caller, the interrupt must actually be triggered
unconditionally, like it was before commit 2fee0088. For example, a
successful read without an implied seek can result
From: Stefan Hajnoczi
Now that AIOPool no longer keeps a freelist, it isn't really a "pool"
anymore. Rename it to AIOCBInfo and make it const since it no longer
needs to be modified.
Signed-off-by: Stefan Hajnoczi
Signed-off-by: Kevin Wolf
---
block.c | 22 +++---
From: Hervé Poussineau
Replace it by directly setting FD_SR0_SEEK if required
Signed-off-by: Hervé Poussineau
Signed-off-by: Kevin Wolf
---
hw/fdc.c | 12 ++--
1 files changed, 2 insertions(+), 10 deletions(-)
diff --git a/hw/fdc.c b/hw/fdc.c
index de8778b..29b5449 100644
--- a/hw/
From: Stefan Hajnoczi
Using appropriate types for variables is a good thing :). All users
simply do sizeof(MyType) and the value is passed to a memory allocator,
it should be size_t.
Signed-off-by: Stefan Hajnoczi
Reviewed-by: Paolo Bonzini
Signed-off-by: Kevin Wolf
---
qemu-aio.h |2 +-
From: Hervé Poussineau
On non-DMA transfers, fdctrl_stop_transfer() used to set FD_SR0_SEEK
no matter if there actually was a seek or not. This is obviously wrong.
fdctrl_seek_to_next_sect() has this information because it performs
the seek itself.
Signed-off-by: Hervé Poussineau
Signed-off-by
From: Hervé Poussineau
Signed-off-by: Hervé Poussineau
Signed-off-by: Kevin Wolf
---
tests/fdc-test.c |9 -
1 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/tests/fdc-test.c b/tests/fdc-test.c
index fa74411..a4303d1 100644
--- a/tests/fdc-test.c
+++ b/tests/fdc-test.c
Calling qemu_aio_flush() directly can hang when combined with I/O
throttling.
Signed-off-by: Kevin Wolf
Acked-by: Paolo Bonzini
---
hw/megasas.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/hw/megasas.c b/hw/megasas.c
index 291ff40..61b6527 100644
--- a/hw/megasas.c
From: Gerhard Wiesinger
Fixed a MAJOR BUG in VMDK files on file boundaries on reads
and ALSO ON WRITES WHICH MIGHT CORRUPT THE IMAGE AND DATA!!
Triggered for example with the following VMDK file (partly listed):
RW 4193792 FLAT "XP-W1-f001.vmdk" 0
RW 2097664 FLAT "XP-W1-f002.vmdk" 0
RW 41937
From: Hervé Poussineau
FD_MSR_CMDBUSY flag is already set in fdctrl_write_data(), just
before calling the command handler (fdctrl_start_transfer() here).
Signed-off-by: Hervé Poussineau
Signed-off-by: Kevin Wolf
---
hw/fdc.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --
ST0 shouldn't include 0x20 (FD_SR0_SEEK) after READ ID.
Signed-off-by: Kevin Wolf
Tested-by: Hervé Poussineau
---
tests/fdc-test.c | 66 ++
1 files changed, 66 insertions(+), 0 deletions(-)
diff --git a/tests/fdc-test.c b/tests/fdc-test.c
i
We don't use qemu's VHD driver in XenServer. Instead, we use blktap2 to create
a block device in dom0 serving the VHD file in question, and have qemu open
that block device instead of the VHD file itself.
> -Original Message-
> From: Stefano Stabellini [mailto:stefano.stabell...@eu.citri
1 - 100 of 177 matches
Mail list logo
