This patch introduce a new bit - enabled in TAPState which tracks whether a
specific queue/fd is enabled. The tap/fd is enabled during initialization and
could be enabled/disabled by tap_enalbe() and tap_disable() which calls platform
specific helpers to do the real work. Polling of a tap fd can on
This patch add migration support for multiqueue virtio-net. Instead of bumping
the version, we conditionally send the info of multiqueue only when the device
support more than one queue to maintain the backward compatibility.
Signed-off-by: Jason Wang
Signed-off-by: Michael S. Tsirkin
---
hw/vi
This patch add basic multiqueue support for Linux. When multiqueue is needed, we
will first check whether kernel support multiqueue tap before creating more
queues. Two new functions tap_fd_enable() and tap_fd_disable() were introduced
to enable and disable a specific queue. Since the multiqueue is
Disable multiqueue support for pre 1.4.
Signed-off-by: Jason Wang
Signed-off-by: Michael S. Tsirkin
---
hw/pc_piix.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/hw/pc_piix.c b/hw/pc_piix.c
index ba09714..0af436c 100644
--- a/hw/pc_piix.c
+++ b/hw/pc_piix.c
@@ -31
This patch adds basic multiqueue support for qemu. The idea is simple, an array
of NetClientStates were introduced in NICState, parse_netdev() were extended to
find and match all NetClientStates belongs to the backend and place their
pointers in NICConf. Then qemu_new_nic can setup a N:N mapping be
In multiqueue, all NetClientState that belongs to the same netdev or nic has the
same id. So this patches introduces an helper qemu_find_net_clients_except()
which finds all NetClientState with the same id. This will be used by multiqueue
networking.
Signed-off-by: Jason Wang
Signed-off-by: Micha
Hi
This includes xbzrle fixes. They are just:
- refactor xbzrle into its own file
- tests for xbzrle
- fix documentanion
- bugfix
- fix error message
please pull
The following changes since commit 8a55ebf01507ab73cc458cfcd5b9cb856aba0b9e:
Merge remote-tracking branch 'afaerber/qom-cpu'
To support multiqueue nic, this patch separate the nic destructor from
qemu_del_net_client() to a new helper qemu_del_nic() since the mapping bettween
NiCState and NetClientState were not 1:1 in multiqueue. The following patches
would refactor this function to support multiqueue nic.
Signed-off-by
On (Wed) 30 Jan 2013 [08:38:50], Anthony Liguori wrote:
> Amit Shah writes:
>
> > Hi Anthony,
> >
> > I did some basic testing of the char flow control patches from your
> > char-flow.2 branch. With the following patch applied, things seem to
> > be working fine. I tested the isa-serial and vir
On 02/01/2013 03:39 PM, Jason Wang wrote:
> Hello all:
>
> This seires is an update of last version of multiqueue virtio-net support.
Hi Anthony:
This series is not applied cleanly on master, could you please pick
those for 1.4?
Thanks
Am 01.02.2013 01:53, schrieb Alexander Graf:
> Hi Blue / Aurelien,
>
> This is my final pre-hard-freeze patch queue for ppc. Please pull.
Hi Alex, was there something wrong with my format fix v2? Oversight?
Andreas
>
> Alex
>
>
> The following changes since commit 6b2578d678497dbce44ed7999d
Hi Alex,
Am 31.01.2013 00:29, schrieb Andreas Färber:
> Prepares for changing cpu_interrupt() argument to CPUState.
>
> Signed-off-by: Andreas Färber
> ---
> Extracted from my qom-cpu-8 queue - could accompany my debug output fixes. :)
>
> hw/ppc.c | 12 ++--
> hw/ppc.h
On 01.02.2013, at 09:41, Andreas Färber wrote:
> Hi Alex,
>
> Am 31.01.2013 00:29, schrieb Andreas Färber:
>> Prepares for changing cpu_interrupt() argument to CPUState.
>>
>> Signed-off-by: Andreas Färber
>> ---
>> Extracted from my qom-cpu-8 queue - could accompany my debug output fixes. :)
On 01.02.2013, at 09:15, Andreas Färber wrote:
> Am 01.02.2013 01:53, schrieb Alexander Graf:
>> Hi Blue / Aurelien,
>>
>> This is my final pre-hard-freeze patch queue for ppc. Please pull.
>
> Hi Alex, was there something wrong with my format fix v2? Oversight?
Let's fix it after rc0.
Alex
Signed-off-by: Jason Wang
Signed-off-by: Michael S. Tsirkin
---
net/tap-linux.c |4 ++--
net/tap-win32.c |2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/net/tap-linux.c b/net/tap-linux.c
index 059f5f3..0a6acc7 100644
--- a/net/tap-linux.c
+++ b/net/tap-linux.c
@@ -1
Signed-off-by: Jason Wang
Signed-off-by: Michael S. Tsirkin
---
hw/virtio-net.c |2 +-
include/net/tap.h |4 ++--
net/tap-win32.c |6 +++---
net/tap.c | 38 ++
4 files changed, 24 insertions(+), 26 deletions(-)
diff --git a/hw/virtio
This patch implements both userspace and vhost support for multiple queue
virtio-net (VIRTIO_NET_F_MQ). This is done by introducing an array of
VirtIONetQueue to VirtIONet.
Signed-off-by: Jason Wang
Signed-off-by: Michael S. Tsirkin
---
hw/virtio-net.c | 301 +++
Am 01.02.2013 09:42, schrieb Alexander Graf:
>
> On 01.02.2013, at 09:41, Andreas Färber wrote:
>
>> Am 31.01.2013 00:29, schrieb Andreas Färber:
>>> Prepares for changing cpu_interrupt() argument to CPUState.
>>>
>>> Signed-off-by: Andreas Färber
>>> ---
>>> Extracted from my qom-cpu-8 queue -
Hi Andreas:
Thanks for the information, and sorry for the mess I've done.
I'll one-by-one re-send all the patches.
However because most of my patches are new files,
should I send-out the patches with detail change log?
For example:
[PATCH] dumb timer
... [PATCH v2 0/2] dumb timer (Cover letter)
Hi,
Am 01.02.2013 02:39, schrieb Kuo-Jung Su:
> 2013/2/1 Igor Mitsyanko
>>
>> On 01/25/2013 12:19 PM, Kuo-Jung Su wrote:
>>>
>>> +/* Board init. */
>>> +static void
>>> +a360_device_init(a360_state *s)
>>> +{
>>> +qemu_irq *pic;
>>> +qemu_irq ack, req;
>>> +qemu_irq cs_line;
>>> +
Am 20.01.2013 19:16, schrieb Andreas Färber:
> Am 20.01.2013 18:56, schrieb Andreas Färber:
>> DeviceState::parent_bus is document as private and should be accessed
>
> "documented" - please fix when applying.
>
>> through qdev_get_parent_bus(). Use a DEVICE() cast instead of accessing
>> ISADevi
Import multiqueue constants from if_tun.h from 3.8-rc3. A new ifr flag
IFF_MULTI_QUEUE were introduced to create a multiqueue backend by calling
TUNSETIFF with the this flag and with the same interface name many times.
A new ioctl TUNSETQUEUE were introduced. When doing this ioctl with
IFF_DETACH_
This patch factors out the common initialization of tap into a new helper
net_init_tap_one(). This will be used by multiqueue tap patches.
Signed-off-by: Jason Wang
Signed-off-by: Michael S. Tsirkin
---
net/tap.c | 130 ++---
1 files chan
Add a queue_index to VirtQueue and a helper to fetch it, this could be used by
multiqueue supported device.
Signed-off-by: Jason Wang
Signed-off-by: Michael S. Tsirkin
---
hw/virtio.c |8
hw/virtio.h |1 +
2 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/hw/virtio
To support multiqueue, this patch introduces a helper qemu_get_nic() to get
NICState from a NetClientState. The following patches would refactor this helper
to support multiqueue.
Signed-off-by: Jason Wang
Signed-off-by: Michael S. Tsirkin
---
hw/cadence_gem.c|8
hw/dp8393x
Am 25.01.2013 09:12, schrieb Andreas Färber:
> It is not being used in hot paths and is obsoleted by I2C_SLAVE()
> QOM cast macro. Clean it up using a scripted conversion, so that it
> doesn't get used in new code.
>
> Some of its callers were combining it with FROM_I2C_SLAVE() macro, which
> is e
嗨 國榮,
Am 01.02.2013 09:57, schrieb Kuo-Jung Su:
> Thanks for the information, and sorry for the mess I've done.
You don't need to apologize for every review comment you get. It's
meant for improvements of results, not personal. :)
> I'll one-by-one re-send all the patches.
>
> However because m
Am 31.01.2013 11:15, schrieb Michal Privoznik:
> Currently, we are enforcing the _FORTIFY_SOURCE=2 without any
> previous detection if the macro has been already defined, e.g.
> by environment, or is just enabled by compiler by default.
>
> Signed-off-by: Michal Privoznik
> ---
> configure | 2 +
Hello all:
During testing, I find doing scp during migration with vhost fails with
warnings in guest like:
Corrupted MAC on input.
Disconnecting: Packet corrupt.
lost connection
Here's the bisect result:
Commit a01672d3968cf91208666d371784110bfde9d4f8 kvm: convert to MemoryListener
API is the
> -Original Message-
> From: Alex Williamson [mailto:alex.william...@redhat.com]
> Sent: Tuesday, January 29, 2013 5:25 AM
> To: Pandarathil, Vijaymohan R
> Cc: Gleb Natapov; Bjorn Helgaas; Blue Swirl; Ortiz, Lance E;
> k...@vger.kernel.org; qemu-devel@nongnu.org; linux-...@vger.kernel.or
Fix the ordering of the endpoint descriptors for superspeed endpoints:
The superspeed companion must come first, possible additional
descriptors for the endpoint after that.
Signed-off-by: Gerd Hoffmann
---
hw/usb/desc.c |9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --g
Hi,
Early patches to add support for usb3 streams to qemu. Covered are
the subsystem core, xhci emulation and usb attached scsi emulation.
It's good enougth to trigger bugs in the linux xhci driver's stream
handling.
enjoy,
Gerd
Gerd Hoffmann (4):
usb: fix endpoint descriptor ordering
---
hw/usb.h | 10 ++
hw/usb/core.c | 10 ++
hw/usb/dev-bluetooth.c|2 +-
hw/usb/dev-hid.c |2 +-
hw/usb/dev-hub.c | 10 +-
hw/usb/dev-network.c |2 +-
hw/usb/dev-smartcard-rea
Signed-off-by: Gerd Hoffmann
---
hw/usb/hcd-xhci.c | 266 +++--
trace-events |6 +-
2 files changed, 220 insertions(+), 52 deletions(-)
diff --git a/hw/usb/hcd-xhci.c b/hw/usb/hcd-xhci.c
index 5fb0c48..ce17e2a 100644
--- a/hw/usb/hcd-xhci
Signed-off-by: Gerd Hoffmann
---
hw/usb/dev-uas.c | 247 --
1 file changed, 205 insertions(+), 42 deletions(-)
diff --git a/hw/usb/dev-uas.c b/hw/usb/dev-uas.c
index d904d1a..1ac5117 100644
--- a/hw/usb/dev-uas.c
+++ b/hw/usb/dev-uas.c
@@ -99,
Il 14/01/2013 19:21, Stefano Stabellini ha scritto:
I did a quick test and it seems that it should be possible to change
the amount of videoram for stdvga too using the same command line
option, however at the moment it just errors out. Therefore I am OK
with this patch only taking care of Cirr
Hi,
This patch series makes usb devices configurable.
cheers,
Gerd
Gerd Hoffmann (4):
usb: Makefile cleanup
fix scripts/make_device_config.sh
make usb devices configurable
allow disabling usb smartcard support
default-configs/alpha-softmmu.mak|1 +
default-configs/arm-softm
Group files, sprinkle in some comments.
Signed-off-by: Gerd Hoffmann
---
hw/usb/Makefile.objs | 27 ---
1 file changed, 20 insertions(+), 7 deletions(-)
diff --git a/hw/usb/Makefile.objs b/hw/usb/Makefile.objs
index d1bbbc0..bfe5e5f 100644
--- a/hw/usb/Makefile.objs
++
Leave the core usb devices (usb hub, tablet, mouse, keyboard)
enabled unconditionally. Make the other ones configurable.
Exceptions:
- bluetooth: not qdevified yet, has a vl.c dependency because
of that, thus disabling isn't as easy as not linking the
object file.
- smardcard: ccid-ca
Make it handle multiple include statements in a file.
Signed-off-by: Gerd Hoffmann
---
scripts/make_device_config.sh |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/make_device_config.sh b/scripts/make_device_config.sh
index 0778fe2..81fe942 100644
--- a/scripts/ma
Signed-off-by: Gerd Hoffmann
---
hw/Makefile.objs |2 ++
hw/usb/Makefile.objs |1 -
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/Makefile.objs b/hw/Makefile.objs
index 447e32a..a1f3a80 100644
--- a/hw/Makefile.objs
+++ b/hw/Makefile.objs
@@ -38,8 +38,10 @@ common-o
Am 27.01.2013 16:20, schrieb Andreas Färber:
> Am 27.01.2013 16:16, schrieb Andreas Färber:
>> The automatic drop of -Werror during the RC phases has in the past led
>> to warnings creeping into submaintainer trees.
>>
>> Last QEMU Summit it was concluded that -Werror should stay on and
>> enabled
Hello,
This series moves more fields from CPU_COMMON / CPU*State to CPUState,
allowing access from target-independent code.
The final patch in this series will help solve some issues (in particular
avoid a dependency on CPU_COMMON TLB refactoring for now) but opens a can
of worms: Since it is ini
Pass CPUState to cpu_exec_{start,end}() functions.
Signed-off-by: Andreas Färber
---
include/exec/cpu-defs.h |1 -
include/qom/cpu.h |2 ++
linux-user/main.c | 37 ++---
3 Dateien geändert, 24 Zeilen hinzugefügt(+), 16 Zeilen entfernt(-)
dif
CPUArchState is no longer needed.
Signed-off-by: Andreas Färber
---
exec.c |2 +-
translate-all.c |9 +
translate-all.h |2 +-
3 Dateien geändert, 7 Zeilen hinzugefügt(+), 6 Zeilen entfernt(-)
diff --git a/exec.c b/exec.c
index dbb893a..a41bcb8 100644
--- a/exec.c
+
Since it was located before breakpoints field, it needs to be reset.
Signed-off-by: Andreas Färber
---
cpu-exec.c |8
exec.c |4 +++-
hw/spapr_hcall.c|5 +++--
include/exec/cpu-defs.h |2 --
include/qom/cpu.h |2 ++
kvm-all
On 01/31/2013 09:10 PM, Michael R. hines wrote:
> Sorry, I didn't go into enough detail about the problem I'm having in the
> loop:
>
> The loop that's not breaking is inside qemu_loadvm_state(), not
> ram_save_block().
I understand now, ram_load always read the addr in the begining of the while
The target-specific ENV_GET_CPU() macros have allowed us to navigate
from CPUArchState to CPUState. The reverse direction was not supported.
Avoid introducing CPU_GET_ENV() macros by initializing an untyped
pointer that is initialized in derived instance_init functions.
The field may not be called
26.01.2013 19:09, Blue Swirl wrote:
Thanks, applied.
Now when I've comments from BALATON Zoltan, I asked Serge to try
out the current qemu in the environment where this buh was easy
to trigger (ubuntu guest with unity desktop) - and he says he
can't reproduce the issue anymore (this does not me
Dear Michael Tokarev,
> 26.01.2013 19:09, Blue Swirl wrote:
> > Thanks, applied.
>
> Now when I've comments from BALATON Zoltan, I asked Serge to try
> out the current qemu in the environment where this buh was easy
> to trigger (ubuntu guest with unity desktop) - and he says he
> can't reproduce
We removed the calculation in commit e4ed1541ac9413eac494a03532e34beaf8a7d1c5
Now we add it back. We need to create dirty_bytes_rate because we
can't include cpu-all.h from migration.c, and there is no other way to
include TARGET_PAGE_SIZE.
Signed-off-by: Juan Quintela
---
arch_init.c
Signed-off-by: Juan Quintela
---
migration.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/migration.c b/migration.c
index d86946e..67abd12 100644
--- a/migration.c
+++ b/migration.c
@@ -681,7 +681,7 @@ static void *buffered_file_thread(void *opaque)
qemu_mutex_unlock
While we are sleeping we are not sending, so we should not use that
time to estimate our bandwidth.
Signed-off-by: Juan Quintela
---
migration.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/migration.c b/migration.c
index 67abd12..64e75ca 100644
--- a/migration.c
+++ b
Explictly NULL it on CPU reset since it was located before breakpoints.
Change vapic_report_tpr_access() argument to CPUState. This also
resolves the use of void* for cpu.h independence.
Change vAPIC patch_instruction() argument to X86CPU.
Signed-off-by: Andreas Färber
---
cpu-exec.c
0 is a very bad initial value, what we are trying to get is
max_downtime, so that is a much better estimation.
Signed-off-by: Juan Quintela
---
migration.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/migration.c b/migration.c
index 77c1971..d86946e 100644
--- a/migration.c
+++ b/migrat
Hi
migration expected_downtime calculation was removed on commit
e4ed1541ac9413eac494a03532e34beaf8a7d1c5.
We add the calculation back. Before doing the calculation we do:
- expected_downtime intial value is max_downtime. Much, much better
intial value than 0.
- we move when we measure the
Change gdbstub's cpu_index() argument to CPUState now that CPUArchState
is no longer used.
Signed-off-by: Andreas Färber
---
dump.c |8 ++--
gdbstub.c | 14 +-
include/exec/cpu-defs.h |1 -
include/exec/gdbstub.h |5 ++---
include/qom
Am 01.02.2013 13:00, schrieb Gerd Hoffmann:
> Make it handle multiple include statements in a file.
>
> Signed-off-by: Gerd Hoffmann
> ---
> scripts/make_device_config.sh |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/make_device_config.sh b/scripts/make_dev
On Fri, Feb 01, 2013 at 01:51:21PM +0800, Wenchao Xia wrote:
> >>+typedef enum QBlockProtocol {
> >>+QB_PROTO_NONE = 0,
> >>+QB_PROTO_FILE,
> >>+QB_PROTO_MAX
> >>+} QBlockProtocol;
> >
> >What prevents libqblock from supporting all protocols?
> >
> I think no problem exist in supporti
Amos Kong reported that file descriptors numbered higher
than 1024 could crash QEMU. This is due to the fixed size of the fd_set type
used for select(2) event polling.
This series converts the main-loop.c and aio-posix.c select(2) calls to
g_poll(3). This eliminates the fd_set type and allows Q
We will need to loop over AioHandlers calling ->io_read()/->io_write()
when aio_poll() is converted from select(2) to g_poll(2).
Luckily the code for this already exists, extract it into the new
aio_dispatch() function.
Two small changes:
* aio_poll() checks !node->deleted to avoid calling hand
Convert iohandler_select_fill() and iohandler_select_poll() to use
GPollFD instead of rfds/wfds/xfds.
Signed-off-by: Stefan Hajnoczi
---
include/qemu/main-loop.h | 4 ++--
iohandler.c | 40 ++--
main-loop.c | 4 ++--
3 files changed
aio-posix.c could not take advantage of G_IO_HUP and G_IO_ERR because
select(2) does not have equivalent events. Now that g_poll(3) is used
we can support G_IO_HUP and G_IO_ERR.
Signed-off-by: Stefan Hajnoczi
---
aio-posix.c | 12 ++--
1 file changed, 2 insertions(+), 10 deletions(-)
d
Now that all *_fill() and *_poll() functions use GPollFD we no longer
need rfds/wfds/xfds or pollfds_from_select()/pollfds_to_select().
>From now on everything uses GPollFD.
Signed-off-by: Stefan Hajnoczi
---
main-loop.c | 77 ++---
1 file
On 01/31/13 19:24, Peter Maydell wrote:
> We should only be reporting one error:
> "we failed because you asked for this bus and it's full" should
> override the default "we failed to find this bus". We can fix
> that by having the recursion stop as soon as we get an error.
> I think the QMP call
Hi all,
I did following operations to install QEMU to my PC.
1. git clone git://git.qemu.org/qemu.git
2. cd qemu
3, git reset --hard v1.3.0
4, ./configure
5, make
6, sudo make install
After step 6, i did following jobs.
a: see size of qemu-img bin in qemu package
jiawei@jiawei-laptop:~/workshop
On Thu, Jan 31, 2013 at 03:40:14PM +0100, Andreas Färber wrote:
> Commit eeb6b45d48800e96f67ef2a5c80332557fd45ddb (block: raw-posix image
> file reopen) broke the build on OpenIndiana.
>
> illumos has no O_ASYNC. Exclude it from flags to be compared
> and instead assert that it is not set where de
Signed-off-by: Stefan Hajnoczi
---
main-loop.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/main-loop.c b/main-loop.c
index 6f52ac3..d0d8fe4 100644
--- a/main-loop.c
+++ b/main-loop.c
@@ -330,7 +330,8 @@ void qemu_fd_register(int fd)
static int os_host_main_loop_wait(uin
Convert glib file descriptor polling from rfds/wfds/xfds to GPollFD.
The Windows code still needs poll_fds[] and n_poll_fds but they can now
become local variables.
Signed-off-by: Stefan Hajnoczi
---
main-loop.c | 71 +++--
1 file changed,
Use g_poll(3) instead of select(2). Well, this is kind of a cheat.
It's true that we're now using g_poll(3) on POSIX hosts but the *_fill()
and *_poll() functions are still using rfds/wfds/xfds.
We've set the scene to start converting *_fill() and *_poll() functions
step-by-step until no more rfd
AioHandler already has a GPollFD so we can directly use its
events/revents.
Add the int pollfds_idx field to AioContext so we can map g_poll(3)
results back to AioHandlers.
Reuse aio_dispatch() to invoke handlers after g_poll(3).
Signed-off-by: Stefan Hajnoczi
---
aio-posix.c | 67
The following changes since commit 8a55ebf01507ab73cc458cfcd5b9cb856aba0b9e:
Merge remote-tracking branch 'afaerber/qom-cpu' into staging (2013-01-31
19:37:33 -0600)
are available in the git repository at:
git://github.com/stefanha/qemu.git block
for you to fetch changes up to fdf263f63fa
From: Kevin Wolf
Return -errno instead of -1 on errors. While touching the
code, fix a memory leak.
Signed-off-by: Kevin Wolf
Signed-off-by: Stefan Hajnoczi
---
block/vpc.c | 42 ++
1 file changed, 30 insertions(+), 12 deletions(-)
diff --git a/block/v
On Fri, Feb 01, 2013 at 03:20:59PM +0800, Amos Kong wrote:
> On Thu, Jan 31, 2013 at 05:43:51PM -0600, Michael Roth wrote:
> > This reverts commit 84dd2120247a7d25ff1bb337de21c0e76816ad2d.
> >
> > I'm not sure what issue the original commit was meant to fix, or if
> > the logic is actually wrong,
From: Liu Yuan
Sheep daemon needs vdi_id to identify which vdi is closed to release resources
such as object cache.
Cc: MORITA Kazutaka
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Signed-off-by: Liu Yuan
Reviewed-by: MORITA Kazutaka
Signed-off-by: Stefan Hajnoczi
---
block/sheepdog.c | 5 +++--
1
From: Philipp Hahn
The previous scanf() format string stopped parsing the file name on the
first white white space, which seems to be allowed at least by VMware
Workstation.
Change the format string to collect everything between the first and
second quote as the file name, disallowing line break
On Fri, Feb 01, 2013 at 03:53:22PM +0800, Amos Kong wrote:
> On Fri, Feb 01, 2013 at 03:20:59PM +0800, Amos Kong wrote:
> > On Thu, Jan 31, 2013 at 05:43:51PM -0600, Michael Roth wrote:
> > > This reverts commit 84dd2120247a7d25ff1bb337de21c0e76816ad2d.
> > >
> > > I'm not sure what issue the orig
Michael Roth writes:
> This reverts commit 84dd2120247a7d25ff1bb337de21c0e76816ad2d.
>
> I'm not sure what issue the original commit was meant to fix, or if
> the logic is actually wrong, but it causes e1000 to stop working
> after a guest issues a reset.
>
>>From what I can tell a guest with an
From: Kevin Wolf
Return -errno instead of -1 on errors and add error checks in some
places that didn't have one. Passing things by reference requires more
correct typing, replaced a few off_ts therefore - with a 32-bit off_t
this is even a fix for truncation bugs.
While touching the code, fix ev
Hi Laurent,
Laurent Vivier writes:
> Hi,
>
>
> Please pull this linux-user patches queue into master.
>
> All these patches have already been sent and reviewed on the mailing list.
>
> The following changes since commit 4c37ef022381e777251d7084591978a4dc622efe:
>
> host-utils: add ffsl (2013-01-
From: KONRAD Frederic
These structures must be made public to avoid two memory allocations for
refactored virtio devices.
Signed-off-by: KONRAD Frederic
---
hw/virtio-balloon.c| 15 ---
hw/virtio-balloon.h| 14 ++
hw/virtio-blk.c| 16
hw
From: Kevin Wolf
Return -errno instead of -1 on errors. While touching the
code, fix a memory leak.
Signed-off-by: Kevin Wolf
Signed-off-by: Stefan Hajnoczi
---
block/cloop.c | 29 +++--
1 file changed, 19 insertions(+), 10 deletions(-)
diff --git a/block/cloop.c b/bl
On 1 February 2013 15:05, Anthony Liguori wrote:
> Laurent Vivier writes:
>> Please pull this linux-user patches queue into master.
>>
>> All these patches have already been sent and reviewed on the mailing list.
> Did you use git-request-pull to generate this? Perhaps your mailer
> munged all
From: Vishvananda Ishaya
In an image chain, if the base image is smaller than the current
image, we need to make sure to use the current images count of
unallocated blocks once we get to the end of the base image. Without
this change the code will return 0 blocks when it gets to the end
of the ba
On 01.02.2013 10:54, Andreas Färber wrote:
> Am 31.01.2013 11:15, schrieb Michal Privoznik:
>> Currently, we are enforcing the _FORTIFY_SOURCE=2 without any
>> previous detection if the macro has been already defined, e.g.
>> by environment, or is just enabled by compiler by default.
>>
>> Signed-o
On 02/01/13 14:38, Andreas Färber wrote:
> Am 01.02.2013 13:00, schrieb Gerd Hoffmann:
>> Make it handle multiple include statements in a file.
>>
>> Signed-off-by: Gerd Hoffmann
>> ---
>> scripts/make_device_config.sh |2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a
From: Vishvananda Ishaya
This test verifies two mirroring issues are fixed with resized images:
* sync='top' creates an image that is the proper size
* sync='full' doesn't cause an assertion failure and crash qemu
Reviewed-by: Paolo Bonzini
Signed-off-by: Stefan Hajnoczi
---
tests/qemu-iot
Thanks for the fixes since the last version. I have a few comments
below. Apologies for not catching some of these the first time through.
On 01/18/2013 11:02 AM, Stefan Berger wrote:
This patch adds support for TPM command line options.
The command line options supported here are
./qemu-...
From: Othmar Pasteka
Introduce a new option "adapter_type" when converting to vmdk images.
It can be one of the following: ide (default), buslogic, lsilogic
or legacyESX (according to the vmdk spec from vmware).
In case of a non-ide adapter, heads is set to 255 instead of the 16.
The latter is u
From: Andreas Färber
Commit eeb6b45d48800e96f67ef2a5c80332557fd45ddb (block: raw-posix image
file reopen) broke the build on OpenIndiana.
illumos has no O_ASYNC. Exclude it from flags to be compared
and instead assert that it is not set where defined.
Cf. e61ab1da7e98357da47c54d8f893b9bd6ff2f7f
Signed-off-by: Gerd Hoffmann
---
qemu-master.cfg | 27 +++
1 file changed, 27 insertions(+)
diff --git a/qemu-master.cfg b/qemu-master.cfg
index 04d26bf..b54ceb4 100644
--- a/qemu-master.cfg
+++ b/qemu-master.cfg
@@ -45,6 +45,32 @@ emailAddressesNotify = ["gol...@b1-sys
From: Kevin Wolf
It turned out that the change in b7ab0fea was actually a real qcow2
corruption fix. This is a reproducer for the bug.
Signed-off-by: Kevin Wolf
Signed-off-by: Stefan Hajnoczi
---
tests/qemu-iotests/047 | 75 ++
tests/qemu-iotest
Signed-off-by: Gerd Hoffmann
---
qemu-master.cfg |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/qemu-master.cfg b/qemu-master.cfg
index 2502ced..04d26bf 100644
--- a/qemu-master.cfg
+++ b/qemu-master.cfg
@@ -699,7 +699,9 @@ def create_build_factory(repourl, branch="HEAD
From: Kevin Wolf
Return -errno instead of -1 on errors. While touching the
code, fix a memory leak.
Signed-off-by: Kevin Wolf
Signed-off-by: Stefan Hajnoczi
---
block/bochs.c | 22 +++---
1 file changed, 15 insertions(+), 7 deletions(-)
diff --git a/block/bochs.c b/block/boch
On 01/24/2013 07:40 AM, Alexander Graf wrote:
I think for now the best choice for get_regs() would be to ignore the
FULL/RESET bits and always keep the syncing as it happens today under the
RUNTIME umbrella only. So all of get_regs() only checks for RUNTIME.
Whenever get_xxx() happens, a bit g
From: Kevin Wolf
Return -errno instead of -1 on errors. Hey, no memory leak to fix here
while we're touching it!
Signed-off-by: Kevin Wolf
Signed-off-by: Stefan Hajnoczi
---
block/parallels.c | 23 +++
1 file changed, 15 insertions(+), 8 deletions(-)
diff --git a/block/pa
Signed-off-by: Gerd Hoffmann
---
qemu-master.cfg | 24 ++--
1 file changed, 18 insertions(+), 6 deletions(-)
diff --git a/qemu-master.cfg b/qemu-master.cfg
index 03db6ee..2f793c1 100644
--- a/qemu-master.cfg
+++ b/qemu-master.cfg
@@ -681,17 +681,29 @@ from buildbot.steps.sh
Hi,
Goes on top of stefans update. Untested. Shoveling
over some goodies from my local buildbot config.
https://github.com/kraxel/buildbot/tree/kraxel
(will create a pull req in a moment)
One question on the buildslave config (which isn't in the tree for
security reasons): Do you have max_b
On Fri, Feb 01, 2013 at 08:49:00AM -0600, mdroth wrote:
> On Fri, Feb 01, 2013 at 03:53:22PM +0800, Amos Kong wrote:
> > On Fri, Feb 01, 2013 at 03:20:59PM +0800, Amos Kong wrote:
> > > On Thu, Jan 31, 2013 at 05:43:51PM -0600, Michael Roth wrote:
> > > > This reverts commit 84dd2120247a7d25ff1bb33
Slirp uses rfds/wfds/xfds more extensively than other QEMU components.
The rarely-used out-of-band TCP data feature is used. That means we
need the full table of select(2) to g_poll(3) events:
rfds -> G_IO_IN | G_IO_HUP | G_IO_ERR
wfds -> G_IO_OUT | G_IO_ERR
xfds -> G_IO_PRI
I came up wit
1 - 100 of 180 matches
Mail list logo