Re: [Xen-devel] [PATCH v2] libxl: Add AHCI support for upstream qemu

2015-06-26 Thread Fabio Fantoni
Il 25/06/2015 14:18, Ian Campbell ha scritto: On Thu, 2015-06-25 at 12:58 +0100, Stefano Stabellini wrote: On Thu, 25 Jun 2015, Fabio Fantoni wrote: Il 25/06/2015 12:21, Ian Campbell ha scritto: On Tue, 2015-06-23 at 11:15 +0200, Fabio Fantoni wrote: Usage: ahci=0|1 (default=0) I think a

Re: [Xen-devel] [PATCH v2] libxl: Add AHCI support for upstream qemu

2015-06-25 Thread Malcolm Crossley
On 25/06/15 12:15, Fabio Fantoni wrote: Il 25/06/2015 12:21, Ian Campbell ha scritto: On Tue, 2015-06-23 at 11:15 +0200, Fabio Fantoni wrote: Usage: ahci=0|1 (default=0) I think a global rather than per disk option is OK (I can't think why a user would want to mix and match) but maybe we

Re: [Xen-devel] [PATCH v2] libxl: Add AHCI support for upstream qemu

2015-06-25 Thread Ian Campbell
On Tue, 2015-06-23 at 11:15 +0200, Fabio Fantoni wrote: Usage: ahci=0|1 (default=0) I think a global rather than per disk option is OK (I can't think why a user would want to mix and match) but maybe we should consider using an enum (with values ide and ahci, defaulting to ide in libxl) so that

Re: [Xen-devel] [PATCH v2] libxl: Add AHCI support for upstream qemu

2015-06-25 Thread Stefano Stabellini
On Thu, 25 Jun 2015, Fabio Fantoni wrote: Il 25/06/2015 12:21, Ian Campbell ha scritto: On Tue, 2015-06-23 at 11:15 +0200, Fabio Fantoni wrote: Usage: ahci=0|1 (default=0) I think a global rather than per disk option is OK (I can't think why a user would want to mix and match) but

Re: [Xen-devel] [PATCH v2] libxl: Add AHCI support for upstream qemu

2015-06-25 Thread Ian Campbell
On Thu, 2015-06-25 at 12:58 +0100, Stefano Stabellini wrote: On Thu, 25 Jun 2015, Fabio Fantoni wrote: Il 25/06/2015 12:21, Ian Campbell ha scritto: On Tue, 2015-06-23 at 11:15 +0200, Fabio Fantoni wrote: Usage: ahci=0|1 (default=0) I think a global rather than per disk option is

Re: [Xen-devel] [PATCH v2] libxl: Add AHCI support for upstream qemu

2015-06-25 Thread Fabio Fantoni
Il 25/06/2015 12:21, Ian Campbell ha scritto: On Tue, 2015-06-23 at 11:15 +0200, Fabio Fantoni wrote: Usage: ahci=0|1 (default=0) I think a global rather than per disk option is OK (I can't think why a user would want to mix and match) but maybe we should consider using an enum (with values

[Xen-devel] [PATCH v2] libxl: Add AHCI support for upstream qemu

2015-06-23 Thread Fabio Fantoni
Usage: ahci=0|1 (default=0) If enabled adds ich9 disk controller in ahci mode and uses it with upstream qemu to emulate disks instead of ide. It doesn't support cdroms which still using ide (cdroms will use -device ide-cd as new qemu parameter) Ahci requires new qemu parameter but for now other

Re: [Xen-devel] [PATCH v2] libxl: Add AHCI support for upstream qemu

2015-06-23 Thread Stefano Stabellini
On Tue, 23 Jun 2015, Fabio Fantoni wrote: Usage: ahci=0|1 (default=0) If enabled adds ich9 disk controller in ahci mode and uses it with upstream qemu to emulate disks instead of ide. It doesn't support cdroms which still using ide (cdroms will use -device ide-cd as new qemu parameter)