Re: [PATCH V1 02/14] Consolidate "post read processing" into a new file

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

To: Chandan Rajendra <chandan@xxxxxxxxxxxxx>
Subject: Re: [PATCH V1 02/14] Consolidate "post read processing" into a new file
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Tue, 23 Apr 2019 22:35:44 -0700
Cc: linux-fsdevel@xxxxxxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, linux-f2fs-devel@xxxxxxxxxxxxxxxxxxxxx, linux-fscrypt@xxxxxxxxxxxxxxx, tytso@xxxxxxx, adilger.kernel@xxxxxxxxx, ebiggers@xxxxxxxxxx, jaegeuk@xxxxxxxxxx, yuchao0@xxxxxxxxxx
In-reply-to: <20190424043730.13683-3-chandan@linux.ibm.com>
References: <20190424043730.13683-1-chandan@linux.ibm.com> <20190424043730.13683-3-chandan@linux.ibm.com>
User-agent: Mutt/1.9.2 (2017-12-15)

On Wed, Apr 24, 2019 at 10:07:18AM +0530, Chandan Rajendra wrote:
> +ifeq (y, $(firstword $(filter y,$(CONFIG_FS_ENCRYPTION) $(CONFIG_FS_VERITY))))
> +obj-y +=	post_read_process.o
> +endif

Please just add a new config option selected by the users.

Also I find the file name rather cumbersome.  Maybe just
read-callbacks.[co] ?

Follow-Ups:
- Re: [PATCH V1 02/14] Consolidate "post read processing" into a new file
  - From: Chandan Rajendra

References:
- [PATCH V1 00/14] Consolidate Post read processing code
  - From: Chandan Rajendra
- [PATCH V1 02/14] Consolidate "post read processing" into a new file
  - From: Chandan Rajendra

Prev by Date: Re: [Qemu-devel] [PATCH v6 5/6] ext4: disable map_sync for async flush
Next by Date: Re: [PATCH v2] ext4: cond_resched in work-heavy group loops
Previous by thread: [PATCH V1 02/14] Consolidate "post read processing" into a new file
Next by thread: Re: [PATCH V1 02/14] Consolidate "post read processing" into a new file
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]