Amit Shah wrote:
> On (Wed) 14 Oct 2015 [17:12:44], Michael S. Tsirkin wrote:
>> On Wed, Oct 14, 2015 at 03:21:15PM +0200, Juan Quintela wrote:
>> > Amit Shah wrote:
>> > > We were announcing the dest host's IP as our new IP a bit too soon -- if
>> > > there were errors detected after this announ
Hi,
I am debugging an issue, where through one of the nics I am not able
to connect to dtcp.
So to dig in deep, i was going thought net.c files, and ended-up
understanding little.
So can i know how does a netdev connects to nic.
nd_table[] has some pointers for netdev devices. Where are this
act
From: Amit Shah
We were announcing the dest host's IP as our new IP a bit too soon -- if
there were errors detected after this announcement was done, the
migration is failed and the VM could continue running on the src host --
causing problems later.
Move around the qemu_announce_self() call so
From: "Dr. David Alan Gilbert"
The current migration-completed event is generated a bit too early,
which means that an eager libvirt that's ready to go as soon
as it sees the event ends up racing with the actual end of migration.
This corresponds to RH bug:
https://bugzilla.redhat.com/show_bug.c
Hi Peter
This is a pull request with the small fixes from the list, please apply.
Later, Juan
The following changes since commit c49d3411faae8ffaab8f7e5db47405a008411c10:
Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2015-10-12' into
staging (2015-10-13 10:42:06 +0100)
are av
From: "Denis V. Lunev"
Release qemu global mutex before call synchronize_rcu().
synchronize_rcu() waiting for all readers to finish their critical
sections. There is at least one critical section in which we try
to get QGM (critical section is in address_space_rw() and
prepare_mmio_access() is tr
Eric Blake writes:
> Commit e36c714e causes 'qemu -netdev ?' to dump core, because the
> call to visit_end_union() is no longer conditional on whether
> *obj was allocated.
>
> Reported by Marc-André Lureau
> Signed-off-by: Eric Blake
> ---
> v2: don't depend on unreleased patches
>
> scripts/
On (Wed) 14 Oct 2015 [17:12:44], Michael S. Tsirkin wrote:
> On Wed, Oct 14, 2015 at 03:21:15PM +0200, Juan Quintela wrote:
> > Amit Shah wrote:
> > > We were announcing the dest host's IP as our new IP a bit too soon -- if
> > > there were errors detected after this announcement was done, the
> >
Quoting Eric Blake (2015-10-14 17:38:56)
> On 10/14/2015 02:08 PM, Michael Roth wrote:
> > From: Yuri Pudgorodskiy
> >
> > Guest-exec rewriten in platform-independant style with glib spawn.
>
> s/rewriten/rewritten/
> s/independant/independent/
>
> >
> > Child process is spawn asynchroneously
Update cpu_model in MachineState for i386, so that the field can be used
for cpu hotplug, instead of using a static variable.
This patch is rebased on the latest master.
Signed-off-by: Zhu Guihua
Reviewed-by: Eduardo Habkost
---
v3:
-use PCMachineState in pc_cpus_init() instead MachineState
v
- Original Message -
> On Tue, Oct 13, 2015 at 06:16:15PM +0800, Fam Zheng wrote:
> > This reverts commit 723c5d93c51bdb3adbc238ce90195c0864aa6cd5.
> >
> > block_job_cb is called by block_job_completed, which is always called in
> > a main loop bottom half in existing block jobs. So we d
On 10/14/2015 10:27 PM, Stefan Hajnoczi wrote:
> On Tue, Oct 13, 2015 at 05:08:17PM +0800, Wen Congyang wrote:
>> On 10/13/2015 12:27 AM, Stefan Hajnoczi wrote:
>>> On Fri, Sep 25, 2015 at 02:17:36PM +0800, Wen Congyang wrote:
+/* start backup job now */
+bdrv_op_unblock(s
On 10/14/2015 10:12 PM, Michael S. Tsirkin wrote:
> On Wed, Oct 14, 2015 at 03:21:15PM +0200, Juan Quintela wrote:
>> Amit Shah wrote:
>>> We were announcing the dest host's IP as our new IP a bit too soon -- if
>>> there were errors detected after this announcement was done, the
>>> migration i
On 10/14/2015 08:07 PM, Amit Shah wrote:
> We were announcing the dest host's IP as our new IP a bit too soon -- if
> there were errors detected after this announcement was done, the
> migration is failed and the VM could continue running on the src host --
> causing problems later.
>
> Move arou
On 10/15/2015 01:06 AM, Eduardo Habkost wrote:
On Wed, Oct 14, 2015 at 10:50:40PM +0800, Xiao Guangrong wrote:
On 10/14/2015 05:40 PM, Stefan Hajnoczi wrote:
On Sun, Oct 11, 2015 at 11:52:59AM +0800, Xiao Guangrong wrote:
static void dsm_write(void *opaque, hwaddr addr,
On 10/14/2015 05:26 PM, Michael S. Tsirkin wrote:
> commit 5be7d9f1b1452613b95c6ba70b8d7ad3d0797991
> vhost-net: tell tap backend about the vnet endianness
> makes vhost net always try to set LE - even if that matches the
> native endian-ness.
>
> This makes it fail on older kernels on x86 wi
This converts vga code to curses code in console_write_bh().
With this changes, we can see line graphics (for example, dialog uses)
correctly.
Signed-off-by: OGAWA Hirofumi
---
include/ui/console.h | 11 ++-
ui/curses.c | 43 +++
2 f
OGAWA Hirofumi writes:
> Gerd Hoffmann writes:
>
>> On Mi, 2015-10-14 at 20:37 +0900, OGAWA Hirofumi wrote:
>>> This converts vga code to curses code in console_write_bh().
>>>
>>> With this changes, we can see line graphics (for example, dialog uses)
>>> correctly.
>>
>> But it breaks line gra
Markus Armbruster writes:
> Copying maintainer. You can use scripts/get_maintainer.pl to find
> maintainers yourself.
Ah, I was missing to use it. Thanks.
> OGAWA Hirofumi writes:
>
>> Hi,
>>
>> This patchset is to improve/fix -curses mode. But this patchset is
>> tested only on linux, so RFC
Gerd Hoffmann writes:
> On Mi, 2015-10-14 at 20:37 +0900, OGAWA Hirofumi wrote:
>> This converts vga code to curses code in console_write_bh().
>>
>> With this changes, we can see line graphics (for example, dialog uses)
>> correctly.
>
> But it breaks line graphics in vga text mode on gtk I sus
From: "Edgar E. Iglesias"
Introduce ARMMMUFaultInfo to propagate MMU Fault information
across the MMU translation code path. This is in preparation for
adding Stage-2 translation.
No functional changes.
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c| 32 -
From: "Edgar E. Iglesias"
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c | 44 +---
1 file changed, 37 insertions(+), 7 deletions(-)
diff --git a/target-arm/helper.c b/target-arm/helper.c
index 69e24e1..9d70ef2 100644
--- a/target-arm/helper.c
From: "Edgar E. Iglesias"
Add support for applying S2 translation to 64bit S1
page-table walks.
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c| 50 --
target-arm/op_helper.c | 4 ++--
2 files changed, 50 insertions(+), 4 deletions(
From: "Edgar E. Iglesias"
Signed-off-by: Edgar E. Iglesias
---
target-arm/op_helper.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/target-arm/op_helper.c b/target-arm/op_helper.c
index d4715f4..2ccd1c9 100644
--- a/target-arm/op_helper.c
+++ b/target-arm/op_hel
From: "Edgar E. Iglesias"
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c | 41 +
1 file changed, 37 insertions(+), 4 deletions(-)
diff --git a/target-arm/helper.c b/target-arm/helper.c
index 8530f7e..d1ffcdf 100644
--- a/target-arm/helper.c
+++
From: "Edgar E. Iglesias"
Add support for applying S2 translation to 32bit S1
page-table walks.
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c | 22 +-
1 file changed, 17 insertions(+), 5 deletions(-)
diff --git a/target-arm/helper.c b/target-arm/helper.c
index 7
From: "Edgar E. Iglesias"
Avoid inline for get_phys_addr() to prepare for future recursive use.
Reviewed-by: Peter Maydell
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/target-arm/helper.c b/target
From: "Edgar E. Iglesias"
The starting level for S2 pagetable walks is computed
differently from the S1 starting level. Implement the S2
variant.
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c| 117 +++--
target-arm/internals.h | 25 ++
From: "Edgar E. Iglesias"
Make t0sz and t1sz signed integers to match tsz and to make
it easier to implement support for AArch32 negative t0sz.
t1sz is changed for consistensy.
No functional change.
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c | 4 ++--
1 file changed, 2 insertion
From: "Edgar E. Iglesias"
Rename granule_sz to stride to better match the reference manuals.
No functional change.
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/target-arm/helper.c
From: "Edgar E. Iglesias"
Add support for AArch32 S2 negative t0sz. In preparation for
using 40bit IPAs on AArch32.
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/target-arm/helper.c b/target-arm/helper.c
index 4e19838.
From: "Edgar E. Iglesias"
Reviewed-by: Alex Bennée
Signed-off-by: Edgar E. Iglesias
---
target-arm/cpu.h| 1 +
target-arm/helper.c | 12
2 files changed, 13 insertions(+)
diff --git a/target-arm/cpu.h b/target-arm/cpu.h
index 493f9d0..aaace9b 100644
--- a/target-arm/cpu.h
++
From: "Edgar E. Iglesias"
Remove the tsz variable and introduce inputsize.
This simplifies the code a little and makes it easier to
compare with the reference manuals.
No functional change.
Signed-off-by: Edgar E. Iglesias
---
target-arm/helper.c | 22 +++---
1 file changed, 1
From: "Edgar E. Iglesias"
Hi,
Another round of patches towards EL2 support. This one adds partial
support for 2-stage MMU. The AArch32/ARMv7 support is untested.
Some of the details of error reporting are intentionally missing, I
was thinking to add those incrementally as they get quite involve
On 10/14/2015 02:08 PM, Michael Roth wrote:
> From: Yuri Pudgorodskiy
>
> Guest-exec rewriten in platform-independant style with glib spawn.
s/rewriten/rewritten/
s/independant/independent/
>
> Child process is spawn asynchroneously and exit status can later
s/asynchroneously/asynchronously/
Commit e36c714e causes 'qemu -netdev ?' to dump core, because the
call to visit_end_union() is no longer conditional on whether
*obj was allocated.
Reported by Marc-André Lureau
Signed-off-by: Eric Blake
---
v2: don't depend on unreleased patches
scripts/qapi-visit.py | 4 +++-
1 file changed,
On 10/14/2015 10:30 AM, Eric Blake wrote:
> Commit e36c714e causes 'qemu -netdev ?' to dump core, because the
> call to visit_end_union() is no longer conditional on whether
> *obj was allocated.
>
> Reported by Marc-André Lureau
> Signed-off-by: Eric Blake
> ---
> scripts/qapi-visit.py | 4 +++
Hi,
I'm profiling memcpy and seeing strange behavior (for me at least) and
wanted to see if some one has an idea what may be happening.
My set up is as follows:
I have a Ubuntu 12.04 Linux host running 3.2.0-23 kernel. It has a four
10-core dual-hyper-threaded CPU with 128GB RAM. I have instant
On 10/15/2015 06:34 AM, Peter Maydell wrote:
This is still the same cryptic comment we have in the
targets which do do this. Can we have something
that is a bit more explanatory about what is going on and
why we need to do this, please?
Suggestions?
(Also explaining what the number you need t
On 10/14/2015 09:54 AM, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini
> ---
> qemu-char.c | 37 +
> 1 file changed, 21 insertions(+), 16 deletions(-)
Reviewed-by: Eric Blake
--
Eric Blake eblake redhat com+1-919-301-3266
Libvirt virtualization l
On 10/14/2015 09:54 AM, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini
> ---
> backends/baum.c | 16 +++-
> include/sysemu/char.h | 3 ---
> qemu-char.c | 4 +---
> stubs/Makefile.objs | 1 -
> stubs/chr-baum-init.c | 7 ---
> 5 files changed, 12 inserti
On 10/14/2015 09:54 AM, Paolo Bonzini wrote:
> Conversion to Error * brings better error messages; before:
>
> qemu-system-x86_64: -chardev id=serial,backend=parallel,path=vl.c: Failed
> to create chardev
>
> After:
>
> qemu-system-x86_64: -chardev id=serial,backend=parallel,path=vl.c:
On Wed, Oct 14, 2015 at 02:00:17PM +0100, Alex Bennée wrote:
>
> Edgar E. Iglesias writes:
>
> > On Wed, Oct 07, 2015 at 05:24:27PM +0100, Alex Bennée wrote:
> >>
> >> Edgar E. Iglesias writes:
> >>
> >> > From: "Edgar E. Iglesias"
> >> >
> >> > Introduce ARMMMUFaultInfo to propagate MMU Fau
Currently POSIX builds rely on 'qemu-ga' target to do qga-only
distributable build. On w32, as with most standalone binary targets,
we rely on 'qemu-ga.exe' target.
Unlike with POSIX, qemu-ga for w32 has a number of related targets
such as VSS DLL and MSI package. We can do the full distributable
From: Marc-André Lureau
Move the default verbosity settings before loading the configuration
file, or it will overwrite it. Found thanks to writing qga tests :)
Signed-off-by: Marc-André Lureau
Reviewed-by: Michael Roth
Signed-off-by: Michael Roth
---
qga/main.c | 4 ++--
1 file changed, 2 i
From: Marc-André Lureau
Those are mostly useful for writing tests.
Signed-off-by: Marc-André Lureau
Reviewed-by: Claudio Fontana
Signed-off-by: Michael Roth
---
include/glib-compat.h | 61 +++
1 file changed, 61 insertions(+)
diff --git a/incl
From: Marc-André Lureau
Having a environment variable allows to override default configuration
path, useful for testing. Note that this can't easily be an argument,
since loading config is done before parsing the arguments.
Signed-off-by: Marc-André Lureau
Reviewed-by: Michael Roth
Signed-off-
From: Marc-André Lureau
Add some local guest agent tests, as it is better than nothing, only
when CONFIG_POSIX (using unix sockets).
With the QGA_TEST_SIDE_EFFECTING environment variable, it will include
tests with side effects, such as freezing/thawing the FS or changing the
time.
(a better te
From: "Denis V. Lunev"
This just makes code shorter and better.
Signed-off-by: Denis V. Lunev
Signed-off-by: Yuri Pudgorodskiy
Reviewed-by: Michael Roth
Signed-off-by: Denis V. Lunev
Signed-off-by: Michael Roth
---
qga/commands-posix.c | 10 +++---
qga/commands-win32.c | 10 +++---
From: Marc-André Lureau
Add a few functions to interact with qmp via a simple fd.
Signed-off-by: Marc-André Lureau
Reviewed-by: Michael Roth
Signed-off-by: Michael Roth
---
tests/libqtest.c | 45 +
tests/libqtest.h | 7 +++
2 files changed, 48
From: Yuri Pudgorodskiy
Implemented with base64-encoded strings in qga json protocol.
Glib portable GIOChannel is used for data I/O.
Optinal stdin parameter of guest-exec command is now used as
stdin content for spawned subprocess.
If capture-output bool flag is specified, guest-exec redirects
From: Yuri Pudgorodskiy
glib may return G_IO_STATUS_AGAIN which is actually not an error.
Also fixed a bug when on incomplete write buf pointer was not adjusted.
Signed-off-by: Yuri Pudgorodskiy
Signed-off-by: Denis V. Lunev
Reviewed-by: Michael Roth
Signed-off-by: Michael Roth
---
qga/chan
From: "Denis V. Lunev"
qemu-ga should not exit on guest-file-write to pipe without read end
but proper error code should be returned. The behavior of the
spawned process should be default thus SIGPIPE processing should be
reset to default after fork() but before exec().
Signed-off-by: Denis V. L
From: Yuri Pudgorodskiy
Guest-exec rewriten in platform-independant style with glib spawn.
Child process is spawn asynchroneously and exit status can later
be picked up by guest-exec-status command.
stdin/stdout/stderr of the child now is redirected to /dev/null
Later we will add ability to spe
Hi Peter,
Please note that 'glib-compat: add 2.38/2.40/2.46 asserts' is also in
Marc-André's recent ivshmem PULL. The 2 versions of the patches are identical,
but let me know if you'd prefer a re-send/re-base later.
The following changes since commit c49d3411faae8ffaab8f7e5db47405a008411c10:
M
From: Markus Armbruster
g_new(T, n) is neater than g_malloc(sizeof(T) * n). It's also safer,
for two reasons. One, it catches multiplication overflowing size_t.
Two, it returns T * rather than void *, which lets the compiler catch
more type errors.
This commit only touches allocations with siz
On 14 October 2015 at 13:45, Alex Bennée wrote:
> While looking I noticed we probably extract more than we need into attrs
> considering the tableattrs is snarfed earlier. Maybe we should only do:
>
> attrs = extract64(descriptor, 2, 10)
> | (extract64(descriptor, 52, 7) << 10)
On 14 October 2015 at 10:35, Sergey Fedorov wrote:
> On 13.10.2015 20:27, Peter Maydell wrote:
>> On 9 October 2015 at 10:43, Sergey Fedorov wrote:
>>> Signed-off-by: Sergey Fedorov
>>> ---
>>>
>>> Changes in v2:
>>> * Reset value is simply made zero
>>>
>>> target-arm/cpu.h| 1 +
>>> tar
On 13 October 2015 at 23:10, Richard Henderson wrote:
> Some targets already had this within their logic, but make sure
> it's present for all targets.
>
> Signed-off-by: Richard Henderson
> ---
> target-alpha/translate.c | 3 +++
> target-cris/translate.c | 3 +++
> target-i386/trans
On 14 October 2015 at 07:52, Pavel Fedin wrote:
> P.S. By the way, what about GICv3 data format? Are you going to
> review this part and do something with it? This would also advance
> Shlomo's work i think.
It's in my queue of things to look at at some point, but
so are a great many other thing
Am 08.10.2015 um 18:44 schrieb John Snow:
>
> On 10/08/2015 08:06 AM, Peter Lieven wrote:
>> Hi all,
>>
>> short summary from my side. The whole thing seems to get complicated,
>> let me explain why:
>>
>> 1) During review I found that the code in ide_atapi_cmd_reply_end can't
>> work correctly if
On Thu, Oct 8, 2015 at 4:30 PM, Alistair Francis
wrote:
> On Wed, Oct 7, 2015 at 5:01 PM, Peter Crosthwaite
> wrote:
>> On Wed, Oct 7, 2015 at 2:34 PM, Alistair Francis
>> wrote:
>>> Connect the sst25wf080 SPI flash to the EP108 board.
>>>
>>> Signed-off-by: Alistair Francis
>>> ---
>>> V2:
>>>
On Thu, Oct 8, 2015 at 4:30 PM, Alistair Francis
wrote:
> On Wed, Oct 7, 2015 at 5:01 PM, Peter Crosthwaite
> wrote:
>> On Wed, Oct 7, 2015 at 2:34 PM, Alistair Francis
>> wrote:
>>> Connect the sst25wf080 SPI flash to the EP108 board.
>>>
>>> Signed-off-by: Alistair Francis
>>> ---
>>> V2:
>>>
On Tue, Oct 13, 2015 at 9:03 PM, Xiao Guangrong
wrote:
>> Label-less DIMMs are tested as part of the unit test [1] and the
>> "memmap=nn!ss" kernel parameter that registers a persistent-memory
>> address range without a DIMM. What error do you see when label
>> support is disabled?
>>
>> [1]: htt
Ping^2
On Thu, Aug 27, 2015 at 2:47 PM, Alistair Francis
wrote:
> Ping!
>
> On Wed, Jul 29, 2015 at 1:24 PM, Alistair Francis
> wrote:
>> From Peter:
>> Hi All. This is a new scheme I've come up with handling device registers in a
>> data driven way. My motivation for this is to factor out a lot
Fix the index used to read the IBAT's vector which results in IBAT0..3 instead
of IBAT4..N.
The bug appeared by saving/restoring contexts including IBATs values.
Signed-off-by: Julio Guerra
---
target-ppc/translate_init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/targe
On 10/14/2015 02:19 PM, Peter Lieven wrote:
> Am 08.10.2015 um 18:44 schrieb John Snow:
>>
>> On 10/08/2015 08:06 AM, Peter Lieven wrote:
>>> Hi all,
>>>
>>> short summary from my side. The whole thing seems to get complicated,
>>> let me explain why:
>>>
>>> 1) During review I found that the cod
On Wed, Oct 14, 2015 at 08:05:34PM +0200, Max Reitz wrote:
> On 14.10.2015 15:16, Jeff Cody wrote:
> > To find a BlockDriverState interface, it can be done via blk_by_name(),
> > bdrv_find_node(), and bdrv_lookup_bs(). The latter can take the place
> > of the other two, in the instances where we a
On 14.10.2015 15:16, Jeff Cody wrote:
> To find a BlockDriverState interface, it can be done via blk_by_name(),
> bdrv_find_node(), and bdrv_lookup_bs(). The latter can take the place
> of the other two, in the instances where we are only concerned with the
> BlockDriverState.
>
> In much of the
On Wed, Oct 14, 2015 at 07:34:43PM +0200, Max Reitz wrote:
> On 14.10.2015 19:29, Max Reitz wrote:
> > On 14.10.2015 15:16, Jeff Cody wrote:
> >> This is a precursor to making bdrv_find_node() static, and internal
> >> to block.c
> >>
> >> To find a BlockDriverState interface, it can be done via bl
On 14.10.2015 15:16, Jeff Cody wrote:
> This patch does two things: it moves bdrv_find_node() up before the
> first usage in block.c, and it makes the function static so that it
> is only internal to block.c.
>
> Signed-off-by: Jeff Cody
> ---
> block.c | 30 +++
On 14.10.2015 19:29, Max Reitz wrote:
> On 14.10.2015 15:16, Jeff Cody wrote:
>> This is a precursor to making bdrv_find_node() static, and internal
>> to block.c
>>
>> To find a BlockDriverState interface, it can be done via blk_by_name(),
>> bdrv_find_node(), and bdrv_lookup_bs(). The latter can
On 14.10.2015 15:16, Jeff Cody wrote:
> This is a precursor to making bdrv_find_node() static, and internal
> to block.c
>
> To find a BlockDriverState interface, it can be done via blk_by_name(),
> bdrv_find_node(), and bdrv_lookup_bs(). The latter can take the place
> of the other two, in the i
On Wed, Oct 14, 2015 at 10:50:40PM +0800, Xiao Guangrong wrote:
> On 10/14/2015 05:40 PM, Stefan Hajnoczi wrote:
> >On Sun, Oct 11, 2015 at 11:52:59AM +0800, Xiao Guangrong wrote:
> >> static void dsm_write(void *opaque, hwaddr addr,
> >>uint64_t val, unsigned size)
> >> {
On 14/10/2015 18:34, Samuel Thibault wrote:
> Paolo Bonzini, le Wed 14 Oct 2015 17:51:51 +0200, a écrit :
>>
>>
>> On 12/10/2015 17:41, Samuel Thibault wrote:
>>> Eric Blake, le Mon 12 Oct 2015 09:30:12 -0600, a écrit :
Also, I assume that brlapi_perror() adds additional information to
On Wed, Oct 14, 2015 at 10:45:00AM +0200, Igor Mammedov wrote:
> On Tue, 13 Oct 2015 16:10:03 -0300
> Eduardo Habkost wrote:
>
> > On Sat, Oct 10, 2015 at 12:00:16AM -0400, Gabriel L. Somlo wrote:
> > > On Thu, Oct 01, 2015 at 01:33:50PM +0200, Igor Mammedov wrote:
[...]
> > > > > >> +if (!pc
Copying maintainer. You can use scripts/get_maintainer.pl to find
maintainers yourself.
OGAWA Hirofumi writes:
> Hi,
>
> This patchset is to improve/fix -curses mode. But this patchset is
> tested only on linux, so RFC. (BTW, tested with xterm/gnome-terminal on
> linux)
>
> ui/curses: Fix m
On Wed, Oct 14, 2015 at 06:06:37PM +0300, Pavel Fedin wrote:
> For GICv3 ITS implementation we are going to use requester IDs in KVM IRQ
> routing code. This patch introduces reusable convenient way to obtain this
> ID from the device pointer.
>
> Since requester ID is an architecture-specific thi
Paolo Bonzini, le Wed 14 Oct 2015 17:51:51 +0200, a écrit :
>
>
> On 12/10/2015 17:41, Samuel Thibault wrote:
> > Eric Blake, le Mon 12 Oct 2015 09:30:12 -0600, a écrit :
> >> Also, I assume that brlapi_perror() adds additional information to
> >> the error message it prints, such as conversion
On Wed, Oct 14, 2015 at 08:06:36AM +0300, Michael S. Tsirkin wrote:
> On Tue, Oct 13, 2015 at 07:43:00PM -0300, Eduardo Habkost wrote:
> > On Wed, Oct 14, 2015 at 12:18:10AM +0300, Michael S. Tsirkin wrote:
> > > On Tue, Oct 13, 2015 at 04:10:03PM -0300, Eduardo Habkost wrote:
> > > > One of the th
Commit e36c714e causes 'qemu -netdev ?' to dump core, because the
call to visit_end_union() is no longer conditional on whether
*obj was allocated.
Reported by Marc-André Lureau
Signed-off-by: Eric Blake
---
scripts/qapi-visit.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --
Conversion to Error * brings better error messages; before:
qemu-system-x86_64: -chardev id=serial,backend=parallel,path=vl.c: Failed
to create chardev
After:
qemu-system-x86_64: -chardev id=serial,backend=parallel,path=vl.c: not a
parallel port: Inappropriate ioctl for device
Signed-
On 10/14/2015 10:01 AM, Marc-André Lureau wrote:
> Hi,
>
> This patch introduces a regression:
>
> $ x86_64-softmmu/qemu-system-x86_64 -netdev ?
>
> Program received signal SIGSEGV, Segmentation fault.
> 0x55a0bc0f in visit_type_NetClientOptions (v=0x564e19e0,
> obj=0x56510438, n
On 14/10/2015 12:29, Cornelia Huck wrote:
> Do we want to change anything for 2.5 about the default?
>
> Currently, we still default scsi to true, and you have to disable it
> explicitly if you want to use virtio-1 compliant virtio-blk devices
> (which is a bit annoying, as scsi passthrough is n
On 10/14/2015 09:54 AM, Paolo Bonzini wrote:
> This series rewrites chardev creation to use a new ->create
> member of the CharDriver struct, and to always signal errors
> via Error*.
>
> The advantage is that backend-specific creation functions need
> not be exported anymore for qemu-char.c's usa
Andrew Jones writes:
> On Wed, Oct 14, 2015 at 04:58:13PM +0200, Eduardo Otubo wrote:
>> On Wed, Oct 14, 2015 at 03=25=00PM +0200, Markus Armbruster wrote:
>> > Andrew Jones writes:
>> >
>> > > On Wed, Oct 14, 2015 at 11:03:37AM +0200, Eduardo Otubo wrote:
>> > >> On Tue, Sep 29, 2015 at 11=39=
On Thu, Oct 08, 2015 at 07:08:49PM +0100, Peter Maydell wrote:
> The code in arm_excp_unmasked() suppresses the ability of PSTATE.AIF
> to mask exceptions from a lower EL targeting EL2 or EL3 if the
> CPU is 64-bit. This is correct for a target of EL3, but not correct
> for targeting EL2. Further,
Eric Blake writes:
> On 10/08/2015 06:26 AM, Markus Armbruster wrote:
>> Struct and union type members are generally named alike in QMP and C,
>> except for a simple union's implicit tag member, which is "type" in
>> QMP, and "kind" in C. Can't change QMP, so rename it in C.
>>
>> Since the imp
On 10/14/2015 09:51 AM, Paolo Bonzini wrote:
>
>
> On 12/10/2015 17:41, Samuel Thibault wrote:
>> Eric Blake, le Mon 12 Oct 2015 09:30:12 -0600, a écrit :
>>> Also, I assume that brlapi_perror() adds additional information to
>>> the error message it prints, such as conversion of a brlapi-specif
This patch provides the building blocks for creating an SR/IOV
PCIe Extended Capability header and register/unregister
SR/IOV Virtual Functions.
Signed-off-by: Knut Omang
---
hw/pci/Makefile.objs| 2 +-
hw/pci/pci.c| 102 -
hw/pci/pcie.c |
Hi,
This patch introduces a regression:
$ x86_64-softmmu/qemu-system-x86_64 -netdev ?
Program received signal SIGSEGV, Segmentation fault.
0x55a0bc0f in visit_type_NetClientOptions (v=0x564e19e0,
obj=0x56510438, name=0x55af7041 "opts", errp=0x7fffd450)
at qapi-visit.c:690
This series rewrites chardev creation to use a new ->create
member of the CharDriver struct, and to always signal errors
via Error*.
The advantage is that backend-specific creation functions need
not be exported anymore for qemu-char.c's usage, and hence do not
need stubs anymore.
Paolo Bonzini (
Signed-off-by: Paolo Bonzini
---
qemu-char.c | 37 +
1 file changed, 21 insertions(+), 16 deletions(-)
diff --git a/qemu-char.c b/qemu-char.c
index ff7722f..47db98f 100644
--- a/qemu-char.c
+++ b/qemu-char.c
@@ -1078,18 +1078,17 @@ static CharDriverState *qemu
Signed-off-by: Paolo Bonzini
---
backends/baum.c | 16 +++-
include/sysemu/char.h | 3 ---
qemu-char.c | 4 +---
stubs/Makefile.objs | 1 -
stubs/chr-baum-init.c | 7 ---
5 files changed, 12 insertions(+), 19 deletions(-)
delete mode 100644 stubs/chr-baum-in
Without this, the devfn argument to pci_create_*()
does not affect the assigned devfn.
Needed to support (VF_STRIDE,VF_OFFSET) values other than (1,1)
for SR/IOV.
Signed-off-by: Knut Omang
Reviewed-by: Marcel Apfelbaum
---
hw/pci/pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
dif
- Fix comment typo in pcie_cap_slot_write_config
- Simplify code in pcie_cap_slot_hot_unplug_request_cb.
Signed-off-by: Knut Omang
---
hw/pci/pcie.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/hw/pci/pcie.c b/hw/pci/pcie.c
index 774b9ed..ba49c0f 100644
--- a/hw/pci
I have encountered that issue recently, and started some analysis.
The issue is due to the fact that in qemu, gdbstub no longer reads the
communication channel once
the debugged process is resumed with "cont".
Another way to say that, is that communication with gdb is only possible once
the pro
On 14/10/2015 17:46, Sergey Fedorov wrote:
> The first argument of g_free_rcu() is a pointer to a structure. But
> foo_reclaim is used as a function name in the previous example along
> with &foo as a pointer to the structure being reclaimed. Make the
> example consistent with the previous one.
Add a small intro + minimal documentation for how to
implement SR/IOV support for an emulated device.
Signed-off-by: Knut Omang
---
docs/pcie_sriov.txt | 115
1 file changed, 115 insertions(+)
create mode 100644 docs/pcie_sriov.txt
diff --gi
This patch set implements generic support for SR/IOV as an extension to the
core PCIe functionality, similar to the way other capabilities such as AER
is implemented.
There is no implementation of any device that provides
SR/IOV support included, but I have implemented a test
example which can be
1 - 100 of 208 matches
Mail list logo