From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH 1/7] FS-Cache: Provide a filesystem-specific sync'able page bit Date: Fri, 21 Apr 2006 03:33:29 -0700 Message-ID: <20060421033329.35f661e1.akpm@osdl.org> References: <20060420171216.4cdd369a.akpm@osdl.org> <20060420165927.9968.33912.stgit@warthog.cambridge.redhat.com> <28997.1145614975@warthog.cambridge.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: aviro@redhat.com, sct@redhat.com, nfsv4@linux-nfs.org, steved@redhat.com, linux-kernel@vger.kernel.org, dhowells@redhat.com, torvalds@osdl.org, linux-cachefs@redhat.com, linux-fsdevel@vger.kernel.org Return-path: To: David Howells In-Reply-To: <28997.1145614975@warthog.cambridge.redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: nfsv4-bounces@linux-nfs.org Errors-To: nfsv4-bounces@linux-nfs.org List-Id: linux-fsdevel.vger.kernel.org David Howells wrote: > > Andrew Morton wrote: > > > It would be better to rename PG_checked to PG_fs_misc kernel-wide. > > So would deleting PG_checked and changing the PageChecked() macros to: > > #define PageChecked(page) PageFsMisc((page)) > #define SetPageChecked(page) SetPageFsMisc((page)) > #define ClearPageChecked(page) ClearPageFsMisc((page)) > > be acceptable? Or would you rather I replaced those too? > PG_checked is presently a misc bit which only filesystems use. So yes, I'd say it's appropriate to remove PageChecked() and friends altogether. That might break out-of-tree filesystems, but they'll work it out.