From 1291b288a1f62c555b0b772fd5b8c6aad4b1132c Mon Sep 17 00:00:00 2001 From: Naphtali Sprei <nsprei@redhat.com> Date: Thu, 15 Apr 2010 11:56:43 -0300 Subject: [PATCH 07/12] read-only: Make CDROM a read-only drive RH-Author: Naphtali Sprei <nsprei@redhat.com> Message-id: <1271332608-5243-7-git-send-email-nsprei@redhat.com> Patchwork-id: 8654 O-Subject: [RHEL 5.6 kvm PATCH v4 06/11] read-only: Make CDROM a read-only drive Bugzilla: 510630 RH-Acked-by: Christoph Hellwig <chellwig@redhat.com> RH-Acked-by: Juan Quintela <quintela@redhat.com> RH-Acked-by: Kevin Wolf <kwolf@redhat.com> based on upstream commit: b196b1532f2981e9c811c96214a44ee9e7d12af3 Signed-off-by: Naphtali Sprei <nsprei@redhat.com> --- qemu/vl.c | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> --- qemu/vl.c | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/qemu/vl.c b/qemu/vl.c index e889c37..96a581a 100644 --- a/qemu/vl.c +++ b/qemu/vl.c @@ -2754,6 +2754,14 @@ int drive_init(struct drive_opt *arg, int snapshot, return -1; } } + /* + * cdrom is read-only. Set it now, after above interface checking + * since readonly attribute not explicitly required, so no error. + */ + if (media == MEDIA_CDROM) { + ro = 1; + } + if (!ro) { bdrv_flags |= BDRV_O_RDWR; } -- 1.7.0.3