Applied to -trivial, thanks!
/mjt
Eric Blake writes:
> On 02/01/2016 05:31 AM, Markus Armbruster wrote:
>
>>> |+visit_start_struct(v, NULL, NULL, "ACPI_DEVICE_OST", 0, &err);
>>> | if (err) {
>>> | goto out;
>>> | }
>>> | visit_type_ACPIOSTInfo(v, &info, "info", &err);
>>> | if (err) {
>>> |-go
On 31/01/16 21:57, Paolo Bonzini wrote:
> On 31/01/2016 18:54, Peter Maydell wrote:
>> On 31 January 2016 at 17:19, Paolo Bonzini wrote:
>>> On 31/01/2016 16:54, Mark Cave-Ayland wrote:
I also notice that with the above commit I lose cycling through history
in the GTK monitor - even wit
Hi Gerd/Alex,
On Mon, Feb 01, 2016 at 02:44:55PM -0700, Alex Williamson wrote:
> On Mon, 2016-02-01 at 14:10 +0100, Gerd Hoffmann wrote:
> > Hi,
> >
> > > > Unfortunately it's not the only one. Another example is, device-model
> > > > may want to write-protect a gfn (RAM). In case that this req
Hi,
> +realloc:
> +opregion = malloc_high(size * 1024);
memalign_high(PAGE_SIZE, size * 1024);
> > I'd have qemu copy the data on 0xfc write then, so things continue to
> > work without updating seabios. So, the firmware has to allocate space,
> > reserve it etc., and programming the 0xf
> From: Kirti Wankhede [mailto:kwankh...@nvidia.com]
> Sent: Tuesday, February 02, 2016 9:48 AM
>
> Resending this mail again, somehow my previous mail didn't reached every
> to everyone's inbox.
>
> On 2/2/2016 3:16 AM, Kirti Wankhede wrote:
> > Design for vGPU Driver:
> > Main purpose of vGPU d
Hi,
> Alternatively can the vgpu
> driver copy it to a private buffer and hardware can execute from that?
Copying is an option, but given the size execbuffers can have this comes
with a noticeable performance difference.
cheers,
Gerd
> -Original Message-
> From: EXT Peter Crosthwaite [mailto:crosthwaitepe...@gmail.com]
> Sent: Monday, February 01, 2016 7:29 PM
> To: Krzeminski, Marcin (Nokia - PL/Wroclaw)
> Cc: Peter Maydell; qemu-devel@nongnu.org Developers; Lenkow, Pawel
> (Nokia - PL/Wroclaw)
> Subject: Re: [Qemu-d
> From: Kay, Allen M
> Sent: Saturday, January 30, 2016 5:58 AM
>
> First of all, I would like to clarify I'm talking about general IGD
> passthrough case - not
> specific to KVMGT. In IGD passthrough configuration, one of the following
> will happen
> when the driver accesses OpRegion:
>
> 1)
01.02.2016 21:01, Peter Maydell wrote:
> On 1 February 2016 at 17:20, Wei Huang wrote:
>> This patch removes float_high field of PL061State, which doesn't seem
>> to be used anywhere.
[]
>> @@ -88,7 +87,6 @@ static const VMStateDescription vmstate_pl061 = {
>> VMSTATE_UINT32(slr, PL061Sta
> From: Gerd Hoffmann
> Sent: Monday, February 01, 2016 8:49 PM
>
> Hi,
>
> > Thanks for the tip that seabios allocated pages automatically become
> > e820 reserved, that simplifies things a bit.
>
> It's common practice for all firmware. The e820 table from qemu is just
> a starting point, i
> From: Kay, Allen M
> Sent: Tuesday, February 02, 2016 8:04 AM
> >
> > David notes in the latter commit above:
> >
> > "We should be able to successfully assign graphics devices to guests too, as
> > long as the initial handling of stolen memory is reconfigured
> > appropriately."
> >
> > What co
The new feature for qcow2: storing bitmaps.
This patch adds new header extension to qcow2 - Bitmaps Extension. It
provides an ability to store virtual disk related bitmaps in a qcow2
image. For now there is only one type of such bitmaps: Dirty Tracking
Bitmap, which just tracks virtual disk change
The proposed IGD OpRegion support in QEMU via vfio maps the host
OpRegion into VM system memory at the address written to the ASL
Storage register (0xFC). The OpRegion contains a 16-byte signature
followed by a 4-byte size field. Therefore SeaBIOS can allocate a
buffer of the typical size (8KB),
On 02/01/2016 08:01 PM, zhanghailiang wrote:
We add each netdev a default buffer filter, and
the default buffer filter is disabled, so it has
no side effect for packets delivering in qemu net layer.
The default buffer filter can be used by COLO or Micro-checkpoint,
The reason we add the defaul
On 02/01/2016 08:01 PM, zhanghailiang wrote:
We add a new helper function netdev_add_filter(), this function
can help adding a filter object to a netdev.
Besides, we add a is_default member for struct NetFilterState
to indicate whether the filter is default or not.
Signed-off-by: zhanghailiang
This is provided via a device specific region, look for it on Intel
VGA class devices, initialize it, and tie it to the config space
register at 0xFC. Note that on bare metal it seems that 0xFC only
points to the memory reserved by the BIOS for the OpRegion, in the
model used here, programming the
Signed-off-by: Alex Williamson
---
linux-headers/linux/vfio.h | 98 +++-
1 file changed, 96 insertions(+), 2 deletions(-)
diff --git a/linux-headers/linux/vfio.h b/linux-headers/linux/vfio.h
index aa276bc..7e955f1 100644
--- a/linux-headers/linux/vfio.h
Both platform and PCI vfio drivers create a "slow", I/O memory region
with one or more mmap memory regions overlayed when supported by the
device. Generalize this to a set of common helpers in the core that
pulls the region info from vfio, fills the region data, configures
slow mapping, and adds he
In preparation for supporting capability chains on regions, wrap
ioctl(VFIO_DEVICE_GET_REGION_INFO) so we don't duplicate the code for
each caller.
Signed-off-by: Alex Williamson
---
hw/vfio/common.c | 18 +
hw/vfio/pci.c | 81 +---
The sparse mmap capability in a vfio region info allows vfio to tell
us which sub-areas of a region may be mmap'd. Thus rather than
assuming a single mmap covers the entire region and later frobbing it
ourselves for things like the PCI MSI-X vector table, we can read that
directly from vfio.
Sign
vfio-pci currently requires a host= parameter, which comes in the
form of a PCI address in [domain:] notation. We
expect to find a matching entry in sysfs for that under
/sys/bus/pci/devices/. vfio-platform takes a similar approach, but
defines the host= parameter to be a string, which can be mat
Match common vfio code with setup, exit, and finalize functions for
BAR, quirk, and VGA management. VGA is also changed to dynamic
allocation to match the other MemoryRegions.
Signed-off-by: Alex Williamson
---
hw/vfio/pci-quirks.c | 38 -
hw/vfio/pci.c| 114 +
This is the QEMU compliment to the vfio kernel capability chain
series. This is RFC since it depends on those non-upstream kernel
changes. Patch 1/ will be posted separately, it's somewhat unrelated,
but is in my build tree so I include it here for anyone that wants to
build this series.
This se
If we have to add a default filter, then I have a suggestion only
for this series:
1. Add a nop filter. filter-nop.c
2. Add a "default-filter=xxx" property to -netdev, if not specified,
default to nop.
On 02/01/2016 08:01 PM, zhanghailiang wrote:
This series is a prerequisite for COLO, here w
From: zhanghailiang
The properties of netfilter object could be changed by 'qom-set'
command, but the output of 'info network' command is not updated,
because it got the old information through nf->info_str, it will
not be updated while we change the value of netfilter's property.
Here we split
From: Laszlo Ersek
The start_xmit() and e1000_receive_iov() functions implement DMA transfers
iterating over a set of descriptors that the guest's e1000 driver
prepares:
- the TDLEN and RDLEN registers store the total size of the descriptor
area,
- while the TDH and RDH registers store the of
From: Li Zhijian
Previously, if we attach more than one filters for a single netdev,
both ingress and egress traffic will go through net filters in same
order like:
ingress: netdev ->filter1 ->filter2 ->...filter[n] ->emulated device
egress: emulated device ->filter1 ->filter2 ->...filter[n] ->n
From: Guillaume Subiron
This patch simply adds a sa_family_t argument to remove the hardcoded
"AF_INET" in the call of qemu_socket().
This prepares for IPv6 support.
Signed-off-by: Guillaume Subiron
Signed-off-by: Samuel Thibault
Reviewed-by: Thomas Huth
Signed-off-by: Jason Wang
---
slirp
From: Vincenzo Maffione
This patch simplifies the netmap backend code by means of the nm_open()
helper function provided by netmap_user.h, which hides the details of
open(), iotcl() and mmap() carried out on the netmap device.
Moreover, the semantic of nm_open() makes it possible to open special
From: Guillaume Subiron
A sa_family_t is now passed in argument to udp_attach instead of using a
hardcoded "AF_INET" to call qemu_socket().
This prepares for IPv6 support.
Signed-off-by: Guillaume Subiron
Signed-off-by: Samuel Thibault
Reviewed-by: Thomas Huth
Signed-off-by: Jason Wang
---
From: Guillaume Subiron
This patch makes solookup() compatible with varying address
families, by using a new sockaddr_equal() function that compares
two sockaddr_storage.
This prepares for IPv6 support.
Signed-off-by: Guillaume Subiron
Signed-off-by: Samuel Thibault
Reviewed-by: Thomas Huth
From: Guillaume Subiron
In if_encap, a switch is added to prepare for the IPv6 case. Some code
is factorized.
This prepares for IPv6 support.
Signed-off-by: Guillaume Subiron
Signed-off-by: Samuel Thibault
Reviewed-by: Thomas Huth
---
slirp/slirp.c | 61 +
From: Guillaume Subiron
This patch factorizes some duplicate code into a new function,
sotranslate_out(). This function perform the address translation when a
packet is transmitted to the host network. If the packet is destinated
to the host, the loopback address is used, and if the packet is
des
From: Guillaume Subiron
This patch replaces foreign and local address/port couples in Socket
structure by 2 sockaddr_storage which can be casted in sockaddr_in.
Direct access to address and port is still possible thanks to some
\#define, so retrocompatibility of the existing code is assured.
The
From: Guillaume Subiron
solookup() was only compatible with TCP. Having the socket list in
argument, it is now compatible with UDP too.
Some optimization code is factorized inside the function (the function
look at the last returned result before browsing the complete socket
list).
This prepare
From: Guillaume Subiron
Before this patch, if sosendto fails, udp_input is executed as if the
packet was sent, recording the packet for icmp errors, which does not
makes sense since the packet was not actually sent, errors would be
related to a previous packet.
This patch adds a goto bad to cut
From: "Michael S. Tsirkin"
gem_transmit copies a packet from guest into an tx_packet[2048]
array on stack, with size limited by descriptor length set by guest. If
guest is malicious and specifies a descriptor length that is too large,
and should packet size exceed array size, this results in a b
From: Prasad J Pandit
While receiving packets in 'gem_receive' routine, if Frame Check
Sequence(FCS) is enabled, it copies the packet into a local
buffer without checking its size. Add check to validate packet
length against the buffer size to avoid buffer overflow.
Reported-by: Ling Liu
Signed
From: Guillaume Subiron
Basically, this patch replaces "arp" by "resolution" every time "arp"
means "mac resolution" and not specifically ARP.
This prepares for IPv6 support.
Signed-off-by: Guillaume Subiron
Signed-off-by: Samuel Thibault
Reviewed-by: Thomas Huth
Signed-off-by: Jason Wang
-
From: Thomas Huth
Since -smb and -redir are deprecated options, we should not
use them as examples in the documentation anymore.
Signed-off-by: Thomas Huth
Signed-off-by: Jason Wang
---
qemu-doc.texi | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/qemu-doc.texi b/
From: Thomas Huth
We don't want to support the legacy -tftp, -bootp, -smb and
-net channel options forever. So let's start telling the users
that they are deprecated and what option should be used instead.
Signed-off-by: Thomas Huth
Signed-off-by: Jason Wang
---
net/slirp.c | 3 +++
os-posix.
The following changes since commit 0430891ce162b986c6e02a7729a942ecd2a32ca4:
hw: Clean up includes (2016-01-29 15:07:25 +)
are available in the git repository at:
https://github.com/jasowang/qemu.git tags/net-pull-request
for you to fetch changes up to e8a7a1a574ed6728422959c8aa79ca584c
Hi Eric,
On 02/01/2016 08:37 AM, Eric Auger wrote:
Hi Shanker, Vikram,
On 01/30/2016 12:00 AM, Shanker Donthineni wrote:
From: Vikram Sethi
This patch introduces a Qualcomm Technologies, Inc HiDMA
device and allows the instantiation of the vfio-qcom-hidma
device from the QEMU command line
(-d
On Mon, 02/01 17:18, John Snow wrote:
> Signed-off-by: John Snow
> ---
> blockjob.c | 8
> 1 file changed, 8 deletions(-)
>
> diff --git a/blockjob.c b/blockjob.c
> index 80adb9d..a692142 100644
> --- a/blockjob.c
> +++ b/blockjob.c
> @@ -278,14 +278,6 @@ void block_job_iostatus_reset(B
With a mirror job running on a virtio-blk dataplane disk, sending "q" to
HMP will cause a dead loop in block_job_finish_sync.
This is because the aio_poll() only processes the AIO context of bs
which has no more work to do, while the main loop BH that is scheduled
for setting the job->completed fl
Resending this mail again, somehow my previous mail didn't reached every
to everyone's inbox.
On 2/2/2016 3:16 AM, Kirti Wankhede wrote:
Design for vGPU Driver:
Main purpose of vGPU driver is to provide a common interface for vGPU
management that can be used by differnt GPU drivers.
This modul
Signed-off-by: Jeff Cody
---
tests/qemu-iotests/143 | 114 +
tests/qemu-iotests/143.out | 24 ++
tests/qemu-iotests/group | 1 +
3 files changed, 139 insertions(+)
create mode 100755 tests/qemu-iotests/143
create mode 100644 tests/qem
This fixes a regression introduced with commit 3f09bfbc7. Multiple
bugs arise in conjunction with live snapshots and mirroring operations
(which include active layer commit).
After a live snapshot occurs, the active layer and the base layer both
have a non-NULL tqe_prev field in the device_list,
Changes from v1:
* Rather than allow insertion when bs->device_listtqe_prev points to
a NULL entry, make sure than we follow the block scheme of enforcing
bs->device_list->tqe_prev is NULL upon deletion. (Thanks Max!)
Bug #1300209 is a regression in 2.5, introduced during the
change away fro
On Mon, Feb 01, 2016 at 10:43:02PM +0100, Max Reitz wrote:
> On 30.01.2016 06:17, Jeff Cody wrote:
> > In change_parent_backing_link(), we only inserted the new
> > BlockDriverState entry into the device_list if the tqe_prev pointer was
> > NULL. However, we must also allow insertion when the BDS
On 22/01/2016 08:46, Fam Zheng wrote:
> On Wed, 01/13 10:50, Fam Zheng wrote:
>> > v10: Fix and simplify mirror_cow_align. [Max]
> Jeff, are you happy to take these patches?
Ping again. I have patches waiting for these to be accepted too.
Paolo
> -Original Message-
> From: Alex Williamson [mailto:alex.william...@redhat.com]
> Sent: Sunday, January 31, 2016 9:42 AM
> To: Kay, Allen M; Gerd Hoffmann; David Woodhouse
> Cc: igv...@ml01.01.org; xen-de...@lists.xensource.com; Eduardo Habkost;
> Stefano Stabellini; qemu-devel@nongnu.or
On 01/22/2016 10:12 AM, Markus Armbruster wrote:
> Eric Blake writes:
>
>> Implement the new type_null() callback for the qmp input visitor.
>> While we don't yet have a use for this in qapi (the generator
>> will need some tweaks first), one usage is already envisioned:
>> when changing blockdev
On Sat, Jan 30, 2016 at 1:35 AM, Michael Tokarev wrote:
> 18.01.2016 21:42, Alistair Francis wrote:
>> The description of object_property_get_int() stated that on an error
>> it returns NULL. This is not the case and the function will return -1
>> if an error occurs. Update the commented documenta
On 02/01/2016 06:07 AM, Markus Armbruster wrote:
> Eric Blake writes:
>
>> No backend was setting an error when ending an implicit struct,
>> or when iterating a list.
>
> Perhaps "when ending the visit of a list or implicit struct, or when
> moving to the next list node" would be more precise.
On 02/01/2016 05:31 AM, Markus Armbruster wrote:
>> |+visit_start_struct(v, NULL, NULL, "ACPI_DEVICE_OST", 0, &err);
>> | if (err) {
>> | goto out;
>> | }
>> | visit_type_ACPIOSTInfo(v, &info, "info", &err);
>> | if (err) {
>> |-goto out;
>> |+goto out_o
Hi! I have in my trace instrumentation queue a series that adds a very simple
but efficient way to trigger code in QEMU from guest code using guest-agnostic
code.
Blue Swirl showed some interest long ago in using it in the test suite (e.g.,
instruct QEMU to check the vCPU state after a series of i
Lluís Vilanova writes:
> Bastian Koppelmann writes:
>> Hi Lluis,
>> On 01/27/2016 07:54 PM, Lluís Vilanova wrote:
[...]
>>>
>>> So, I'd say that such support is on the list of current developments (at
>>> least
>>> mine, specially now that I have a bit more time for it). But getting the
>>> cor
On 02/01/2016 03:18 PM, John Snow wrote:
> Signed-off-by: John Snow
> ---
> blockjob.c | 8
> 1 file changed, 8 deletions(-)
Reviewed-by: Eric Blake
>
> diff --git a/blockjob.c b/blockjob.c
> index 80adb9d..a692142 100644
> --- a/blockjob.c
> +++ b/blockjob.c
> @@ -278,14 +278,6 @@ v
Signed-off-by: John Snow
---
blockjob.c | 8
1 file changed, 8 deletions(-)
diff --git a/blockjob.c b/blockjob.c
index 80adb9d..a692142 100644
--- a/blockjob.c
+++ b/blockjob.c
@@ -278,14 +278,6 @@ void block_job_iostatus_reset(BlockJob *job)
}
}
-struct BlockFinishData {
-B
On Mon, 2016-02-01 at 13:49 +0100, Gerd Hoffmann wrote:
> > > Maybe we should define the interface as "guest writes 0xfc to pick
> > > address, qemu takes care to place opregion there". That gives us the
> > > freedom to change the qemu implementation (either copy host opregion or
> > > map the ho
This series contains fixes to the SD card emulation that are needed to
unblock Tianocore EDK2 UEFI (specifically, the bootloader for Windows
on Raspberry Pi 2).
Changes in v2, based on feedback from Peter Crosthwaite:
* correct implementation of CMD23 to switch to transfer state on completion
*
> From: Peter Maydell [mailto:peter.mayd...@linaro.org]
> Sent: Monday, 25 January 2016 10:37
>
> On 25 January 2016 at 18:06, Andrew Baumann
> wrote:
> > This is the most recent version of the patch series. However,
> > there was an unresolved question about migration compatibility
> > for the v
CMD23 is optional for SD but required for MMC, and the UEFI bootloader
used for Windows on Raspberry Pi 2 issues it.
Reviewed-by: Peter Crosthwaite
Signed-off-by: Andrew Baumann
---
hw/sd/sd.c | 37 +
1 file changed, 37 insertions(+)
diff --git a/hw/sd/sd.c
The SD spec for ACMD41 says that a zero argument is an "inquiry"
ACMD41, which does not start initialisation and is used only for
retrieving the OCR. However, Tianocore EDK2 (UEFI) has a bug [1]: it
first sends an inquiry (zero) ACMD41. If that first request returns an
OCR value with the power up b
Some of these errors may be harmless (e.g. probing unimplemented
commands, or issuing CMD12 in the wrong state), and may also be quite
frequent. Spamming the standard error output isn't desirable in such
cases.
Reviewed-by: Peter Crosthwaite
Signed-off-by: Andrew Baumann
---
It might also be des
On Mon, 2016-02-01 at 14:10 +0100, Gerd Hoffmann wrote:
> Hi,
>
> > > Unfortunately it's not the only one. Another example is, device-model
> > > may want to write-protect a gfn (RAM). In case that this request goes
> > > to VFIO .. how it is supposed to reach KVM MMU?
> >
> > Well, let's work
On 30.01.2016 06:17, Jeff Cody wrote:
> In change_parent_backing_link(), we only inserted the new
> BlockDriverState entry into the device_list if the tqe_prev pointer was
> NULL. However, we must also allow insertion when the BDS pointed
> to by the tqe_prev pointer is NULL as well.
>
> This fi
On 27.01.2016 16:52, Vladimir Sementsov-Ogievskiy wrote:
> The new feature for qcow2: storing bitmaps.
>
> This patch adds new header extension to qcow2 - Bitmaps Extension. It
> provides an ability to store virtual disk related bitmaps in a qcow2
> image. For now there is only one type of such bi
Le 29/01/2016 23:35, John Snow a écrit :
Accidentally, I removed a "feature" where empty drives had geometry
values applied to them, which allows seek on empty drives to work
"by accident," as QEMU actually tries to disallow that.
Seeks on empty drives should work, though, but the easiest thing
Current QEMU doesn't clear PL061 state after reset. This causes a
weird issue with guest reboot via GPIO. Here is the device state
description with two reboot requests:
(PL061State fields) data old_in_data istate
VM boot 0 0 0
After 1st ACPI
This patch removes the float_high field of PL061State, which doesn't
seem to be used anywhere. Because this changes the device state, the
version ID is also bumped up for the reason of compatiblity.
Signed-off-by: Wei Huang
---
hw/gpio/pl061.c | 6 ++
1 file changed, 2 insertions(+), 4 delet
On 01/29/2016 10:17 PM, Jeff Cody wrote:
> Bug #1300209 is a regression in 2.5, introduced during the
> change away from bdrv_swap().
>
> When we change the parent backing link (change_parent_backing_link),
> we must also accomodate non-NULL tqe_prev pointers that point to a
> NULL entry. Please
Signed-off-by: Andrew Jones
---
kvm-all.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kvm-all.c b/kvm-all.c
index 9148889921197..330f509a0da84 100644
--- a/kvm-all.c
+++ b/kvm-all.c
@@ -2362,7 +2362,7 @@ int kvm_set_one_reg(CPUState *cs, uint64_t id, void
*source)
On Mon, Feb 1, 2016 at 3:25 AM, Ruslan Bilovol wrote:
> On Wed, Jan 27, 2016 at 7:39 PM, Peter Crosthwaite
> wrote:
>> On Tue, Jan 26, 2016 at 4:05 AM, Ruslan Bilovol
>> wrote:
>>> On Mon, Jan 25, 2016 at 6:07 PM, Peter Maydell
>>> wrote:
On 25 January 2016 at 13:51, Ruslan Bilovol
This fixes double-definitions in bsd-user builds when using the UST
tracing backend (which indirectly includes the system's "syscall.h").
Signed-off-by: Lluís Vilanova
Reviewed-by: Peter Maydell
---
bsd-user/i386/target_syscall.h|4
bsd-user/qemu.h |2 +-
bsd-
This fixes double-definitions in linux-user builds when using the UST
tracing backend (which indirectly includes the system's "syscall.h").
Signed-off-by: Lluís Vilanova
---
linux-user/aarch64/target_syscall.h|5 +
linux-user/alpha/target_syscall.h |5 +
linux-user/arm/t
This fixes double-definitions in *-user builds when using the UST
tracing backend (which indirectly includes the system's "syscall.h").
Changes in v3
=
* Fix stray new files in generic target code [Peter Maydell].
Changes in v2
=
* Fix includes for bsd-user too [Peter
Peter Maydell writes:
> On 1 February 2016 at 14:18, Lluís Vilanova wrote:
>> This fixes double-definitions in linux-user builds when using the UST
>> tracing backend (which indirectly includes the system's "syscall.h").
>>
>> Signed-off-by: Lluís Vilanova
>> ---
>> linux-user/aarch64/target_sy
On Mon, Dec 21, 2015 at 5:39 AM, Krzeminski, Marcin (Nokia -
PL/Wroclaw) wrote:
>
>
> W dniu 21.12.2015 o 12:04, Peter Crosthwaite pisze:
>> On Wed, Dec 16, 2015 at 4:57 AM, wrote:
>>> From: Marcin Krzeminski
>>>
>>> Signed-off-by: Marcin Krzeminski
>>> ---
>>> hw/block/m25p80.c | 38
On 1 February 2016 at 17:20, Wei Huang wrote:
> This patch removes float_high field of PL061State, which doesn't seem
> to be used anywhere.
>
> Signed-off-by: Wei Huang
> ---
> hw/gpio/pl061.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/hw/gpio/pl061.c b/hw/gpio/pl061.c
> index 34
Stefan Berger/Watson/IBM wrote on 01/21/2016 07:31:10 AM:
>
> "Dr. David Alan Gilbert" wrote on 01/21/2016
> 06:40:35 AM:
> >
> > >
> > > There is one issue in case of resume of a snapshot. If the
> permanent state
> > > of the TPM is modified during snapshotting, like ownership is
> taken
On 01/26/2016 06:08 PM, Alex Williamson wrote:
> On Tue, 2016-01-26 at 16:03 +0100, Eric Auger wrote:
>>
>> Hi Alex,
>>
>> I did a try with both legacy cmd line and new one and it works fine for
>> vfio platform too:
>> -device vfio-calxeda-xgmac,host="fff51000.ethernet"
>> -device
>> vfio-calxeda-
On 02/01/2016 04:17 AM, Igor Mammedov wrote:
> On Fri, 29 Jan 2016 09:13:15 -0600
> Wei Huang wrote:
>
>> On 01/29/2016 08:50 AM, Peter Maydell wrote:
>>> On 29 January 2016 at 14:46, Shannon Zhao wrote:
On 2016/1/29 22:35, Wei Huang wrote:
> On 01/29/2016 04:10 AM, Shannon Zhao w
On 02/01/2016 06:07 AM, Markus Armbruster wrote:
> Eric Blake writes:
>
>> No backend was setting an error when ending an implicit struct,
>> or when iterating a list.
>
> Perhaps "when ending the visit of a list or implicit struct, or when
> moving to the next list node" would be more precise.
Current QEMU doesn't clear PL061 state after reset. This causes a
weird issue with guest reboot via GPIO. Here is the device state
description with two reboot requests:
(PL061State fields) data old_in_data istate
VM boot 0 0 0
After 1st ACPI
This patch removes float_high field of PL061State, which doesn't seem
to be used anywhere.
Signed-off-by: Wei Huang
---
hw/gpio/pl061.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/hw/gpio/pl061.c b/hw/gpio/pl061.c
index 342a70d..2c08e88 100644
--- a/hw/gpio/pl061.c
+++ b/hw/gpio/pl061.c
Marc-André Lureau writes:
> Hi
>
> On Fri, Jan 29, 2016 at 5:23 PM, Markus Armbruster wrote:
>> marcandre.lur...@redhat.com writes:
>>
>>> From: Marc-André Lureau
>>>
>>> Simplify the interrupt handling by having a single callback on irq&msi
>>> cases. Remove usage of CharDriver, replace it wit
On Mon, Feb 1, 2016 at 10:22 AM, Eric Auger wrote:
> Hi Rob,
> On 02/01/2016 05:15 PM, Rob Herring wrote:
>> On Mon, Feb 1, 2016 at 7:51 AM, Eric Auger wrote:
>>> This function returns the host device tree blob from sysfs
>>> (/proc/device-tree). It uses a recursive function inspired
>>> from dtc
Marc-André Lureau writes:
> Hi
>
> On Fri, Jan 29, 2016 at 4:59 PM, Markus Armbruster wrote:
>> marcandre.lur...@redhat.com writes:
>>
>>> From: Marc-André Lureau
>>>
>>> Call ivshmem_setup_interrupts() with or without MSI, always allocate
>>> msi_vectors that is going to be used in all case in
Marc-André Lureau writes:
> Hi
>
> On Fri, Jan 29, 2016 at 4:43 PM, Markus Armbruster wrote:
>> The existing users pass a func that saves dev, and free the saved dev
>> later. Works as long as we call func() at most once. If multiple
>> devices match, all but the last one are leaked. Can this
From: Marc-André Lureau
qpci_device_find() returns allocated data, don't leak it.
Signed-off-by: Marc-André Lureau
---
tests/libqos/pci.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/libqos/pci.c b/tests/libqos/pci.c
index 4e630c2..80b1a21 100644
--- a/tests/libqos/pci.c
+++ b/t
From: Marc-André Lureau
Simplify the interrupt handling by having a single callback on irq&msi
cases. Remove usage of CharDriver, replace it with
qemu_set_fd_handler(). Use event_notifier_test_and_clear() to read the
eventfd.
Before this patch, ivshmem writes the first byte received to
s->intrst
On 02/01/2016 05:27 PM, Rob Herring wrote:
> On Mon, Feb 1, 2016 at 10:22 AM, Eric Auger wrote:
>> Hi Rob,
>> On 02/01/2016 05:15 PM, Rob Herring wrote:
>>> On Mon, Feb 1, 2016 at 7:51 AM, Eric Auger wrote:
This function returns the host device tree blob from sysfs
(/proc/device-tree).
From: Marc-André Lureau
Recent commit 660c97ee introduced a regression in irq case, make
sure this code path is also tested.
Signed-off-by: Marc-André Lureau
---
tests/ivshmem-test.c | 53
1 file changed, 37 insertions(+), 16 deletions(-)
d
From: Marc-André Lureau
This is a ivshmem series with various bits:
- add a test for msi=off regression (the fix is included in the series
but was sent separatly to cc -stable)
- get rid of CharDriver usage for eventfd & fix regression introduced
by d0d7708ba by Daniel Berrange
- fix some tes
From: Marc-André Lureau
Broken since d0d7708ba29cbc, since the backend is NULL.
And now no longer needed by ivshmem.
Signed-off-by: Marc-André Lureau
---
include/sysemu/char.h | 3 ---
qemu-char.c | 13 -
2 files changed, 16 deletions(-)
diff --git a/include/sysemu/cha
Hi Rob,
On 02/01/2016 05:15 PM, Rob Herring wrote:
> On Mon, Feb 1, 2016 at 7:51 AM, Eric Auger wrote:
>> This function returns the host device tree blob from sysfs
>> (/proc/device-tree). It uses a recursive function inspired
>> from dtc read_fstree.
>>
>> Signed-off-by: Eric Auger
>>
>> ---
>>
From: Marc-André Lureau
Add a cleanup_vm() function to free QPCIDevice & QPCIBus when cleaning
up the IVState.
Signed-off-by: Marc-André Lureau
---
tests/ivshmem-test.c | 28 +---
1 file changed, 17 insertions(+), 11 deletions(-)
diff --git a/tests/ivshmem-test.c b/tes
1 - 100 of 204 matches
Mail list logo
