On Tue, Sep 20, 2016 at 10:53 PM, Jeff Cody wrote:
> On Tue, Sep 20, 2016 at 06:07:45PM -0700, Ashish Mittal wrote:
>> This patch adds support for a new block device type called "vxhs".
>> Source code for the library that this code loads can be downloaded from:
>> https://github.com/MittalAshish/l
Hi,
I have submitted a new patch (V6) after fixing the problem of
libqnioshim referencing symbols from inside qemu vxhs.c code.
(1) libqnioshim code has now been brought inside vxhs.c
(2) vxhs.c now needs to link only with libqnio.so.
(3) libqnio.so does not have any unreferenced symbols coming f
> -Original Message-
> From: Peter Xu [mailto:pet...@redhat.com]
> Sent: Wednesday, September 21, 2016 2:12 PM
> To: Wu, Feng
> Cc: qemu-devel@nongnu.org; pbonz...@redhat.com
> Subject: Re: [Qemu-devel] [PATCH] intel-iommu: Check IOAPIC's Trigger Mode
> against the one in IRTE
>
> On We
On Thu, Sep 15, 2016 at 02:45:57PM +0200, Cédric Le Goater wrote:
> On a real POWER8 system, the Pervasive Interconnect Bus (PIB) serves
> as a backbone to connect different units of the system. The host
> firmware connects to the PIB through a bridge unit, the
> Alter-Display-Unit (ADU), which giv
On Thu, Sep 15, 2016 at 02:45:59PM +0200, Cédric Le Goater wrote:
> From: Benjamin Herrenschmidt
>
> This version of the LPC controller model doesn't yet implement
> support for the SerIRQ deserializer present in the Naples version
> of the chip though some preliminary work is there.
>
> Signed-
On Thu, Sep 15, 2016 at 02:45:58PM +0200, Cédric Le Goater wrote:
> Now that we are using real HW ids for the cores in PowerNV chips, we
> can route the XSCOM accesses to them. We just need to attach a
> specific XSCOM memory region to each core in the appropriate window
> for the core number.
>
>
On Fri, Sep 16, 2016 at 08:11:45AM +1000, Benjamin Herrenschmidt wrote:
> On Thu, 2016-09-15 at 14:45 +0200, Cédric Le Goater wrote:
> > - The PCB translation is too much of a constraint for a specific
> > XSCOM address space, unless someone can explain me how to address 8
> > bytes at 0xb00
On Thu, Sep 15, 2016 at 02:46:00PM +0200, Cédric Le Goater wrote:
> As Qemu only supports a single instance of the ISA bus, we use the LPC
> controller of chip 0 to create one and plug in a couple of useful
> devices, like an UART and RTC. An IPMI BT device, which is also an ISA
> device, can be de
"Pavel Dovgalyuk" writes:
>> From: Felipe Franciosi [mailto:fel...@nutanix.com]
>> If compiling with -Werror=unused-result, replay-internal.c won't build
>> due to a call to fwrite() where the returned value is ignored. A simple
>> cast to (void) is not sufficient on recent GCCs, so this fixes it
Marc-André Lureau writes:
> - Original Message -
>> Hi Alberto
>>
>> - Original Message -
>> > On Mon, Sep 12, 2016 at 01:19:10PM +0400, Marc-André Lureau wrote:
>> > > Replace the old manual dispatch and validation code by the generic one
>> > > provided by qapi common code.
>>
On Wed, Sep 21, 2016 at 05:54:40AM +, Wu, Feng wrote:
>
>
> > -Original Message-
> > From: Peter Xu [mailto:pet...@redhat.com]
> > Sent: Wednesday, September 21, 2016 1:45 PM
> > To: Wu, Feng
> > Cc: qemu-devel@nongnu.org; pbonz...@redhat.com
> > Subject: Re: [Qemu-devel] [PATCH] int
Alistair Francis writes:
> This work is based on the original work by Li Guang with extra
> features added by Peter C and myself.
>
> The idea of this loader is to allow the user to load multiple images
> or values into QEMU at startup.
>
> Memory values can be loaded like this: -device
> loader
On Wed, Sep 21, 2016 at 12:27:26PM +0800, Fam Zheng wrote:
> Signed-off-by: Fam Zheng
> ---
> tests/.gitignore | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tests/.gitignore b/tests/.gitignore
> index b4a9cfc..24ac6cf 100644
> --- a/tests/.gitignore
> +++ b/tests/.gitignore
> @@ -70,6
> -Original Message-
> From: Peter Xu [mailto:pet...@redhat.com]
> Sent: Wednesday, September 21, 2016 1:45 PM
> To: Wu, Feng
> Cc: qemu-devel@nongnu.org; pbonz...@redhat.com
> Subject: Re: [Qemu-devel] [PATCH] intel-iommu: Check IOAPIC's Trigger Mode
> against the one in IRTE
>
> On Th
On Tue, Sep 20, 2016 at 06:07:45PM -0700, Ashish Mittal wrote:
> This patch adds support for a new block device type called "vxhs".
> Source code for the library that this code loads can be downloaded from:
> https://github.com/MittalAshish/libqnio.git
>
> Sample command line using JSON syntax:
>
On Thu, Sep 22, 2016 at 12:12:17AM +0800, Feng Wu wrote:
> The Trigger Mode field of IOAPIC must match the Trigger Mode in
> the IRTE according to VT-d Spec 5.1.5.1.
>
> Signed-off-by: Feng Wu
Reviewed-by: Peter Xu
Could I ask why we want this now? I know that both vector and trigger
mode shou
On Wed, Sep 21, 2016 at 10:18:00AM +0530, Bharata B Rao wrote:
> CPU unplug doesn't work in TCG mode currently and causes frequent system
> freeze. In addition to other potential problems, the main problem arises
> of out the requirement to support synchronous removal of a CPU thread.
> The CPU thr
On Thu, Sep 15, 2016 at 02:45:56PM +0200, Cédric Le Goater wrote:
> On PowerNV, CPU ids start at 0x8 or 0x20, we don't have a CPU 0
> anymore. So let's use the first_cpu index to initialize the monitor.
>
> Signed-off-by: Cédric Le Goater
I've rewritten the commit message to make the case for th
On 09/20/2016 10:07 PM, Igor Mammedov wrote:
On Fri, 12 Aug 2016 14:54:03 +0800
Xiao Guangrong wrote:
Currently, 'RLEN' is the totally buffer size written by QEMU and it is
ACPI internally used only. The buffer size returned to guest should
not include 'RLEN' itself
Do you see any errors in
Now that we allow CPU hot unplug on a few platforms, we can end up in a
situation where we don't have a CPU with index 0. Or at least we could,
if we didn't have code to explicitly prohibit unplug of CPU 0.
Longer term we want to allow CPU 0 unplug, this patch is an early step in
allowing this, b
Functions of type FindSysbusDeviceFunc currently return an integer.
However, this return value is always ignored by the caller in
find_sysbus_device().
This changes the function type to return void, to avoid confusion over
the function semantics.
Signed-off-by: David Gibson
---
hw/arm/sysbus-fd
This patch introduces asymmetric crypto service into virtio crypto
device. The asymmetric crypto service can be referred as signature,
verification, encryption, decryption, key generation and key exchange.
This patch depends on another virtio crypto device spec patch:
https://lists.gnu.org/archive/
> From: Felipe Franciosi [mailto:fel...@nutanix.com]
> If compiling with -Werror=unused-result, replay-internal.c won't build
> due to a call to fwrite() where the returned value is ignored. A simple
> cast to (void) is not sufficient on recent GCCs, so this fixes it.
>
> Signed-off-by: Felipe Fra
On Tue, 09/20 22:02, no-re...@ec2-52-6-146-230.compute-1.amazonaws.com wrote:
> BUILD fedora
> ARCHIVE qemu.tgz
> ARCHIVE dtc.tgz
> COPY RUNNER
> RUN test-mingw in fedora
> tests/docker/Makefile.include:107: recipe for target
> 'docker-run-test-mingw@fedora' failed
> make: *** [docker-ru
On 09/21/2016 12:51 PM, Alex Williamson wrote:
> On Wed, 21 Sep 2016 11:19:17 +0800
> Jike Song wrote:
>
>> On 09/21/2016 12:24 AM, Alex Williamson wrote:
>>> On Tue, 20 Sep 2016 10:50:47 +0800
>>> Jike Song wrote:
>>
>> /* trim the quotations */
>>
Even performing a lightweight sanity ch
Intel vIOMMU is still lacking of a complete IOMMU notifier mechanism.
Before that is achieved, let's open a door for vhost DMAR support, which
only requires cache invalidations (UNMAP operations).
Meanwhile, converting hw_error() to error_report() and exit(1), to make
the error messages clean and
Hi,
Your series failed automatic build test. Please find the testing commands and
their output below. If you have docker installed, you can probably reproduce it
locally.
Type: series
Message-id: 1474433280-25652-1-git-send-email-bhar...@linux.vnet.ibm.com
Subject: [Qemu-devel] [RFC PATCH v0] spa
IOMMU Notifier list is used for notifying IO address mapping changes.
Currently VFIO is the only user.
However it is possible that future consumer like vhost would like to
only listen to part of its notifications (e.g., cache invalidations).
This patch introduced IOMMUNotifier and IOMMUNotfierFla
Hi, Paolo,
If this is to be merged, please take either patch 1-2 or the whole
series (1-3). Both work for me. Thanks!
V6:
- use IOMMUTLBEntry directly for IOMMUNotifier.notify() [David]
- add explicit comment for patch 1 that CHANGE should be treated as a
MAP then an UNMAP [David]
- add a-b for
The new interface can be used to replace the old notify_started() and
notify_stopped(). Meanwhile it provides explicit flags so that IOMMUs
can know what kind of notifications it is requested for.
Acked-by: David Gibson
Signed-off-by: Peter Xu
---
hw/i386/intel_iommu.c | 6 --
hw/ppc/spapr
On Wed, Sep 21, 2016 at 01:49:10PM +1000, David Gibson wrote:
> On Tue, Sep 20, 2016 at 03:28:08PM +0800, Peter Xu wrote:
> > On Tue, Sep 20, 2016 at 04:14:09PM +1000, David Gibson wrote:
> > > On Wed, Sep 14, 2016 at 04:25:48PM +0800, Peter Xu wrote:
> > > > Intel vIOMMU is still lacking of a comp
On Wed, 21 Sep 2016 11:19:17 +0800
Jike Song wrote:
> On 09/21/2016 12:24 AM, Alex Williamson wrote:
> > On Tue, 20 Sep 2016 10:50:47 +0800
> > Jike Song wrote:
>
> /* trim the quotations */
>
> >> Even performing a lightweight sanity check, would require vfio-mdev
> >> to be able to decode
CPU unplug doesn't work in TCG mode currently and causes frequent system
freeze. In addition to other potential problems, the main problem arises
of out the requirement to support synchronous removal of a CPU thread.
The CPU thread that performs the cleanup of the unplugged CPU, kicks and
waits for
On Wed, 21 Sep 2016 04:10:53 +
"Tian, Kevin" wrote:
> > From: Kirti Wankhede [mailto:kwankh...@nvidia.com]
> > Sent: Wednesday, September 21, 2016 12:23 AM
> >
> >
> > On 9/20/2016 8:13 PM, Alex Williamson wrote:
> > > On Tue, 20 Sep 2016 19:51:58 +0530
> > > Kirti Wankhede wrote:
> > >
On Tue, 09/20 09:43, Eric Blake wrote:
> Commits 9ef8112a and efad6682 introduced new tests, but forgot
> to ignore the built executables from an in-tree build.
>
> Signed-off-by: Eric Blake
> ---
> tests/.gitignore | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/tests/.gitignore b/t
On Wed, 21 Sep 2016 03:56:21 +
"Tian, Kevin" wrote:
> > From: Kirti Wankhede [mailto:kwankh...@nvidia.com]
> > Sent: Tuesday, September 20, 2016 10:22 PM
> > >>
> > >> 'max_instance':
> > >> Read-Only file. Mandatory.
> > >> Returns integer. Returns maximum mdev device could be cre
I understand that we've been keeping eyes on the uncovered files. Since
I'm adding some more files I volunteer to look after them in the futuer.
Signed-off-by: Fam Zheng
Reviewed-by: Jeff Cody
---
MAINTAINERS | 6 ++
1 file changed, 6 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
in
Update all qemu_uuid users as well, especially get rid of the duplicated
low level g_strdup_printf, sscanf and snprintf calls with QEMU UUID API.
Since qemu_uuid_parse is quite tangled with qemu_uuid, its switching to
QemuUUID is done here too to keep everything in sync and avoid code
churn.
Sign
Signed-off-by: Fam Zheng
---
tests/.gitignore | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/.gitignore b/tests/.gitignore
index b4a9cfc..24ac6cf 100644
--- a/tests/.gitignore
+++ b/tests/.gitignore
@@ -70,6 +70,7 @@ test-string-output-visitor
test-thread-pool
test-throttle
test-tim
sscanf is relatively loose (tolerate) on some invalid formats that we
should fail instead of generating a wrong uuid structure, like with
whitespaces and short strings.
Add and use a helper function to first check the format.
Signed-off-by: Fam Zheng
Reviewed-by: Eric Blake
Reviewed-by: Jeff Co
Signed-off-by: Fam Zheng
Reviewed-by: Jeff Cody
---
tests/Makefile.include | 2 +
tests/test-uuid.c | 177 +
2 files changed, 179 insertions(+)
create mode 100644 tests/test-uuid.c
diff --git a/tests/Makefile.include b/tests/Makefile.inclu
All code now uses built-in UUID implementation. Remove the code of
libuuid and make --enable-uuid and --disable-uuid only print a message.
Signed-off-by: Fam Zheng
Reviewed-by: Eric Blake
Reviewed-by: Jeff Cody
---
configure | 43 ---
1 file changed, 4 i
crypto now uses built-in uuid implementation, so this check is not
needed.
Signed-off-by: Fam Zheng
Reviewed-by: Eric Blake
Reviewed-by: Jeff Cody
---
tests/test-crypto-block.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/test-crypto-block.c b/tests/test-crypto-blo
The uuid generation doesn't return error, so update the function
signature and calling code accordingly.
Signed-off-by: Fam Zheng
Reviewed-by: Eric Blake
Reviewed-by: Jeff Cody
---
crypto/block-luks.c | 26 +++---
1 file changed, 7 insertions(+), 19 deletions(-)
diff --git
A number of different places across the code base use CONFIG_UUID. Some
of them are soft dependency, some are not built if libuuid is not
available, some come with dummy fallback, some throws runtime error.
It is hard to maintain, and hard to reason for users.
Since UUID is a simple standard with
Previously we conditionally generated footer->uuid, when libuuid was
available. Now that we have a built-in implementation, we can switch to
it.
Signed-off-by: Fam Zheng
Reviewed-by: Eric Blake
Reviewed-by: Jeff Cody
---
block/vpc.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions
This removes our dependency to libuuid, so that the driver can always be
built.
Similar to how we handled data plane configure options, --enable-vhdx
and --disable-vhdx are also changed to a nop with a message saying it's
obsolete.
Signed-off-by: Fam Zheng
Reviewed-by: Eric Blake
Reviewed-by: J
So that it doesn't have to be zeroed at each call.
Suggested-by: Eric Blake
Signed-off-by: Fam Zheng
Reviewed-by: Jeff Cody
---
util/uuid.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/util/uuid.c b/util/uuid.c
index 6815904..f0c1eeb 100644
--- a/util/uuid.c
+++ b/util/u
The UUID operations we need from libuuid are fully supported by QEMU UUID
implementation. Use it, and remove the unused code.
Signed-off-by: Fam Zheng
Reviewed-by: Eric Blake
Reviewed-by: Jeff Cody
---
block/vdi.c | 73 ++---
1 file chang
v9: Add .gitignore patch. [Eric]
Add Jeff's and Eric's r-b in other patches.
The facts how we use libuuid now are not particularly pleasant.
- VHDX driver depends on uuid, but is unconditionally checked in iotests 109.
If it is not built, the test would fail, leaving no hint about that. In
> From: Kirti Wankhede [mailto:kwankh...@nvidia.com]
> Sent: Wednesday, September 21, 2016 12:23 AM
>
>
> On 9/20/2016 8:13 PM, Alex Williamson wrote:
> > On Tue, 20 Sep 2016 19:51:58 +0530
> > Kirti Wankhede wrote:
> >
> >> On 9/20/2016 3:06 AM, Alex Williamson wrote:
> >>> On Tue, 20 Sep 2016
On Tue, 09/20 14:56, Alex Bennée wrote:
> This re-factors the docker makefile to include a docker-run target which
> can be controlled entirely from environment variables specified on the
> make command line. This allows us to run against any given docker image
> we may have in our repository, for
> From: Kirti Wankhede [mailto:kwankh...@nvidia.com]
> Sent: Tuesday, September 20, 2016 10:22 PM
> >>
> >> 'max_instance':
> >> Read-Only file. Mandatory.
> >> Returns integer. Returns maximum mdev device could be created
> >> at the moment when this file is read. This count would be upda
Add a make variable SHOW_ENV. When it's set to non empty, print the
package information and environment variables.
Signed-off-by: Fam Zheng
---
tests/docker/Makefile.include | 2 +-
tests/docker/run | 11 +++
2 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/te
On Tue, Sep 20, 2016 at 03:16:33PM +0800, Peter Xu wrote:
> On Tue, Sep 20, 2016 at 04:12:05PM +1000, David Gibson wrote:
> > On Wed, Sep 14, 2016 at 04:25:46PM +0800, Peter Xu wrote:
> > > IOMMU Notifier list is used for notifying IO address mapping changes.
> > > Currently VFIO is the only user.
This makes the configure command more obvious which usually has useful
information.
Signed-off-by: Fam Zheng
---
tests/docker/common.rc | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/tests/docker/common.rc b/tests/docker/common.rc
index 0c6d8d5..510a3ad 100755
Now that 23 is becoming an "old" release with 24 available. Fedora has a
quick release cycle, so use latest to follow more closely.
Signed-off-by: Fam Zheng
---
tests/docker/dockerfiles/fedora.docker | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/docker/dockerfiles/fed
This is the last command to run (unless DEBUG), make it 'exec' to
simplify the process tree.
Signed-off-by: Fam Zheng
---
tests/docker/run | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/tests/docker/run b/tests/docker/run
index ed7dd31..c1e4513 100755
--- a/t
Put the list of package names in an environment, and output their
package names to the target file in the end.
Signed-off-by: Fam Zheng
---
tests/docker/dockerfiles/ubuntu.docker | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tests/docker/dockerfiles/ubuntu.docker
b/test
Previously it is expanded to a whitespace separated list which is not
the most appropriate format. Since it's only two items, flatten it.
Signed-off-by: Fam Zheng
---
tests/docker/test-quick | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/docker/test-quick b/tests/docke
Put the list of package names in an environment, and output their
package names to the target file in the end.
Signed-off-by: Fam Zheng
---
tests/docker/dockerfiles/fedora.docker | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/tests/docker/dockerfiles/fedora.d
Put the list of package names in an environment, and output their
package names to the target file in the end.
Signed-off-by: Fam Zheng
---
tests/docker/dockerfiles/centos6.docker | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tests/docker/dockerfiles/centos6.docker
b
Signed-off-by: Fam Zheng
---
tests/docker/docker.py | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/tests/docker/docker.py b/tests/docker/docker.py
index b85c165..552608e 100755
--- a/tests/docker/docker.py
+++ b/tests/docker/docker.py
@@ -21,6 +21,7 @@ import uuid
import
On Tue, Sep 20, 2016 at 03:28:08PM +0800, Peter Xu wrote:
> On Tue, Sep 20, 2016 at 04:14:09PM +1000, David Gibson wrote:
> > On Wed, Sep 14, 2016 at 04:25:48PM +0800, Peter Xu wrote:
> > > Intel vIOMMU is still lacking of a complete IOMMU notifier mechanism.
> > > Before that is achieved, let's op
The first patches add a new feature that if SHOW_ENV is set when running a
test, (explicitly installed) package list and env vars will be printed.
The last pactches (hopefully) improves termination of docker instance, so that
in a patchew tester env, building and testing procecess (that lies under
David Gibson writes:
> [ Unknown signature status ]
> On Tue, Sep 20, 2016 at 10:40:03PM +0530, Nikunj A Dadhania wrote:
>> David Gibson writes:
>>
>> > [ Unknown signature status ]
>> > On Mon, Sep 19, 2016 at 04:06:40PM +0530, Nikunj A Dadhania wrote:
>> >> David Gibson writes:
>> >> > [ Unk
Perhaps include a base set of resolutions that the actual mac99 machine
supported, implemented in OpenBIOS, and also adding the --video-resolutions
option so the user may add custom entries that would be read by the VGA
device if so desired. I believe that is the best compromise vs. having a
huge
On Mon, Aug 15, 2016 at 10:11:53AM +1000, Benjamin Herrenschmidt wrote:
> On Mon, 2016-08-15 at 09:59 +1000, Benjamin Herrenschmidt wrote:
> > Bla bla bla bla ...
> >
> > Seriously, we have nazi stylebot on the list now ?
> >
> > I made conscious decisions to style it this way and as far as the t
On Tue, Aug 30, 2016 at 01:02:47AM +, Nathan Whitehorn wrote:
> These are mandatory per PAPR and available on Linux 4.3 and newer kernels.
> The calls in question are required to run FreeBSD guests with reasonable
> performance, so enable them if possible.
>
> Signed-off-by: Nathan Whitehorn
On Tue, 09/20 10:37, Eric Blake wrote:
> On 09/17/2016 11:25 PM, Fam Zheng wrote:
> > Signed-off-by: Fam Zheng
> > ---
> > tests/Makefile.include | 2 +
> > tests/test-uuid.c | 177
> > +
> > 2 files changed, 179 insertions(+)
> > create mo
On 09/21/2016 12:24 AM, Alex Williamson wrote:
> On Tue, 20 Sep 2016 10:50:47 +0800
> Jike Song wrote:
/* trim the quotations */
>> Even performing a lightweight sanity check, would require vfio-mdev
>> to be able to decode the ppos into a particular region, that means
>> information of all regi
On Wed, Sep 21, 2016 at 12:05:47PM +1000, Benjamin Herrenschmidt wrote:
> On Wed, 2016-09-21 at 11:51 +1000, David Gibson wrote:
> > Ok, as noted elsewhere, I think you need to disassociate the PIR value
> > from the cpu_index. It may be a little less elegant, but it'll make
> > your life much eas
On Sep 20, 2016, at 8:41 PM, Benjamin Herrenschmidt wrote:
On Tue, 2016-09-20 at 20:35 -0400, G 3 wrote:
When I use the -prom-env option I can easily add properties to the
options node.
Can something like this be done with the chosen node?
We can make it so. That or we can add code to OpenB
This patchset adds a pmu=[on/off] option to enable/disable vPMU support
for guest VM. There are several reasons to justify this option. First,
vPMU can be problematic for cross-migration between different SoC as perf
counters are architecture-dependent. It is more flexible to have an option
to tur
CPU vPMU is now turned off by default, but it was ON in virt-2.7
machine type. To solve this problem, this patch adds a PMU option
in machine state, which is used to control CPU's vPMU status. This
PMU option is not exposed to command line and is turned on in
virt-2.7 machine type to make sure it i
This patch adds a pmu=[on/off] option to enable/disable vPMU support
in guest vCPU. This option is only available for cortex-a57/cortex-53/
host under both TCG and KVM modes, but unavailable on ARMv7 and other
processors. It allows virt tools, such as libvirt, to determine the
exsitence of vPMU and
Hi Michael,
> -Original Message-
> From: virtio-...@lists.oasis-open.org [mailto:virtio-...@lists.oasis-open.org]
> On Behalf Of Michael S. Tsirkin
> Sent: Wednesday, September 21, 2016 3:11 AM
> Subject: [virtio-dev] Re: [PATCH v10 1/2] virtio-crypto: Add virtio crypto
> device
> specif
The Trigger Mode field of IOAPIC must match the Trigger Mode in
the IRTE according to VT-d Spec 5.1.5.1.
Signed-off-by: Feng Wu
---
hw/i386/intel_iommu.c | 12
1 file changed, 12 insertions(+)
diff --git a/hw/i386/intel_iommu.c b/hw/i386/intel_iommu.c
index 28c31a2..f32ad5c 100644
On Tue, Sep 20, 2016 at 10:40:03PM +0530, Nikunj A Dadhania wrote:
> David Gibson writes:
>
> > [ Unknown signature status ]
> > On Mon, Sep 19, 2016 at 04:06:40PM +0530, Nikunj A Dadhania wrote:
> >> David Gibson writes:
> >> > [ Unknown signature status ]
> >> > On Mon, Sep 19, 2016 at 04:19:3
On Wed, 2016-09-21 at 11:51 +1000, David Gibson wrote:
> Ok, as noted elsewhere, I think you need to disassociate the PIR value
> from the cpu_index. It may be a little less elegant, but it'll make
> your life much easier in the short and medium term.
>
> Apart from that, this looks pretty good,
On Thu, Sep 15, 2016 at 02:45:55PM +0200, Cédric Le Goater wrote:
> This is largy inspired by sPAPRCPUCore with some simplification, no
> hotplug for instance. But the differences are small and the objects
> could possibly be merged.
>
> A set of PnvCore objects is added to the PnvChip and the dev
On Wed, 2016-09-21 at 11:29 +1000, David Gibson wrote:
> On Thu, Sep 15, 2016 at 02:45:54PM +0200, Cédric Le Goater wrote:
> >
> > P9 and P8 have some differences in the CPU PIR encoding.
>
> The thread id isn't in the PIR at all?
Yes it is. Though on P9 there could be different encodings depend
On Thu, Sep 15, 2016 at 02:45:54PM +0200, Cédric Le Goater wrote:
> P9 and P8 have some differences in the CPU PIR encoding.
The thread id isn't in the PIR at all?
>
> Signed-off-by: Cédric Le Goater
> ---
> hw/ppc/pnv.c | 14 ++
> include/hw/ppc/pnv.h | 1 +
> 2 files cha
On Tue, Sep 20, 2016 at 10:04:58PM +0530, Nikunj A Dadhania wrote:
> PowerPC failed to handle broadcast TLB flush operations. Executing
> instructions that are defined architecturally as synchronizing global TLB
> should have a global effect.
>
> * tlbie on BookS
> * tlbivax on BookE
> * H_CALLs
This patch adds support for a new block device type called "vxhs".
Source code for the library that this code loads can be downloaded from:
https://github.com/MittalAshish/libqnio.git
Sample command line using JSON syntax:
./qemu-system-x86_64 -name instance-0008 -S -vnc 0.0.0.0:0 -k en-us -vg
On 09/08/2016 03:32 PM, Michael Rolnik wrote:
+decode_fdecode[32] = {
There's no reason to use an array here, and worse, a non-static non-const
array. Just use a switch here
+return decode[curr_major](ctx, curr_opcode);
r~
On Tue, 2016-09-20 at 20:35 -0400, G 3 wrote:
>
> When I use the -prom-env option I can easily add properties to the
> options node.
> Can something like this be done with the chosen node?
We can make it so. That or we can add code to OpenBIOS to copy the list
of resolutions into the device-nod
On 09/08/2016 03:32 PM, Michael Rolnik wrote:
+int arc_gen_UNIMP(DisasCtxt *ctx)
+{
+/* PC = INT_VECTOR_BACE + 0x10 */
+tcg_gen_andi_tl(cpu_pc, cpu_intvec, 0xffc0);
+tcg_gen_addi_tl(cpu_pc, cpu_pc, 0x10);
+
+return BS_EXCP;
+}
This should be done via an exception, with the
On 09/08/2016 03:32 PM, Michael Rolnik wrote:
+int arc_gen_ASLS(DisasCtxt *ctx, TCGv dest, TCGv src1, TCGv src2)
Just do these as helpers.
r~
On Sep 20, 2016, at 8:18 PM, Benjamin Herrenschmidt wrote:
On Tue, 2016-09-20 at 00:01 -0400, G 3 wrote:
Something is wrong with the options node in OpenBIOS. It is
inaccessible from Mac OS X. When trying to access the options node,
IORegistryExplorer always crashes. The other nodes are acces
> -Original Message-
> From: virtio-...@lists.oasis-open.org [mailto:virtio-...@lists.oasis-open.org]
> On Behalf Of Xin Zeng
> Sent: Tuesday, September 20, 2016 11:47 PM
> Subject: [virtio-dev] [PATCH] virtio crypto device specification
>
> Fix spelling mistakes of keywords to make xela
On 09/08/2016 03:32 PM, Michael Rolnik wrote:
+int arc_gen_LR(DisasCtxt *ctx, TCGv dest, TCGv src1)
+{
+TCGv cpc = tcg_const_local_i32((ctx->cpc + 3) & 0xfffc);
+TCGv npc = tcg_const_local_i32((ctx->npc + 3) & 0xfffc);
No need for _local here.
r~
On 09/08/2016 03:31 PM, Michael Rolnik wrote:
+caseARC_COND_HI: {
+ARC_COND_IF_1(C, label_skip);
+ARC_COND_IF_1(Z, label_skip);
+} break;
Why aren't you using AND / OR to combine these conditions?
r~
On 09/08/2016 03:31 PM, Michael Rolnik wrote:
+static void arc_gen_exec_delayslot(DisasCtxt *ctx)
+{
+if (ctx->opt.limm == 0) {
+uint32_t cpc = ctx->cpc;
+uint32_t npc = ctx->npc;
+uint32_t dpc = ctx->dpc;
+uint32_t pcl = ctx->pcl;
+options_t opt = ctx-
On 09/08/2016 03:31 PM, Michael Rolnik wrote:
Signed-off-by: Michael Rolnik
---
target-arc/translate-inst.c | 105
target-arc/translate-inst.h | 4 ++
2 files changed, 109 insertions(+)
diff --git a/target-arc/translate-inst.c b/target-arc/transla
On Tue, 2016-09-20 at 00:01 -0400, G 3 wrote:
>
> Something is wrong with the options node in OpenBIOS. It is
> inaccessible from Mac OS X. When trying to access the options node,
> IORegistryExplorer always crashes. The other nodes are accessible.
> I'm not certain what the problem could be
On Tue, 2016-09-20 at 09:51 -0400, G 3 wrote:
>
> The malloc() function used in this driver is used to allocate a
> very
> small amount of space at most. We are realistically talking under
> 2k.
> Running out of space is highly unlikely. I'm sure the user could do
> something evil to try to
On 09/08/2016 03:31 PM, Michael Rolnik wrote:
+tcg_gen_ext_i32_i64(srcA, src1);
+tcg_gen_ext_i32_i64(srcB, src2);
+
+tcg_gen_mul_i64(rslt, srcA, srcB);
+
+tcg_gen_trunc_i64_tl(dest, rslt);
+
+if (ctx->opt.f) {
+TCGv_i64 temp = tcg_temp_new_i64();
+
+tcg_gen_set
On 09/08/2016 03:31 PM, Michael Rolnik wrote:
+target_ulong helper_norm(CPUARCState *env, uint32_t src1)
+{
+if (src1 == 0x || src1 == 0x) {
+return 31;
+} else {
+if ((src1 & 0x8000) == 0x8000) {
+src1 = ~src1;
+}
+retur
On 09/08/2016 03:31 PM, Michael Rolnik wrote:
+int arc_gen_BCLR(DisasCtxt *ctx, TCGv dest, TCGv src1, TCGv src2)
+{
+TCGv rslt = dest;
+
+if (TCGV_EQUAL(dest, src1) || TCGV_EQUAL(dest, src2)) {
+rslt = tcg_temp_new_i32();
+}
+
+tcg_gen_andi_tl(rslt, src2, 0x3f);
0x1f, in
1 - 100 of 466 matches
Mail list logo
