From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7D794C76195 for ; Mon, 20 Mar 2023 13:01:05 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231315AbjCTNBD (ORCPT ); Mon, 20 Mar 2023 09:01:03 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49544 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231414AbjCTNAq (ORCPT ); Mon, 20 Mar 2023 09:00:46 -0400 Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:3::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5C8C5193D4; Mon, 20 Mar 2023 06:00:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Transfer-Encoding :Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Sender:Reply-To:Content-ID:Content-Description; bh=+crC4dGYsz6T67x08q4z29IFmwO1ZpfQLhOdC3e6Aw8=; b=4YcwwUhnutcuyzWQ8rvJd5hPw8 yNdL1CfyZXNDmF9BYcuE7B35iFZMmSHv3SLi9AGW+B07w6xasyrV8hM43RLboep9DG7iXLTkyvVUw hRdJLkRyTn8t9jyw3ruR6+X29aPCnXdaYX4nZ1gnldqUuZRLD57P08bkyecBNKVYAe+B89uJ4pnhH Z27mgh2IwuxyfBSkA6r7tEF20rH5mA5bpP+M4Aami04F7HWSRsU85+Ueu/J8ahVSPz9rnUrOa1jGC xKas1swgd+vCVxxEH1EwXV3YgL6B39nRF85qGzdA+7DRtRlrI/mgUHNi4fMn3eB8wxO6EG2THwvaj rWEtdvvQ==; Received: from hch by bombadil.infradead.org with local (Exim 4.96 #2 (Red Hat Linux)) id 1peF7f-0092BU-2v; Mon, 20 Mar 2023 13:00:23 +0000 Date: Mon, 20 Mar 2023 06:00:23 -0700 From: Christoph Hellwig To: Chao Yu Cc: Christoph Hellwig , martin.petersen@oracle.com, jejb@linux.ibm.com, bvanassche@acm.org, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v5] scsi: support packing multi-segment in UNMAP command Message-ID: References: <20230310123604.1820231-1-chao@kernel.org> <9e3ffff0-6cc8-b01a-4d2e-5ec49a936415@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <9e3ffff0-6cc8-b01a-4d2e-5ec49a936415@kernel.org> X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Mar 13, 2023 at 09:44:47AM +0800, Chao Yu wrote: > IIUC,originally, HPB driver tries to lookup LBA range{,s} from WRITE/DISCARD > request, and will dirty mapped HPB regions based on LBA range{,s}, do you mean > HPB driver should not parse DISCARD request? Eww.. This just means we really need to drop this bogus HPB code. It will otherwise break every single time when a new SCSI command get added that modifies LBA content.