Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move / Xen on Xen nested virt

2013-02-20 Thread Ian Jackson
Pasi Kärkkäinen writes (Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move / Xen on Xen nested virt): Ping again? I wonder if I've gotten into some blacklist already for nagging about this.. This is a required patch for making Xen-on-Xen Nested virt working

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move / Xen on Xen nested virt

2013-02-20 Thread Pasi Kärkkäinen
On Wed, Feb 20, 2013 at 03:42:15PM +, Ian Jackson wrote: Pasi Kärkkäinen writes (Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move / Xen on Xen nested virt): Ping again? I wonder if I've gotten into some blacklist already for nagging about

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move / Xen on Xen nested virt

2013-02-19 Thread Pasi Kärkkäinen
and cpu_ioreq_move Ian Campbell writes (Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move): On Fri, 2012-12-07 at 16:14 +, Ian Jackson wrote: +target_phys_addr_t offset = (target_phys_addr_t)req-size * i; +if (req-df) addr

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move / Xen on Xen nested virt

2013-01-30 Thread Xu, Dongxiao
[mailto:ian.jack...@eu.citrix.com] Sent: Saturday, December 08, 2012 12:34 AM To: Ian Campbell Cc: Stefano Stabellini; Xu, Dongxiao; xen-de...@lists.xensource.com; qemu-devel@nongnu.org Subject: Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move / Xen on Xen nested virt

2013-01-24 Thread Pasi Kärkkäinen
: simplify cpu_ioreq_pio and cpu_ioreq_move Ian Campbell writes (Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move): On Fri, 2012-12-07 at 16:14 +, Ian Jackson wrote: +target_phys_addr_t offset = (target_phys_addr_t)req-size * i

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2013-01-08 Thread Pasi Kärkkäinen
-devel@nongnu.org Subject: Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move Ian Campbell writes (Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move): On Fri, 2012-12-07 at 16:14 +, Ian Jackson wrote

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2013-01-07 Thread Xu, Dongxiao
-Original Message- From: Xu, Dongxiao Sent: Tuesday, December 11, 2012 1:50 PM To: Ian Jackson; Ian Campbell Cc: Stefano Stabellini; xen-de...@lists.xensource.com; qemu-devel@nongnu.org Subject: RE: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2012-12-10 Thread Stefano Stabellini
On Fri, 7 Dec 2012, Ian Jackson wrote: Stefano Stabellini writes ([Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move): after reviewing the patch fix multiply issue for int and uint types with Ian Jackson, we realized that cpu_ioreq_pio and cpu_ioreq_move

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2012-12-10 Thread Ian Jackson
Stefano Stabellini writes (Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move): On Fri, 7 Dec 2012, Ian Jackson wrote: ... +if (req-df) addr -= offset; +else addr -= offset; This can't be right, can it? Indeed not. v2 has this fixed. The search

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2012-12-10 Thread Xu, Dongxiao
cpu_ioreq_pio and cpu_ioreq_move Ian Campbell writes (Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move): On Fri, 2012-12-07 at 16:14 +, Ian Jackson wrote: +target_phys_addr_t offset = (target_phys_addr_t)req-size * i; +if (req-df) addr -= offset

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2012-12-07 Thread Ian Jackson
Stefano Stabellini writes ([Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move): after reviewing the patch fix multiply issue for int and uint types with Ian Jackson, we realized that cpu_ioreq_pio and cpu_ioreq_move are in much need for a simplification as well

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2012-12-07 Thread Ian Jackson
Ian Campbell writes (Re: [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move): On Fri, 2012-12-07 at 16:14 +, Ian Jackson wrote: +target_phys_addr_t offset = (target_phys_addr_t)req-size * i; +if (req-df) addr -= offset; +else addr -= offset

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2012-12-07 Thread Ian Campbell
On Fri, 2012-12-07 at 16:14 +, Ian Jackson wrote: +target_phys_addr_t offset = (target_phys_addr_t)req-size * i; +if (req-df) addr -= offset; +else addr -= offset; One of these -= should be a += I presume? [...] +write_phys_req_item((target_phys_addr_t)

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2012-11-29 Thread Ian Campbell
On Thu, 2012-11-29 at 13:21 +, Stefano Stabellini wrote: Also I think 4.2.1 need these patches to enable the basic Xen on Xen nested virtualization usage scenario. I agree. Nested virt was a tech preview in 4.2.0, is it really worth backporting ? Ian.

Re: [Qemu-devel] [Xen-devel] [PATCH 0/2] QEMU/xen: simplify cpu_ioreq_pio and cpu_ioreq_move

2012-11-29 Thread Pasi Kärkkäinen
On Thu, Nov 29, 2012 at 01:57:11PM +, Ian Campbell wrote: On Thu, 2012-11-29 at 13:21 +, Stefano Stabellini wrote: Also I think 4.2.1 need these patches to enable the basic Xen on Xen nested virtualization usage scenario. I agree. Nested virt was a tech preview in 4.2.0, is