Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261769AbVCGPln (ORCPT ); Mon, 7 Mar 2005 10:41:43 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261782AbVCGPln (ORCPT ); Mon, 7 Mar 2005 10:41:43 -0500 Received: from mx1.redhat.com ([66.187.233.31]:15283 "EHLO mx1.redhat.com") by vger.kernel.org with ESMTP id S261769AbVCGPlm (ORCPT ); Mon, 7 Mar 2005 10:41:42 -0500 From: David Howells In-Reply-To: References: <20050307041047.59c24dec.akpm@osdl.org> <20050307034747.4c6e7277.akpm@osdl.org> <20050307033734.5cc75183.akpm@osdl.org> <20050303123448.462c56cd.akpm@osdl.org> <20050302135146.2248c7e5.akpm@osdl.org> <20050302090734.5a9895a3.akpm@osdl.org> <9420.1109778627@redhat.com> <31789.1109799287@redhat.com> <13767.1109857095@redhat.com> <9268.1110194624@redhat.com> <9741.1110195784@redhat.com> <9947.1110196314@redhat.com> <22447.1110204304@redhat.com> To: Miklos Szeredi Cc: akpm@osdl.org, torvalds@osdl.org, davidm@snapgear.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] BDI: Provide backing device capability information X-Mailer: MH-E 7.82; nmh 1.0.4; GNU Emacs 21.3.50.1 Date: Mon, 07 Mar 2005 15:41:21 +0000 Message-ID: <24382.1110210081@redhat.com> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 945 Lines: 21 Miklos Szeredi wrote: > > The attached patch replaces backing_dev_info::memory_backed with > > capabilitied bitmap. The capabilities available include: > > Wouldn't it be better to reverse the meaning of BDI_CAP_ACCOUNT_DIRTY > and BDI_CAP_WRITEBACK_DIRTY (BDI_CAP_NO_ACCOUNT_DIRTY...)? That way > out of tree filesystems that implicitly zero bdi->memory_backed > wouldn't _silently_ break. E.g. fuse does this, though it would not > actually break since it doesn't dirty any pages currently. I have no > idea whether there are other filesystems that are affected. It shouldn't silently break... It will refuse to compile. I renamed "memory_backed" to "capabilities". David - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/