From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) (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 F1B1E3FC0 for ; Thu, 19 Aug 2021 20:35:39 +0000 (UTC) Received: by mail-pl1-f173.google.com with SMTP id x1so2068986plg.10 for ; Thu, 19 Aug 2021 13:35:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=2lq1/9dzMwse8hgj+nLsRtNNVHy5Ra8cI/pFGIqflik=; b=2RKmo3Q2iKNSxDasfmEsswnoQ8z3YAyLtAaZ/HjlU037dI8WGM89auY+HT0DGdfQG5 U/MVHxJdBMIcF+Bd+xSvtL95UEX2dwFl/L/z3oPUnO/aigHxiu57hkL3swk7sYKK8eoK OPWWqbTNA4eO3pFQE3oPGvv1E3fjsnrOGOVy65plV9Zqf0TYsbeSGghf1ADPutP9iZR1 5qcOPhlcQ0vwLOx7+tVxh274/kANbc4PzXIIj1zUrQt/iw6E5iILNWMZccVLZCykIsql szt7T9dgmeVXyvOI4dVbuIMMnv9/0iFbmrxmbHF8Fdu9z40bAQVIYwD2gzFvHS2c+7l1 Qchg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=2lq1/9dzMwse8hgj+nLsRtNNVHy5Ra8cI/pFGIqflik=; b=Km+mRghHdGDZbAuC6OtiRFeBpawc+VtCLrIyWax4DdwP41PLWV4vbyqTzfTquFmnNm K62vviabzb9Rze5RJwbxRn5C/pmC3I3wEGppPFjm/3WfRjQNlEhYcdRkO8nK8o8gY/wA JiaVn/vlxYj8izQXe2wXpvklakkGCJHUjveWkw2Y+1uorifG3IZvlC6++dPa7EpFKHR4 4VM2LmWH8H6Wr14fuYb/z8rw0mvdi5+7A1VWOqgwXvujmvs9Qkpkg565N7Cxz3Jbiju5 S/3wbltd2MyoypIxDR6RREfFtBMyOZdB0y1kljIxBeIBr6+ZGCEPc4JcimUufgCh9rIt vOoQ== X-Gm-Message-State: AOAM531ylVJXE+34RQ87xsn0/edHxWZh6d9mXLzqWYVDCBVd6nwbKpIB uKwKWO5Ztyoz7o/l3sO0uxdLxoUdniWUPhbYgKGeiw== X-Google-Smtp-Source: ABdhPJxSp6xtccEDJJJY44GOHyKJniTxZ6vHD+7V+PQJrY3uECT6oRKOpiFktsbnpDGqKIO/PuciS0G+rhFl437R2uk= X-Received: by 2002:a17:90a:708c:: with SMTP id g12mr628765pjk.13.1629405339323; Thu, 19 Aug 2021 13:35:39 -0700 (PDT) Precedence: bulk X-Mailing-List: nvdimm@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20210809061244.1196573-1-hch@lst.de> <20210809061244.1196573-8-hch@lst.de> In-Reply-To: <20210809061244.1196573-8-hch@lst.de> From: Dan Williams Date: Thu, 19 Aug 2021 13:35:28 -0700 Message-ID: Subject: Re: [PATCH 07/30] fsdax: mark the iomap argument to dax_iomap_sector as const To: Christoph Hellwig Cc: "Darrick J. Wong" , Matthew Wilcox , Andreas Gruenbacher , Shiyang Ruan , linux-xfs , linux-fsdevel , linux-btrfs , Linux NVDIMM , cluster-devel@redhat.com Content-Type: text/plain; charset="UTF-8" On Sun, Aug 8, 2021 at 11:19 PM Christoph Hellwig wrote: I'd prefer a non-empty changelog that said something like: "In preparation for iomap_iter() update iomap helpers to treat the iomap argument as read-only." Just to leave a breadcrumb for the motivation for this change, but no major worry if you don't respin for that. > > Signed-off-by: Christoph Hellwig > Reviewed-by: Darrick J. Wong Otherwise, LGTM Reviewed-by: Dan Williams