[Expired for QEMU because there has been no activity for 60 days.]
** Changed in: qemu
Status: Incomplete => Expired
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/961757
Title:
wrong error
[Expired for QEMU because there has been no activity for 60 days.]
** Changed in: qemu
Status: Incomplete => Expired
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/822408
Title:
Unable to ac
[Expired for QEMU because there has been no activity for 60 days.]
** Changed in: qemu
Status: Incomplete => Expired
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/899664
Title:
Bad internet
On 07/07/2017 08:21 PM, Alistair Francis wrote:
As we don't regard error messages as a stable API the let's add a
'error: ' prefix to the original error_report() messages.
Signed-off-by: Alistair Francis
Reviewed-by: Philippe Mathieu-Daudé
---
util/qemu-error.c | 1 +
1 file changed, 1
The real kernel has TASK_SIZE as 0x7c00, due to quirks with
a couple of SH parts. But nominally user-space is limited to 2GB.
Signed-off-by: Richard Henderson
---
target/sh4/cpu.h | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/target/sh4/cpu.h b/target/sh4/cpu.h
in
We had a check using TARGET_VIRT_ADDR_SPACE_BITS to make sure
that the allocation coming in from the command-line option was
not too large, but that didn't include target-specific knowledge
about other restrictions on user-space.
Remove several target-specific hacks in linux-user/main.c.
For MIPS
At first I was simply going to add TARGET_SH to the existing
set of defines that trigger a 31-bit address space. But then
I realized that one could create non-working va configurations
from the command-line.
r~
Richard Henderson (3):
tcg: Fix off-by-one in assert in page_set_flags
linux-us
Most of the users of page_set_flags offset (page, page + len) as
the end points. One might consider this an error, since the other
users do supply an endpoint as the last byte of the region.
However, the first thing that page_set_flags does is round end UP
to the start of the next page. Which me
> "Alistair" == Alistair Francis writes:
Alistair> Convert all uses of error_report("[Ww]arning:"... to use
Alistair> warn_report() instead. This helps standardise on a single
Alistair> method of printing warnings to the user.
Alistair> All of the warnings were changed using this command: fi
On 07/07/2017 05:13 AM, Vladimir Sementsov-Ogievskiy wrote:
> 06.07.2017 20:53, John Snow wrote:
>>
>> On 07/06/2017 04:05 AM, Vladimir Sementsov-Ogievskiy wrote:
>>> 06.07.2017 00:46, John Snow wrote:
On 07/05/2017 05:24 AM, Vladimir Sementsov-Ogievskiy wrote:
> 16.02.2017 16:04, Fam Zh
As we don't regard error messages as a stable API the let's add a
'error: ' prefix to the original error_report() messages.
Signed-off-by: Alistair Francis
---
util/qemu-error.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/util/qemu-error.c b/util/qemu-error.c
index b555247e24..cf1eb8e8b
Convert all uses of error_report*_err("Warning:"... to use
warn_report*_err() instead. This helps standardise on a single
method of printing warnings to the user.
Signed-off-by: Alistair Francis
Cc: Paolo Bonzini
Cc: Richard Henderson
Cc: Eduardo Habkost
Cc: "Michael S. Tsirkin"
Reviewed-by:
When QEMU is waiting for a TCP socket connection it reports that message as
an error. This isn't an error it is just information so let's change the
report to use info_report() instead.
Signed-off-by: Alistair Francis
Reviewed-by: Thomas Huth
Reviewed-by: Philippe Mathieu-Daudé
---
chardev/ch
Convert all uses of error_report("[Ww]arning:"... to use warn_report()
instead. This helps standardise on a single method of printing warnings
to the user.
All of the warnings were changed using this command:
find ./* -type f -exec sed -i \
's|error_report.*[Ww]arning: |warn_report("|g' {}
Signed-off-by: Alistair Francis
Suggested-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 7 ---
hw/i386/pc.c | 9 -
hw/i386/pc_q35.c | 4 ++--
3 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 6b7bade183..f
Add warn_report(), warn_vreport() for reporting warnings, and
info_report(), info_vreport() for informational messages.
These are implemented them with a helper function factored out of
error_vreport(), suitably generalized. This patch makes no changes
to the output of the original error_report()
Implement warn_report_err() and warn_reportf_err() functions which
are the same as the error_report_err() and error_reportf_err()
functions except report a warning instead of an error.
Signed-off-by: Alistair Francis
---
include/qapi/error.h | 11 +++
scripts/checkpatch.pl | 1 +
util
QEMU currently has a standard method to report errors with
error_repot(). This ensure a sane and standard format when printing
errors. This series is attempting to extend this functionality for
warnings and information as well.
This patch renames error_print_loc() function to be more clear, but I
Rename the error_print_loc() function in preparation for using it to
print warnings as well.
Signed-off-by: Alistair Francis
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Thomas Huth
---
util/qemu-error.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/util/qemu-err
On 2017-07-01 13:25, Richard Henderson wrote:
> 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 ++
> 4 files
On 2017-07-01 13:25, Richard Henderson wrote:
> 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.
>
On 2017-07-06 16:21, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/sh4/helper.h| 1 +
> target/sh4/op_helper.c | 16
> target/sh4/translate.c | 2 ++
> 3 files changed, 19 insertions(+)
>
> diff --git a/target/sh4/helper.h b/target/sh4/helper.h
Last time, on DRAGON BALL Q:
On 07/06/2017 10:36 AM, Kashyap Chamarthy wrote:
[A lot of good documentation]
> +
> +So, the following is the flow for `block-commit_Case-3`_ -- to convert a
> +disk image chain such as this::
> +
Maybe drop the "So, "
"The following is the flow for 'Block Commit
On 2017-07-06 16:21, Richard Henderson wrote:
> Both frchg and fschg require PR == 0, otherwise undefined_operation.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 2 ++
> 1 file changed, 2 insertions(+)
Reviewed-by: Aurelien Jarno
--
Aurelien Jarno
On 2017-07-06 16:21, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 5 +
> 1 file changed, 5 insertions(+)
Reviewed-by: Aurelien Jarno
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
aurel...@aurel32.net http://w
On 2017-07-06 16:21, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 64
> +++---
> 1 file changed, 29 insertions(+), 35 deletions(-)
Reviewed-by: Aurelien Jarno
--
Aurelien Jarno
On 2017-07-06 16:21, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 57
> +++---
> 1 file changed, 31 insertions(+), 26 deletions(-)
Reviewed-by: Aurelien Jarno
--
Aurelien Jarno
On 2017-07-06 16:21, Richard Henderson wrote:
> We do not need to emit N copies of raising an exception.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 24 ++--
> 1 file changed, 14 insertions(+), 10 deletions(-)
Reviewed-by: Aurelien Jarno
--
Aurel
On 2017-07-06 16:21, Richard Henderson wrote:
> Now that we have a do_illegal label, use goto in order
> to self-document the forcing of the exception.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 22 +-
> 1 file changed, 13 insertions(+), 9 deletions
On 2017-07-06 16:21, Richard Henderson wrote:
> We do not need to emit N copies of raising an exception.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 14 --
> 1 file changed, 4 insertions(+), 10 deletions(-)
Reviewed-by: Aurelien Jarno
--
Aurelien Jarno
On 2017-07-06 16:21, Richard Henderson wrote:
> We do not need to emit N copies of raising an exception.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 11 +--
> 1 file changed, 5 insertions(+), 6 deletions(-)
>
Reviewed-by: Aurelien Jarno
--
Aurelien Jarno
On 2017-07-06 16:21, Richard Henderson wrote:
> We do not need to form full 64-bit quantities in order to perform
> the move. This reduces code expansion on 64-bit hosts.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 8
> 1 file changed, 4 insertions(+), 4 delet
On 2017-07-06 16:21, Richard Henderson wrote:
> This enforces proper alignment and makes the register update
> more natural. Note that there is a more serious bug fix for
> fmov {DX}Rn,@(R0,Rn) to use a store instead of a load.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c
Hello Jean,
Thanks for the information.
Is someone already working on implementing this, or is this something I can
look into and implementing ?
Also, as Michael mentioned and I looked in the vfio iommu (type1) driver
implementation that it supports mainly
static mappings of user-space process
On Fri, 7 Jul 2017, Paul Durrant wrote:
> > -Original Message-
> > From: Stefano Stabellini [mailto:sstabell...@kernel.org]
> > Sent: 22 June 2017 23:15
> > To: Paul Durrant
> > Cc: xen-de...@lists.xenproject.org; qemu-devel@nongnu.org; qemu-
> > bl...@nongnu.org; Stefano Stabellini ; Anth
On 2017-07-06 16:20, Richard Henderson wrote:
> Also add a debugging assert that we did signal illegal opc
> for odd double-precision registers.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 26 +++---
> 1 file changed, 15 insertions(+), 11 deletions(-
On 2017-07-06 16:20, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 1 -
> 1 file changed, 1 deletion(-)
Reviewed-by: Aurelien Jarno
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
aurel...@aurel32.net http://www.aur
On 2017-07-06 16:20, Richard Henderson wrote:
> Compute which register bank to use once at the start of translation.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 8 +---
> 1 file changed, 5 insertions(+), 3 deletions(-)
Reviewed-by: Aurelien Jarno
--
Aurelien Jar
On 2017-07-06 16:20, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 26 +-
> 1 file changed, 13 insertions(+), 13 deletions(-)
Reviewed-by: Aurelien Jarno
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
aurel
On 2017-07-06 16:20, Richard Henderson wrote:
> We were treating FREG as an index and REG as a TCGv.
> Making FREG return a TCGv is both less confusing and
> a step toward cleaner banking of cpu_fregs.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 125
> +
On 2017-07-06 16:20, Richard Henderson wrote:
> Compute which register bank to use once at the start of translation.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 21 +++--
> 1 file changed, 11 insertions(+), 10 deletions(-)
Reviewed-by: Aurelien Jarno
On 2017-07-06 16:20, Richard Henderson wrote:
> If we mask off any out-of-band bits before we assign to the
> variable, then we don't need to clean it up when reading.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/cpu.h | 2 +-
> target/sh4/cpu.c | 2 +-
> 2 files changed, 2 insertions
On 2017-07-06 16:20, Richard Henderson wrote:
> We'll be putting more things into this bitmask soon.
> Let's have a name that covers all possible uses.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/cpu.h | 4 +++-
> target/sh4/translate.c | 4 ++--
> 2 files changed, 5 insertions
On 2017-07-06 16:20, Richard Henderson wrote:
> Don't leave an unused bit after DELAY_SLOT_MASK.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/cpu.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
Reviewed-by: Aurelien Jarno
--
Aurelien Jarno
On 2017-07-06 16:20, Richard Henderson wrote:
> We can fold 3 different tests within the decode loop
> into a more accurate computation of max_insns to start.
>
> Signed-off-by: Richard Henderson
> ---
> target/sh4/translate.c | 29 +
> 1 file changed, 17 insertions(+
object_resolve_path*() ambiguous path detection breaks when
ambiguous==NULL and the object tree have 3 objects of the same type and
only 2 of them are under the same parent. e.g.:
/container/obj1 (TYPE_FOO)
/container/obj2 (TYPE_FOO)
/obj2 (TYPE_FOO)
With the above tree, object_resolve_path_t
Test for partial path lookup using object_resolve_path*().
Signed-off-by: Eduardo Habkost
---
tests/check-qom-proplist.c | 41 +
1 file changed, 41 insertions(+)
diff --git a/tests/check-qom-proplist.c b/tests/check-qom-proplist.c
index 8e432e9..abafbd7 1
object_resolve_path*() ambiguous path detection breaks when
ambiguous==NULL and the object tree have 3 objects of the same type and
only 2 of them are under the same parent. e.g.:
/container/obj1 (TYPE_FOO)
/container/obj2 (TYPE_FOO)
/obj2 (TYPE_FOO)
With the above tree, object_resolve_path_t
On 07/07/2017 09:30 AM, Pavel Butsykin wrote:
> This patch add shrinking of the image file for qcow2. As a result, this allows
> us to reduce the virtual image size and free up space on the disk without
> copying the image. Image can be fragmented and shrink is done by punching
> holes
> in the im
Anyway, this was more of an experiment than a serious fix proposal.
I believe the general idea is solid but it'll get a couple of spins to
get a
good patch for pushing. I'll send the POC as a RFC to get things
started and
we'll see what happens.
I've sent the POC as RFC here:
"[RFC drcVI P
On 07/07/2017 10:15 AM, Daniel P. Berrange wrote:
> The semantics around handling ipv4=on|off & ipv6=on|off are quite
> subtle to understand in combination with the various hostname addresses
> and backend types. Introduce a massive test matrix that launches QEMU
> and validates the ability to conn
"spapr: Remove 'awaiting_allocation' DRC flag" removed the flag that
was originally was being used to prevent a race condition between
hot unplug and hotplug. The DRC code base got simplified and more
robust over time, eliminating the conditions that led to this race.
Thus the awaiting_allocation e
On 07/07/2017 09:30 AM, Pavel Butsykin wrote:
> The flag as additional precaution of data loss. Perhaps in the future the
s/as/is/ s/of/against/
> operation shrink without this flag will be blocked for all formats, but while
s/while/for now/
> we need to maintain compatibility with raw.
>
> Si
On 07/07/2017 06:46 AM, Kashyap Chamarthy wrote:
> Thanks for your patience in review. Much appreciated (including the
> grammar corrections)!
>
> I'll wait for the rest of your comments before I spin a v6.
>
> [...]
Thank *you* for exercising the necessary patience to make incredibly
nice d
On 07/07/2017 09:53 AM, Markus Armbruster wrote:
> Vladimir Sementsov-Ogievskiy writes:
>
>> 07.07.2017 12:00, Markus Armbruster wrote:
>>> "Daniel P. Berrange" writes:
>>>
On Fri, Jul 07, 2017 at 10:05:22AM +0200, Markus Armbruster wrote:
> Vladimir Sementsov-Ogievskiy writes:
>
On 06/23/2017 08:02 AM, Tomáš Golembiovský wrote:
> This is just a draft, or a request for comments if you will.
>
> This patch sets drafts the support of sending events by QEMU Guest Agent.
> Events can plan important role in monitoring of the guest OS behaviour. The
> range of use cases ranges f
On 06/23/2017 08:02 AM, Tomáš Golembiovský wrote:
> Events can play an integral role when monitoring internal state of the
> guest OS. This patch adds the core functionality for adding events to
> QEMU Guest Agent.
Will sending events confuse existing clients that aren't expecting them?
Do we need
On 07/07/2017 03:30 PM, Eric Blake wrote:
> The NBD protocol has several constants defined in various extensions
> that we are about to implement. Expose them to the code, along with
> an easy way to map various constants to strings during diagnostic
> messages.
>
> Signed-off-by: Eric Blake
>
Hi,
On Sun, 25 Jun 2017 23:25:17 +0200
Tomáš Golembiovský wrote:
> Hi,
>
> On Fri, 23 Jun 2017 13:25:34 +
> Marc-André Lureau wrote:
>
> > Hi
> >
> > On Fri, Jun 23, 2017 at 3:03 PM Tomáš Golembiovský
> > wrote:
> >
> > > This is just a draft, or a request for comments if you will.
>
On 07/07/2017 03:30 PM, Eric Blake wrote:
> NBD_OPT_EXPORT_NAME is lousy: per the NBD protocol, any failure
> requires us to close the connection rather than report an error.
> Therefore, upstream NBD recently added NBD_OPT_GO as the improved
> version of the option that does what we want [1], alon
The upstream NBD Protocol has defined a new extension to allow
the server to advertise block sizes to the client, as well as
a way for the client to inform the server whether it intends to
obey block sizes.
When using the block layer as the client, we will obey block
sizes; but when used as 'qemu-
NBD_OPT_EXPORT_NAME is lousy: per the NBD protocol, any failure
requires us to close the connection rather than report an error.
Therefore, upstream NBD recently added NBD_OPT_GO as the improved
version of the option that does what we want [1], along with
NBD_OPT_INFO that returns the same informat
The NBD Protocol is introducing some additional information
about exports, such as minimum request size and alignment, as
well as an advertised maximum request size. It will be easier
to feed this information back to the block layer if we gather
all the information into a struct, rather than addin
The upstream NBD Protocol has defined a new extension to allow
the server to advertise block sizes to the client, as well as
a way for the client to inform the server that it intends to
obey block sizes.
Thanks to a recent fix (commit df7b97ff), our real minimum
transfer size is always 1 (the bloc
The NBD protocol has several constants defined in various extensions
that we are about to implement. Expose them to the code, along with
an easy way to map various constants to strings during diagnostic
messages.
Signed-off-by: Eric Blake
---
v5: rebase to master, add command name lookup; enoug
NBD_OPT_EXPORT_NAME is lousy: per the NBD protocol, any failure
requires the server to close the connection rather than report an
error to us. Therefore, upstream NBD recently added NBD_OPT_GO as
the improved version of the option that does what we want [1]: it
reports sane errors on failures, and
Simplify the tracing of client flags in the server, and return -EINVAL
instead of -EIO if we successfully read but don't like those flags.
Signed-off-by: Eric Blake
---
v5: new patch
---
nbd/server.c | 6 ++
nbd/trace-events | 4 +---
2 files changed, 3 insertions(+), 7 deletions(-)
di
We really don't care if our spec-compliant reply to NBD_OPT_ABORT
was received, so shave off some lines of code by not even tracing it.
Signed-off-by: Eric Blake
---
v5: new patch
---
nbd/server.c | 15 ---
1 file changed, 4 insertions(+), 11 deletions(-)
diff --git a/nbd/server.c
Reply directly in nbd_negotiate_handle_export_name(), rather than
waiting until nbd_negotiate_options() completes. This will make it
easier to implement NBD_OPT_GO. Pass additional parameters around,
rather than stashing things inside NBDClient.
Signed-off-by: Eric Blake
---
v5: new patch
---
The NBD protocol has now finalized the extension that fixes
several shortcomings with NBD_OPT_EXPORT_NAME (namely, no error
reporting, no way for the server to advertise block sizes to the
client):
https://github.com/NetworkBlockDevice/nbd/blob/extension-info/doc/proto.md
This sort of overlaps wit
On 07/07/17 21:54, Eduardo Habkost wrote:
> On Fri, Jul 07, 2017 at 04:30:29PM -0300, Eduardo Habkost wrote:
>> On Fri, Jul 07, 2017 at 08:18:20PM +0200, Igor Mammedov wrote:
>>> On Fri, 7 Jul 2017 12:09:56 -0300
>>> Eduardo Habkost wrote:
>>>
On Fri, Jul 07, 2017 at 04:58:17PM +0200, Igor Ma
On Fri, Jul 07, 2017 at 04:30:29PM -0300, Eduardo Habkost wrote:
> On Fri, Jul 07, 2017 at 08:18:20PM +0200, Igor Mammedov wrote:
> > On Fri, 7 Jul 2017 12:09:56 -0300
> > Eduardo Habkost wrote:
> >
> > > On Fri, Jul 07, 2017 at 04:58:17PM +0200, Igor Mammedov wrote:
> > > > On Fri, 7 Jul 2017 10
On Fri, Jul 07, 2017 at 08:18:20PM +0200, Igor Mammedov wrote:
> On Fri, 7 Jul 2017 12:09:56 -0300
> Eduardo Habkost wrote:
>
> > On Fri, Jul 07, 2017 at 04:58:17PM +0200, Igor Mammedov wrote:
> > > On Fri, 7 Jul 2017 10:13:00 -0300
> > > "Eduardo Habkost" wrote:
> > [...]
> > > > I don't disagr
From: Vladimir Sementsov-Ogievskiy
[eblake minor tweaks to a couple of traces]
Signed-off-by: Eric Blake
---
If you are okay with this, I plan to squash this into your patch 10
when I send my pull request
nbd/client.c | 3 +--
nbd/server.c | 5 +++--
nbd/trace-events | 4 +---
3 files
On 07/07/2017 07:57 AM, Richard Henderson wrote:
+/* ??? The SH4 kernel checks for and address above 0xC000.
+ However, the page mappings in qemu linux-user aren't as restricted
+ and we wind up with the normal stack mapped above 0xF000.
+ That said, there is no reas
On 07/07/2017 08:29 AM, Alex Bennée wrote:
Naming all of these different exit conditions is certainly
non-trivial.
Given the variation of usage this is something that should probably be
done after Lluís common run loop goes in and we can beef up the
semantics of the various exit conditions.
D
From: Anoob Soman
xen_pt_pci_config_access_check checks if addr >= 0xFF. 0xFF is a valid
address and should not be ignored.
Signed-off-by: Anoob Soman
Acked-by: Anthony PERARD
Signed-off-by: Stefano Stabellini
---
hw/xen/xen_pt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --
We are promising more than just odd fixes, and Paolo is hoping
to offload the pull requests to me. Also, enough of NBD is related
to the block layer that it is worth including qemu-block on patches.
Signed-off-by: Eric Blake
---
If there are no objections, I'll include this patch and Vladimir's
On Fri, 7 Jul 2017 12:09:56 -0300
Eduardo Habkost wrote:
> On Fri, Jul 07, 2017 at 04:58:17PM +0200, Igor Mammedov wrote:
> > On Fri, 7 Jul 2017 10:13:00 -0300
> > "Eduardo Habkost" wrote:
> [...]
> > > I don't disagree with adding the assert(), but it looks like
> > > making fw_cfg_find() retur
On 07/07/2017 07:36 AM, Peter Maydell wrote:
On 7 July 2017 at 18:32, Alex Bennée wrote:
Given the wording of:
/* is_jmp field values */
#define DISAS_NEXT0 /* next instruction can be analyzed */
#define DISAS_JUMP1 /* only pc was modified dynamically */
#define DISAS_UPDATE 2 /* cpu
On 07/07/2017 01:56 AM, Lluís Vilanova wrote:
+void translator_loop(const TranslatorOps *ops, DisasContextBase *db,
+ CPUState *cpu, TranslationBlock *tb)
+{
+int max_insns;
+
+/* Initialize DisasContext */
+db->tb = tb;
+db->pc_first = tb->pc;
+db->pc_next
On 07/07/2017 07:33 AM, Peter Maydell wrote:
On 7 July 2017 at 16:26, Richard Henderson wrote:
That is the purpose of DISAS_EXC too. We've called a noreturn helper to
raise an exception and all following code is unreached. If there *was* any
code being emitted afterward, that is arguably a bu
From: Ross Lagerwall
When the guest unplugs the emulated NICs, cleanup the peer for each NIC
as it is not needed anymore. Most importantly, this allows the tap
interfaces which QEMU holds open to be closed and removed.
Signed-off-by: Ross Lagerwall
Acked-by: Anthony PERARD
Signed-off-by: Stefa
m.git tags/xen-20170707-tag
for you to fetch changes up to 4daf62594d13dfca2ce3a74dd3bddee5f54d7127:
xen/pt: Fixup addr validation in xen_pt_pci_config_access_check (2017-07-07
11:13:10 -0700)
Xen
On 07/07/2017 07:32 AM, Lluís Vilanova wrote:
That said, even for AArch32 we know by the TB flags whether or not we're going
to be generating arm or thumb code. I think that these hooks will allow arm and
thumb mode to finally be split apart cleanly, instead of the tangle that they
are now.
I s
Initialize xenfb properly, as all other backends, from its own
"initialise" function.
Remove the dependency of vkbd on vfb: use qemu_console_lookup_by_index
to find the principal console (to get the size of the screen) instead of
relying on a vfb backend to be available (which adds a dependency
be
On 7 July 2017 at 18:32, Alex Bennée wrote:
> Given the wording of:
>
> /* is_jmp field values */
> #define DISAS_NEXT0 /* next instruction can be analyzed */
> #define DISAS_JUMP1 /* only pc was modified dynamically */
> #define DISAS_UPDATE 2 /* cpu state was modified dynamically */
> #
Richard Henderson writes:
> On 07/07/2017 07:36 AM, Peter Maydell wrote:
>> On 7 July 2017 at 18:32, Alex Bennée wrote:
>>> Given the wording of:
>>>
>>> /* is_jmp field values */
>>> #define DISAS_NEXT0 /* next instruction can be analyzed */
>>> #define DISAS_JUMP1 /* only pc was modif
Alex Bennée writes:
> Previously DISAS_JUMP did ensure this but with the optimisation of
> 8a6b28c7 (optimize indirect branches) we might not leave the loop.
> This means if any pending interrupts are cleared by changing IRQ flags
> we might never get around to servicing them. You usually notice
On Fri, Jul 07, 2017 at 06:17:57PM +0300, Michael S. Tsirkin wrote:
> On Fri, Jul 07, 2017 at 10:39:49AM -0300, Eduardo Habkost wrote:
> > On Wed, Jul 05, 2017 at 12:32:10PM +0300, Marcel Apfelbaum wrote:
> > > On 05/07/2017 11:14, Thomas Huth wrote:
> > > > Hi,
> > > >
> > >
> > > Hi,
> > >
>
Bah. Should have gone to the list as well.
r~
-- Forwarded message --
From: Richard Henderson
Date: Fri, Jul 7, 2017 at 6:38 AM
Subject: [PATCH v2.5] fixup! linux-user/sh4: Notice gUSA regions during
signal delivery
To: glaub...@physik.fu-berlin.de
Cc: laur...@vivier.eu
This fi
From: Vladimir Sementsov-Ogievskiy
We should release them here to reload on invalidate cache.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Max Reitz
Message-id: 20170628120530.31251-31-vsement...@virtuozzo.com
Signed-off-by: Max Reitz
---
block.c | 4
b
From: "Daniel P. Berrange"
While the qemu-img dd command does accept --image-opts
this is not sufficient to make it work with the LUKS
image yet. This is because bdrv_create() still always
requires the non-image-opts syntax.
Thus we must skip 159/170 with luks for now
Reviewed-by: Max Reitz
Re
On 07/07/2017 07:18 AM, Lluís Vilanova wrote:
There was no code being generated after this specific case, but I haven't
checked if DISAS_EXC is set in any other place that is not immediately followed
by a "goto done_generating".
Typically we haven't actually done a goto, but simply exit the loo
On Fri, Jul 7, 2017 at 5:06 AM, Eduardo Habkost wrote:
> On Thu, Jul 06, 2017 at 04:49:44PM -0700, Alistair Francis wrote:
>> Convert all uses of error_report("[Ww]arning:"... to use warn_report()
>> instead. This helps standardise on a single method of printing warnings
>> to the user.
>>
>> All
From: Vladimir Sementsov-Ogievskiy
Interface for removing persistent bitmap from its storage.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Max Reitz
Reviewed-by: John Snow
Message-id: 20170628120530.31251-28-vsement...@virtuozzo.com
Signed-off-by: Max Reitz
---
block/dirty-bitma
Richard Henderson writes:
> On 07/07/2017 01:18 AM, Lluís Vilanova wrote:
diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c
index 9c870f6d07..586a01a2de 100644
--- a/target/arm/translate-a64.c
+++ b/target/arm/translate-a64.c
@@ -11244,6 +11244,9 @@ stat
Emilio G Cota writes:
> On Fri, Jul 07, 2017 at 05:46:19 -1000, Richard Henderson wrote:
>> I do wonder if we should provide a generic empty hook, so that a target that
>> does not need a particular hook need not define an empty function. It could
>> just put e.g. "translator_noop" into the struc
On 7 July 2017 at 16:26, Richard Henderson wrote:
> That is the purpose of DISAS_EXC too. We've called a noreturn helper to
> raise an exception and all following code is unreached. If there *was* any
> code being emitted afterward, that is arguably a bug.
One exception to that is a conditional
From: Vladimir Sementsov-Ogievskiy
Store bitmaps and mark them read-only on reopening image as read-only.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Reviewed-by: Max Reitz
Message-id: 20170628120530.31251-21-vsement...@virtuozzo.com
Signed-off-by: Max Reitz
---
block/qcow2-bitmap.c | 22 +++
1 - 100 of 482 matches
Mail list logo
