* 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