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>
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: [RESEND, PATCH 3/3] fuse: Add FOPEN_STREAM and use stream_open() if filesystem returned that from open handler
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] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [Samba] [Device Mapper] [CEPH Development]