From nobody Sat Feb 7 05:30:54 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1564485966; cv=none; d=zoho.com; s=zohoarc; b=HkqV543GuTBagy4E0Mp8sVcQxbFPFWsp1UAbiHijveDNJVal3W88NEjRxDD/ElJaWxRoeovG76gdsod37Ds74mvQPD9CVTU4QEoyJWlzZvi8q8mcoCx+W5e27gNOp7JqJzu2AjgFvPPad1jOf+iasg8vtcyq1mUwHHXOaJE6Dc8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1564485966; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=sIlJoxJkZn/WPmUh9JXJicSMGkOCu8GqAOsbYq/zTlI=; b=X3x6NZFcNb+pev0lDGgwDFpNtcwW4OzFuoVIx3rgQdDVI3kVpLiUkH302eSgiXqzZipmsSCOXaosar4FN8YzLAvbl5Ky0N3xpQC0VKNX0+qav6Z0U3B9ZoDnqF+sIil/URQQAnD5WQh+L7m5PLbh/DtaxzJbiuRMrjGhkgPYGn0= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1564485965971834.0407763444291; Tue, 30 Jul 2019 04:26:05 -0700 (PDT) Received: from localhost ([::1]:59830 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsQGa-000562-3k for importer@patchew.org; Tue, 30 Jul 2019 07:26:04 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:52200) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsQFD-0003Xe-Ft for qemu-devel@nongnu.org; Tue, 30 Jul 2019 07:24:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hsQFB-0006fp-1L for qemu-devel@nongnu.org; Tue, 30 Jul 2019 07:24:38 -0400 Received: from mx1.redhat.com ([209.132.183.28]:36856) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hsQF5-0006dB-S1; Tue, 30 Jul 2019 07:24:32 -0400 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 16AE081DF1; Tue, 30 Jul 2019 11:24:31 +0000 (UTC) Received: from localhost.localdomain.com (ovpn-117-75.ams2.redhat.com [10.36.117.75]) by smtp.corp.redhat.com (Postfix) with ESMTP id 0AAEF60623; Tue, 30 Jul 2019 11:24:29 +0000 (UTC) From: Kevin Wolf To: qemu-block@nongnu.org Date: Tue, 30 Jul 2019 13:24:22 +0200 Message-Id: <20190730112425.21497-2-kwolf@redhat.com> In-Reply-To: <20190730112425.21497-1-kwolf@redhat.com> References: <20190730112425.21497-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Tue, 30 Jul 2019 11:24:31 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [PULL 1/4] tests/multiboot: Fix load address of test kernels X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, peter.maydell@linaro.org, qemu-devel@nongnu.org Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" While older toolchains produced binaries where the physical load address of ELF segments was the same as the virtual address, newer versions seem to choose a different physical address if it isn't specified explicitly. The means that the test kernel doesn't use the right addresses to access e.g. format strings any more and the whole output disappears, causing all test cases to fail. Fix this by specifying the physical load address of sections explicitly. Signed-off-by: Kevin Wolf --- tests/multiboot/link.ld | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/multiboot/link.ld b/tests/multiboot/link.ld index 3d49b58c60..2eafcffc4f 100644 --- a/tests/multiboot/link.ld +++ b/tests/multiboot/link.ld @@ -3,14 +3,14 @@ ENTRY(_start) SECTIONS { . =3D 0x100000; - .text : { + .text : AT(ADDR(.text)) { *(multiboot) *(.text) } - .data ALIGN(4096) : { + .data ALIGN(4096) : AT(ADDR(.data)) { *(.data) } - .rodata ALIGN(4096) : { + .rodata ALIGN(4096) : AT(ADDR(.rodata)) { *(.rodata) } .bss ALIGN(4096) : { --=20 2.20.1 From nobody Sat Feb 7 05:30:54 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1564486080; cv=none; d=zoho.com; s=zohoarc; b=ZX+ij8+R1UFV0mu+A9y2EDLRnZ9mNa9G0IpIzrbR0T1DmnBRGsGlr4hdEfV/fkg/nbm7UV2bIex6hLZU8W1v1BWO/TenzOEZkAAm1ukHD4Nmdge1yQZRaeeNfZryViBGFST7n9aJoqlTudT5U+KwKQOgNzyTJ/i/conH59IXXKI= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1564486080; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=SPZIxHkvPCg8jk3WzrAG/uhF+DyEMY5DIsExzt5RUd8=; b=mBUzbo8faZRmBVgEOhXNZ8mRKgVkU+BJeWHwUDWBZUxsjxBpJoW54KkEGVZLrF4ZL2U/8DXmvwxR8PumHYNgixN/8AZRm0MxMLSlgZgeMyS+p/pXFeHFGYYL/z2AxRTh5lAvQdCV14L9ihdXTD5mQaF8sgyK2IGkXDqJUjEweYk= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1564486080885562.4928359525313; Tue, 30 Jul 2019 04:28:00 -0700 (PDT) Received: from localhost ([::1]:59860 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsQIS-0000pg-06 for importer@patchew.org; Tue, 30 Jul 2019 07:28:00 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:52251) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsQFJ-0003al-Cp for qemu-devel@nongnu.org; Tue, 30 Jul 2019 07:24:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hsQFH-0006jY-Hl for qemu-devel@nongnu.org; Tue, 30 Jul 2019 07:24:44 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38328) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hsQF8-0006dq-DV; Tue, 30 Jul 2019 07:24:34 -0400 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 6EED1308FBAF; Tue, 30 Jul 2019 11:24:32 +0000 (UTC) Received: from localhost.localdomain.com (ovpn-117-75.ams2.redhat.com [10.36.117.75]) by smtp.corp.redhat.com (Postfix) with ESMTP id 62F1C6092F; Tue, 30 Jul 2019 11:24:31 +0000 (UTC) From: Kevin Wolf To: qemu-block@nongnu.org Date: Tue, 30 Jul 2019 13:24:23 +0200 Message-Id: <20190730112425.21497-3-kwolf@redhat.com> In-Reply-To: <20190730112425.21497-1-kwolf@redhat.com> References: <20190730112425.21497-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.43]); Tue, 30 Jul 2019 11:24:32 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [PULL 2/4] Fixes: add read-zeroes to 051.out X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, peter.maydell@linaro.org, qemu-devel@nongnu.org Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" From: Andrey Shinkevich The patch "iotests: Set read-zeroes on in null block driver for Valgrind" with the commit ID a6862418fec4072 needs the change in 051.out when compared against on the s390 system. Fixes: a6862418fec40727b392c86dc13d9ec980efcb15 Reported-by: Christian Borntraeger Signed-off-by: Andrey Shinkevich Tested-by: Christian Borntraeger Reviewed-by: John Snow Signed-off-by: Kevin Wolf --- tests/qemu-iotests/051.out | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/qemu-iotests/051.out b/tests/qemu-iotests/051.out index 8993835b94..554c5ca90a 100644 --- a/tests/qemu-iotests/051.out +++ b/tests/qemu-iotests/051.out @@ -149,23 +149,23 @@ QEMU X.Y.Z monitor - type 'help' for more information =20 =3D=3D=3D Cache modes =3D=3D=3D =20 -Testing: -drive driver=3Dnull-co,cache=3Dnone +Testing: -drive driver=3Dnull-co,read-zeroes=3Don,cache=3Dnone QEMU X.Y.Z monitor - type 'help' for more information (qemu) quit =20 -Testing: -drive driver=3Dnull-co,cache=3Ddirectsync +Testing: -drive driver=3Dnull-co,read-zeroes=3Don,cache=3Ddirectsync QEMU X.Y.Z monitor - type 'help' for more information (qemu) quit =20 -Testing: -drive driver=3Dnull-co,cache=3Dwriteback +Testing: -drive driver=3Dnull-co,read-zeroes=3Don,cache=3Dwriteback QEMU X.Y.Z monitor - type 'help' for more information (qemu) quit =20 -Testing: -drive driver=3Dnull-co,cache=3Dwritethrough +Testing: -drive driver=3Dnull-co,read-zeroes=3Don,cache=3Dwritethrough QEMU X.Y.Z monitor - type 'help' for more information (qemu) quit =20 -Testing: -drive driver=3Dnull-co,cache=3Dunsafe +Testing: -drive driver=3Dnull-co,read-zeroes=3Don,cache=3Dunsafe QEMU X.Y.Z monitor - type 'help' for more information (qemu) quit =20 --=20 2.20.1 From nobody Sat Feb 7 05:30:54 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1564486020; cv=none; d=zoho.com; s=zohoarc; b=TPukRjA7TZlrSxolwmiJEsSdhOUSpzvvkxw38P7KVKMBIsv7JM/6bh38bv9I7rPeeUR1ichO94/BnYrXPbU9RvkwOWVRFS/2R0JSWpxDG42QwrLWup9JpTdl21/2SgpOgFYIC6+9PBxzhRh2qQaX1em8GCcnZIO+hPv6O9yOrzE= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1564486020; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=JLGDY5H9TKRd9sLWAWnocal55NBQMoOlLkL9IIbVGJI=; b=fWYCPL/+CfxRKtI9a0BdZosZCGxSjJ3PCtKMmRX9nEAcb3AOqBJzAGEML2/dUxs5+meKG94+YFrJYlpObA47w24raDCXRS1Jiu4+d5E60QQFhMR/DorzaRfBDWpobtwiBXcFv80aaDYAlHYhjvxd5mB8lfVz6YfPc3v7/O/jSEE= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 156448602042619.709027267280135; Tue, 30 Jul 2019 04:27:00 -0700 (PDT) Received: from localhost ([::1]:59848 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsQHT-0007YK-H6 for importer@patchew.org; Tue, 30 Jul 2019 07:26:59 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:52252) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsQFJ-0003am-DH for qemu-devel@nongnu.org; Tue, 30 Jul 2019 07:24:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hsQFH-0006jd-Hq for qemu-devel@nongnu.org; Tue, 30 Jul 2019 07:24:44 -0400 Received: from mx1.redhat.com ([209.132.183.28]:43288) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hsQF8-0006eH-FU; Tue, 30 Jul 2019 07:24:35 -0400 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id C3D6B88313; Tue, 30 Jul 2019 11:24:33 +0000 (UTC) Received: from localhost.localdomain.com (ovpn-117-75.ams2.redhat.com [10.36.117.75]) by smtp.corp.redhat.com (Postfix) with ESMTP id B9BB260623; Tue, 30 Jul 2019 11:24:32 +0000 (UTC) From: Kevin Wolf To: qemu-block@nongnu.org Date: Tue, 30 Jul 2019 13:24:24 +0200 Message-Id: <20190730112425.21497-4-kwolf@redhat.com> In-Reply-To: <20190730112425.21497-1-kwolf@redhat.com> References: <20190730112425.21497-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Tue, 30 Jul 2019 11:24:33 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [PULL 3/4] block/copy-on-read: Fix permissions for inactive node X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, peter.maydell@linaro.org, qemu-devel@nongnu.org Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" The copy-on-read drive must not request the WRITE_UNCHANGED permission for its child if the node is inactive, otherwise starting a migration destination with -incoming will fail because the child cannot provide write access yet: qemu-system-x86_64: -blockdev copy-on-read,file=3Dimg,node-name=3Dcor: Bl= ock node is read-only Earlier QEMU versions additionally ran into an abort() on the migration source side: bdrv_inactivate_recurse() failed to update permissions. This is silently ignored today because it was only supposed to loosen restrictions. This is the symptom that was originally reported here: https://bugzilla.redhat.com/show_bug.cgi?id=3D1733022 Signed-off-by: Kevin Wolf Reviewed-by: Max Reitz --- block/copy-on-read.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/block/copy-on-read.c b/block/copy-on-read.c index 22f24fd0db..6631f30205 100644 --- a/block/copy-on-read.c +++ b/block/copy-on-read.c @@ -56,16 +56,14 @@ static void cor_child_perm(BlockDriverState *bs, BdrvCh= ild *c, uint64_t perm, uint64_t shared, uint64_t *nperm, uint64_t *nshared) { - if (c =3D=3D NULL) { - *nperm =3D (perm & PERM_PASSTHROUGH) | BLK_PERM_WRITE_UNCHANGED; - *nshared =3D (shared & PERM_PASSTHROUGH) | PERM_UNCHANGED; - return; - } + *nperm =3D perm & PERM_PASSTHROUGH; + *nshared =3D (shared & PERM_PASSTHROUGH) | PERM_UNCHANGED; =20 - *nperm =3D (perm & PERM_PASSTHROUGH) | - (c->perm & PERM_UNCHANGED); - *nshared =3D (shared & PERM_PASSTHROUGH) | - (c->shared_perm & PERM_UNCHANGED); + /* We must not request write permissions for an inactive node, the chi= ld + * cannot provide it. */ + if (!(bs->open_flags & BDRV_O_INACTIVE)) { + *nperm |=3D BLK_PERM_WRITE_UNCHANGED; + } } =20 =20 --=20 2.20.1 From nobody Sat Feb 7 05:30:54 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1564486018; cv=none; d=zoho.com; s=zohoarc; b=fLYYyAstHA9JLigwmvBEkIC4xPg3sJfkwcFmWrWd5AwIL7I3Z1M7B5aBxNQkSmQm+WF//4Y/D9KWE/7JDssxzLY2qbab93FlEbNRKWE9m3VseJBRFQu8uhmKdjszYt4/QLqqpGI1BgmcKtrvBAYbjrp3bpxYB8mvv0OzEOSz2AQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1564486018; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=PJN1l1Mb09O/ObF15qRz5EzxDV2bXi1isiHp5WRCSeY=; b=jTqV5gQQoiBgce2Afp5MPA76fIU7rPtTl7JArtaJ5mrfEjojtZOMFoFb+1/ajdc6ILVQX8U9ttxrfahrRVZTdFLLTZEpJpzD2jYZTHwn2fRNOs9bX5dQEjUJ389twH7uQOyLrp9jiLhaeAy7a5qF4B9zZj00biFa+wj+P8v5jgM= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1564486018705327.2856753967659; Tue, 30 Jul 2019 04:26:58 -0700 (PDT) Received: from localhost ([::1]:59846 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsQHP-0007RA-Nl for importer@patchew.org; Tue, 30 Jul 2019 07:26:55 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:52250) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hsQFJ-0003ak-Ct for qemu-devel@nongnu.org; Tue, 30 Jul 2019 07:24:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hsQFH-0006ji-KM for qemu-devel@nongnu.org; Tue, 30 Jul 2019 07:24:44 -0400 Received: from mx1.redhat.com ([209.132.183.28]:55636) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hsQFB-0006et-8O; Tue, 30 Jul 2019 07:24:38 -0400 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 26886300BEA8; Tue, 30 Jul 2019 11:24:35 +0000 (UTC) Received: from localhost.localdomain.com (ovpn-117-75.ams2.redhat.com [10.36.117.75]) by smtp.corp.redhat.com (Postfix) with ESMTP id 1BA5060623; Tue, 30 Jul 2019 11:24:33 +0000 (UTC) From: Kevin Wolf To: qemu-block@nongnu.org Date: Tue, 30 Jul 2019 13:24:25 +0200 Message-Id: <20190730112425.21497-5-kwolf@redhat.com> In-Reply-To: <20190730112425.21497-1-kwolf@redhat.com> References: <20190730112425.21497-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.49]); Tue, 30 Jul 2019 11:24:35 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [PULL 4/4] scsi-cd: Fix inserting read-only media in empty drive X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, peter.maydell@linaro.org, qemu-devel@nongnu.org Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" scsi-disks decides whether it has a read-only device by looking at whether the BlockBackend specified as drive=3D... is read-only. In the case of an anonymous BlockBackend (with a node name specified in drive=3D...), this is the read-only flag of the attached node. In the case of an empty anonymous BlockBackend, it's always read-write because nothing prevented it from being read-write. This is a problem because scsi-cd would take write permissions on the anonymous BlockBackend of an empty drive created without a drive=3D... option. Using blockdev-insert-medium with a read-only node fails then with the error message "Block node is read-only". Fix scsi_realize() so that scsi-cd devices always take read-only permissions on their BlockBackend instead. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=3D1733920 Signed-off-by: Kevin Wolf Reviewed-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: Max Reitz Reviewed-by: Markus Armbruster --- hw/scsi/scsi-disk.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c index 8e95e3e38d..af3e622dc5 100644 --- a/hw/scsi/scsi-disk.c +++ b/hw/scsi/scsi-disk.c @@ -2318,6 +2318,7 @@ static void scsi_disk_unit_attention_reported(SCSIDev= ice *dev) static void scsi_realize(SCSIDevice *dev, Error **errp) { SCSIDiskState *s =3D DO_UPCAST(SCSIDiskState, qdev, dev); + bool read_only; =20 if (!s->qdev.conf.blk) { error_setg(errp, "drive property not set"); @@ -2351,8 +2352,13 @@ static void scsi_realize(SCSIDevice *dev, Error **er= rp) return; } } - if (!blkconf_apply_backend_options(&dev->conf, - blk_is_read_only(s->qdev.conf.blk), + + read_only =3D blk_is_read_only(s->qdev.conf.blk); + if (dev->type =3D=3D TYPE_ROM) { + read_only =3D true; + } + + if (!blkconf_apply_backend_options(&dev->conf, read_only, dev->type =3D=3D TYPE_DISK, errp)) { return; } --=20 2.20.1