Le 14/07/2017 à 15:51, Igor Mammedov a écrit :
now cpu_mips_init() reimplements subset of cpu_generic_init()
tasks, so just drop it and use cpu_generic_init() directly.
Signed-off-by: Igor Mammedov
Reviewed-by: Hervé Poussineau
---
CC: Aurelien
Le 14/07/2017 à 15:52, Igor Mammedov a écrit :
it's just a wrapper, drop it and use cpu_generic_init() directly
Signed-off-by: Igor Mammedov
Reviewed-by: Hervé Poussineau
---
CC: Alexander Graf
CC: David Gibson
Am Fri, 14 Jul 2017 15:52:05 +0200
schrieb Igor Mammedov :
> call register_m68k_insns() at realize time which makes
> cpu_m68k_init() typical object creation function.
> As result we can replace it with cpu_generic_init()
> which does the same job, reducing code duplication a
From: Lluís Vilanova
Signed-off-by: Lluís Vilanova
Message-Id: <150002073981.22386.9870422422367410100.st...@frigg.lan>
[rth: Moved max_insns adjustment from tb_start to init_disas_context.
Removed pc_next return from translate_insn.
Removed
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Emilio G. Cota
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Alex Benneé
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Alex Benneé
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Alex Benneé
Message-Id:
We can check for single-step just once.
Signed-off-by: Richard Henderson
---
target/arm/translate.c | 13 -
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/target/arm/translate.c b/target/arm/translate.c
index 4b1230b..ebe1c1a 100644
---
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Alex Benneé
Message-Id:
We need not check for ARM vs Thumb state in order to dispatch
disassembly of every instruction.
Signed-off-by: Richard Henderson
---
target/arm/translate.c | 134 +++--
1 file changed, 86 insertions(+), 48 deletions(-)
diff --git
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Message-Id:
On 15 July 2017 at 03:42, David Gibson wrote:
> On Fri, Jul 14, 2017 at 04:13:05PM +0100, Peter Maydell wrote:
>> Hi. I'm afraid this fails to build on the w32 compiler:
>> CC ppc64-softmmu/hw/ppc/spapr.o
>> /home/petmay01/linaro/qemu-for-merges/hw/ppc/spapr.c:
Richard Henderson writes:
> On 07/13/2017 11:26 PM, Lluís Vilanova wrote:
>> Incrementally paves the way towards using the generic instruction translation
>> loop.
>>
>> Signed-off-by: Lluís Vilanova
>> Reviewed-by: Richard Henderson
>> ---
>>
On 2017-07-11 08:23, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> v3.5: Added even register checks in the translator [thuth].
> ---
> target/s390x/helper.h | 6 +
> target/s390x/mem_helper.c | 310
> +
>
We have generated an exception, so use DISAS_NORETURN.
Signed-off-by: Richard Henderson
---
target/arm/translate-a64.c | 1 +
target/arm/translate.c | 4 +++-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/target/arm/translate-a64.c
From: Lluís Vilanova
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Emilio G. Cota
Tested-by: Emilio G. Cota
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Emilio G. Cota
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Alex Benneé
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Message-Id: <150002534291.22386.13499916738708680298.st...@frigg.lan>
Signed-off-by: Richard Henderson
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Alex Benneé
Message-Id:
Since AArch64 uses a fixed-width ISA, we can pre-compute the number of
insns remaining on the page. Also, we can check for single-step once.
Signed-off-by: Richard Henderson
---
target/arm/translate-a64.c | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Message-Id: <150002485863.22386.13949856269576226529.st...@frigg.lan>
[rth: Adjust for translate_insn interface
Hi,
This series seems to have some coding style problems. See output below for
more information:
Message-id: 20170715094243.28371-1-...@twiddle.net
Subject: [Qemu-devel] [PATCH v14 00/34] Generic translation framework
Type: series
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total=$(git
Richard Henderson writes:
> On 07/14/2017 07:26 AM, Richard Henderson wrote:
>> On 07/13/2017 11:26 PM, Lluís Vilanova wrote:
>>> Incrementally paves the way towards using the generic instruction
>>> translation
>>> loop.
>>>
>>> Signed-off-by: Lluís Vilanova
>>>
This is my take on Lluis' v13. I've also written patches for
Alpha, SH4, and HPPA as examples of more complicated cases.
I have not included them here, but for the record:
git://github.com/rth7680/qemu.git tgt-axp-2 tgt-sh4-2 tgt-pa
have the patches. In the case of Alpha and SH4, there were
From: Lluís Vilanova
Needed to implement a target-agnostic gen_intermediate_code() in the
future.
Signed-off-by: Lluís Vilanova
Reviewed-by: David Gibson
Reviewed-by: Richard Henderson
Reviewed-by: Alex
This will allow some amount of cleanup to happen before
switching the backends over to enum DisasJumpType.
Signed-off-by: Richard Henderson
---
include/exec/exec-all.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h
index
There's nothing magic about the exception that we generate in order
to execute the magic kernel page. We can and should allow gdb to
set a breakpoint at this location.
Signed-off-by: Richard Henderson
---
target/arm/translate.c | 22 +++---
1 file changed, 11
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Emilio G. Cota
Reviewed-by: Richard Henderson
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Emilio G. Cota
Reviewed-by: Richard Henderson
Reviewed-by: Alex
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Message-Id: <150002510079.22386.10164419868911710218.st...@frigg.lan>
[rth: Adjust for translate_insn interface
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Alex Benneé
Message-Id:
Impliment the vhost-crypto's funtions, such as startup,
stop and notification etc. Introduce an enum
QCryptoCryptoDevBackendOptionsType in order to
identify the cryptodev vhost backend is vhost-user
or vhost-kernel-module (If exist).
At this point, the cryptdoev-vhost-user works.
Signed-off-by:
Introduce two vhost-user meassges: VHOST_USER_CREATE_CRYPTO_SESSION
and VHOST_USER_CLOSE_CRYPTO_SESSION. At this point, the QEMU side
support crypto operation in cryptodev host-user backend.
Signed-off-by: Gonglei
Signed-off-by: Longpeng(Mike)
Because the cryptodev-builtin beckend doesn't fit
in production environment as its' poor performance.
And the Intel guys are working on the DPDK vhost-crypto
implementation. So let us push the cryptodev-vhost-user
backend upstream so that the whole process can be
accelerated.
This is a RFC
Richard Henderson writes:
> On 07/13/2017 10:25 PM, Lluís Vilanova wrote:
>> +/* Pass breakpoint hits to target for further processing */
>> +if (unlikely(!QTAILQ_EMPTY(>breakpoints))) {
>> +CPUBreakpoint *bp;
>> +QTAILQ_FOREACH(bp, >breakpoints, entry) {
Richard Henderson writes:
> On 07/13/2017 11:42 PM, Lluís Vilanova wrote:
>> -dc->base.is_jmp = DISAS_SKIP;
>> +dc->base.is_jmp = DISAS_NORETURN;
> Really should fold this back into the patch that introduced SKIP temporarily.
Sorry, I don't understand what you mean.
Thanks,
On 2017-07-14 14:23, Richard Henderson wrote:
> On 07/14/2017 11:08 AM, Aurelien Jarno wrote:
> > On 2017-07-11 17:18, Thomas Huth wrote:
> > > On 10.07.2017 22:45, Richard Henderson wrote:
> > > > Signed-off-by: Richard Henderson
> > > > ---
> > > > target/s390x/helper.h
On 07/14/2017 10:25 PM, Aurelien Jarno wrote:
That said I still wonder if we can get generators like that:
| static void in1_r1n(DisasContext *s, DisasFields *f, DisasOps *o)
| {
| int r1 = get_field(s->fields, r1);
| o->in1 = tcg_const_i32(r1);
|
| #define SPEC_in1_r1n 0
and
| static
On 2017-07-14 14:22, Richard Henderson wrote:
> On 07/14/2017 11:01 AM, Aurelien Jarno wrote:
> > > +if (parallel_cpus) {
> > > +int mask = 0;
> > > +#if !defined(CONFIG_ATOMIC64)
> > > +mask = -8;
> > > +#elif !defined(CONFIG_ATOMIC128)
> > > +mask = -16;
> > > +#endif
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Emilio G. Cota
Reviewed-by: Richard Henderson
Reviewed-by: Alex
This target is not sophisticated in its use of cleanups at the
end of the translation loop. For the most part, any condition
that exits the TB is dealt with by emitting the exiting opcode
right then and there. Therefore the only is_jmp indicator that
is needed is DISAS_NORETURN.
For two stack
Fold DISAS_EXC and DISAS_TB_JUMP into DISAS_NORETURN.
In both cases all following code is dead. In the first
case because we have exited the TB via exception; in the
second case because we have exited the TB via goto_tb
and its associated machinery.
Signed-off-by: Richard Henderson
From: Lluís Vilanova
Used later. An enum makes expected values explicit and
bounds the value space of switches.
Signed-off-by: Lluís Vilanova
Reviewed-by: Emilio G. Cota
Reviewed-by: Richard Henderson
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Emilio G. Cota
Reviewed-by: Richard Henderson
Message-Id:
From: Lluís Vilanova
Signed-off-by: Lluís Vilanova
Message-Id: <150002631325.22386.10348327185029496649.st...@frigg.lan>
Signed-off-by: Richard Henderson
---
target/arm/translate.h | 8 +---
target/arm/translate-a64.c | 107
ARM is a fixed-length ISA and we can compute the page crossing
condition exactly once during init_disas_context.
Signed-off-by: Richard Henderson
---
target/arm/translate.c | 57 +-
1 file changed, 33 insertions(+), 24
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Message-Id:
From: Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation
loop.
Signed-off-by: Lluís Vilanova
Reviewed-by: Richard Henderson
Reviewed-by: Alex Benneé
Message-Id:
Usage:
-chardev socket,id=charcrypto0,path=/path/to/your/socket
-object cryptodev-vhost-user,id=cryptodev0,chardev=charcrypto0
-device virtio-crypto-pci,id=crypto0,cryptodev=cryptodev0
Signed-off-by: Gonglei
Signed-off-by: Longpeng(Mike)
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [Qemu-devel] [RFC 0/3] cryptodev: add vhost support
Message-id: 1500114070-181004-1-git-send-email-arei.gong...@huawei.com
Type: series
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
qemu-backup will be a command-line tool for performing full and
incremental disk backups on running VMs. It is intended as a
reference implementation for management stack and backup developers
to see QEMU's backup features in action. The following commit is an
initial implementation of manpage
Hi,
This series failed build test on s390x host. Please find the details below.
Message-id: 1500118925-10569-1-git-send-email-chugh.ish...@research.iiit.ac.in
Type: series
Subject: [Qemu-devel] [PATCH v5] Add manpage for QEMU Backup Tool
=== TEST SCRIPT BEGIN ===
#!/bin/bash
# Testing script
From: Marc-André Lureau
I used the clang-tidy qemu-round check to generate the fix:
https://github.com/elmarco/clang-tools-extra
Signed-off-by: Marc-André Lureau
Signed-off-by: Samuel Thibault
---
Write support works again when image contains non-ASCII names. It is either the
case when user created a non-ASCII filename, or when initial directory contained
a non-ASCII filename (since 0c36111f57ec2188f679e7fa810291b7386bdca1)
Signed-off-by: Hervé Poussineau
---
In the first line of run_agent,it has set ga_state = s,don't need
set ga_state = s again behind.
Signed-off-by: Peng Hao
---
qga/main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/qga/main.c b/qga/main.c
index 405c129..dcd6104 100644
---
This prevents some host to guest memory content leaks.
Fixes: https://bugs.launchpad.net/qemu/+bug/1599539
Signed-off-by: Hervé Poussineau
---
block/vvfat.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/block/vvfat.c b/block/vvfat.c
index afc6170a69..7340decef3
Also add links to related compatibility problems.
Signed-off-by: Hervé Poussineau
---
block/vvfat.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/block/vvfat.c b/block/vvfat.c
index c2674d7703..e585a8e0be 100644
--- a/block/vvfat.c
+++
Dr. David Alan Gilbert, on ven. 14 juil. 2017 15:24:19 +0100, wrote:
> * Peter Maydell (peter.mayd...@linaro.org) wrote:
> > sosendoob() can return a failure code, but all its callers ignore it.
> > This is OK in sbappend(), as the comment there states -- we will try
> > again later in sowrite().
From: Peter Maydell
The code in sosendoob() assumes that slirp_send() always
succeeds, but it might return an OS error code (for instance
if the other end has disconnected). Catch these and return
the caller either -1 on error or the number of urgent bytes
actually
From: Peter Maydell
sosendoob() can return a failure code, but all its callers ignore it.
This is OK in sbappend(), as the comment there states -- we will try
again later in sowrite(). Add a (void) cast to tell Coverity so.
In sowrite() we do need to check the return
warning: redirection vers https://people.debian.org/~sthibault/qemu.git/
The following changes since commit 4871b51b9241b10f4fd8e04bbb21577886795e25:
vmgenid-test: use boot-sector infrastructure (2017-07-14 17:03:03 +0100)
are available in the git repository at:
From: Peter Maydell
In a fork_exec() error path we try to closesocket(s) when s might
be a negative number because the thing that failed was the
qemu_socket() call. Add a guard so we don't do this.
(Spotted by Coverity: CID 1005727 issue 1 of 2.)
Signed-off-by: Peter
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [Qemu-devel] [PULL 0/4] slirp updates
Message-id: 20170715123057.8529-1-samuel.thiba...@ens-lyon.org
Type: series
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total=$(git log
Signed-off-by: Hervé Poussineau
---
block/vvfat.c | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/block/vvfat.c b/block/vvfat.c
index 4fd28e1e87..c2674d7703 100644
--- a/block/vvfat.c
+++ b/block/vvfat.c
@@ -71,6 +71,11 @@ void
Hi,
This patchset is a follow-up for patch series sent here:
http://lists.gnu.org/archive/html/qemu-devel/2017-05/msg05017.html
Patches 1 and 2 define and use some constants to make the code more clear.
Patch 3 make read-write mode work when using non-ASCII filenames.
Patch 4 fixes the last
With "-netdev user,id=net0,dns=1.2.3.4"
error was:
qemu-system-i386: -netdev user,id=net0,dns=1.2.3.4: Device 'user' could not be
initialized
Error is now:
qemu-system-i386: -netdev user,id=net0,dns=1.2.3.4: DNS doesn't belong to
network
Signed-off-by: Hervé Poussineau
HEllo,
Hervé Poussineau, on sam. 15 juil. 2017 18:43:50 +0200, wrote:
> With "-netdev user,id=net0,dns=1.2.3.4"
> error was:
> qemu-system-i386: -netdev user,id=net0,dns=1.2.3.4: Device 'user' could not
> be initialized
>
> Error is now:
> qemu-system-i386: -netdev user,id=net0,dns=1.2.3.4: DNS
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/mem_helper.c | 41 +
target/s390x/translate.c | 13 +
target/s390x/insn-data.def | 2
Reviewed-by: Aurelien Jarno
Reviewed-by: Thomas Huth
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 6 +
target/s390x/mem_helper.c | 310 +
target/s390x/translate.c |
:
vmgenid-test: use boot-sector infrastructure (2017-07-14 17:03:03 +0100)
are available in the git repository at:
git://github.com/rth7680/qemu.git tags/pull-s390-20170715
for you to fetch changes up to 4fb4047e74c21a3692360b9b1df473d7512d8055:
target/s390x: Fix risbg handling (2017-07-15
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/cpu_models.c | 2 +
target/s390x/mem_helper.c | 189 +
target/s390x/translate.c | 13 +++-
From: David Hildenbrand
STFL bit 4 and 5 are just indications to the guest, which TLB entries an
IDTE call will clear. These are performance indicators for the guest.
STFL bit 4:
INVALIDATE DAT TABLE ENTRY (IDTE) performs
the invalidation-and-clearing operation by
Since we require all registers saved on input, read R0 from ENV instead
of passing it manually. Recognize the specification exception when R0
contains incorrect data. Keep high bits of result registers unmodified
when in 31 or 24-bit mode.
Reviewed-by: Aurelien Jarno
Drop TRT from the set of insns handled internally by EXECUTE.
It's more important to adjust the existing helper to handle
both TRT and TRTR.
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
The rotation is to the left, but extract shifts to the right.
The computation of the extract parameters needs adjusting.
For the entry condition, simplify
64 - rot + len <= 64
-rot + len <= 0
len <= rot
Reviewed-by: Aurelien Jarno
Reported-by:
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/cpu_models.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
index 8433010..9e9e649 100644
---
Hello all,
Could you please create an account for me?
account: tim3385
email: tim3...@gmail.com
thanks
best
Dong Hao
Sent from YoMail for Gmail
On 07/14/2017 09:56 PM, Lluís Vilanova wrote:
Another possibility is is_jmp = DISAS_TOO_MANY, and exit the translation loop
after the breakpoint check only for is_jmp > DISAS_TOO_MANY. That allows all of
the DISAS_TARGET_N values to exit as well.
After a quick check, I see that arm uses both
On 07/14/2017 09:56 PM, Lluís Vilanova wrote:
Richard Henderson writes:
On 07/13/2017 11:42 PM, Lluís Vilanova wrote:
-dc->base.is_jmp = DISAS_SKIP;
+dc->base.is_jmp = DISAS_NORETURN;
Really should fold this back into the patch that introduced SKIP temporarily.
Sorry, I
On 07/14/2017 03:52 AM, Igor Mammedov wrote:
cpu_alpha_init() used to provide default fallback if invalid
(i.e. non existent) cpu_model were provided.
dp264 machine provides its own default so sole user of fallback
is [bsd|linux]-user targets which specifies 'any' cpu model that
fallbacks to
On 07/14/2017 03:52 AM, Igor Mammedov wrote:
drop custom cpu_hppa_init() in favor of cpu_generic_init(),
to make cpu_generic_init() work all we need is to provide
cc->class_by_name callback that would resolve any cpu_model
to the sole TYPE_HPPA_CPU to match current behaviour.
Signed-off-by:
On 07/14/2017 03:52 AM, Igor Mammedov wrote:
@@ -230,6 +230,8 @@ static void m68k_cpu_realizefn(DeviceState *dev, Error
**errp)
M68kCPUClass *mcc = M68K_CPU_GET_CLASS(dev);
Error *local_err = NULL;
+register_m68k_insns(>env);
+
I think it would make more sense to do this
The value is set to uint16_t by mistake. The value is passed to Qemu in
bytes, but then it is divided by the sector size and passed to the vm in
sectors through a 16 bit register field. The vm kernel then multiplies
it again by sector size and shows (through /sys/block/x/queue) the value
in bytes.
Hello all,
- QEMU is a great work. Thanks for your effor.
- I am a begginer of QEMU. I dont't found the way to support AMD acceleration
on Windows host. I want to run x86 ABI android rom(or Android-x86) on AMD CPU
quickly.
- As I know:
- KQEMU isnot support HW CPU Virtualization
This is a Request For Comments patch for qemu backup tool. As an
Outreachy intern, I am assigned to the project for creating a backup
tool. qemu-backup will be a command-line tool for performing full and
incremental disk backups on running VMs. It is intended as a
reference implementation for
Le 15/07/2017 à 20:08, Richard Henderson a écrit :
> On 07/14/2017 03:52 AM, Igor Mammedov wrote:
>> @@ -230,6 +230,8 @@ static void m68k_cpu_realizefn(DeviceState *dev,
>> Error **errp)
>> M68kCPUClass *mcc = M68K_CPU_GET_CLASS(dev);
>> Error *local_err = NULL;
>> +
On 07/15/2017 10:28 AM, Hervé Poussineau wrote:
Also add links to related compatibility problems.
Signed-off-by: Hervé Poussineau
Reviewed-by: Philippe Mathieu-Daudé
---
block/vvfat.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
Michael,
Thanks for the review and comments.
> -邮件原件-
> 发件人: Michael S. Tsirkin [mailto:m...@redhat.com]
> 主题: Re: [PATCH v5 2/3] ACPI: Add APEI GHES Table Generation support
>
> On Wed, Jul 12, 2017 at 10:08:16AM +0800, Dongjiu Geng wrote:
> > This implements APEI GHES Table by
On Wed, Jul 12, 2017 at 13:06:23 -1000, Richard Henderson wrote:
> You've got a problem here in that you're not including CF_COUNT_MASK in the
> hash and you dropped the flush when changing to parallel_cpus = true. That
> means you could find an old TB with CF_COUNT > 1.
>
> Not required for
On 07/16/2017 12:14 AM, Aurelien Jarno wrote:
> Do you have actually have a good documentation about gUSA? I have found
> a few documents (some of them in Japanese), the most complete one being
> the LinuxTag paper. The ABI is also described in the kernel and the
> glibc. That said I am missing
On 2017-07-06 16:20, Richard Henderson wrote:
> If a signal is delivered during the execution of a delay slot,
> or a gUSA region, clear those bits from the environment so that
> the signal handler does not start in that same state.
How are signals delivered in linux-user? At least in system mode
On 07/15/2017 12:59 PM, Aurelien Jarno wrote:
On 2017-07-06 16:20, Richard Henderson wrote:
If a signal is delivered during the execution of a delay slot,
or a gUSA region, clear those bits from the environment so that
the signal handler does not start in that same state.
How are signals
Le 16/07/2017 à 00:24, Philippe Mathieu-Daudé a écrit :
Hi Hervé,
On 07/15/2017 10:28 AM, Hervé Poussineau wrote:
This prevents some host to guest memory content leaks.
Fixes: https://bugs.launchpad.net/qemu/+bug/1599539
Signed-off-by: Hervé Poussineau
---
On 07/14/2017 10:51 AM, Igor Mammedov wrote:
Register separate QOM types for each mips cpu model,
so it would be possible to reuse generic CPU creation
routines.
Signed-off-by: Igor Mammedov
Reviewed-by: Philippe Mathieu-Daudé
---
CC: Aurelien Jarno
On 07/14/2017 10:51 AM, Igor Mammedov wrote:
it will help to replace custom cpu_mips_init() with cpu_generic_init().
Signed-off-by: Igor Mammedov
Reviewed-by: Philippe Mathieu-Daudé
---
CC: Aurelien Jarno
CC: Yongbok Kim
On 07/14/2017 10:51 AM, Igor Mammedov wrote:
now cpu_mips_init() reimplements subset of cpu_generic_init()
tasks, so just drop it and use cpu_generic_init() directly.
Signed-off-by: Igor Mammedov
Reviewed-by: Philippe Mathieu-Daudé
---
CC: Aurelien
On 2017-07-06 16:20, Richard Henderson wrote:
> We translate gUSA regions atomically in a parallel context.
> But in a serial context a gUSA region may be interrupted.
> In that case, restart the region as the kernel would.
>
> Signed-off-by: Richard Henderson
> ---
>
1 - 100 of 108 matches
Mail list logo