From owner-freebsd-emulation Wed Oct 9 0:29:55 2002 Delivered-To: freebsd-emulation@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 088BF37B411; Wed, 9 Oct 2002 00:29:51 -0700 (PDT) Received: from baraca.united.net.ua (ns.united.net.ua [193.111.8.193]) by mx1.FreeBSD.org (Postfix) with ESMTP id 0F9A143EAC; Wed, 9 Oct 2002 00:29:43 -0700 (PDT) (envelope-from max@vega.com) Received: from vega.vega.com (xDSL-2-2.united.net.ua [193.111.9.226]) by baraca.united.net.ua (8.11.6/8.11.6) with ESMTP id g997TNg07866; Wed, 9 Oct 2002 10:29:24 +0300 (EEST) (envelope-from max@vega.com) Received: from vega.vega.com (max@localhost [127.0.0.1]) by vega.vega.com (8.12.6/8.12.5) with ESMTP id g997TKTs081308; Wed, 9 Oct 2002 10:29:20 +0300 (EEST) (envelope-from sobomax@FreeBSD.org) Received: (from max@localhost) by vega.vega.com (8.12.6/8.12.5/Submit) id g997TG4h081307; Wed, 9 Oct 2002 10:29:16 +0300 (EEST) Date: Wed, 9 Oct 2002 10:29:16 +0300 From: Maxim Sobolev To: Marcel Moolenaar Cc: Alfred Perlstein , emulation@FreeBSD.ORG, audit@FreeBSD.ORG Subject: Re: Linuxlator patches for review [was: Who currently maintains Linuxlator?] Message-ID: <20021009072916.GA81270@vega.vega.com> References: <20021006110243.GA70993@vega.vega.com> <20021006183650.GY95327@elvis.mu.org> <3DA2BE32.BDEC28B5@FreeBSD.org> <20021009010309.GA607@dhcp01.pn.xcllnt.net> Mime-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: <20021009010309.GA607@dhcp01.pn.xcllnt.net> User-Agent: Mutt/1.4i Sender: owner-freebsd-emulation@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org On Tue, Oct 08, 2002 at 06:03:09PM -0700, Marcel Moolenaar wrote: > On Tue, Oct 08, 2002 at 02:14:58PM +0300, Maxim Sobolev wrote: > > See attached. As it was suggested, I'm also CC'ing marcel, > > freebsd-audit and freebsd-emulation. I'd like to hear any suggestion > > or comments. Please note that changes were not tested on alpha, > > because I don't have any axp hardware. > > I can test on Alpha, provided the patch is against -current. Do you > have some test cases? Unfortunately no, I don't have any test cases. However it should be OK to only check that the code actually compiles on alpha and existing apps still work fine. > > /* > > + * VFAT > > + */ > > +#define LINUX_VFAT_READDIR_BOTH 0x7201 > > + > > +#define LINUX_IOCTL_VFAT_MIN LINUX_VFAT_READDIR_BOTH > > +#define LINUX_IOCTL_VFAT_MAX LINUX_VFAT_READDIR_BOTH > > + > > +/* > > Silly question: The VFAT ioctl is not implemented. Is there a bigger > picture? > Also: is it worth having a VFAT handler if there's only 1 ioctl (again, > there might be a bigger picture)? I've added it just to avoid bogus "ioctl not implemented" kernel messages, which slows down a machine quite a bit. It should be OK, because VFAT ioctl doesn't make sense for UFS (on FreeBSD) or extfs (on Linux) anyway. > > > case LINUX_IPC_INFO: > > case LINUX_SEM_INFO: > > - error = copyin((caddr_t)args->arg.buf, &linux_seminfo, > > - sizeof(linux_seminfo) ); > > - if (error) > > - return error; > > bcopy(&seminfo, &linux_seminfo, sizeof(linux_seminfo) ); > > /* XXX BSD equivalent? > > #define used_semids 10 > > > Should the copyin be replaced with something or was the copyin bogus? > > Other than that: looks good. I merely skimmed over it, so I may have > missed a detail.... There is no point to do copyin(), because this particular system call doesn't use information from user supplied buffer, but fills this buffer with information instead. I've figured out this from reading Linux kernel sources. -Maxim To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-emulation" in the body of the message