On 17/07/2016 04:06, Ayaz Akram wrote:
> Hi all !
>
> I ran a program with qemu in user mode emulation and generated trace for
> generated host instructions using (-d OUT_ASM) on two different linux
> systems.I expected that the addresses in two trace files can be different.
> But the total numb
Linux requires that you specify a WWN for the disk (through the wwn
property of the scsi-disk device).
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1603693
Title:
Disks in mptsas1068 scsi controll
Public bug reported:
At least qemu-i368 and qemu-x86_64 hang in floatx80_sqrt in versions
2.6.0 and git (2.6.50) for some input values, likely due to an infinite
loop at fpu/softfloat.c:6569.
Steps to reproduce:
1) Compile attached code: gcc -o test test.c -lm
2) `qemu-i368 test` and `qemu-x86_64
On 2016 Jul 14 (Thu) 23:34, Stefano Stabellini wrote:>
Hi Quan,
>
> thanks for CC'ing me. sstabell...@kernel.org is the right address to
> reach me now.
>
> I am also CC'ing Anthony Perard who is Xen co-maintainer in QEMU.
>
> Cheers,
>
> Stefano
thanks in advance!! :):)Quan
[Quan:]: comment starts with [Quan:]
The purpose of the new file is to store generic functions shared by frontendand
backends such as xenstore operations, xendevs.
Signed-off-by: Quan Xu
Signed-off-by: Emil Condrea
---
hw/xen/Makefile.objs | 2 +-
hw/xen/xen_backend.c | 12
Public bug reported:
While working with qtest test cases generated via fuzzing with QEMU
2.5.0, I discovered some odd behavior for the AC97 virtual device with
qemu-system-i386. If AC97_MIC_ADC_RATE is set to the value of 1, the
QEMU process allocates over 500MB of additional host RAM. You probabl
Public bug reported:
Running qemu with tracing (-D ~/qemu_trace -d trace:\*) will result in a
trace file with unprintable characters.
example: usb_port_attach bus 0, port 1, devspeed <90>l.U,
portspeed full+high
The problem is in hw/usb/bus.c usb_mask_to_str. If speedmask doesn't
match any of th
v4 -> v5:
Addressed the pull request issues: (Peter Maydell)
See: https://lists.gnu.org/archive/html/qemu-devel/2016-07/msg00882.html
- cland warning -> "hw/pci/pci.c:196:23: runtime error: shift exponent -1 is
negative":
The PCIe Root port was not initialized properly, the interrupt pi
Specify the root port interrupt pin as part of the init
process for cases when msi/msix are not enabled.
Fixes "hw/pci/pci.c:196:23: runtime error: shift exponent -1 is negative"
warning from clang's sanitizer.
Reported-by: Peter Maydell
Signed-off-by: Marcel Apfelbaum
---
hw/pci-bridge/ioh342
Instead of always passing both IO and MEM ranges when
computing CRS ranges, define a new CrsRangeSet structure
that include them both.
This is done before introducing a third type of range,
64-bit MEM, so it will be easier to pass them all around.
Reviewed-by: Igor Mammedov
Signed-off-by: Marcel
Add an ivshmem device with 1G shared memory to
pxb in order to check the ACPI code of 64bit MMIO allocation.
Suggested-by: Igor Mammedov
Signed-off-by: Marcel Apfelbaum
Tested-by: Laszlo Ersek
---
tests/bios-tables-test.c | 37 +
1 file changed, 37 insertion
Signed-off-by: Marcel Apfelbaum
---
tests/acpi-test-data/pc/DSDT.pxb | Bin 0 -> 6286 bytes
tests/acpi-test-data/q35/DSDT.pxb_pcie | Bin 0 -> 9098 bytes
2 files changed, 0 insertions(+), 0 deletions(-)
create mode 100644 tests/acpi-test-data/pc/DSDT.pxb
create mode 100644 tests/acpi-test
PXBs do not support hotplug so they don't have a PCNT function.
Since the PXB's PCI root-bus is a child bus of bus 0, the
build_dsdt code will add a call to the corresponding PCNT function.
Fix this by skipping the PCNT call for the above case.
While at it skip also PCIe child buses.
Reported-by:
Prevent future issues when hotplug will work for devices
attached to pxbs.
Suggested-by: Igor Mammedov
Signed-off-by: Marcel Apfelbaum
Tested-by: Laszlo Ersek
---
hw/pci-bridge/pci_expander_bridge.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/pci-bridge/pci_expander_bridge.c
b/hw
In build_crs(), the calculation and merging of the ranges already happens
in 64-bit, but the entry boundaries are silently truncated to 32-bit in the
call to aml_dword_memory(). Fix it by handling the 64-bit MMIO ranges
separately.
This fixes 64-bit BARs behind PXBs.
Reported-by: Laszlo Ersek
Re
On Jul 17, 2016 10:41, "Quan Xu" wrote:
>
>
> [Quan:]: comment starts with [Quan:]
>
Thanks, Quan for your comments.
The first patches from this series just move some code from xen_backend to
xen_pvdev file. I would not group the reorg from xen_backend with
refactoring in the same patch. Eventual
On 07/15/2016 08:30 PM, Alex Williamson wrote:
QEMU supports ARI on downstream ports and assigned devices may support
ARI in their extended capabilities. The endpoint ARI capability
specifies the next function, such that the OS doesn't need to walk
each possible function, however this next funct
On Fri, Jul 15, 2016 at 05:39:38PM +0200, Thomas Huth wrote:
> The rather random list of check-qtest-xxx entries caused some
> confusion in the past, where to use "=" and where to use "+="
> (see commits 0ccac16f59462b8e2b9afbc1 and 1f5c1cfbaec0792cd2e5da
> for example).
> Sorting the check-qtest-x
On Fri, Jul 15, 2016 at 05:39:39PM +0200, Thomas Huth wrote:
> Some of the machines that we have got a firmware image for write
> some output to the serial console while booting up. We can use
> this output to make sure that the machine is basically working,
> so this adds a test that checks the ou
On Fri, Jul 15, 2016 at 05:22:10PM +0200, Paolo Bonzini wrote:
> ps->pte_enc is a 32-bit value, which is shifted left and then compared
> to a 64-bit value. It needs a cast before the shift.
>
> Reported by Coverity.
>
> Signed-off-by: Paolo Bonzini
Applied to ppc-for-2.7, thanks.
> ---
> ta
On Sat, Jul 16, 2016 at 12:11:56AM +0200, Greg Kurz wrote:
> On Thu, 14 Jul 2016 21:59:45 +1000
> David Gibson wrote:
>
> > On Thu, Jul 14, 2016 at 03:50:56PM +0530, Bharata B Rao wrote:
> > > On Thu, Jul 14, 2016 at 3:24 PM, Peter Maydell
> > > wrote:
> > > > On 14 July 2016 at 08:57, David
On Fri, Jul 15, 2016 at 10:10:25AM +0200, Thomas Huth wrote:
> Commit 86b50f2e1bef ("Disable huge page support if it is not available
> for main RAM") already made sure that huge page support is not announced
> to the guest if the normal RAM of non-NUMA configurations is not backed
> by a huge page
On Fri, Jul 15, 2016 at 06:12:05PM +0200, Lluís Vilanova wrote:
> Needed to implement a target-agnostic gen_intermediate_code() in the
> future.
>
> Signed-off-by: Lluís Vilanova
> ---
> include/exec/exec-all.h |2 +-
> target-alpha/translate.c | 11 +--
> target-arm/tra
On Fri, Jul 15, 2016 at 03:55:20PM +0800, Zhangfei Gao wrote:
> Dear Dave
>
> On Wed, Jul 13, 2016 at 7:03 AM, Dave Chinner wrote:
> > On Tue, Jul 12, 2016 at 12:43:24PM -0400, Theodore Ts'o wrote:
> >> On Tue, Jul 12, 2016 at 03:14:38PM +0800, Zhangfei Gao wrote:
> >> > Some update:
> >> >
> >>
Hi~ All~~
Can you give me some feedback for this patch?
We need more comments~~
COLO project depend on this patch to work.
Because this patch colo-compare can make handler of qemu_chr_add_handlers()
run in compare thread, reduce workload of main_loop in network busy
situation.
This idea fr
On Tue, Jul 12, 2016 at 11:33:20PM +0530, Nikunj A Dadhania wrote:
> ISA 3.0 Compare Ranged Byte instruction useful for
> isupper/islower/isaplha kind of operation.
At least until you have locale-aware versions of those...
> Signed-off-by: Nikunj A Dadhania
Reviewed-by: David Gibson
> ---
>
On Tue, Jul 12, 2016 at 11:33:17PM +0530, Nikunj A Dadhania wrote:
> From: "Aneesh Kumar K.V"
>
> Signed-off-by: Aneesh Kumar K.V
> [ rebased and added POWER9 alias ]
> Signed-off-by: Nikunj A Dadhania
> ---
> target-ppc/cpu-models.c | 5 +++
> target-ppc/cpu-models.h | 2 ++
> targe
On Tue, Jul 12, 2016 at 11:33:18PM +0530, Nikunj A Dadhania wrote:
> This flag will be used for POWER9 instructions.
>
> Signed-off-by: Nikunj A Dadhania
Reviewed-by: David Gibson
> ---
> target-ppc/cpu.h| 5 -
> target-ppc/translate_init.c | 2 +-
> 2 files changed, 5 inserti
On Tue, Jul 12, 2016 at 11:33:21PM +0530, Nikunj A Dadhania wrote:
> Adding following instructions:
>
> moduw: Modulo Unsigned Word
> modsw: Modulo Signed Word
>
> Signed-off-by: Nikunj A Dadhania
Hrm.. any reason you're not using the TCG inbuilt remainder ops
(tcg_gen_rem_i32() etc.)?
> ---
>
On Tue, Jul 12, 2016 at 11:33:19PM +0530, Nikunj A Dadhania wrote:
> ISA 3.0 instruction for adding immediate value with next instruction
> address and return the result in the target register.
>
> Signed-off-by: Nikunj A Dadhania
Reviewed-by: David Gibson
> ---
> target-ppc/translate.c | 27
Welp. Yeah now I see it, it was in the test case I linked. Thanks.
Vmware doesn't seem to need this. Seems like it assigns a WWN of
0x5000c293944837df to my disk (not in the vm config files as far as i
can see, seems to persist across reboots)
[2.305111] ioc0: LSISAS1068 B0: Capabilities={Ini
A static analysis tool called BEAM detected possible side effect from
assert() calling a helper which may change an XHCI ring after every call.
This moves xhci_ring_fetch() out of assert() so it will be called
with and without enabled debug.
Signed-off-by: Alexey Kardashevskiy
---
hw/usb/hcd-xh
No need duplicate the judgment, there is one in function entry.
Cc: Stefan Hajnoczi
Cc: Kevin Wolf
Cc: Max Reitz
Signed-off-by: Cao jin
---
hw/block/dataplane/virtio-blk.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/hw/block/dataplane/virtio-blk.c b/hw/block/data
From: Thomas Huth
No need to include error-report.h twice here.
Signed-off-by: Thomas Huth
Signed-off-by: David Gibson
---
target-ppc/mmu-hash64.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/target-ppc/mmu-hash64.c b/target-ppc/mmu-hash64.c
index 82c2186..f6ffe35 100644
--- a/target-pp
From: Mark Cave-Ayland
Otherwise MacOS 9 hangs upon shutdown.
Signed-off-by: Mark Cave-Ayland
Acked-by: Benjamin Herrenschmidt
Signed-off-by: David Gibson
---
hw/misc/macio/mac_dbdma.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/misc/macio/mac_dbdma.c b/hw/misc/macio/mac_dbdma.c
i
From: Mark Cave-Ayland
Enabling DBDMA_DPRINTF unconditionally ensures that any errors in debug
statements are picked up immediately.
Signed-off-by: Mark Cave-Ayland
Acked-by: Benjamin Herrenschmidt
Signed-off-by: David Gibson
---
hw/misc/macio/mac_dbdma.c | 15 +++
1 file changed
From: Mark Cave-Ayland
This fixes MacOS 9 whereby it continually flushes and polls the status bits
until they are set to indicate a successful flush.
Signed-off-by: Mark Cave-Ayland
Acked-by: Benjamin Herrenschmidt
Signed-off-by: David Gibson
---
hw/misc/macio/mac_dbdma.c | 10 ++
1
The following changes since commit 6b92bbfe812746fe7841a24c24e6460f5359ce72:
Merge remote-tracking branch 'remotes/mcayland/tags/qemu-openbios-signed'
into staging (2016-07-15 16:56:08 +0100)
are available in the git repository at:
git://github.com/dgibson/qemu.git tags/ppc-for-2.7-20160718
From: Thomas Huth
Commit 86b50f2e1bef ("Disable huge page support if it is not available
for main RAM") already made sure that huge page support is not announced
to the guest if the normal RAM of non-NUMA configurations is not backed
by a huge page filesystem. However, there is one more case that
This ioctl() call to VFIO_IOMMU_SPAPR_TCE_REMOVE was left over from an
earlier version of the code and has since been folded into
vfio_spapr_remove_window().
It wasn't caught because although the argument structure has been removed,
the libc function remove() means this didn't trigger a compile fa
From: Mark Cave-Ayland
By default large amounts of DBDMA debugging are produced when often it is just
1 or 2 channels that are of interest. Introduce DEBUG_DBDMA_CHANMASK to allow
the developer to select the channels of interest at compile time, and then
further add the extra channel information
From: Greg Kurz
If the host has 8 threads/core and the guest is started with:
-smp cores=1,threads=4,maxcpus=12
It is possible to crash QEMU by doing:
(qemu) device_add host-spapr-cpu-core,core-id=16,id=foo
(qemu) device_del foo
Segmentation fault
This happens because spapr_core_unplug() assu
From: Greg Kurz
It is not possible to set the compat property to an unknown value with
powerpc_set_compat(). Something must have gone terribly wrong in QEMU,
if we detect an "Internal error" in powerpc_get_compat(). Let's abort then.
This patch also drops the "max_compat ? *max_compat : -1" cons
From: Bharata B Rao
If CPU core addition or removal is allowed in random order leading to
holes in the core id range (and hence in the cpu_index range), migration
can fail as migration with holes in cpu_index range isn't yet handled
correctly.
Prevent this situation by enforcing the addition in
From: Paolo Bonzini
ps->pte_enc is a 32-bit value, which is shifted left and then compared
to a 64-bit value. It needs a cast before the shift.
Reported by Coverity.
Signed-off-by: Paolo Bonzini
Signed-off-by: David Gibson
---
target-ppc/mmu-hash64.c | 2 +-
1 file changed, 1 insertion(+),
From: Benjamin Herrenschmidt
MacOS uses an architecturally illegal MSR combination that
seems nonetheless supported by 32-bit processors, which is
to have MSR[PR]=1 and one or more of MSR[DR/IR/EE]=0.
This adds support for it. To work properly we need to also
properly include support for PR=1,{I
From: Mark Cave-Ayland
The current DBDMA command is stored in little-endian format, so make sure
we convert it to match our CPU when updating the DBDMA_CMDPTR_LO register.
Signed-off-by: Mark Cave-Ayland
Acked-by: Benjamin Herrenschmidt
Signed-off-by: David Gibson
---
hw/misc/macio/mac_dbdma
From: Mark Cave-Ayland
The values to read/write to/from physical memory are copied directly to the
physical address with no endian swapping required.
Also add some extra information to debugging output while we are here.
Signed-off-by: Mark Cave-Ayland
Acked-by: Benjamin Herrenschmidt
Signed-
David Gibson writes:
> [ Unknown signature status ]
> On Tue, Jul 12, 2016 at 11:33:21PM +0530, Nikunj A Dadhania wrote:
>> Adding following instructions:
>>
>> moduw: Modulo Unsigned Word
>> modsw: Modulo Signed Word
>>
>> Signed-off-by: Nikunj A Dadhania
>
> Hrm.. any reason you're not using
On Sun, Jul 17, 2016 at 9:37 PM, 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 boot using
David Gibson writes:
> [ Unknown signature status ]
> On Tue, Jul 12, 2016 at 11:33:17PM +0530, Nikunj A Dadhania wrote:
>> From: "Aneesh Kumar K.V"
>>
>> Signed-off-by: Aneesh Kumar K.V
>> [ rebased and added POWER9 alias ]
>> Signed-off-by: Nikunj A Dadhania
>> ---
>> target-ppc/cpu-models
On Mon, 07/18 12:05, Cao jin wrote:
> No need duplicate the judgment, there is one in function entry.
>
> Cc: Stefan Hajnoczi
> Cc: Kevin Wolf
> Cc: Max Reitz
> Signed-off-by: Cao jin
> ---
> hw/block/dataplane/virtio-blk.c | 6 ++
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> di
On 2016年07月18日 09:55, Zhang Chen wrote:
Hi~ All~~
Can you give me some feedback for this patch?
We need more comments~~
COLO project depend on this patch to work.
Because this patch colo-compare can make handler of
qemu_chr_add_handlers()
run in compare thread, reduce workload of main_lo
On 07/18/2016 01:31 PM, Jason Wang wrote:
On 2016年07月18日 09:55, Zhang Chen wrote:
Hi~ All~~
Can you give me some feedback for this patch?
We need more comments~~
COLO project depend on this patch to work.
Because this patch colo-compare can make handler of
qemu_chr_add_handlers()
run
On 2016年07月13日 10:42, Jason Wang wrote:
e1000e needs net_tx_pkt.o and net_rx_pkt.o too.
Cc: Dmitry Fleytman
Cc: Leonid Bloch
Signed-off-by: Jason Wang
---
hw/net/Makefile.objs | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/net/Makefile.objs b/hw/net/Makefile.objs
index fe61e9f..6
On 2016年07月15日 16:41, Paolo Bonzini wrote:
Coverity reports a "suspicious sizeof" which is indeed wrong.
Signed-off-by: Paolo Bonzini
---
net/eth.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/eth.c b/net/eth.c
index 95fe15c..0be59c2 100644
--- a/net/eth.c
++
On 2016年07月15日 16:43, Paolo Bonzini wrote:
This is not dereferencing the pointer, and instead checking only
the value of the pointer.
Signed-off-by: Paolo Bonzini
---
net/eth.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/eth.c b/net/eth.c
index 0be59c2..df81efb
On 2016年07月15日 16:44, Paolo Bonzini wrote:
This is not dereferencing the pointer, and instead checking only
the value of the pointer.
Signed-off-by: Paolo Bonzini
---
hw/net/e1000e_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/net/e1000e_core.c b/hw/net/
On 2016年07月15日 16:56, Paolo Bonzini wrote:
Reported by Coverity.
Signed-off-by: Paolo Bonzini
---
net/tap.c | 22 --
1 file changed, 16 insertions(+), 6 deletions(-)
diff --git a/net/tap.c b/net/tap.c
index e9c32f3..6a2cedc 100644
--- a/net/tap.c
+++ b/net/tap.c
@@ -78
On 2016年07月11日 15:49, Jiri Pirko wrote:
From: Jiri Pirko
As requested by Scott, removing him.
Signed-off-by: Scott Feldman
Signed-off-by: Jiri Pirko
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 1d0e2c3..5928f22 100644
--- a/MAINTAINE
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 boot using pstore filesystem. It currently deals with
kernel log (print
From: Namhyung Kim
Add virtio pstore device to allow kernel log files saved on the host.
It will save the log files on the directory given by pstore device
option.
$ qemu-system-x86_64 -device virtio-pstore,directory=dir-xx ...
(guest) # echo c > /proc/sysrq-trigger
$ ls dir-xx
dmesg-0
> The non-working vmware config says `scsi0.virtualDev = "lsilogic"`
> (that's mptspi, LSI53C1030 or "LSI Logic Ultra 320"). For the mptsas
> tests above, I changed it to `scsi0.virtualDev = "lsisas1068"`.
>
> Is it correct to say that the LSI53C1030 parts of [1] were never applied?
Yes, that's co
Hello,
This patchset is a proof of concept of virtio-pstore idea [1]. It has
some rough edges and I'm not familiar with this area, so please give
me feedbacks and advices if I'm going to a wrong direction.
It started from the fact that dumping ftrace buffer at kernel
oops/panic takes too much ti
Hello,
On Sun, Jul 17, 2016 at 10:12:26PM -0700, Kees Cook wrote:
> On Sun, Jul 17, 2016 at 9:37 PM, 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
COLO-compare is a part of COLO project. It is used
to compare the network package to help COLO decide
whether to do checkpoint.
The full version in this github:
https://github.com/zhangckid/qemu/tree/colo-v2.7-proxy-mode-compare-with-colo-base-jul18
v7:
p5:
- add [PATCH]qemu-char: Fix contex
In this patch we use kernel jhash table to track
connection, and then enqueue net packet like this:
+ CompareState ++
| |
+---+ +---+ +---+
|conn list +--->conn +->conn |
+---+ +-
Jhash used by colo-compare and filter-rewriter
to save and lookup net connection info
Signed-off-by: Zhang Chen
Signed-off-by: Li Zhijian
Signed-off-by: Wen Congyang
---
include/qemu/jhash.h | 61
1 file changed, 61 insertions(+)
create mod
If primary packet is same with secondary packet,
we will send primary packet and drop secondary
packet, otherwise notify COLO frame to do checkpoint.
If primary packet comes and secondary packet not,
after REGULAR_PACKET_CHECK_MS milliseconds we set
the primary packet as old_packet,then do a checkp
We add TCP,UDP,ICMP packet comparison to replace
IP packet comparison. This can increase the
accuracy of the package comparison.
less checkpoint more efficiency.
Signed-off-by: Zhang Chen
Signed-off-by: Li Zhijian
Signed-off-by: Wen Congyang
---
net/colo-compare.c | 174 +++
We want to poll and handle chardev in another thread
other than main loop. But qemu_chr_add_handlers() can only
work for global default context other than thread default context.
So we use g_source_attach(xx, g_main_context_get_thread_default())
replace g_source_attach(xx, NULL) to attach g_source.
COLO-base used by colo-compare and filter-rewriter.
this can share common data structure like:net packet,
and share other functions.
Signed-off-by: Zhang Chen
Signed-off-by: Li Zhijian
Signed-off-by: Wen Congyang
---
net/Makefile.objs | 1 +
net/colo-base.c| 74
This a COLO net ascii figure:
Primary qemu
Secondary qemu
+--+
++
| +--
73 matches
Mail list logo
