Re: [PATCH v4 6/8] ext4: move inode extension checks out from ext4_iomap_alloc()

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

To: Ritesh Harjani <riteshh@xxxxxxxxxxxxx>
Subject: Re: [PATCH v4 6/8] ext4: move inode extension checks out from ext4_iomap_alloc()
From: Matthew Bobrowski <mbobrowski@xxxxxxxxxxxxxx>
Date: Wed, 9 Oct 2019 21:39:37 +1100
Cc: tytso@xxxxxxx, jack@xxxxxxx, adilger.kernel@xxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, hch@xxxxxxxxxxxxx, david@xxxxxxxxxxxxx, darrick.wong@xxxxxxxxxx
In-reply-to: <20191009063023.E332942047@d06av24.portsmouth.uk.ibm.com>
References: <cover.1570100361.git.mbobrowski@mbobrowski.org> <d1ca9cc472175760ef629fb66a88f0c9b0625052.1570100361.git.mbobrowski@mbobrowski.org> <20191009063023.E332942047@d06av24.portsmouth.uk.ibm.com>
User-agent: Mutt/1.10.1 (2018-07-13)

On Wed, Oct 09, 2019 at 12:00:22PM +0530, Ritesh Harjani wrote:
> On 10/3/19 5:04 PM, Matthew Bobrowski wrote:
> This looks good. Should solve our previous lengthy discussion
> about orphan handling :)

Yeah, although I'm still not the biggest fan of this approach.

> You may add:
> Reviewed-by: Ritesh Harjani <riteshh@xxxxxxxxxxxxx>

Thanks Ritesh!

--<M>--

References:
- [PATCH v4 0/8] ext4: port direct I/O to iomap infrastructure
  - From: Matthew Bobrowski
- [PATCH v4 6/8] ext4: move inode extension checks out from ext4_iomap_alloc()
  - From: Matthew Bobrowski
- Re: [PATCH v4 6/8] ext4: move inode extension checks out from ext4_iomap_alloc()
  - From: Ritesh Harjani

Prev by Date: Re: [PATCH v4 6/8] ext4: move inode extension checks out from ext4_iomap_alloc()
Next by Date: Re: [PATCH v4 7/8] ext4: reorder map.m_flags checks in ext4_set_iomap()
Previous by thread: Re: [PATCH v4 6/8] ext4: move inode extension checks out from ext4_iomap_alloc()
Next by thread: [PATCH v4 7/8] ext4: reorder map.m_flags checks in ext4_set_iomap()
Index(es):
- Date
- Thread

[Index of Archives] [Reiser Filesystem Development] [Ceph FS] [Kernel Newbies] [Security] [Netfilter] [Bugtraq] [Linux FS] [Yosemite National Park] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Device Mapper] [Linux Media]