From nobody Sun Nov 16 02:26:42 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; 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=pass(p=quarantine dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1744223442; cv=none; d=zohomail.com; s=zohoarc; b=ksPhbV1F7d7TZoQNrG0wXHrfXIZaFMe/EHNXB6NAxzOnbnGz2NLY8VdDyvV+UEVYH1tNkprBd2NDiFPImhNvs8yFLBHbEtx7nffaYyAyfq/Rlqtd3IEWNVMAxZq/Wo8Bxw5kyYVJDnYy4hBpqEifDlerbBRtQG1aimEy1EN9EoY= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1744223442; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=rVLuzkg7JMYXg8j4lbQ3tRgxU43XzF0oeTKvVx+oPlk=; b=SljUPnwS70C5Y5k24lX24jykR/MxUk1ILpjz26Bs54AwCoEstn9/bF3yVCmowcBPoXhRcb0cMRSP8y4Foe+T7Oh0oN3DdbwssENzPK5BgBXCELMU87qPcMnAnV9hbvhQLXudbjVb5tX9O5tOPklgVKYeM30pgl+7je8IYJdNUnA= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; 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=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 17442234419431022.5618910190954; Wed, 9 Apr 2025 11:30:41 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1u2aB2-0003bN-8C; Wed, 09 Apr 2025 14:29:32 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1u2aB0-0003ZV-Dp for qemu-devel@nongnu.org; Wed, 09 Apr 2025 14:29:30 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1u2aAx-0003D0-Dq for qemu-devel@nongnu.org; Wed, 09 Apr 2025 14:29:30 -0400 Received: from mx-prod-mc-08.mail-002.prod.us-west-2.aws.redhat.com (ec2-35-165-154-97.us-west-2.compute.amazonaws.com [35.165.154.97]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-97-D4oGWfX1NF6a9bWB0yW5Xg-1; Wed, 09 Apr 2025 14:29:24 -0400 Received: from mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.17]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-08.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id D54B1180882E; Wed, 9 Apr 2025 18:29:22 +0000 (UTC) Received: from localhost (unknown [10.2.16.144]) by mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 6A7E91955DCE; Wed, 9 Apr 2025 18:29:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1744223366; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=rVLuzkg7JMYXg8j4lbQ3tRgxU43XzF0oeTKvVx+oPlk=; b=G1KXcjyemnS5xeXb1Q41tzB5d2GCjKIHXavR3H0/zVy3T+qDtXe/fs338omNiHeqK+cTvK y3WUYGzzIsnjajAQeQ7ghFfoI3IItkhIz2gGTs6Ql44d7+OHQapOiZk/VpcpbEIhzOCSfi wdczcNtTuQGx7ni17+xlSQbur5f/HSc= X-MC-Unique: D4oGWfX1NF6a9bWB0yW5Xg-1 X-Mimecast-MFC-AGG-ID: D4oGWfX1NF6a9bWB0yW5Xg_1744223363 From: Stefan Hajnoczi To: qemu-devel@nongnu.org Cc: Fam Zheng , , Stefan Hajnoczi , Hanna Czenczek , Kevin Wolf Subject: [PATCH 2/2] block/io: skip head/tail requests on EINVAL Date: Wed, 9 Apr 2025 14:29:14 -0400 Message-ID: <20250409182914.4725-3-stefanha@redhat.com> In-Reply-To: <20250409182914.4725-1-stefanha@redhat.com> References: <20250409182914.4725-1-stefanha@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.0 on 10.30.177.17 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=170.10.129.124; envelope-from=stefanha@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -25 X-Spam_score: -2.6 X-Spam_bar: -- X-Spam_report: (-2.6 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.505, 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_VALIDITY_CERTIFIED_BLOCKED=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1744223444267019100 Content-Type: text/plain; charset="utf-8" When guests send misaligned discard requests, the block layer breaks them up into a misaligned head, an aligned main body, and a misaligned tail. The file-posix block driver on Linux returns -EINVAL on misaligned discard requests. This causes bdrv_co_pdiscard() to fail and guests configured with werror=3Dstop will pause. Add a special case for misaligned head/tail requests. Simply continue when EINVAL is encountered so that the aligned main body of the request can be completed and the guest is not paused. This is the best we can do when guest discard limits do not match the host discard limits. Fixes: https://issues.redhat.com/browse/RHEL-86032 Signed-off-by: Stefan Hajnoczi --- block/io.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/block/io.c b/block/io.c index 1ba8d1aeea..5975f4e9a3 100644 --- a/block/io.c +++ b/block/io.c @@ -3180,7 +3180,11 @@ int coroutine_fn bdrv_co_pdiscard(BdrvChild *child, = int64_t offset, } } if (ret && ret !=3D -ENOTSUP) { - goto out; + if (ret =3D=3D -EINVAL && (offset % align =3D=3D 0 || num % al= ign =3D=3D 0)) { + /* Silently skip rejected unaligned head/tail requests */ + } else { + goto out; /* bail out */ + } } =20 offset +=3D num; --=20 2.49.0