logo       

Re: v9fs release: msg#00595

os.plan9.general

Subject: Re: v9fs release

Please don't bother. We are not proposing to merge this with any code base, nor are we looking for a review of the design or code. This is simply a work-in-progress that we wish to make available to those who are interested. Constructive feedback is always appreciated.

It is entirely likely that this will morph into something that is of no interest to the 9p folks, since we are primarily concerned with developing for our local environment. Expect to see all sorts of add-ons in the source tree.

Greg

On Thu, 28 Nov 2002, Greg Watson wrote:

...and we are planning to mount users home directories, etc. using
v9fs. There's an automounter that mounts the filesystems in a private
namespace each time a process is executed on a node. When the process
terminates the filesystems are auto unmounted.

Umm... Why the devil was that stuff mixed with 9P support, again? Do one
thing and do it well and all such...

Frankly, after the look through the source... Folks, it's unmergable.
There are 4 or 5 unrelated patches trying to get out of that beast and
as it is the combination will be vetoed.

And please, please, use the sane coding style. Either one described in
Documentation/CodingStyle or the style common in Plan 9 kernel - they
are close enough.

I can post detailed review either on fsdevel or here tomorrow (probably
the former - code and design review of that stuff doesn't look on-topic for
9fans). And no, it won't be pleasant - aside of very sloppy programming in
a lot of places, there are outright races aplenty.

/me sadly shakes head and ponders adding "Required reading: K&P" into the
place where required versions of cc/binutils/etc. are described...




<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

News | FAQ | advertise