* Re: [PATCHSET 0/8 version 4] exofs for kernel 2.6.30
[not found] <49C1331D.1080805@panasas.com>
@ 2009-03-30 21:22 ` Andrew Morton
2009-03-31 3:01 ` Stephen Rothwell
2009-03-31 7:20 ` Boaz Harrosh
0 siblings, 2 replies; 6+ messages in thread
From: Andrew Morton @ 2009-03-30 21:22 UTC (permalink / raw)
To: Boaz Harrosh
Cc: avishay, jeff, linux-fsdevel, osd-dev, zbr, James.Bottomley,
linux-kernel, fujita.tomonori, linux-next
On Wed, 18 Mar 2009 19:45:01 +0200
Boaz Harrosh <bharrosh@panasas•com> wrote:
> What's new since last iteration:
>
> * I completely re-wrote the [PATCH 4/8] exofs: address_space_operations
> in which we actually write/read to/from osd-storage. The difference is
> that now we try to accumulate as many contiguous pages as possible and
> send them as one large request. As opposed to writing each page at a
> time, in the previous patchset.
>
> * [PATCH 5/8] exofs: dir_inode and directory operations received lots
> of love thanks to Evgeniy Polyakov's grate comments.
>
> exofs is a file system that uses an OSD device as it's back store.
>
> OSD is a new T10 command set that views storage devices not as a large/flat
> array of sectors but as a container of objects, each having a length, quota,
> time attributes and more. Each object is addressed by a 64bit ID, and is
> contained in a 64bit ID partition. Each object has associated attributes
> attached to it, which are integral part of the object and provide metadata about
> the object. The standard defines some common obligatory attributes, but user
> attributes can be added as needed.
>
> Here is the list of patches
> [PATCH 1/8] exofs: Kbuild, Headers and osd utils
> [PATCH 2/8] exofs: file and file_inode operations
> [PATCH 3/8] exofs: symlink_inode and fast_symlink_inode operations
> [PATCH 4/8] exofs: address_space_operations
> [PATCH 5/8] exofs: dir_inode and directory operations
> [PATCH 6/8] exofs: super_operations and file_system_type
> [PATCH 7/8] exofs: Documentation
> [PATCH 8/8] fs: Add exofs to Kernel build
Are all the prerequisites for exofs now in mainline?
> This patchset is also available on:
> git-clone git://git.open-osd.org/linux-open-osd.git linux-next
> or on the web at:
> http://git.open-osd.org/gitweb.cgi?p=linux-open-osd.git;a=shortlog;h=refs/heads/linux-next
Well I could merge them, but given that you have a git tree, a more
convenient path would be for us to include your tree in linux-next and
then you ask Linus to pull it directly when the time comes.
I'm unsure when that time will come. Who has reviewed this work and
what was the result?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCHSET 0/8 version 4] exofs for kernel 2.6.30
2009-03-30 21:22 ` [PATCHSET 0/8 version 4] exofs for kernel 2.6.30 Andrew Morton
@ 2009-03-31 3:01 ` Stephen Rothwell
2009-03-31 7:13 ` Evgeniy Polyakov
2009-03-31 7:20 ` Boaz Harrosh
1 sibling, 1 reply; 6+ messages in thread
From: Stephen Rothwell @ 2009-03-31 3:01 UTC (permalink / raw)
To: Andrew Morton
Cc: Boaz Harrosh, avishay, jeff, linux-fsdevel, osd-dev, zbr,
James.Bottomley, linux-kernel, fujita.tomonori, linux-next
[-- Attachment #1: Type: text/plain, Size: 1096 bytes --]
On Mon, 30 Mar 2009 14:22:00 -0700 Andrew Morton <akpm@linux-foundation•org> wrote:
>
> > This patchset is also available on:
> > git-clone git://git.open-osd.org/linux-open-osd.git linux-next
> > or on the web at:
> > http://git.open-osd.org/gitweb.cgi?p=linux-open-osd.git;a=shortlog;h=refs/heads/linux-next
>
> Well I could merge them, but given that you have a git tree, a more
> convenient path would be for us to include your tree in linux-next and
> then you ask Linus to pull it directly when the time comes.
$ git cat-file -p next-20090330:Next/Trees | grep osd
osd git git://git.open-osd.org/linux-open-osd.git#linux-next
i.e. its in there. Those particular patches have been in linux-next
since next-20090326 and presumably have been built by the
all{yes,mod}config builds I do.
> I'm unsure when that time will come. Who has reviewed this work and
> what was the result?
That is a good question. The other is "Is this destined for 2.6.30?"
--
Cheers,
Stephen Rothwell sfr@canb•auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCHSET 0/8 version 4] exofs for kernel 2.6.30
2009-03-31 3:01 ` Stephen Rothwell
@ 2009-03-31 7:13 ` Evgeniy Polyakov
0 siblings, 0 replies; 6+ messages in thread
From: Evgeniy Polyakov @ 2009-03-31 7:13 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Andrew Morton, Boaz Harrosh, avishay, jeff, linux-fsdevel,
osd-dev, James.Bottomley, linux-kernel, fujita.tomonori,
linux-next
Hi.
On Tue, Mar 31, 2009 at 02:01:24PM +1100, Stephen Rothwell (sfr@canb•auug.org.au) wrote:
> > I'm unsure when that time will come. Who has reviewed this work and
> > what was the result?
>
> That is a good question. The other is "Is this destined for 2.6.30?"
There was a lengthy discussion in scsi list.
I made another review of the previous revision and we talked a bit about
ducumented features/bugs for the current one.
--
Evgeniy Polyakov
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCHSET 0/8 version 4] exofs for kernel 2.6.30
2009-03-30 21:22 ` [PATCHSET 0/8 version 4] exofs for kernel 2.6.30 Andrew Morton
2009-03-31 3:01 ` Stephen Rothwell
@ 2009-03-31 7:20 ` Boaz Harrosh
2009-03-31 7:41 ` Boaz Harrosh
1 sibling, 1 reply; 6+ messages in thread
From: Boaz Harrosh @ 2009-03-31 7:20 UTC (permalink / raw)
To: Andrew Morton, Stephen Rothwell, Linus Torvalds
Cc: avishay, jeff, linux-fsdevel, osd-dev, zbr, James.Bottomley,
linux-kernel, fujita.tomonori, linux-next
On 03/31/2009 12:22 AM, Andrew Morton wrote:
> On Wed, 18 Mar 2009 19:45:01 +0200
> Boaz Harrosh <bharrosh@panasas•com> wrote:
>
>> What's new since last iteration:
>>
>> * I completely re-wrote the [PATCH 4/8] exofs: address_space_operations
>> in which we actually write/read to/from osd-storage. The difference is
>> that now we try to accumulate as many contiguous pages as possible and
>> send them as one large request. As opposed to writing each page at a
>> time, in the previous patchset.
>>
>> * [PATCH 5/8] exofs: dir_inode and directory operations received lots
>> of love thanks to Evgeniy Polyakov's grate comments.
>>
>> exofs is a file system that uses an OSD device as it's back store.
>>
>> OSD is a new T10 command set that views storage devices not as a large/flat
>> array of sectors but as a container of objects, each having a length, quota,
>> time attributes and more. Each object is addressed by a 64bit ID, and is
>> contained in a 64bit ID partition. Each object has associated attributes
>> attached to it, which are integral part of the object and provide metadata about
>> the object. The standard defines some common obligatory attributes, but user
>> attributes can be added as needed.
>>
>> Here is the list of patches
>> [PATCH 1/8] exofs: Kbuild, Headers and osd utils
>> [PATCH 2/8] exofs: file and file_inode operations
>> [PATCH 3/8] exofs: symlink_inode and fast_symlink_inode operations
>> [PATCH 4/8] exofs: address_space_operations
>> [PATCH 5/8] exofs: dir_inode and directory operations
>> [PATCH 6/8] exofs: super_operations and file_system_type
>> [PATCH 7/8] exofs: Documentation
>> [PATCH 8/8] fs: Add exofs to Kernel build
>
> Are all the prerequisites for exofs now in mainline?
>
Yes they are all in
>> This patchset is also available on:
>> git-clone git://git.open-osd.org/linux-open-osd.git linux-next
>> or on the web at:
>> http://git.open-osd.org/gitweb.cgi?p=linux-open-osd.git;a=shortlog;h=refs/heads/linux-next
>
> Well I could merge them, but given that you have a git tree, a more
> convenient path would be for us to include your tree in linux-next
As Stephan said they are there since 2.6.29-rc1
> and
> then you ask Linus to pull it directly when the time comes.
>
I was hoping the time is now
> I'm unsure when that time will come. Who has reviewed this work and
> what was the result?
>
>
The patches have been reveiwed on linux-kernel and linux-fsdevel for
5-6 rounds. Each round drew it's comments which I fixed and so on.
The code is pretty well as far as styling and layout. And I'm not sure
what else can be done for them.
As far as quality robustness and performance, that's hard to say since
it has not been used outside of the labs yet. Perhaps being in mainline
will give the exposure it needs to stabilize.
Exofs is the only current candidate user for the osd in mainline, and in a
shape that does things relatively well as far as we could test it.
It has lots of work still to do on it in order to make it the pNFS-Objects
filesystem it needs to be. One reason it is very important for me that it
will go in mainline is because of the pNFS patches that need to come. pNFS
is an out-of-tree staging area that holds the up coming pNFS stuff that will
be added to Linux. It currently holds patches that will eventually go through
4 different git trees / maintainers. if exofs is left outside it is another
headache added. And of course is the hassle of keeping out-of-tree code for
another round, that's depressing. I don't see what is the danger of inclusion
and what is to be gained with keeping this out-of-tree? Please advise you have
much longer experience with these thing then me.
For long term plans By next kernel exofs should be pNFS capable, and a
pNFS-object-layout driver should be added for the pNFS client side. The
layout driver is based on the same infrastructure as exofs. Then farther
down exofs will need to be multi-device raids and all.
Lots of more work to do, but first thing first, can this be included in
2.6.30?
Thanks
Boaz
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCHSET 0/8 version 4] exofs for kernel 2.6.30
2009-03-31 7:20 ` Boaz Harrosh
@ 2009-03-31 7:41 ` Boaz Harrosh
2009-03-31 8:04 ` Andrew Morton
0 siblings, 1 reply; 6+ messages in thread
From: Boaz Harrosh @ 2009-03-31 7:41 UTC (permalink / raw)
To: Andrew Morton, Evgeniy Polyakov, Stephen Rothwell, Linus Torvalds
Cc: avishay, jeff, linux-fsdevel, osd-dev, James.Bottomley,
linux-kernel, fujita.tomonori, linux-next
On 03/31/2009 10:20 AM, Boaz Harrosh wrote:
> On 03/31/2009 12:22 AM, Andrew Morton wrote:
>> On Wed, 18 Mar 2009 19:45:01 +0200
>> Boaz Harrosh <bharrosh@panasas•com> wrote:
>>
>>> What's new since last iteration:
>>>
>>> * I completely re-wrote the [PATCH 4/8] exofs: address_space_operations
>>> in which we actually write/read to/from osd-storage. The difference is
>>> that now we try to accumulate as many contiguous pages as possible and
>>> send them as one large request. As opposed to writing each page at a
>>> time, in the previous patchset.
>>>
>>> * [PATCH 5/8] exofs: dir_inode and directory operations received lots
>>> of love thanks to Evgeniy Polyakov's grate comments.
>>>
>>> exofs is a file system that uses an OSD device as it's back store.
>>>
>>> OSD is a new T10 command set that views storage devices not as a large/flat
>>> array of sectors but as a container of objects, each having a length, quota,
>>> time attributes and more. Each object is addressed by a 64bit ID, and is
>>> contained in a 64bit ID partition. Each object has associated attributes
>>> attached to it, which are integral part of the object and provide metadata about
>>> the object. The standard defines some common obligatory attributes, but user
>>> attributes can be added as needed.
>>>
>>> Here is the list of patches
>>> [PATCH 1/8] exofs: Kbuild, Headers and osd utils
>>> [PATCH 2/8] exofs: file and file_inode operations
>>> [PATCH 3/8] exofs: symlink_inode and fast_symlink_inode operations
>>> [PATCH 4/8] exofs: address_space_operations
>>> [PATCH 5/8] exofs: dir_inode and directory operations
>>> [PATCH 6/8] exofs: super_operations and file_system_type
>>> [PATCH 7/8] exofs: Documentation
>>> [PATCH 8/8] fs: Add exofs to Kernel build
>> Are all the prerequisites for exofs now in mainline?
>>
>
> Yes they are all in
>
>>> This patchset is also available on:
>>> git-clone git://git.open-osd.org/linux-open-osd.git linux-next
>>> or on the web at:
>>> http://git.open-osd.org/gitweb.cgi?p=linux-open-osd.git;a=shortlog;h=refs/heads/linux-next
>> Well I could merge them, but given that you have a git tree, a more
>> convenient path would be for us to include your tree in linux-next
>
> As Stephan said they are there since 2.6.29-rc1
>
>> and
>> then you ask Linus to pull it directly when the time comes.
>>
>
> I was hoping the time is now
>
>> I'm unsure when that time will come. Who has reviewed this work and
>> what was the result?
>>
>>
>
> The patches have been reveiwed on linux-kernel and linux-fsdevel for
> 5-6 rounds. Each round drew it's comments which I fixed and so on.
>
I forgot to say. Some of the people that sent comments where:
Marcin Slusarz <marcin.slusarz@gmail•com>
Evgeniy Polyakov <zbr@ioremap•net>
Jeff Garzik <jeff@garzik•org>
Pavel Machek <pavel@suse•cz>
Benny Halevy <bhalevy@panasas•com>
Alan Cox <alan@lxorguk•ukuu.org.uk>
Andrew Morton <akpm@linux-foundation•org>
And then there was a long flame war about user-mode API, which I now
have, and all in kernel utilities are gone.
To the best of my knowledge I have addressed all comments, at least
no one complained.
<snip>
Thanks
Boaz
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCHSET 0/8 version 4] exofs for kernel 2.6.30
2009-03-31 7:41 ` Boaz Harrosh
@ 2009-03-31 8:04 ` Andrew Morton
0 siblings, 0 replies; 6+ messages in thread
From: Andrew Morton @ 2009-03-31 8:04 UTC (permalink / raw)
To: Boaz Harrosh
Cc: Evgeniy Polyakov, Stephen Rothwell, Linus Torvalds, avishay, jeff,
linux-fsdevel, osd-dev, James.Bottomley, linux-kernel,
fujita.tomonori, linux-next
On Tue, 31 Mar 2009 10:41:29 +0300 Boaz Harrosh <bharrosh@panasas•com> wrote:
>
> I forgot to say. Some of the people that sent comments where:
> Marcin Slusarz <marcin.slusarz@gmail•com>
> Evgeniy Polyakov <zbr@ioremap•net>
> Jeff Garzik <jeff@garzik•org>
> Pavel Machek <pavel@suse•cz>
> Benny Halevy <bhalevy@panasas•com>
> Alan Cox <alan@lxorguk•ukuu.org.uk>
> Andrew Morton <akpm@linux-foundation•org>
>
> And then there was a long flame war about user-mode API, which I now
> have, and all in kernel utilities are gone.
>
> To the best of my knowledge I have addressed all comments, at least
> no one complained.
>
Let me take a quick look...
<quickly looks>
OK, I foud a few minor things, but the code looks fine to me. However
I cannot speak to the overall design and to the actual desirability of
adding the feature to Linux.
I'd say that unless someone pipes up with a showstopper objection in
the next few days, please send the pull request to Linus near the end of
the week.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-03-31 8:14 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <49C1331D.1080805@panasas.com>
2009-03-30 21:22 ` [PATCHSET 0/8 version 4] exofs for kernel 2.6.30 Andrew Morton
2009-03-31 3:01 ` Stephen Rothwell
2009-03-31 7:13 ` Evgeniy Polyakov
2009-03-31 7:20 ` Boaz Harrosh
2009-03-31 7:41 ` Boaz Harrosh
2009-03-31 8:04 ` Andrew Morton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox