Re: [Xen-devel] [PATCH v6] libxl: allow 'phy' backend to use empty files

2016-04-05 Thread Alex Braunegg
Hi George, > I take it that if he used a phy backend for a cdrom, that "xl cd-eject" > failed? No - I was always using ISO images for cd-rom devices. In the 4.4 configuration I was specifying it as a file. In Xen 4.4 and 4.6 (before patching) whenever I attempted to perform an 'xl cd-eject'

Re: [Xen-devel] [PATCH v6] libxl: allow 'phy' backend to use empty files

2016-04-05 Thread George Dunlap
On Wed, Feb 17, 2016 at 5:20 PM, Roger Pau Monne wrote: > This was introduced by 97ee1f (~5 years ago), but was probably never > surfaced because most people used regular files as CDROM images, so the PHY > backend was actually never selected. A year ago this was changed,

Re: [Xen-devel] [PATCH v6] libxl: allow 'phy' backend to use empty files

2016-02-19 Thread Roger Pau Monné
El 19/2/16 a les 18:30, Ian Jackson ha escrit: > Roger Pau Monne writes ("[PATCH v6] libxl: allow 'phy' backend to use empty > files"): >> This was introduced by 97ee1f (~5 years ago), but was probably never >> surfaced because most people used regular files as CDROM images, so the PHY >> backend

Re: [Xen-devel] [PATCH v6] libxl: allow 'phy' backend to use empty files

2016-02-19 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v6] libxl: allow 'phy' backend to use empty files"): > This was introduced by 97ee1f (~5 years ago), but was probably never > surfaced because most people used regular files as CDROM images, so the PHY > backend was actually never selected. A year ago this was

Re: [Xen-devel] [PATCH v6] libxl: allow 'phy' backend to use empty files

2016-02-18 Thread Alex Braunegg
Hi Roger, I have tested and applied the patch locally, and can confirm that the issue reported is now resolved. Best regards, Alex -Original Message- From: Roger Pau Monne [mailto:roger@citrix.com] Sent: Thursday, 18 February 2016 4:21 AM To: xen-de...@lists.xenproject.org Cc:

[Xen-devel] [PATCH v6] libxl: allow 'phy' backend to use empty files

2016-02-17 Thread Roger Pau Monne
This was introduced by 97ee1f (~5 years ago), but was probably never surfaced because most people used regular files as CDROM images, so the PHY backend was actually never selected. A year ago this was changed, and now regular RAW files are also handled by the PHY backend, which has made this bug