From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f177.google.com (mail-pf1-f177.google.com [209.85.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 016F425F99F for ; Mon, 24 Nov 2025 23:48:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.177 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764028103; cv=none; b=bdilbOGee/uVioTklW61YgIQZW659B5kl8I46gO2AqY2HYy3DLA+urll69lSlJEWvnIFNxv6/OpY+pXwdFXWarpK5t2hO1EoWrWiqOMjcvJ+ITaG0rdu/khWHWtFRuSAUaaVN4pOCxai7U6cssBxJdqXziOLXDRo9t78x10I3dY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764028103; c=relaxed/simple; bh=0HZiUPVVqiDv8ZEl67pEJba49nGpbGLpzJLHrHVWkJs=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=DHaWa/35A6msEPs1Lp2SgKn4V9aA0jdJz4bgQY12ZQq56zQvToMO8HsDRJuYBq7Csr9woK8FXIOKpXv4ovvhiISCPxXIcAgxQkQjgFmESdovFoQvU1abviiS3LWh8mrRHa91q2U+y6uQFmjUluYPFE45DBhieqIEl4UFVE71CeI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=bPL86y/i; arc=none smtp.client-ip=209.85.210.177 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="bPL86y/i" Received: by mail-pf1-f177.google.com with SMTP id d2e1a72fcca58-7b852bb31d9so5645398b3a.0 for ; Mon, 24 Nov 2025 15:48:21 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1764028101; x=1764632901; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=L8zwVcuAssRnEQZ8e7ewIMIsPfvOXMAA4RWGmHyZ+Rs=; b=bPL86y/iBZjOidhBAl6VEV+QifNIHQrW4C3Ep/nwnKCbwUYO59IpTBFsDTtTEXt8yx FJpoB8nln2MJ0VffyNsLuxsbAfCrlFlqVQdFMMBONxF0RNtxGWy73OnTLTVzfvdxyT/l 1a5MQ5SILc0M6AkSTMRTtkdDAAw9Lhar88tFes+HRhPfsfPrH0YQpWIRbMdqpN8JxpT5 pn89jOdNkpwNkPjwlZ7wuYN9CfRt97F+rlWz4bReRFYlh2UbtSsd3m2VD8CYzxEdUPwJ 1hrdyU71mzSM4j22YMWsrdkPTHruHM0KynglrtZuDsfqrhVEhrIXc/umLfuVoTM6pkLZ 4c1w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1764028101; x=1764632901; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=L8zwVcuAssRnEQZ8e7ewIMIsPfvOXMAA4RWGmHyZ+Rs=; b=DVAIbhhtnWW+5Msb4eqkjew5Aay57x1hft3xko1ctAUq+/ywfDeB3B3GmTlbR+bH4u psZk6SMt/bwq30a62rTGgz9jtC1Md8ToPvNd9vAv9c0Zbx5KooTCqPaPJCDqCj6nUKoF aAvi6lXO10A0Ul/+Kw2OIujzaD1Ts5ZYvj8iUlLArHzD/NQQZEHXBlyX1zsaE+FMs7/l dkZV1QsEYIkP4gHtjpdG7x+AAb06oN27NzY1+AIhoQCiGWuwiTaqTqh2W24BHz0k/7N6 RwceVJ4/6FLRoG/kZQyhPoIyuMSrqLWlgZKUYrH42wlSS7ZA14SePxwircUhGtlNi6+B SCFw== X-Forwarded-Encrypted: i=1; AJvYcCXLmXyY8zCX1B2X8xOCekqeePzS/admneUAIYY3McxIaubyBLECgb5RXedAsMBe+laZZjsICFxSDThBo/k=@vger.kernel.org X-Gm-Message-State: AOJu0YxxE+O1BJGhZP3LXN68vOMoTdJsUznoOnKXaS9W9q44m/zwgvI9 SOYlef/1BwrWOV++QKaxH5fqHLi6tGxHmG0JouTxiOv0LrYeXqnMKPAe X-Gm-Gg: ASbGncsd3aS3+qu2UyyxHafgWjbGRcFFocBz/Om+NEKFmkj1BXfJ8C8zZ48tZerJn7+ 788d/fqbzZjiJLSDR7y60oqwSN0g6JaGHY5yAxOiZf161SyzV/x//EMe5UdokNjyMfP/tciPHUk JD4DGZJ5Su1uiFjoYCvMY6X9qUX4WakUp/99sFigu/hm3g7b4LrCwXpCA6LSBGDeHU2n5jG/2NV qHikmyHjyQfMY0ZtIwxsdNISFHu0ulR0+wbyZ23sk4ySoylRoPR1m/LLvHTE78n2DUuClWYM4MR AA8ZaOLtHzXqSFq7mBd1q6nrD/QAes9Pf9eXw2qU+YgF5DsLJ3cb+XylHs6gMadL8LezA7pzSun r3L/eDyWDZ9bQjoz2mR2huuSHTdpsSDXkCSiqbR24pVN9NK5K4xkd0Exh2y7J+bPIJhcF1MM0Vi KanW1NkTKI4EDdmqnKwXAv2hi2SO8KVs+5i2ltLaxgTd4Bdkw= X-Google-Smtp-Source: AGHT+IEWVgvs7HfQh1Jftv0/sg1EMetPp1p1pbe1p7lp0IBRgw/xws/S7S4QHSHoK5/x2kKFyy9WJQ== X-Received: by 2002:a05:7022:1093:b0:11a:2ec0:dd02 with SMTP id a92af1059eb24-11c9d85fed4mr7996088c88.33.1764028101157; Mon, 24 Nov 2025 15:48:21 -0800 (PST) Received: from localhost (ip70-175-132-216.oc.oc.cox.net. [70.175.132.216]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-11c93db556csm74670928c88.1.2025.11.24.15.48.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 24 Nov 2025 15:48:20 -0800 (PST) From: Chaitanya Kulkarni To: axboe@kernel.dk, agk@redhat.com, snitzer@kernel.org, mpatocka@redhat.com, song@kernel.org, yukuai@fnnas.com, hch@lst.de, sagi@grimberg.me, kch@nvidia.com, jaegeuk@kernel.org, chao@kernel.org, cem@kernel.org Cc: linux-block@vger.kernel.org, linux-kernel@vger.kernel.org, dm-devel@lists.linux.dev, linux-raid@vger.kernel.org, linux-nvme@lists.infradead.org, linux-f2fs-devel@lists.sourceforge.net, linux-xfs@vger.kernel.org, bpf@vger.kernel.org, Chaitanya Kulkarni Subject: [PATCH V3 0/6] block: ignore __blkdev_issue_discard() ret value Date: Mon, 24 Nov 2025 15:48:00 -0800 Message-Id: <20251124234806.75216-1-ckulkarnilinux@gmail.com> X-Mailer: git-send-email 2.40.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Hi, __blkdev_issue_discard() only returns value 0, that makes post call error checking code dead. This patch series revmoes this dead code at all the call sites and adjust the callers. Please note that it doesn't change the return type of the function from int to void in this series, it will be done once this series gets merged smoothly. For f2fs and xfs I've ran following test which includes discard they produce same PASS and FAIL output with and without this patch series. for-next (lblk-fnext) discard-ret (lblk-discard) --------------------- -------------------------- FAIL f2fs/008 FAIL f2fs/008 FAIL f2fs/014 FAIL f2fs/014 FAIL f2fs/015 FAIL f2fs/015 PASS f2fs/017 PASS f2fs/017 PASS xfs/016 PASS xfs/016 PASS xfs/288 PASS xfs/288 PASS xfs/432 PASS xfs/432 PASS xfs/449 PASS xfs/449 PASS xfs/513 PASS xfs/513 PASS generic/033 PASS generic/033 PASS generic/038 PASS generic/038 PASS generic/098 PASS generic/098 PASS generic/224 PASS generic/224 PASS generic/251 PASS generic/251 PASS generic/260 PASS generic/260 PASS generic/288 PASS generic/288 PASS generic/351 PASS generic/351 PASS generic/455 PASS generic/455 PASS generic/457 PASS generic/457 PASS generic/470 PASS generic/470 PASS generic/482 PASS generic/482 PASS generic/500 PASS generic/500 PASS generic/537 PASS generic/537 PASS generic/608 PASS generic/608 PASS generic/619 PASS generic/619 PASS generic/746 PASS generic/746 PASS generic/757 PASS generic/757 For NVMeOF taret I've testing blktest with nvme_trtype=nvme_loop and all the testcases are passing. -ck Changes from V2:- 1. Add Reviewed-by: tags. 2. Split patch 2 into two separate patches dm and md. 3. Condense __blkdev_issue_discard() parameters for in nvmet patch. 4. Condense __blkdev_issue_discard() parameters for in f2fs patch. Chaitanya Kulkarni (6): block: ignore discard return value md: ignore discard return value dm: ignore discard return value nvmet: ignore discard return value f2fs: ignore discard return value xfs: ignore discard return value block/blk-lib.c | 6 +++--- drivers/md/dm-thin.c | 12 +++++------- drivers/md/md.c | 4 ++-- drivers/nvme/target/io-cmd-bdev.c | 28 +++++++--------------------- fs/f2fs/segment.c | 10 +++------- fs/xfs/xfs_discard.c | 27 +++++---------------------- fs/xfs/xfs_discard.h | 2 +- 7 files changed, 26 insertions(+), 63 deletions(-) -- 2.40.0