From owner-freebsd-emulation Wed Oct 9 19:11:48 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 DA70737B401; Wed, 9 Oct 2002 19:11:46 -0700 (PDT) Received: from kayak.xcllnt.net (209-128-86-226.BAYAREA.NET [209.128.86.226]) by mx1.FreeBSD.org (Postfix) with ESMTP id A125843E65; Wed, 9 Oct 2002 19:11:45 -0700 (PDT) (envelope-from marcel@xcllnt.net) Received: from dhcp01.pn.xcllnt.net (dhcp01.pn.xcllnt.net [192.168.4.201]) by kayak.xcllnt.net (8.12.6/8.12.6) with ESMTP id g9A2Bj0N006634; Wed, 9 Oct 2002 19:11:45 -0700 (PDT) (envelope-from marcel@kayak.pn.xcllnt.net) Received: from dhcp01.pn.xcllnt.net (localhost [127.0.0.1]) by dhcp01.pn.xcllnt.net (8.12.6/8.12.6) with ESMTP id g9A2Dfj3000727; Wed, 9 Oct 2002 19:13:41 -0700 (PDT) (envelope-from marcel@dhcp01.pn.xcllnt.net) Received: (from marcel@localhost) by dhcp01.pn.xcllnt.net (8.12.6/8.12.6/Submit) id g9A2DeWF000726; Wed, 9 Oct 2002 19:13:40 -0700 (PDT) Date: Wed, 9 Oct 2002 19:13:40 -0700 From: Marcel Moolenaar To: Maxim Sobolev Cc: Alfred Perlstein , emulation@FreeBSD.ORG, audit@FreeBSD.ORG Subject: Re: Linuxlator patches for review [was: Who currently maintains Linuxlator?] Message-ID: <20021010021340.GA656@dhcp01.pn.xcllnt.net> References: <20021006110243.GA70993@vega.vega.com> <20021006183650.GY95327@elvis.mu.org> <3DA2BE32.BDEC28B5@FreeBSD.org> <20021009010309.GA607@dhcp01.pn.xcllnt.net> <20021009072916.GA81270@vega.vega.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20021009072916.GA81270@vega.vega.com> User-Agent: Mutt/1.5.1i 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 Wed, Oct 09, 2002 at 10:29:16AM +0300, Maxim Sobolev wrote: > 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. It compiles cleanly and loads on Alpha. I'm happy... > > > + * 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. Hmmm.... That sort of defeats the purpose of the message and we do have msdosfs (not that it has the support yet). The intend is that people get annoyed enough to actually implement missing functionality... If you keep it, please rename LINUX_VFAT_READDIR_BOTH to LINUX_VFAT_IOCTL_READDIR_BOTH to match the name used in the Linux sources (better cross reference). -- Marcel Moolenaar USPA: A-39004 marcel@xcllnt.net To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-emulation" in the body of the message