From nobody Mon Feb 9 11:30:38 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.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=1593544836; cv=none; d=zohomail.com; s=zohoarc; b=NLIqGoaiefjTSaePpUyr9ArkAk31gluf+7KYceciw+GYRPWA8imKNjnH/ZAPrl13yPrP+1Yvm6uBtGvAqExrxBU3kj1kNXBEmrSi15B59WJgnK6ssoO/j1C+PSM63lbQ16vBCo9cGqpOlpDR1YH4Hj15YXKIQJSTldtXFyjdo4E= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1593544836; 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; bh=zJXkiU+zUDdCxJIrGhqvrdqSAXvINFmtZ8OHT+4C18g=; b=ImGuLUvfJCrvWUiDjCP/QK9kNRLG6j/NcN17C9PSQYnMkmCM1m0LRJmQ7HkrQTgPbeuoVIVIFQhSaxOTDSTYNB2JDWtgL2lHzHBUwueWreCINayHU/HVVDVPkJBZJ2ev6f4r1SaGZzfYiwPuYitPJhnzR8ZP5Wco/P7/6chywZE= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.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 1593544836173754.109909177399; Tue, 30 Jun 2020 12:20:36 -0700 (PDT) Received: from localhost ([::1]:39060 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jqLo2-0008BD-P4 for importer@patchew.org; Tue, 30 Jun 2020 15:20:34 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:42346) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jqLhr-0006ru-GT for qemu-devel@nongnu.org; Tue, 30 Jun 2020 15:14:11 -0400 Received: from us-smtp-delivery-1.mimecast.com ([205.139.110.120]:39317 helo=us-smtp-1.mimecast.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1jqLho-0005ZK-LK for qemu-devel@nongnu.org; Tue, 30 Jun 2020 15:14:11 -0400 Received: from mail-wm1-f72.google.com (mail-wm1-f72.google.com [209.85.128.72]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-292-OofM12ocMBG9_h7OqaGezg-1; Tue, 30 Jun 2020 15:14:00 -0400 Received: by mail-wm1-f72.google.com with SMTP id v11so2744922wmb.1 for ; Tue, 30 Jun 2020 12:14:00 -0700 (PDT) Received: from localhost.localdomain (1.red-83-51-162.dynamicip.rima-tde.net. [83.51.162.1]) by smtp.gmail.com with ESMTPSA id s8sm4634990wru.38.2020.06.30.12.13.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 30 Jun 2020 12:13:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1593544447; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=zJXkiU+zUDdCxJIrGhqvrdqSAXvINFmtZ8OHT+4C18g=; b=UlVkjj4bdTnewlXaE3T5TbutfBmFoyHCA8dlkUnXLRgIW2/A/AFXJIoohmawPwyHslxS6N cE44C8jggXpBaMWJbtrZ0QMQnViWfmYPf6ALeq1AJDHUDxu8T7etFXY0cd8axxXhpg460x TL392i0WVT95IMZi0KnU+hZ6NU3pBfA= X-MC-Unique: OofM12ocMBG9_h7OqaGezg-1 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=zJXkiU+zUDdCxJIrGhqvrdqSAXvINFmtZ8OHT+4C18g=; b=QdU/clwmZ3tg0sfcvSyAOEWJHQaFgAMOYAeVMLAvsHJe/7TKxgj+LL+HpdmxKcurm9 P/O6UW22K7v3IHsWcj9qbIQAxMLoa5V3eWeTYLnXETMXTfWxcu1e0kSSia+WTRgsitrX Es0//R8p+QbQ4r1aAhUoMqb+oXimAiLFpm4uO7YsQBHibc9ZxqdvmCPZEk0F1JwoHIOB CLYMUCjCxv0wg+2cRpmCAggTO1dWDx3bwtkbLo41YpcvGy9Ezd9VMfy8X4+1bzZZsK4+ R7qt/BAPCUocsKnpad2On1PJNZYjxz9buJa5Rq9u2grBdlcm1SdaqsspMzm1MRGIS3Os kbBA== X-Gm-Message-State: AOAM532zMyE64h6G8hmCW9ucxjBTfjELohWlhY3hNqHdV1xPVBynXtgZ jvoJzzumZcg2hAYjdZHzQnIU0jOxbBcZaiHVyhgzpOK2d+9wkbxaCCm74oW3+mLpGQOtRVfgdYW wMl/2ziM5TJ3pbxs= X-Received: by 2002:a1c:ab56:: with SMTP id u83mr22295319wme.94.1593544439292; Tue, 30 Jun 2020 12:13:59 -0700 (PDT) X-Google-Smtp-Source: ABdhPJziHrzDf2mF4+g9g4C/qcXZkOdsUL16wMN3Diry8DYUA539wBsUBlqrJgGg1aJq2Agr7gv6Pw== X-Received: by 2002:a1c:ab56:: with SMTP id u83mr22295299wme.94.1593544439095; Tue, 30 Jun 2020 12:13:59 -0700 (PDT) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: Stefan Hajnoczi , qemu-devel@nongnu.org Subject: [PATCH v2 08/12] block/nvme: Replace qemu_try_blockalign(bs) by qemu_try_memalign(pg_sz) Date: Tue, 30 Jun 2020 21:13:14 +0200 Message-Id: <20200630191318.30021-9-philmd@redhat.com> X-Mailer: git-send-email 2.21.3 In-Reply-To: <20200630191318.30021-1-philmd@redhat.com> References: <20200630191318.30021-1-philmd@redhat.com> MIME-Version: 1.0 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=philmd@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8"; text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: pass client-ip=205.139.110.120; envelope-from=philmd@redhat.com; helo=us-smtp-1.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/06/30 03:55:26 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -30 X-Spam_score: -3.1 X-Spam_bar: --- X-Spam_report: (-3.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-1, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=_AUTOLEARN X-Spam_action: no action 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: Kevin Wolf , Fam Zheng , qemu-block@nongnu.org, Maxim Levitsky , Max Reitz , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) qemu_try_blockalign() is a generic API that call back to the block driver to return its page alignment. As we call from within the very same driver, we already know to page alignment stored in our state. Remove indirections and use the value from BDRVNVMeState. This change is required to later remove the BlockDriverState argument, to make nvme_init_queue() per hardware, and not per block driver. Signed-off-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: Stefan Hajnoczi --- block/nvme.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/block/nvme.c b/block/nvme.c index 5b0bb9a8d7..8b4d957a8e 100644 --- a/block/nvme.c +++ b/block/nvme.c @@ -174,7 +174,7 @@ static void nvme_init_queue(BlockDriverState *bs, NVMeQ= ueue *q, =20 bytes =3D ROUND_UP(nentries * entry_bytes, s->page_size); q->head =3D q->tail =3D 0; - q->queue =3D qemu_try_blockalign(bs, bytes); + q->queue =3D qemu_try_memalign(s->page_size, bytes); if (!q->queue) { error_setg(errp, "Cannot allocate queue"); return; @@ -223,7 +223,7 @@ static NVMeQueuePair *nvme_create_queue_pair(BlockDrive= rState *bs, if (!q) { return NULL; } - q->prp_list_pages =3D qemu_try_blockalign(bs, + q->prp_list_pages =3D qemu_try_memalign(s->page_size, s->page_size * NVME_QUEUE_SIZE); if (!q->prp_list_pages) { goto fail; @@ -522,7 +522,7 @@ static void nvme_identify(BlockDriverState *bs, int nam= espace, Error **errp) .cdw10 =3D cpu_to_le32(0x1), }; =20 - id =3D qemu_try_blockalign(bs, sizeof(*id)); + id =3D qemu_try_memalign(s->page_size, sizeof(*id)); if (!id) { error_setg(errp, "Cannot allocate buffer for identify response"); goto out; @@ -1140,7 +1140,7 @@ static int nvme_co_prw(BlockDriverState *bs, uint64_t= offset, uint64_t bytes, return nvme_co_prw_aligned(bs, offset, bytes, qiov, is_write, flag= s); } trace_nvme_prw_buffered(s, offset, bytes, qiov->niov, is_write); - buf =3D qemu_try_blockalign(bs, bytes); + buf =3D qemu_try_memalign(s->page_size, bytes); =20 if (!buf) { return -ENOMEM; @@ -1284,7 +1284,7 @@ static int coroutine_fn nvme_co_pdiscard(BlockDriverS= tate *bs, =20 assert(s->nr_queues > 1); =20 - buf =3D qemu_try_blockalign(bs, s->page_size); + buf =3D qemu_try_memalign(s->page_size, s->page_size); if (!buf) { return -ENOMEM; } --=20 2.21.3