Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 2 Apr 2007 10:38:03 -0700
From:      Alfred Perlstein <alfred@freebsd.org>
To:        Robert Watson <rwatson@FreeBSD.org>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, Julian Elischer <julian@FreeBSD.org>, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/compat/linux linux_file.c linux_util.h src/sys/i386/linux linux.h linux_dummy.c linux_proto.h linux_syscall.h linux_sysent.c syscalls.master src/sys/amd64/linux32 linux.h linux32_dummy.c linux32_proto.h ...
Message-ID:  <20070402173803.GV61362@elvis.mu.org>
In-Reply-To: <20070401104935.C75869@fledge.watson.org>
References:  <200703290211.l2T2BlHv081735@repoman.freebsd.org> <20070401104935.C75869@fledge.watson.org>

next in thread | previous in thread | raw e-mail | index | archive | help
* Robert Watson <rwatson@FreeBSD.org> [070401 07:57] wrote:
> 
> On Thu, 29 Mar 2007, Julian Elischer wrote:
> 
> >julian      2007-03-29 02:11:46 UTC
> >
> > FreeBSD src repository
> >
> > Modified files:
> >   sys/compat/linux     linux_file.c linux_util.h
> >   sys/i386/linux       linux.h linux_dummy.c linux_proto.h
> >                        linux_syscall.h linux_sysent.c
> >                        syscalls.master
> >   sys/amd64/linux32    linux.h linux32_dummy.c linux32_proto.h
> >                        linux32_syscall.h linux32_sysent.c
> >                        syscalls.master
> > Log:
> > Implement the openat() linux syscall
> > Submitted by:   Roman Divacky (rdivacky@)
> > MFC after:      2 weeks
> 
> The locking and reference counting in this patch is highly dubious; also, 
> vn_fullpath() should really not be used this way, as it fails whenever 
> vnodes fall out of the name cache or a file system doesn't use the 
> namecache (i.e., it is purely advisory).  If we're going to do openat() in 
> the Linux emulation layer, let's instead implement it properly in the 
> FreeBSD system call code as a first class service and then wrap it the way 
> we wrap other system calls. Please do not MFC this patch as-is.

While I really appreciate the work here, I do have to agree with
Robert, this should have been implemented in FreeBSD proper and then
wrapped in the emulation layer.

-- 
- Alfred Perlstein



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070402173803.GV61362>