Re: [Qemu-devel] intel-iommu and vhost: Do we need 'device-iotlb' and 'ats'?

2018-02-22 Thread Peter Xu
On Tue, Feb 20, 2018 at 11:03:46PM -0500, Jintack Lim wrote: > Hi, > > I'm using vhost with the virtual intel-iommu, and this page[1] shows > the QEMU command line example. > > qemu-system-x86_64 -M q35,accel=kvm,kernel-irqchip=split -m 2G \ >-device intel-iommu,intremap=on,de

Re: [Qemu-devel] [PATCH v2] linux-user: Support f_flags in statfs when available.

2018-02-22 Thread no-reply
Hi, This series failed build test on s390x host. Please find the details below. Type: series Message-id: 20180219024555.26467-1-s...@shealevy.com Subject: [Qemu-devel] [PATCH v2] linux-user: Support f_flags in statfs when available. === TEST SCRIPT BEGIN === #!/bin/bash # Testing script will be

[Qemu-devel] [PATCH v3] linux-user: Support f_flags in statfs when available.

2018-02-22 Thread Shea Levy
Signed-off-by: Shea Levy --- configure | 21 +++ linux-user/syscall.c | 3 +++ linux-user/syscall_defs.h | 53 +++ 3 files changed, 77 insertions(+) diff --git a/configure b/configure index 913e14839d..91082aa1dc 1

Re: [Qemu-devel] [PATCH v3] linux-user: Support f_flags in statfs when available.

2018-02-22 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20180223045735.1288-1-s...@shealevy.com Subject: [Qemu-devel] [PATCH v3] linux-user: Support f_flags in statfs when available. === TEST SCRIPT BEGIN === #!/bin/bash BASE=ba

Re: [Qemu-devel] intel-iommu and vhost: Do we need 'device-iotlb' and 'ats'?

2018-02-22 Thread Jintack Lim
Hi Peter, Hope you had great holidays! On Thu, Feb 22, 2018 at 10:55 PM, Peter Xu wrote: > On Tue, Feb 20, 2018 at 11:03:46PM -0500, Jintack Lim wrote: >> Hi, >> >> I'm using vhost with the virtual intel-iommu, and this page[1] shows >> the QEMU command line example. >> >> qemu-system-x86_64 -M

Re: [Qemu-devel] intel-iommu and vhost: Do we need 'device-iotlb' and 'ats'?

2018-02-22 Thread Peter Xu
On Fri, Feb 23, 2018 at 12:32:13AM -0500, Jintack Lim wrote: > Hi Peter, > > Hope you had great holidays! > > On Thu, Feb 22, 2018 at 10:55 PM, Peter Xu wrote: > > On Tue, Feb 20, 2018 at 11:03:46PM -0500, Jintack Lim wrote: > >> Hi, > >> > >> I'm using vhost with the virtual intel-iommu, and th

Re: [Qemu-devel] [PATCH 1/2] xilinx_spips: Enable only two slaves when reading/writing with stripe

2018-02-22 Thread francisco iglesias
On 22 February 2018 at 23:38, Alistair Francis wrote: > On Thu, Feb 22, 2018 at 2:28 PM, Francisco Iglesias > wrote: > > Assert only the lower cs on bus 0 and upper cs on bus 1 when both buses > and > > chip selects are enabled (e.g reading/writing with stripe). > > > > Signed-off-by: Francisco

Re: [Qemu-devel] intel-iommu and vhost: Do we need 'device-iotlb' and 'ats'?

2018-02-22 Thread Jintack Lim
On Fri, Feb 23, 2018 at 1:10 AM Peter Xu wrote: > On Fri, Feb 23, 2018 at 12:32:13AM -0500, Jintack Lim wrote: > > Hi Peter, > > > > Hope you had great holidays! > > > > On Thu, Feb 22, 2018 at 10:55 PM, Peter Xu wrote: > > > On Tue, Feb 20, 2018 at 11:03:46PM -0500, Jintack Lim wrote: > > >> Hi

Re: [Qemu-devel] [PATCH v3] rtc: placing RTC memory region outside BQL

2018-02-22 Thread Gonglei (Arei)
Ping... Regards, -Gonglei > -Original Message- > From: Gonglei (Arei) > Sent: Monday, February 12, 2018 4:58 PM > To: qemu-devel@nongnu.org > Cc: pbonz...@redhat.com; Huangweidong (C); peter.mayd...@linaro.org; > Gonglei (Arei) > Subject: [PATCH v3] rtc: placing RTC memory region outsid

Re: [Qemu-devel] [RFC, PATCH, v1] hw/audio/opl2lpt: add support for OPL2LPT

2018-02-22 Thread no-reply
Hi, This series failed docker-quick@centos6 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: 20180218144021.11641-1-vinc...@bernat.im Subject: [Qemu-devel] [RFC, PATCH, v1] hw/audio/

Re: [Qemu-devel] [PATCH v4 0/7] vfio: add display support

2018-02-22 Thread Zhang, Tina
> -Original Message- > From: Zhenyu Wang [mailto:zhen...@linux.intel.com] > Sent: Friday, February 23, 2018 10:43 AM > To: Gerd Hoffmann > Cc: Alex Williamson ; Kirti Wankhede > ; intel-gvt-...@lists.freedesktop.org; qemu- > de...@nongnu.org; Zhang, Tina > Subject: Re: [Qemu-devel] [PAT

Re: [Qemu-devel] intel-iommu and vhost: Do we need 'device-iotlb' and 'ats'?

2018-02-22 Thread Peter Xu
On Fri, Feb 23, 2018 at 06:34:04AM +, Jintack Lim wrote: > On Fri, Feb 23, 2018 at 1:10 AM Peter Xu wrote: > > > On Fri, Feb 23, 2018 at 12:32:13AM -0500, Jintack Lim wrote: > > > Hi Peter, > > > > > > Hope you had great holidays! > > > > > > On Thu, Feb 22, 2018 at 10:55 PM, Peter Xu wrote:

Re: [Qemu-devel] intel-iommu and vhost: Do we need 'device-iotlb' and 'ats'?

2018-02-22 Thread Tian, Kevin
> From: Peter Xu > Sent: Friday, February 23, 2018 3:09 PM > > > > > Right. I think my question was not clear. My question was that why don’t > > IOMMU invalidate device-iotlb along with its mappings in one go. Then > IOMMU > > device driver doesn’t need to flush device-iotlb explicitly. Maybe the

[Qemu-devel] [PATCH v3 1/3] util/uri.c: replace TAB with whitespace

2018-02-22 Thread Su Hang
Formating with clang-format. Change back few code by hand, to make sure only include whitespace change. Signed-off-by: Su Hang --- util/uri.c | 1456 ++-- 1 file changed, 729 insertions(+), 727 deletions(-) diff --git a/util/uri.c b/util/u

[Qemu-devel] [PATCH v3 3/3] util/uri.c: add brackets to `if` statements

2018-02-22 Thread Su Hang
Add brackets that wrap `if`, `else`, `while` that hold single statements. In order to do this, I write a simple python regex script. Since then, all complaints rised by checkpatch.pl has been suppressed. Signed-off-by: Su Hang --- util/uri.c | 462 ++

[Qemu-devel] [PATCH v3 2/3] util/uri.c: only remove `return`'s brackets

2018-02-22 Thread Su Hang
only remove brackets that wrap `return` statements' content. use `perl -pi -e "s/return \((.*?)\);/return \1;/g" util/uri.c` to remove pattern like this: "return (1);" Signed-off-by: Su Hang --- util/uri.c | 160 ++--- 1 file changed, 80 i

<    1   2   3   4