[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [v3][PATCH 1/5] xen, gfx passthrough: basic graphics pa
From: |
Chen, Tiejun |
Subject: |
Re: [Qemu-devel] [v3][PATCH 1/5] xen, gfx passthrough: basic graphics passthrough support |
Date: |
Wed, 28 May 2014 01:43:25 +0000 |
> -----Original Message-----
> From: Stefano Stabellini [mailto:address@hidden
> Sent: Wednesday, May 28, 2014 1:48 AM
> To: Chen, Tiejun
> Cc: address@hidden; address@hidden;
> address@hidden; address@hidden; address@hidden;
> address@hidden; address@hidden;
> address@hidden; Kay, Allen M; Zhang, Yang Z
> Subject: Re: [v3][PATCH 1/5] xen, gfx passthrough: basic graphics passthrough
> support
>
[snip]
> > + * register VGA resources for the domain with assigned gfx */ int
> > +xen_pt_register_vga_regions(XenHostPCIDevice *dev) {
> > + int i = 0;
> > +
> > + if (!is_vga_passthrough(dev)) {
> > + return -1;
>
> Given that xen_pt_register_vga_regions is called unconditionally, this is not
> an
> error condition, isn't it? It should return 0.
Okay.
>
[snip]
> > + * unregister VGA resources for the domain with assigned gfx */ int
> > +xen_pt_unregister_vga_regions(XenHostPCIDevice *dev) {
> > + int i = 0;
> > +
> > + if (!is_vga_passthrough(dev)) {
> > + return -1;
>
> same here
>
Fixed.
>
[snip]
> > const ram_addr_t default_ram_size =
> (ram_addr_t)DEFAULT_RAM_SIZE *
> > 1024 * 1024;
> > -
>
> Spurious change
>
Removed.
Thanks
Tiejun
- [Qemu-devel] [v3][PATCH 0/5] xen: add Intel IGD passthrough support, Tiejun Chen, 2014/05/26
- [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D, Tiejun Chen, 2014/05/26
- Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D, Stefano Stabellini, 2014/05/27
- Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D, Chen, Tiejun, 2014/05/27
- Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D, Stefano Stabellini, 2014/05/28
- Re: [Qemu-devel] [v3][PATCH 3/5] xen, gfx passthrough: support Intel IGD passthrough with VT-D, Chen, Tiejun, 2014/05/28