On Thu, 2016-09-22 at 11:45 +0530, Bharata B Rao wrote:
> On Thu, Sep 22, 2016 at 04:07:21PM +1000, Benjamin Herrenschmidt wrote:
> >
> > On Thu, 2016-09-22 at 10:51 +0530, Bharata B Rao wrote:
> > >
> > > The flag values are expected to remain same for a machine version for
> > > the migration t
On 22/09/2016 07:41, Pavel Dovgalyuk wrote:
>> From: Paolo Bonzini [mailto:pbonz...@redhat.com]
>> On 21/09/2016 13:33, Pavel Dovgalyuk wrote:
>>> This set of patches includes several fixes for replay and
>>> adds network record/replay for network devices. It also makes possible
>>> saving/restor
On 22/09/2016 10:36, Peter Xu wrote:
> It's 2.8 now, and maybe it's time to switch IOAPIC default version to
> 0x20.
>
> Signed-off-by: Peter Xu
> ---
> hw/intc/ioapic.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/intc/ioapic.c b/hw/intc/ioapic.c
> index 31791b
On 22/09/2016 09:02, Thomas Huth wrote:
> On Wed, 21 Sep 2016 22:08:10 +0200
> Paolo Bonzini wrote:
>
>> On 21/09/2016 21:42, Thomas Huth wrote:
>>> Basic idea of this script is to check the git log for URLs
>>> to the QEMU bugtracker at launchpad.net and to figure out
>>> whether the related b
Benjamin Herrenschmidt writes:
> On Thu, 2016-09-22 at 11:45 +0530, Bharata B Rao wrote:
>> On Thu, Sep 22, 2016 at 04:07:21PM +1000, Benjamin Herrenschmidt wrote:
>> >
>> > On Thu, 2016-09-22 at 10:51 +0530, Bharata B Rao wrote:
>> > >
>> > > The flag values are expected to remain same for a m
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 12
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 5a91d77..c97dcb5 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qmp-c
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 12
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index c97dcb5..1c3e69f 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qmp-c
CCtests/test-crypto-tlscredsx509.o
CCtests/crypto-tls-x509-helpers.o
CCtests/pkix_asn1_tab.o
tests/pkix_asn1_tab.c:7:22: warning: libtasn1.h: No such file or directory
tests/pkix_asn1_tab.c:9: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or
‘__attribute__’ before ‘pkix_asn1_tab’
make: **
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 14 --
qapi-schema.json | 7 +++
2 files changed, 7 insertions(+), 14 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 1c3e69f..c4caf9c 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qm
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 15 ---
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 15 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index c4caf9c..18a15dc 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qm
On 22/09/2016 09:12, Markus Armbruster wrote:
> Paolo Bonzini writes:
>
>> ---
>> scripts/checkpatch.pl | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
>> index dde3f5f..3afa19a 100755
>> --- a/scripts/checkpatch.pl
>> +
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 12
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index c3b813f..158dcbd 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qmp-c
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 15 ---
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 15 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 158dcbd..b73bcac 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qm
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 33 -
qapi-schema.json | 17 ++---
2 files changed, 14 insertions(+), 36 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 6d43415..c3b813f 100644
--- a/docs/
Paolo Bonzini writes:
> On 21/09/2016 17:46, Alistair Francis wrote:
>>> > I know it's way too late for design questions, but the thought just
>>> > occured to me: -device gives you what you need without defining yet
>>> > another command line option (good!), but is it appropriate? It's not
>>>
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 14 --
qapi-schema.json | 8 +++-
2 files changed, 7 insertions(+), 15 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index b73bcac..9ef6ce1 100644
--- a/docs/qmp-commands.txt
+++ b/docs/q
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 16
qapi-schema.json | 7 +++
2 files changed, 7 insertions(+), 16 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 724c3a9..0082101 100644
--- a/docs/qmp-commands.txt
+++ b/docs/
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 30 --
qapi-schema.json | 8
2 files changed, 8 insertions(+), 30 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 0359516..0fc4cde 100644
--- a/docs/qmp-commands.
On 22/09/2016 11:19, Markus Armbruster wrote:
>> > I think -device is okay for something that isn't a "backend" but is
>> > directly guest-visible.
> Well, the contents of a block device is just as guest-visible. We split
> the device in a frontend and a backend, and the contents comes from the
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 15 ---
qapi-schema.json | 7 +++
2 files changed, 7 insertions(+), 15 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index cc90d57..dc7c2f11 100644
--- a/docs/qmp-commands.txt
+++ b/docs/
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 13 -
qapi-schema.json | 8 +++-
2 files changed, 7 insertions(+), 14 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 9ef6ce1..b829dc1 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qm
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 22 --
qapi-schema.json | 7 +++
2 files changed, 7 insertions(+), 22 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 03d8c54..37ff37d 100644
--- a/docs/qmp-commands.txt
+++ b
Hi,
The QAPI documentation is currently done in two places, the json
schema and a more human-friendly text file. The goal is to avoid
duplication and to generate friendly versions from the schema (pdf,
man etc). Thus, all documentation should be moved to the schema.
In order to facilitate the re
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 11 ---
qapi-schema.json | 7 +++
2 files changed, 7 insertions(+), 11 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 0fc4cde..b84446f 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qmp-c
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 17 -
qapi-schema.json | 7 +++
2 files changed, 7 insertions(+), 17 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index b829dc1..724c3a9 100644
--- a/docs/qmp-commands.txt
+++ b/docs
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 19 ---
qapi-schema.json | 9 +
2 files changed, 9 insertions(+), 19 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 280e337..5a91d77 100644
--- a/docs/qmp-commands.txt
+++ b/
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 27 ---
qapi-schema.json | 17 +
2 files changed, 17 insertions(+), 27 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 0082101..d5ad1de 100644
--- a/docs/qmp-co
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 14 --
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 14 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index b84446f..051bbfd 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qmp
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 15 ---
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 15 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index d41a60a..0bc2b1a 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qm
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 14 --
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 14 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 18a15dc..dadcc20 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qmp
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 24
qapi-schema.json | 16
2 files changed, 16 insertions(+), 24 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index d5ad1de..799ac72 100644
--- a/docs/qmp-comman
Hi Thomas,
> So if you like, I can try to come up with a small patch series that
> cleans up this mess - and I could also include an updated versions of
> Anton's patch there unless he wants to redo the changes on his own...?
Thanks for looking at this. I'm travelling (stuck in an airport at the
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 86 ---
qapi-schema.json | 45 +++
2 files changed, 45 insertions(+), 86 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index dadcc20
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 18 --
qapi-schema.json | 8
2 files changed, 8 insertions(+), 18 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 32e5b0a..cc90d57 100644
--- a/docs/qmp-commands.txt
+++ b/do
On 22/09/2016 09:51, Herongguang (Stephen) wrote:
> After making memory consistent between source and destination
> (https://lists.gnu.org/archive/html/qemu-devel/2016-09/msg03069.html),
> there can
> still reproduce instruction emulation failure in destination side if
> migration when VM’s in gr
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 20
qapi-schema.json | 20 +---
2 files changed, 17 insertions(+), 23 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 799ac72..32e5b0a 100644
--- a/docs/qmp-comman
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 19 ---
qapi-schema.json | 11 ++-
2 files changed, 10 insertions(+), 20 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index dc7c2f11..0359516 100644
--- a/docs/qmp-commands.txt
++
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 22 --
qapi-schema.json | 8
2 files changed, 8 insertions(+), 22 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 051bbfd..d41a60a 100644
--- a/docs/qmp-commands.txt
+++
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 28
qapi-schema.json | 8
2 files changed, 8 insertions(+), 28 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 52bec44..93858b2 100644
--- a/docs/qmp-commands.tx
Marc-André Lureau writes:
> Since commit ad739706bbadee49, user_creatable_add_type() expects to be
> given a qdict. However, if object-add is called without props, you reach
> the assert: "qemu/qom/object_interfaces.c:115: user_creatable_add_type:
> Assertion `qdict' failed.", because the qdict i
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 17 -
qapi-schema.json | 7 +++
2 files changed, 7 insertions(+), 17 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 0bc2b1a..ff83ea2 100644
--- a/docs/qmp-commands.txt
+++ b/docs
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 15 ---
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 15 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index ff83ea2..03d8c54 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qm
When enabling the sanitizer build it will complain about control
reaching a non-void function. Normally the compiler should detect that
there is only one possible exit given a static VNC_SERVER_FB_BYTES.
As we always expect a static VNC_SERVER_FB_BYTES I've added a compile
time assert and just cal
The guest client's use of the glib's g_atomic primitives causes newer
GCC's to barf when built on Travis. As QEMU has its own primitives with
well understood semantics we might as well use them.
The use of atomics was a little inconsistent so I've also ensure the
values are correctly set with atom
Signed-off-by: Marc-André Lureau
---
docs/qmp-commands.txt | 14 --
qapi-schema.json | 6 ++
2 files changed, 6 insertions(+), 14 deletions(-)
diff --git a/docs/qmp-commands.txt b/docs/qmp-commands.txt
index 37ff37d..52bec44 100644
--- a/docs/qmp-commands.txt
+++ b/docs/qmp
This is clearly a bug, but your suggested change won't deal with the
problem, which is that we're trying to set a bool so the ? 32 : 64
construct is just wrong.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.ne
As it seems easy to break the ThreadSanitizer build we should defend it to
ensure that fixes get applied when it breaks. We use the Ubuntu GCC PPA
to get the latest GCC goodness.
As we need to use the -fuse-ld=gold work around we have to disable the
linux-user targets as these trip up the linker.
* Bharata B Rao (bhar...@linux.vnet.ibm.com) wrote:
> On Thu, Sep 22, 2016 at 03:30:08PM +1000, David Gibson wrote:
> > On Thu, Sep 22, 2016 at 10:51:05AM +0530, Bharata B Rao wrote:
> > > Hi,
> > >
> > > Nikunj and I realized that migrating pseries-2.6 guest from QEMU-2.6
> > > to newer QEMU-2.7
On Thu, 2016-09-22 at 14:34 +0530, Nikunj A Dadhania wrote:
> Something like this works for KVM:
>
> diff --git a/target-ppc/machine.c b/target-ppc/machine.c
> index 4820f22..1cf3779 100644
> --- a/target-ppc/machine.c
> +++ b/target-ppc/machine.c
> @@ -563,8 +563,8 @@ const VMStateDescription vms
Am 21.09.2016 um 00:27 hat Eric Blake geschrieben:
> On 09/20/2016 04:08 PM, Kevin Wolf wrote:
> > The option whether or not to use a native AIO interface really isn't a
> > generic option for all drivers, but only applies to the native file
> > protocols. This patch moves the option in blockdev-ad
On 09/22/2016 09:18 AM, Thomas Huth wrote:
> On Thu, 22 Sep 2016 07:30:52 +0200
> Thomas Huth wrote:
>
>> On Thu, 22 Sep 2016 11:57:15 +1000
>> David Gibson wrote:
>>
>>> On Wed, Sep 21, 2016 at 10:22:11AM +0200, Thomas Huth wrote:
On 20.09.2016 16:39, Cédric Le Goater wrote:
> On 09/20
"Dr. David Alan Gilbert" writes:
> * Bharata B Rao (bhar...@linux.vnet.ibm.com) wrote:
>> On Thu, Sep 22, 2016 at 03:30:08PM +1000, David Gibson wrote:
>> > On Thu, Sep 22, 2016 at 10:51:05AM +0530, Bharata B Rao wrote:
>> > > Hi,
>> > >
>> > > Nikunj and I realized that migrating pseries-2.6 gu
* Nikunj A Dadhania (nik...@linux.vnet.ibm.com) wrote:
> "Dr. David Alan Gilbert" writes:
>
> > * Bharata B Rao (bhar...@linux.vnet.ibm.com) wrote:
> >> On Thu, Sep 22, 2016 at 03:30:08PM +1000, David Gibson wrote:
> >> > On Thu, Sep 22, 2016 at 10:51:05AM +0530, Bharata B Rao wrote:
> >> > > Hi,
ThreadSanitizer picks up potential races although we already use
barriers to ensure things are in the correct order when processing exit
requests. For now we just use the relaxed atomic_set/atomic_read semantics
to reassure tsan that we can't tear the value.
Signed-off-by: Alex Bennée
---
cpu-ex
From: Prasad J Pandit
i.MX Fast Ethernet Controller uses buffer descriptors to manage
data flow to/fro receive & transmit queues. While transmitting
packets, it could continue to read buffer descriptors if a buffer
descriptor has length of zero and has crafted values in bd.flags.
Set an upper lim
Hi,
This is v2 of the ThreadSanitizer fixes. Changes from the last
version:
- added Marc-André's review tags
- added qga/command: use QEMU atomic primitives
- simplified ui/vnc-enc-tight: remove switch and have single return
- fixed the Travis CI build (that was painful)
There is sti
On 22/09/16 19:04, Nikunj A Dadhania wrote:
> Benjamin Herrenschmidt writes:
>
>> On Thu, 2016-09-22 at 11:45 +0530, Bharata B Rao wrote:
>>> On Thu, Sep 22, 2016 at 04:07:21PM +1000, Benjamin Herrenschmidt wrote:
On Thu, 2016-09-22 at 10:51 +0530, Bharata B Rao wrote:
>
> The f
On 22/09/2016 12:31, P J P wrote:
> From: Prasad J Pandit
>
> i.MX Fast Ethernet Controller uses buffer descriptors to manage
> data flow to/fro receive & transmit queues. While transmitting
> packets, it could continue to read buffer descriptors if a buffer
> descriptor has length of zero and
This is to appease sanitizer builds which complain that:
"error: control reaches end of non-void function"
Signed-off-by: Alex Bennée
Reviewed-by: Marc-André Lureau
---
tcg/optimize.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tcg/optimize.c b/tcg/optimize.c
index
From: Prasad J Pandit
ColdFire Fast Ethernet Controller uses buffer descriptors to manage
data flow to/fro receive & transmit queues. While transmitting
packets, it could continue to read buffer descriptors if a buffer
descriptor has length of zero and has crafted values in bd.flags.
Set upper li
On 22/09/2016 12:04, Benjamin Herrenschmidt wrote:
> On Thu, 2016-09-22 at 14:34 +0530, Nikunj A Dadhania wrote:
>> Something like this works for KVM:
>>
>> diff --git a/target-ppc/machine.c b/target-ppc/machine.c
>> index 4820f22..1cf3779 100644
>> --- a/target-ppc/machine.c
>> +++ b/target-ppc/
A list of blacklists for tsan instrumentation. One hopes more can be
removed over time as tsan improves.
The path needs to be absolute so it doesn't break when directories
change during the build:
./configure --with-coroutine=gthread --disable-pie \
--extra-cflags="-g3 -O0 -fsanitize=thread
The idiom CPU_GET_CLASS(cpu) is fairly extensively used in various
threads and trips of ThreadSanitizer due to the fact it updates
obj->class->object_cast_cache behind the scenes. As this is just a
fast-path cache there is no need to lock updates just ensure that we
don't get torn-updates from two
ThreadSanitizer detects a possible race between reading/writing the
hashes. As ordering semantics are already documented for qht we just
need to ensure a race can't tear the hash value so we can use the
relaxed atomic_set/read functions.
Signed-off-by: Alex Bennée
---
util/qht.c | 10 +-
From: Paolo Bonzini
There is a data race if the sequence is written concurrently to the
read. In C11 this has undefined behavior. Use atomic_set; the
read side is already using atomic_read.
Reported-by: Alex Bennée
Signed-off-by: Paolo Bonzini
Signed-off-by: Alex Bennée
---
include/qemu/se
On 22/09/2016 12:32, P J P wrote:
> From: Prasad J Pandit
>
> ColdFire Fast Ethernet Controller uses buffer descriptors to manage
> data flow to/fro receive & transmit queues. While transmitting
> packets, it could continue to read buffer descriptors if a buffer
> descriptor has length of zero
Marc-André Lureau writes:
> Hi
>
> - Original Message -
>> Aha, we got a different bug fix! The old code fails to fail when the
>> parameter doesn't exist. Instead, it sets *obj = NULL, which seems very
>> likely to crash QEMU. Let me try... yup:
>>
>> { "execute": "object-add",
>
Benjamin Herrenschmidt writes:
> On Thu, 2016-09-22 at 14:34 +0530, Nikunj A Dadhania wrote:
>> Something like this works for KVM:
>>
>> diff --git a/target-ppc/machine.c b/target-ppc/machine.c
>> index 4820f22..1cf3779 100644
>> --- a/target-ppc/machine.c
>> +++ b/target-ppc/machine.c
>> @@ -56
On Thu, 2016-09-22 at 20:34 +1000, Alexey Kardashevskiy wrote:
> > diff --git a/target-ppc/machine.c b/target-ppc/machine.c
> > index 4820f22..1cf3779 100644
> > --- a/target-ppc/machine.c
> > +++ b/target-ppc/machine.c
> > @@ -563,8 +563,8 @@ const VMStateDescription vmstate_ppc_cpu = {
> >
> >
"Daniel P. Berrange" writes:
> On Thu, Sep 22, 2016 at 10:36:45AM +0200, Markus Armbruster wrote:
>> Don't make up a description in user_creatable_help_func(), improve the
>> description infrastructure and its use so you get more useful ones
>> there.
>>
>> The existing description infrastructur
On 20 September 2016 at 19:57, Richard Henderson wrote:
> Annoying that a bug that Peter reported for aarch64 is also
> visible on x86_64. Reported for a ppc64 target, it takes
> a while to trigger the bug.
>
>
> r~
>
>
>
> The following changes since commit a008535b9fa396226ff9cf78b8ac5f3584bda5
On Thu, Sep 22, 2016 at 02:15:08PM +0800, Peter Xu wrote:
> pci-testdev is used mostly in kvm-unit-test for some eventfd tests.
> However I see it a good framework for other tests as well (e.g., the
> IOMMU unit test in the future). So enhanced it to support more
> testcases.
>
> The original memo
"Dr. David Alan Gilbert" writes:
>> > You might find the first two patches in:
>> >https://lists.gnu.org/archive/html/qemu-devel/2016-08/msg03681.html
>> > useful in debugging this; it prints the values when the _EQUAL macros fail
>> > and prints
>> > the field name that fails.
>>
>> Thanks,
Marc-André Lureau writes:
> Hi
>
> - Original Message -
>> Marc-André Lureau writes:
>>
>> > Hi
>> >
>> > - Original Message -
>> >> On 09/13/2016 08:01 AM, Marc-André Lureau wrote:
>> >> > Signed-off-by: Marc-André Lureau
>> >> > ---
>> >> > docs/qmp-commands.txt | 29 ---
On 09/22/2016 01:07 PM, Nikunj A Dadhania wrote:
> Benjamin Herrenschmidt writes:
>
>> On Thu, 2016-09-22 at 14:34 +0530, Nikunj A Dadhania wrote:
>>> Something like this works for KVM:
>>>
>>> diff --git a/target-ppc/machine.c b/target-ppc/machine.c
>>> index 4820f22..1cf3779 100644
>>> --- a/ta
On Thu, Sep 22, 2016 at 01:12:22PM +0200, Markus Armbruster wrote:
> "Daniel P. Berrange" writes:
>
> > On Thu, Sep 22, 2016 at 10:36:45AM +0200, Markus Armbruster wrote:
> >> Don't make up a description in user_creatable_help_func(), improve the
> >> description infrastructure and its use so you
Hi Alberto,
On Fri 16 Sep 2016 10:33:36 AM CEST, Pradeep Jagadeesh wrote:
Hi,
first of all, sorry for the late reply! Here are my comments:
--- a/fsdev/qemu-fsdev-opts.c
+++ b/fsdev/qemu-fsdev-opts.c
@@ -37,6 +37,82 @@ static QemuOptsList qemu_fsdev_opts = {
}, {
.name
On Thu, 2016-09-22 at 12:32 +0200, Paolo Bonzini wrote:
> *However* a better fix would be to preserve the old flags for
> pseries-2.6, and only set the newer flags for pseries-2.7. I'm not
> saying you have to do this, but if it's not hard (no idea) why not learn
> how to do it right.
>
> The des
On Thu, 2016-09-22 at 13:27 +0200, Cédric Le Goater wrote:
> > TCG migration succeeds and proceeds ahead. But fails somewhere
> > ahead in
> > powerpc exception handler:
> >
> > [qemu]$ ./ppc64-softmmu/qemu-system-ppc64 -machine pseries-
> > 2.6,usb=off -vga none -nographic -m 2G ../../imgs/gu
Hi Greg,
On Wed, 21 Sep 2016 12:22:25 +0200
Pradeep Jagadeesh wrote:
Hi Greg,
Thanks for having a look at patchset.
See the replies below.
On Fri, 16 Sep 2016 04:33:36 -0400
Pradeep Jagadeesh wrote:
Uses throttling APIs to limit I/O bandwidth and number of operations on the
fsdev device
Paolo Bonzini writes:
> On 22/09/2016 11:19, Markus Armbruster wrote:
>>> > I think -device is okay for something that isn't a "backend" but is
>>> > directly guest-visible.
>> Well, the contents of a block device is just as guest-visible. We split
>> the device in a frontend and a backend, and
"Daniel P. Berrange" writes:
> On Wed, Sep 21, 2016 at 01:18:58PM -0500, Eric Blake wrote:
>> On 09/21/2016 10:28 AM, Markus Armbruster wrote:
>>
>> >> The compiler.h file has no license header, just a comment
>> >> saying "public domain", which is obviously not the case
>> >> if you add this ma
Paolo Bonzini writes:
> On 22/09/2016 09:12, Markus Armbruster wrote:
>> Paolo Bonzini writes:
>>
>>> ---
>>> scripts/checkpatch.pl | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
>>> index dde3f5f..3afa19a 100755
>>>
Hi
- Original Message -
> Marc-André Lureau writes:
>
> > Hi
> >
> > - Original Message -
> >> Marc-André Lureau writes:
> >>
> >> > Hi
> >> >
> >> > - Original Message -
> >> >> On 09/13/2016 08:01 AM, Marc-André Lureau wrote:
> >> >> > Signed-off-by: Marc-André Lureau
Xiao Long Jiang writes:
> This fixes a Qemu crash that introduced by commit ad73970
> ("qom: Wrap prop visit in visit_start_struct").
>
> Not all of the callers of user_creatable_add_type always pass qdict,
> so we should check qdict against NULL, but not raise an assertion.
>
> The problem can b
On Sun, Sep 04, 2016 at 11:38:58PM +0900, Namhyung Kim wrote:
> The virtio pstore driver provides interface to the pstore subsystem so
> that the guest kernel's log/dump message can be saved on the host
> machine. Users can access the log file directly on the host, or on the
> guest at the next bo
Uses throttling APIs to limit I/O bandwidth and number of operations on the
devices which use 9p-local driver.
Signed-off-by: Pradeep Jagadeesh
---
fsdev/Makefile.objs | 1 +
fsdev/file-op-9p.h | 3 +
fsdev/qemu-fsdev-opts.c | 76 +++
fsdev/qemu-fsd
"Daniel P. Berrange" writes:
> On Thu, Sep 22, 2016 at 01:12:22PM +0200, Markus Armbruster wrote:
>> "Daniel P. Berrange" writes:
>>
>> > On Thu, Sep 22, 2016 at 10:36:45AM +0200, Markus Armbruster wrote:
>> >> Don't make up a description in user_creatable_help_func(), improve the
>> >> descrip
Vhost-user requires an early GET_FEATURES call to determine if the
slave supports protocol feature negotiation. An extra GET_FEATURES
call is made after vhost_backend_init() to actually set the device
features.
This patch moves the actual setting of the device features to both
implementations (ker
From: Peter Maydell
The POSIX standard mandates that for a connected socket recvfrom()
must ignore the msg_name and msg_namelen fields. This is awkward
for QEMU because we will attempt to copy them from guest address
space. Handle this by not immediately returning a TARGET_EFAULT
if the copy fail
From: Riku Voipio
The following changes since commit a008535b9fa396226ff9cf78b8ac5f3584bda58e:
build-sys: fix make install regression (2016-09-20 11:32:43 +0100)
are available in the git repository at:
git://git.linaro.org/people/riku.voipio/qemu.git tags/pull-linux-user-20160915
for you
From: Peter Maydell
Implement the FS_IOC_GETFLAGS and FS_IOC_SETFLAGS ioctls, as used
by chattr.
Note that the type information encoded in these ioctl numbers
is at odds with the actual type the kernel accesses, as discussed
in http://thread.gmane.org/gmane.linux.file-systems/80164.
Signed-off-
From: Peter Maydell
The shmat() handling needs to do target-specific handling
of the attach address for shmat():
* if the SHM_RND flag is passed, the address is rounded
down to a SHMLBA boundary
* if SHM_RND is not passed, then the call is failed EINVAL
if the address is not a multiple of
From: Peter Maydell
Native strace reports when the process being traced takes a signal:
--- SIGSEGV {si_signo=SIGSEGV, si_code=SI_KERNEL, si_addr=0} ---
Report something similar when QEMU is doing its internal strace of
the guest process and is about to deliver it a signal.
Signed-off-by: Pe
From: Peter Maydell
The linux utimensat syscall differs in semantics from the
libc function because the syscall combines the features
of utimensat() and futimens(). Rather than trying to
split these apart in order to call the two libc functions
which then call the same underlying syscall, just al
From: Peter Maydell
do_ioctl_dm() should return target errno values, not host ones;
correct an accidental use of a host errno in an error path.
Signed-off-by: Peter Maydell
Signed-off-by: Riku Voipio
---
linux-user/syscall.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/
From: Peter Maydell
The sendmsg and recvmsg syscalls use a different errno to indicate
an overlarge iovec length from readv and writev. Handle this
special case in do_sendrcvmsg_locked() to avoid getting the
default errno returned by lock_iovec().
Signed-off-by: Peter Maydell
Signed-off-by: Rik
From: Peter Maydell
The kernel checks that the maxevents parameter to epoll_wait
is non-negative and not larger than EP_MAX_EVENTS. Add this
check to our implementation, so that:
* we fail these cases EINVAL rather than EFAULT
* we don't pass negative or overflowing values to the
lock_user()
From: Peter Maydell
A target_mmap() call in load_elf_binary() was missing the MAP_ANONYMOUS
flag. (Spotted by Coverity, because target_mmap() will try to use
-1 as the filedescriptor in this case.)
This has never been noticed because the code in question is for
handling ancient SVr4 iBCS2 binari
From: Peter Maydell
In the kernel the length of an iovec is generally handled as
an unsigned long, not an integer; fix the parameter to
lock_iovec() accordingly.
Signed-off-by: Peter Maydell
Signed-off-by: Riku Voipio
---
linux-user/syscall.c | 8
1 file changed, 4 insertions(+), 4 d
101 - 200 of 659 matches
Mail list logo