From: "Jelinek, Sarah" <sarah.jelinek@intel.com>
To: "faibish_sorin@emc.com" <faibish_sorin@emc.com>,
"adilger@whamcloud.com" <adilger@whamcloud.com>
Cc: "linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>
Subject: Re: Inode metadata and file data syncing
Date: Thu, 19 Jul 2012 13:18:05 +0000 [thread overview]
Message-ID: <CC2D6417.1AA12%sarah.jelinek@intel.com> (raw)
In-Reply-To: <F0897B83F4BC10458936F08DADF233160D547861B1@MX02A.corp.emc.com>
No, I am sorry I can't give you any details at this time. Its still under
development.
On 7/19/12 7:12 AM, "faibish_sorin@emc.com" <faibish_sorin@emc.com> wrote:
>Involving in writing a new file system as a job requirement? There should
>be some value or special features that the file system has, maybe? Could
>you tell what new FS features this introduces?
>
>/Sorin
>
>-----Original Message-----
>From: linux-fsdevel-owner@vger.kernel.org
>[mailto:linux-fsdevel-owner@vger.kernel.org] On Behalf Of Jelinek, Sarah
>Sent: Thursday, July 19, 2012 8:45 AM
>To: Andreas Dilger
>Cc: linux-fsdevel@vger.kernel.org
>Subject: Re: Inode metadata and file data syncing
>
>I am doing a project for my company.
>
>On 7/18/12 7:48 PM, "Andreas Dilger" <adilger@whamcloud.com> wrote:
>
>>On 2012-07-18, at 9:53, "Jelinek, Sarah" <sarah.jelinek@intel.com> wrote:
>>> I am in the process of writing a file system in Linux. This file system
>>> has a separate mechanism by which we manage metadata so I do not want
>>>to
>>> write the file inode metadata to disk without explicitly requesting an
>>> update. I do need the file data pages to be written to disk as per the
>>> normal writeback process.
>>
>>The first, most important, question is why are you writing a new
>>filesystem for Linux? There are lots of filesystems already, and the
>>amount of effort to write a complete filesystem (instead of a simple
>>filesystem with only basic functionality) is fairly high.
>>
>>Unless there is an overwhelmingly good reason to implement a new
>>filesystem, it is better to improve some other existing filesystem to
>>have the feature(s) that you are missing, instead of creating a new one.
>>That helps you avoid a lot of effort, and adds value to everyone else
>>that is using the existing filesystem, instead of making a niche
>>filesystem only useful to yourself and needing ongoing maintenance.
>>
>>> If I use the common mechanism of creating an inode and inserting it
>>>into
>>> the hash via insert_inode_locked(), the inode will be in the I_NEW
>>>state
>>> and when the inode is marked dirty it will be put on the dirty list and
>>> eventually flushed out to disk. One way I thought I could get around
>>>this
>>> is by initializing the inode to i_state = I_DIRTY, skipping I_NEW, and
>>> using insert_inode_hash() instead, so that if mark_inode_dirty() is
>>>called
>>> it won't get put on the dirty list. The issue with this approach is
>>>that
>>> it looks like this inode's pages will not get flushed to disk either
>>>since
>>> it won't ever get on the dirty list. I need the pages written just not
>>>the
>>> inode itself.
>>>
>>> I am handling directory inodes differently. Looking at shmem I see that
>>> the backing_dev_info is set to:
>>>
>>> struct backing_dev_info brnl_backing_dev_info = {
>>> .ra_pages = 0,
>>> .capabilities = BDI_CAP_NO_ACCT_AND_WRITEBACK |
>>>BDI_CAP_SWAP_BACKED,
>>> };
>>>
>>>
>>> I have done the same in my code to prevent directory inodes from being
>>> written to disk.
>>>
>>> Can I manage the inode->i_state with the I_DIRTY flag and then somehow
>>> mark the inode pages dirty and add them to the dirty page list
>>> independently? What I am worried about is what affect doing this will
>>>have
>>> on the processing of anything in page cache or inode cache related to
>>>this
>>> inode.
>>>
>>> Thank you for your help,
>>> Sarah Jelinek
>>>
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe
>>>linux-fsdevel" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>--
>To unsubscribe from this list: send the line "unsubscribe linux-fsdevel"
>in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2012-07-19 13:18 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-18 16:53 Inode metadata and file data syncing Jelinek, Sarah
2012-07-18 17:21 ` Josef Bacik
2012-07-18 17:52 ` Jelinek, Sarah
2012-07-19 1:48 ` Andreas Dilger
2012-07-19 12:44 ` Jelinek, Sarah
2012-07-19 13:12 ` faibish_sorin
2012-07-19 13:18 ` Jelinek, Sarah [this message]
2012-07-19 13:50 ` faibish_sorin
2012-07-19 16:49 ` Zach Brown
2012-07-19 17:11 ` Andreas Dilger
2012-07-19 18:08 ` Matthew Wilcox
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CC2D6417.1AA12%sarah.jelinek@intel.com \
--to=sarah.jelinek@intel.com \
--cc=adilger@whamcloud.com \
--cc=faibish_sorin@emc.com \
--cc=linux-fsdevel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).