From owner-freebsd-emulation@FreeBSD.ORG Sat Apr 9 19:30:23 2011 Return-Path: <owner-freebsd-emulation@FreeBSD.ORG> Delivered-To: freebsd-emulation@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 57651106564A for <freebsd-emulation@hub.freebsd.org>; Sat, 9 Apr 2011 19:30:23 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 2B6108FC0A for <freebsd-emulation@hub.freebsd.org>; Sat, 9 Apr 2011 19:30:23 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.4/8.14.4) with ESMTP id p39JUMks093081 for <freebsd-emulation@freefall.freebsd.org>; Sat, 9 Apr 2011 19:30:22 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.4/8.14.4/Submit) id p39JUMnE093069; Sat, 9 Apr 2011 19:30:22 GMT (envelope-from gnats) Date: Sat, 9 Apr 2011 19:30:22 GMT Message-Id: <201104091930.p39JUMnE093069@freefall.freebsd.org> To: freebsd-emulation@FreeBSD.org From: Mark Linimon <linimon@lonesome.com> Cc: Subject: Re: kern/155903: FreeBSD32 emulation patch to support i386 X11 Server X-BeenThere: freebsd-emulation@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Mark Linimon <linimon@lonesome.com> List-Id: Development of Emulators of other operating systems <freebsd-emulation.freebsd.org> List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-emulation>, <mailto:freebsd-emulation-request@freebsd.org?subject=unsubscribe> List-Archive: <http://lists.freebsd.org/pipermail/freebsd-emulation> List-Post: <mailto:freebsd-emulation@freebsd.org> List-Help: <mailto:freebsd-emulation-request@freebsd.org?subject=help> List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-emulation>, <mailto:freebsd-emulation-request@freebsd.org?subject=subscribe> X-List-Received-Date: Sat, 09 Apr 2011 19:30:23 -0000 The following reply was made to PR kern/155903; it has been noted by GNATS. From: Mark Linimon <linimon@lonesome.com> To: bug-followup@FreeBSD.org Cc: Subject: Re: kern/155903: FreeBSD32 emulation patch to support i386 X11 Server Date: Sat, 9 Apr 2011 14:27:22 -0500 ----- Forwarded message from John Wehle <john@feith.com> ----- Date: Thu, 24 Mar 2011 17:26:08 -0400 (EDT) From: John Wehle <john@feith.com> To: kostikbel@gmail.com Cc: amd64@freebsd.org Subject: Re: amd64/155903: FreeBSD32 emulation patch to support i386 X11 Server > First, please split the patch into smaller, logically self-contained > parts. E.g. the change to handle fdrop() in one place should be committed > separately. Will do. Okay to just submit the series of patches under amd64/155903 or do you want them file under separate bug reports? > The last commit is the most controversial, in fact. I understand the > reason to get the user memory for calling into pciconf ioctls, but this > is somewhat ugly. Ideally, the pci_ioctl() would be changed into wrapper > and core code, and two wrappers produced, one for the native call path, > other for compat32. I don't necessarily disagree, however that's more work than I'm planning on at the moment. > BTW, would you do the shims for other pciconf ioctls, while there ? I would have if necesary (since I was there). However at a quick glance of pciio.h it didn't appear to me to be necessary. Also I do suspect that the i386 X11 Server is making successfuly use of some of the other calls. Keep in mind that the freebsd32 layer has generic handling for those ioctl calls that don't require anything special. I believe PCIOCREAD, PCIOCWRITE, and friends fall into that category since it appears the structures don't change size or alignment between i386 and amd64 (mind you this was based just on a quick glance at the header). -- John ------------------------------------------------------------------------- | Feith Systems | Voice: 1-215-646-8000 | Email: john@feith.com | | John Wehle | Fax: 1-215-540-5495 | | ------------------------------------------------------------------------- ----- End forwarded message -----