On Fri, 2018-08-17 at 10:18 -0300, Eduardo Habkost wrote:
> Thanks for the patch, comments below:
>
> On Fri, Aug 10, 2018 at 10:06:28PM +0800, Robert Hoo wrote:
> > Add kvm_get_supported_feature_msrs() to get supported MSR feature index
> > list.
> > Add kvm_arch_get_supported_msr_feature() to g
- Original Message -
> From: "Robert Hoo"
> To: "Eduardo Habkost"
> Cc: "robert hu" , "robert hu"
> , pbonz...@redhat.com, r...@twiddle.net,
> "thomas lendacky" , qemu-devel@nongnu.org, "jingqi
> liu"
> Sent: Saturday, August 18, 2018 7:48:43 AM
> Subject: Re: [Qemu-devel] [PATCH v3
On Fri, 2018-08-17 at 10:28 -0300, Eduardo Habkost wrote:
> On Fri, Aug 10, 2018 at 10:06:29PM +0800, Robert Hoo wrote:
> > Add an util function feature_word_description(), which help construct the
> > string
> > describing the feature word (both CPUID and MSR types).
> >
> > report_unavailable_f
On 2018-08-17 18:14, Peter Maydell wrote:
> On a SPARC host that I'm using as a build test machine, the
> boot-serial-test for the SPARC guest machines takes about 65
> seconds to execute. This means that it hits the current
> 60 second timer on these tests. Push the timeout up so
> that it doesn't
On 18 August 2018 at 02:09, Philippe Mathieu-Daudé wrote:
> On 08/09/2018 10:01 AM, Peter Maydell wrote:
>> The AN505 FPGA image includes four PL081 DMA controllers, each
>> of which is gated by a Master Security Controller that allows
>> the guest to prevent a non-secure DMA controller from acces
On 18 August 2018 at 01:37, Philippe Mathieu-Daudé wrote:
> On 08/09/2018 10:01 AM, Peter Maydell wrote:
>> The IoTKit does not have any Master Security Contollers itself,
>> but it does provide registers in the secure privilege control
>> block which allow control of MSCs in the external system.
On 17 August 2018 at 18:56, Hans-Erik Floryd
wrote:
>
>
> Den fre 17 aug 2018 19:51Peter Maydell skrev
>>
>> (Your indentation was off-by-one -- we use 4-space indent -- but I fixed
>> that.)
>>
>> -- PMM
>
>
> Thanks! I think i fixed that in a new patch so please ignore v3.
You don't need to re
On 18 August 2018 at 01:00, Philippe Mathieu-Daudé wrote:
> Hi Peter,
>
> On 08/09/2018 10:01 AM, Peter Maydell wrote:
>> Wire up the system control element's register banks.
>>
>> This is the last of the previously completely unimplemented
>> components in the IoTKit.
>>
>> Signed-off-by: Peter M
On 18 August 2018 at 01:23, Philippe Mathieu-Daudé wrote:
> Hi Peter,
>
> On 08/09/2018 10:01 AM, Peter Maydell wrote:
>> The Arm IoTKit includes a system control element which
>> provides a block of read-only ID registers and a block
>> of read-write control registers. Implement a minimal
>> vers
On 18 August 2018 at 10:07, Thomas Huth wrote:
> 6 minutes is really a lot already. I guess most users will hit CTRL-C
> before waiting so long if there is a realy problem here ... If the
> current tests just takes a little bit more than 1 minute on the Sparc
> machine, maybe 2 or 3 minutes would
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180817150559.16243-1-arm...@redhat.com
Subject: [Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting
improvements, cleanups
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BA
Hi,
This series failed docker-quick@centos7 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: 20180809042206.15726-1-richard.hender...@linaro.org
Subject: [Qemu-devel] [PATCH 00/20] t
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180817170246.14641-1-kw...@redhat.com
Subject: [Qemu-devel] [RFC PATCH 0/5] Fix some jobs/drain/aio_poll related hangs
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180816025452.21358-1-richard.hender...@linaro.org
Subject: [Qemu-devel] [PATCH 0/5] tcg: Reorg 128-bit atomic operations
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=
On Fri, 2018-08-17 at 17:52 +0200, Paolo Bonzini wrote:
> On 10/08/2018 16:06, Robert Hoo wrote:
> > x86_cpu_get_feature_words(): limit to CPUID_FEATURE_WORD only.
>
> This should also grow support for MSR feature words.
>
> My suggestion is that you add another patch after patch 1 that expands
>
Hi Yuval,
On 08/14/2018 01:00 PM, Yuval Shaia wrote:
Function create_ah might return NULL, let's exit with an error.
Signed-off-by: Yuval Shaia
---
hw/rdma/rdma_backend.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/rdma/rdma_backend.c b/hw/rdma/rdma_backend.c
index 35726bda2e
On Sat, Aug 18, 2018 at 04:46:29PM +0300, Marcel Apfelbaum wrote:
> Hi Yuval,
>
> On 08/14/2018 01:00 PM, Yuval Shaia wrote:
> > Function create_ah might return NULL, let's exit with an error.
> >
> > Signed-off-by: Yuval Shaia
> > ---
> > hw/rdma/rdma_backend.c | 4
> > 1 file changed,
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180817173104.14691-1-marcandre.lur...@redhat.com
Subject: [Qemu-devel] [PATCH] vnc: call sasl_server_init() only when required
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=b
On 08/18/2018 06:55 AM, Peter Maydell wrote:
> On 18 August 2018 at 01:00, Philippe Mathieu-Daudé wrote:
>> Hi Peter,
>>
>> On 08/09/2018 10:01 AM, Peter Maydell wrote:
>>> Wire up the system control element's register banks.
>>>
>>> This is the last of the previously completely unimplemented
>>>
On Sat, Aug 18, 2018 at 03:27:01PM +0800, Robert Hoo wrote:
> On Fri, 2018-08-17 at 10:18 -0300, Eduardo Habkost wrote:
> > Thanks for the patch, comments below:
> >
> > On Fri, Aug 10, 2018 at 10:06:28PM +0800, Robert Hoo wrote:
> > > Add kvm_get_supported_feature_msrs() to get supported MSR feat
On Sat, Aug 18, 2018 at 05:01:45PM +0800, Robert Hoo wrote:
> On Fri, 2018-08-17 at 10:28 -0300, Eduardo Habkost wrote:
> > On Fri, Aug 10, 2018 at 10:06:29PM +0800, Robert Hoo wrote:
> > > Add an util function feature_word_description(), which help construct the
> > > string
> > > describing the
On 08/18/2018 07:05 AM, Peter Maydell wrote:
> On 18 August 2018 at 01:37, Philippe Mathieu-Daudé wrote:
>> On 08/09/2018 10:01 AM, Peter Maydell wrote:
>>> The IoTKit does not have any Master Security Contollers itself,
>>> but it does provide registers in the secure privilege control
>>> block w
Hi,
This series failed docker-mingw@fedora 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: 20180817190457.8292-1-js...@redhat.com
Subject: [Qemu-devel] [PATCH 0/7] jobs: remove job_
> On Aug 17, 2018, at 4:59 PM, Eric Blake wrote:
>
> On 08/17/2018 02:28 PM, Programmingkid wrote:
>
-o Used with a comma separated list of format specific
options in a
name=value format. Use "-o ?" for an overview of the
options
>>>
>>> Plea
Thank you very much. The following invocation fixed the mouse problem
(mostly) and made the Ethernet device available to the guest:
./qemu-system-x86_64 -m 2047 -boot d -usb -device usb-tablet -smp 3
-netdev user,id=n0 -device rtl8139,netdev=n0 -hda
/bkpool/qemuimages/XP.img -cdrom /bkpool/qemui
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20180809042206.15726-1-richard.hender...@linaro.org
Subject: [Qemu-devel] [PATCH 00/20] target/arm: sve system mode patches
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n
On Sat, Aug 18, 2018 at 14:45:54 +0800, Fam Zheng wrote:
> On Fri, 08/17 19:14, no-re...@patchew.org wrote:
(snip)
> > /tmp/qemu-test/src/include/qemu/thread.h:101:1: error: all paths through
> > this function will call itself [-Werror,-Winfinite-recursion]
(snip)
> > subprocess.CalledProcessError
Public bug reported:
The output for the "-cpu help" on the Sparc executables is not
generating accurate information.
Running
./qemu-sparc64 -cpu help
produces:
Sparc Fujitsu Sparc64 IU 00040002 FPU MMU NWINS 4
Sparc Fujitsu Sparc64 III IU 00040003 FPU 000
Signed-off-by: Richard Henderson
---
linux-user/syscall-file.inc.c | 98 +++
linux-user/syscall.c | 75 ++-
linux-user/strace.list| 12 -
3 files changed, 102 insertions(+), 83 deletions(-)
diff --git a/linux-user/sysca
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Signed-off-by: Richard Henderson
---
linux-user/syscall.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 23c3cd5ddd..812fb27fa1 100644
--- a/linux-u
Version 4 continues the split into multiple files, but for
inclusion rather than separate compilation. This allows us
to get warnings if there are mistakes in the switch statement
that looks up the structures.
r~
Richard Henderson (16):
linux-user: Remove DEBUG
linux-user: Split out do_sys
This is redundant with both -strace and actual tracing.
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Signed-off-by: Richard Henderson
---
linux-user/syscall.c | 10 --
1 file changed, 10 deletions(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index bb4
There was supposed to be a single point of return for do_syscall
so that tracing works properly. However, there are a few bugs
in that area. It is significantly simpler to simply split out
an inner function to enforce this.
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Signed
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Signed-off-by: Richard Henderson
---
linux-user/syscall.c | 60
1 file changed, 22 insertions(+), 38 deletions(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index ef3b9b62
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Signed-off-by: Richard Henderson
---
linux-user/syscall.c | 341 ++-
1 file changed, 175 insertions(+), 166 deletions(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index f92a24
This includes close, open, openat, read, readlink, readlinkat, write.
Signed-off-by: Richard Henderson
---
linux-user/strace.c | 64 -
linux-user/syscall-file.inc.c | 440 ++
linux-user/syscall.c | 393 ++
linux-
There is no point in listing a syscall if you want the same effect as
not listing it. In one less trivial case, the goto was demonstrably
not reachable.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
linux-user/syscall.c | 144 +--
1 fil
Transform outermost "break" to "return ret". If the immediately
preceeding statement was an assignment to ret, return the value
directly.
Reviewed-by: Laurent Vivier
Signed-off-by: Richard Henderson
---
linux-user/syscall.c | 972 +--
1 file changed, 390
For the linux-user syscall split, we have static const structs
that must be matched up with a switch statement that uses them.
By default, gcc will not warn for such a variable, but silently
remove them.
For C++, such objects are sometimes declared for their constructor
side effects. Do not propa
Signed-off-by: Richard Henderson
---
linux-user/syscall.h | 45 +
linux-user/syscall-file.inc.c | 62 ++--
linux-user/syscall.c | 76 +++
linux-user/strace.list| 6 ---
4 files changed, 110 inse
Defines a unified structure for implementation and strace.
Signed-off-by: Richard Henderson
---
linux-user/syscall.h | 114 +
linux-user/strace.c | 386 +++
linux-user/syscall.c | 59 ++-
3 files changed, 446 insertions(+), 113 deletions(
Signed-off-by: Richard Henderson
---
linux-user/syscall.h | 1 +
linux-user/strace.c | 5 +-
linux-user/syscall-file.inc.c | 84 +++
linux-user/syscall.c | 104 ++
linux-user/strace.list| 3 -
5 fil
This includes mmap, mmap2, munmap, mlock, mlockall, munlock,
munlockall, mprotect, mremap, msync.
Signed-off-by: Richard Henderson
---
linux-user/syscall.h | 2 +
linux-user/strace.c | 55 ++-
linux-user/syscall-mem.inc.c | 185 +++
lin
Am 17.08.2018 um 09:32 schrieb David Hildenbrand:
> On 18.07.2018 08:33, Howard Spoelstra wrote:
>> Hi all,
>>
>> I have two issues when cross compiling current master for Windows with
>> mingw 8.1. Host is Fedora29. See further below for gcc and mingw
>> versions.
>>
>> Reproduce with:
>> ./config
Signed-off-by: Richard Henderson
---
linux-user/strace.c | 83 ---
linux-user/syscall-ipc.inc.c | 1085 ++
linux-user/syscall.c | 1006 ++-
linux-user/strace.list | 42 --
4 files changed, 1122 insertions(+), 1
This includes clone, getgroups, gettid, setfsgid, setfsuid,
setgroups, setsid, setuid, fork, getegid, getegid32, geteuid,
geteuid32, getgid, getgid32, getgroups32, getpgrp, getpid,
getppid, getresgid, getresgid32, getresuid, getresuid32,
getuid, getuid32, getxgid, getxpid, getxuid, setfsgid32,
setg
Hi,
On 08/18/2018 04:09 PM, Stefan Weil wrote:
> Am 17.08.2018 um 09:32 schrieb David Hildenbrand:
>> On 18.07.2018 08:33, Howard Spoelstra wrote:
>>> Hi all,
>>>
>>> I have two issues when cross compiling current master for Windows with
>>> mingw 8.1. Host is Fedora29. See further below for gcc a
On 08/18/2018 07:04 AM, Peter Maydell wrote:
> On 18 August 2018 at 01:23, Philippe Mathieu-Daudé wrote:
>> Hi Peter,
>>
>> On 08/09/2018 10:01 AM, Peter Maydell wrote:
>>> The Arm IoTKit includes a system control element which
>>> provides a block of read-only ID registers and a block
>>> of read
Hi all,
As part of testing my OpenBIOS virtio-blk implementation, I've been
trying to use it under the 40p machine and have some questions related
to how address spaces are interpreted.
The 40p PCI address space is aliased onto the CPU physical address space
with an offset of 0x8000 i.e.
On Wed, Aug 15, 2018 at 19:54:48 -0700, Richard Henderson wrote:
> GCC7+ will no longer advertise support for 16-byte __atomic operations
> if only cmpxchg is supported, as for x86_64. Fortunately, x86_64 still
> has support for __sync_compare_and_swap_16 and we can make use of that.
> AArch64 doe
On Sat, Aug 18, 2018 at 9:09 PM, Stefan Weil wrote:
> Am 17.08.2018 um 09:32 schrieb David Hildenbrand:
>> On 18.07.2018 08:33, Howard Spoelstra wrote:
>>> Hi all,
>>>
>>> I have two issues when cross compiling current master for Windows with
>>> mingw 8.1. Host is Fedora29. See further below for
I'm also seeing this behaviour using the MSYS2 packaged mingw64 compiler, GCC
8.2. I have managed to cross compile QEMU successfully under Fedora 28 however
and the resulting binary works. Certainly seems like a tool chain issue,
unfortunately MSYS2 doesn't package any older versions of GCC.
Be
Hi Marcel,
Marcel Apfelbaum 于2018年8月18日周六 上午1:14写道：
>
> Hi Zihan,
>
> On 08/09/2018 09:33 AM, Zihan Yang wrote:
> > The inner host bridge created by pxb-pcie is TYPE_PXB_PCI_HOST by default,
> > change it to a new type TYPE_PXB_PCIE_HOST to better utilize ECAM of PCIe
> >
> > Signed-off-by: Zihan
Marcel Apfelbaum 于2018年8月18日周六 上午1:41写道：
>
> Hi Zihan,
>
> On 08/09/2018 09:33 AM, Zihan Yang wrote:
> > Allocate new segment for pxb-pcie host bridges in MCFG table, and reserve
> > corresponding MCFG space for them. This allows user-defined pxb-pcie
> > host bridges to be placed in different pci
Marcel Apfelbaum 于2018年8月18日周六 上午1:49写道：
>
>
>
> On 08/09/2018 09:34 AM, Zihan Yang wrote:
> > Describe new pci segments of host bridges in AML as new pci devices,
> > with _SEG and _BBN to let them be in DSDT.
> >
> > Besides, bus_nr indicates the bus number of pxb-pcie under pcie.0 bus,
> > but
Marcel Apfelbaum 于2018年8月18日周六 上午1:52写道：
>
>
>
> On 08/09/2018 09:35 AM, Zihan Yang wrote:
> > This enables seabios to read config file in pxb host bus other than sysbus
> >
> > Signed-off-by: Zihan Yang
> > ---
> > hw/pci-bridge/pci_expander_bridge.c | 15 +++
> > include/
Marcel Apfelbaum 于2018年8月18日周六 上午1:54写道：
>
>
>
> On 08/09/2018 09:35 AM, Zihan Yang wrote:
> > The bus_nr indicates the bus number of pxb-pcie under pcie.0, but since pxb
> > host can be put into different pci domain, the start bus should always be 0
>
> I am not sure about this limitation. Do yo
When the user moves the mouse and moves the scroll wheel at the same time, the
mouse cursor's movement becomes erratic in Windows 3.1. With this patch if the
mouse is in ps/2 mode and the scroll wheel is used, the command queue is reset.
This does not fix the erratic mouse problem in Windows NT
On 18/08/18 21:21, Mark Cave-Ayland wrote:
> Hi all,
>
> As part of testing my OpenBIOS virtio-blk implementation, I've been
> trying to use it under the 40p machine and have some questions related
> to how address spaces are interpreted.
>
> The 40p PCI address space is aliased onto the CPU phy
59 matches
Mail list logo
