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

[Ian Campbell]

On Fri, 2012-12-07 at 16:14 +0000, 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) req->data, req, i, 
> &tmp);

This seems to be the only one with this cast, why?

write_phys_req_item takes a target_phys_addr_t so this will happen
regardless I think.

Ian.