Re: [PATCH 3/6] iomap: drop pos param from __iomap_[get|put]_folio()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH 3/6] iomap: drop pos param from __iomap_[get|put]_folio()
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Mon, 5 May 2025 02:01:33 -0700
Cc: linux-fsdevel@xxxxxxxxxxxxxxx, linux-xfs@xxxxxxxxxxxxxxx
In-reply-to: <20250430190112.690800-4-bfoster@redhat.com>
References: <20250430190112.690800-1-bfoster@redhat.com> <20250430190112.690800-4-bfoster@redhat.com>

On Wed, Apr 30, 2025 at 03:01:09PM -0400, Brian Foster wrote:
> Both helpers take the iter and pos as parameters. All callers
> effectively pass iter->pos, so drop the unnecessary pos parameter.

Looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

References:
- [PATCH 0/6] iomap: misc buffered write path cleanups and prep
  - From: Brian Foster
- [PATCH 3/6] iomap: drop pos param from __iomap_[get|put]_folio()
  - From: Brian Foster

Prev by Date: Re: [PATCH 2/6] iomap: drop unnecessary pos param from iomap_write_[begin|end]
Next by Date: Re: [PATCH 4/6] iomap: helper to trim pos/bytes to within folio
Previous by thread: Re: [PATCH 3/6] iomap: drop pos param from __iomap_[get|put]_folio()
Next by thread: [PATCH 5/6] iomap: push non-large folio check into get folio path
Index(es):
- Date
- Thread

[Index of Archives] [XFS Filesystem Development (older mail)] [Linux Filesystem Development] [Linux Audio Users] [Yosemite Trails] [Linux Kernel] [Linux RAID] [Linux SCSI]

Powered by Linux