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
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: [PATCH V1 14/14] ext4: Enable encryption for subpage-sized blocks
Next by Date: Re: [PATCH V1 02/14] Consolidate "post read processing" into a new file
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 Cryptography] [Asterisk App Development] [PJ SIP] [Gnu Gatekeeper] [IETF Sipping] [Info Cyrus] [ALSA User] [Fedora Linux Users] [Linux SCTP] [DCCP] [Gimp] [Yosemite News] [Deep Creek Hot Springs] [Yosemite Campsites] [ISDN Cause Codes]

Powered by Linux