[PATCH] Missing CDROM media (w/Signed-off-by)

[PATCH] Missing CDROM media (w/Signed-off-by)

[Ross Maxfield]

[-- Attachment #1: Type: text/plain, Size: 240 bytes --]

All,
Being a neophyte to open-source checkins, I apologize for the failure to include the 'Signed-off-by' statement, and resubmit the missing-media patch in conformation.

- Ross 

Signed-off-by: Ross Maxfield <rmaxfiel@novell.com>


[-- Attachment #2: xen-missing-media.diff --]
[-- Type: application/octet-stream, Size: 828 bytes --]

Index: xen-3.0-testing/tools/ioemu/vl.c
===================================================================
--- xen-3.0-testing.orig/tools/ioemu/vl.c
+++ xen-3.0-testing/tools/ioemu/vl.c
@@ -3243,8 +3243,17 @@ int main(int argc, char **argv)
     /* we always create the cdrom drive, even if no disk is there */
     bdrv_init();
     if (has_cdrom) {
-        bs_table[2] = bdrv_new("cdrom");
-        bdrv_set_type_hint(bs_table[2], BDRV_TYPE_CDROM);
+	int fd;
+        if ( (fd = open(hd_filename[2], O_RDONLY | O_BINARY)) < 0) {
+  		fprintf(stderr, "Could not open CD %s.\n", hd_filename[i]);
+		hd_filename[2]=NULL;
+		bs_table[2]=NULL;
+	}
+	else {
+		close(fd);
+	        bs_table[2] = bdrv_new("cdrom");
+	        bdrv_set_type_hint(bs_table[2], BDRV_TYPE_CDROM);
+	}
     }
 
     /* open the virtual block devices */

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Re: [PATCH] Missing CDROM media (w/Signed-off-by)

[Anthony Liguori]

Thanks!

Regards,

Anthony Liguori

Ross Maxfield wrote:
> All,
> Being a neophyte to open-source checkins, I apologize for the failure to include the 'Signed-off-by' statement, and resubmit the missing-media patch in conformation.
>
> - Ross 
>
> Signed-off-by: Ross Maxfield <rmaxfiel@novell.com>
>
>   
> ------------------------------------------------------------------------
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
>