[Qemu-devel] [PATCH v2 23/27] target/sh4: Introduce CHECK_SH4A

2017-07-06 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/sh4/translate.c | 64 +++--- 1 file changed, 29 insertions(+), 35 deletions(-) diff --git a/target/sh4/translate.c b/target/sh4/translate.c index 3d8ac59..d164e62 100644 --- a/target/sh4/translate.c +++ b/tar

[Qemu-devel] [PATCH v2 12/27] target/sh4: Pass DisasContext to fpr64 routines

2017-07-06 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/sh4/translate.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/target/sh4/translate.c b/target/sh4/translate.c index b521cff..878c0bd 100644 --- a/target/sh4/translate.c +++ b/target/sh4/translate.c @@ -

[Qemu-devel] [PATCH v2 19/27] target/sh4: Unify code for CHECK_PRIVILEGED

2017-07-06 Thread Richard Henderson
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(-) diff --git a/target/sh4/translate.c b/target/sh4/translate.c index 41157a0..dd14b43 100644 --- a/target/sh4/

[Qemu-devel] [PATCH v2 18/27] target/sh4: Unify code for CHECK_NOT_DELAY_SLOT

2017-07-06 Thread Richard Henderson
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(-) diff --git a/target/sh4/translate.c b/target/sh4/translate.c index 3453f19..41157a0 100644 --- a/target/sh4/tran

[Qemu-devel] [PATCH v2 24/27] target/sh4: Implement fpchg

2017-07-06 Thread Richard Henderson
Signed-off-by: Richard Henderson --- target/sh4/translate.c | 5 + 1 file changed, 5 insertions(+) diff --git a/target/sh4/translate.c b/target/sh4/translate.c index d164e62..35a5c91 100644 --- a/target/sh4/translate.c +++ b/target/sh4/translate.c @@ -492,6 +492,11 @@ static void _decode_opc

[Qemu-devel] [PATCH v2 21/27] target/sh4: Tidy misc illegal insn checks

2017-07-06 Thread Richard Henderson
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(-) diff --git a/target/sh4/translate.c b/target/sh4/transl

[Qemu-devel] [PATCH v2 25/27] target/sh4: Add missing FPSCR.PR == 0 checks

2017-07-06 Thread Richard Henderson
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(+) diff --git a/target/sh4/translate.c b/target/sh4/translate.c index 35a5c91..2b62e39 100644 --- a/target/sh4/translate.c +++ b/

[Qemu-devel] [PATCH v2 26/27] target/sh4: Implement fsrra

2017-07-06 Thread Richard Henderson
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 index 6c6fa04..ea92dc0 100644 --- a/target/sh4/helper.h +++ b

[Qemu-devel] [PATCH v2.2 3/4] doc: add item for "-M enforce-config-section"

2017-07-06 Thread Peter Xu
It's never documented, and now we have one more parameter for it (which obsoletes this one). Document it properly. Although now when enforce-config-section is set, it'll override the other "-global" parameter, that is not necessarily a rule. Forbid that usage in the document. Suggested-by: Eduard

Re: [Qemu-devel] [PATCH v4 19/21] block: Make bdrv_is_allocated() byte-based

2017-07-06 Thread Eric Blake
On 07/06/2017 11:02 AM, Kevin Wolf wrote: >> +++ b/qemu-img.c >> @@ -3229,6 +3229,7 @@ static int img_rebase(int argc, char **argv) >> int64_t new_backing_num_sectors = 0; >> uint64_t sector; >> int n; >> +int64_t count; >> float local_progress = 0; >> >

[Qemu-devel] [PATCH] tcg/mips: Bugfix for crash when running program with qemu-i386.

2017-07-06 Thread Jiang Biao
When running a helloworld program with qemu-i386 in linux-user mode on Loongson 3A3000, it will crash. This patch fix the bug. Signed-off-by: Jiang Biao --- tcg/mips/tcg-target.inc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tcg/mips/tcg-target.inc.c b/tcg/mips/tcg-targ

Re: [Qemu-devel] [PATCH] tcg/mips: Bugfix for crash when running program with qemu-i386.

2017-07-06 Thread Richard Henderson
On 07/06/2017 05:15 PM, Jiang Biao wrote: When running a helloworld program with qemu-i386 in linux-user mode on Loongson 3A3000, it will crash. This patch fix the bug. Signed-off-by: Jiang Biao --- tcg/mips/tcg-target.inc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/

[Qemu-devel] 答复: Re: [PATCH] tcg/mips: Bugfix for crash when runningprogram with qemu-i386.

2017-07-06 Thread jiang.biao2
> > This is wrong, because you're not allowed to modify the input operands. > > Try this, just a few lines lower in the function: > > -tcg_out_movi(s, TCG_TYPE_PTR, base, guest_base) > -tcg_out_opc_reg(s, ALIAS_PADD, base, base, addr_regl) > +tcg_out_movi(s, TCG_TYPE_PT

[Qemu-devel] [PATCH 0/4] Optimize COLO-compare performance

2017-07-06 Thread Zhang Chen
In this serise, we do a lot of job to optimize COLO net performance. Mainly focus on TCP protocol. Zhang Chen (4): net/colo-compare.c: Add checkpoint min period to optimize performance net/colo-compare.c: compare tcp packet base on sequence number net/colo-compare.c: Optimize unpredictable

[Qemu-devel] [PATCH 3/4] net/colo-compare.c: Optimize unpredictable tcp options comparison

2017-07-06 Thread Zhang Chen
When network is busy, some tcp options(like sack) will unpredictable occur in primary side or secondary side. it will make packet size not same, but the two packet's payload is identical. colo just care about packet payload, so we skip the option field. Signed-off-by: Zhang Chen --- net/colo-com

[Qemu-devel] [PATCH 2/4] net/colo-compare.c: compare tcp packet base on sequence number

2017-07-06 Thread Zhang Chen
If primary packet's sequence number not same with secondary packet's sequence number, no need to compare the packet other field. Signed-off-by: Zhang Chen --- net/colo-compare.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/net/colo-compare.c b/net/colo-compare.c index 0f8e198..2caeb

[Qemu-devel] [PATCH 4/4] net/colo-compare.c: Adjust net queue pop order for performance

2017-07-06 Thread Zhang Chen
The packet_enqueue() use g_queue_push_tail() to enqueue net packet, so it is more efficent way use g_queue_pop_head() to get packet for compare. That will improve the success rate of comparison. In my test the performance of ftp put 1000M file will increase 10% Signed-off-by: Zhang Chen --- net/

[Qemu-devel] [PATCH 1/4] net/colo-compare.c: Add checkpoint min period to optimize performance

2017-07-06 Thread Zhang Chen
If colo-compare find out the first different packet that means the following packet almost is different. we needn't do a lot of checkpoint in this time, so we set the no-need-checkpoint peroid, default just set 3 second. Signed-off-by: Zhang Chen --- net/colo-compare.c | 13 - 1 file

Re: [Qemu-devel] [PATCH 1/4] net/colo-compare.c: Add checkpoint min period to optimize performance

2017-07-06 Thread Li Zhijian
On 07/07/2017 01:43 PM, Zhang Chen wrote: If colo-compare find out the first different packet at this situation, why not trigger the checkpoint, and then flush all primary packets and drop secondary packets? that means the following packet almost is different. we needn't do a lot of chec

Re: [Qemu-devel] [PATCH v7 0/6] fsdev: qmp interface for io throttling

2017-07-06 Thread Markus Armbruster
Pradeep Jagadeesh writes: > These patches provide the qmp interface, to query the io throttle > status of the all fsdev devices that are present in a vm. > also, it provides an interface to set the io throttle parameters of a > fsdev to a required value. some of the patches also remove the dupli

Re: [Qemu-devel] [PATCH 2/4] net/colo-compare.c: compare tcp packet base on sequence number

2017-07-06 Thread Li Zhijian
this title confused me i think you want to express "only compare the packets that has the same sequence number" and i think this optimization can not reduce this checkpoint optimization. Thanks On 07/07/2017 01:43 PM, Zhang Chen wrote: If primary packet's sequence number not same with secon

Re: [Qemu-devel] [RFC v2 0/8] VIRTIO-IOMMU device

2017-07-06 Thread Tian, Kevin
> From: Jean-Philippe Brucker [mailto:jean-philippe.bruc...@arm.com] > Sent: Wednesday, July 5, 2017 8:45 PM > > On 05/07/17 08:14, Tian, Kevin wrote: > >> From: Jean-Philippe Brucker [mailto:jean-philippe.bruc...@arm.com] > >> Sent: Monday, June 19, 2017 6:15 PM > >> > >> On 19/06/17 08:54, Bhara

Re: [Qemu-devel] [PATCH v7 0/6] fsdev: qmp interface for io throttling

2017-07-06 Thread Markus Armbruster
PS: Sorry for the late review, got a bit overwhelmed...

Re: [Qemu-devel] [RFC v2 0/8] VIRTIO-IOMMU device

2017-07-06 Thread Bharat Bhushan
Hi Eric, > -Original Message- > From: Auger Eric [mailto:eric.au...@redhat.com] > Sent: Friday, July 07, 2017 2:47 AM > To: Bharat Bhushan ; Jean-Philippe Brucker > ; eric.auger@gmail.com; > peter.mayd...@linaro.org; alex.william...@redhat.com; m...@redhat.com; > qemu-...@nongnu.org; q

Re: [Qemu-devel] [PATCH 1/4] net/colo-compare.c: Add checkpoint min period to optimize performance

2017-07-06 Thread Zhang Chen
On 07/07/2017 02:06 PM, Li Zhijian wrote: On 07/07/2017 01:43 PM, Zhang Chen wrote: If colo-compare find out the first different packet at this situation, why not trigger the checkpoint, and then flush all primary packets and drop secondary packets? In this time we already do this job, b

Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report()

2017-07-06 Thread Thomas Huth
On 07.07.2017 01:49, 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 of the warnings were found using this regex expression: > error_report.*[Ww]arn

Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report()

2017-07-06 Thread Greg Kurz
On Thu, 6 Jul 2017 16:49:44 -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 of the warnings were found using this regex expression: > error_

Re: [Qemu-devel] [PATCH 2/4] net/colo-compare.c: compare tcp packet base on sequence number

2017-07-06 Thread Zhang Chen
On 07/07/2017 02:20 PM, Li Zhijian wrote: this title confused me i think you want to express "only compare the packets that has the same sequence number" OK, I will fix it in next version. and i think this optimization can not reduce this checkpoint optimization. In my test it can r

Re: [Qemu-devel] [RFC 5/5] vifo: introduce new VFIO ioctl VFIO_DEVICE_PCI_GET_DIRTY_BITMAP

2017-07-06 Thread Tian, Kevin
> From: Alex Williamson [mailto:alex.william...@redhat.com] > Sent: Saturday, July 1, 2017 1:00 AM > > On Fri, 30 Jun 2017 05:14:40 + > "Tian, Kevin" wrote: > > > > From: Alex Williamson [mailto:alex.william...@redhat.com] > > > Sent: Friday, June 30, 2017 4:57 AM > > > > > > On Thu, 29 Jun

Re: [Qemu-devel] [PATCH v14] migration: spapr: migrate pending_events of spapr state

2017-07-06 Thread David Gibson
On Tue, Jun 20, 2017 at 03:40:17PM -0300, Daniel Henrique Barboza wrote: > In racing situations between hotplug events and migration operation, > a rtas hotplug event could have not yet be delivered to the source > guest when migration is started. In this case the pending_events of > spapr state ne

Re: [Qemu-devel] [RFC 1/1] qemu-ga: add missing libpcre to MSI build

2017-07-06 Thread Thomas Lamprecht
Hi, On 06/02/2017 01:42 PM, Marc-André Lureau wrote: Hi On Thu, Jun 1, 2017 at 5:08 PM Thomas Lamprecht wrote: glib depends on libpcre which was not shipped with the MSI, thus starting of the qemu-ga.exe failed with the respective error message. Tell WIXL to ship this library with the MSI to

Re: [Qemu-devel] [PATCH v2.2 3/4] doc: add item for "-M enforce-config-section"

2017-07-06 Thread Greg Kurz
On Fri, 7 Jul 2017 10:54:08 +0800 Peter Xu wrote: > It's never documented, and now we have one more parameter for it (which > obsoletes this one). Document it properly. > > Although now when enforce-config-section is set, it'll override the > other "-global" parameter, that is not necessarily a

Re: [Qemu-devel] [Qemu devel v6 PATCH 1/5] msf2: Add Smartfusion2 System timer

2017-07-06 Thread sundeep subbaraya
Hi Alistair, On Wed, Jul 5, 2017 at 11:26 PM, Alistair Francis wrote: > On Sun, Jul 2, 2017 at 9:45 PM, Subbaraya Sundeep > wrote: > > Modelled System Timer in Microsemi's Smartfusion2 Soc. > > Timer has two 32bit down counters and two interrupts. > > > > Signed-off-by: Subbaraya Sundeep > > -

<    1   2   3   4