From nobody Sat May 4 08:32:03 2024 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=intel.com Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1549951030785442.5579410532433; Mon, 11 Feb 2019 21:57:10 -0800 (PST) Received: from localhost ([127.0.0.1]:33812 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gtR47-0004KL-67 for importer@patchew.org; Tue, 12 Feb 2019 00:57:07 -0500 Received: from eggs.gnu.org ([209.51.188.92]:56234) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gtR29-0002ow-4l for qemu-devel@nongnu.org; Tue, 12 Feb 2019 00:55:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gtQrT-0003Us-1I for qemu-devel@nongnu.org; Tue, 12 Feb 2019 00:44:05 -0500 Received: from mga14.intel.com ([192.55.52.115]:53245) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gtQrM-0002vl-Mr for qemu-devel@nongnu.org; Tue, 12 Feb 2019 00:43:57 -0500 Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Feb 2019 21:43:33 -0800 Received: from fedora.sh.intel.com ([10.67.114.176]) by fmsmga008.fm.intel.com with ESMTP; 11 Feb 2019 21:43:31 -0800 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.58,361,1544515200"; d="scan'208";a="123782125" From: Changpeng Liu To: qemu-devel@nongnu.org Date: Tue, 12 Feb 2019 14:01:44 +0800 Message-Id: <1549951304-26100-1-git-send-email-changpeng.liu@intel.com> X-Mailer: git-send-email 1.9.3 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 192.55.52.115 Subject: [Qemu-devel] [PATCH v2] virtio-blk: set correct config size for the host driver X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: ldoktor@redhat.com, mst@redhat.com, dgilbert@redhat.com, stefanha@redhat.com, changpeng.liu@intel.com, sgarzare@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Commit caa1ee43 "vhost-user-blk: add discard/write zeroes features support" introduced extra fields to existing struct virtio_blk_config, when migration was executed from older QEMU version to current head, it will break the migration. While here, set the correct config size when initializing the host driver, for now, discard/write zeroes are not supported by virtio-blk host driver, so set the config size as before, users can change config size when adding the new feature bits support. Signed-off-by: Changpeng Liu Reviewed-by: Greg Kurz Tested-by: Greg Kurz --- hw/block/virtio-blk.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c index 9a87b3b..846b7b9 100644 --- a/hw/block/virtio-blk.c +++ b/hw/block/virtio-blk.c @@ -28,6 +28,9 @@ #include "hw/virtio/virtio-bus.h" #include "hw/virtio/virtio-access.h" =20 +#define VIRTIO_BLK_CFG_SIZE (offsetof(struct virtio_blk_config, num_queues= ) + \ + sizeof_field(struct virtio_blk_config, num_qu= eues)) + static void virtio_blk_init_request(VirtIOBlock *s, VirtQueue *vq, VirtIOBlockReq *req) { @@ -761,7 +764,7 @@ static void virtio_blk_update_config(VirtIODevice *vdev= , uint8_t *config) blkcfg.alignment_offset =3D 0; blkcfg.wce =3D blk_enable_write_cache(s->blk); virtio_stw_p(vdev, &blkcfg.num_queues, s->conf.num_queues); - memcpy(config, &blkcfg, sizeof(struct virtio_blk_config)); + memcpy(config, &blkcfg, VIRTIO_BLK_CFG_SIZE); } =20 static void virtio_blk_set_config(VirtIODevice *vdev, const uint8_t *confi= g) @@ -769,7 +772,7 @@ static void virtio_blk_set_config(VirtIODevice *vdev, c= onst uint8_t *config) VirtIOBlock *s =3D VIRTIO_BLK(vdev); struct virtio_blk_config blkcfg; =20 - memcpy(&blkcfg, config, sizeof(blkcfg)); + memcpy(&blkcfg, config, VIRTIO_BLK_CFG_SIZE); =20 aio_context_acquire(blk_get_aio_context(s->blk)); blk_set_enable_write_cache(s->blk, blkcfg.wce !=3D 0); @@ -952,8 +955,7 @@ static void virtio_blk_device_realize(DeviceState *dev,= Error **errp) return; } =20 - virtio_init(vdev, "virtio-blk", VIRTIO_ID_BLOCK, - sizeof(struct virtio_blk_config)); + virtio_init(vdev, "virtio-blk", VIRTIO_ID_BLOCK, VIRTIO_BLK_CFG_SIZE); =20 s->blk =3D conf->conf.blk; s->rq =3D NULL; --=20 1.9.3