From owner-freebsd-amd64@FreeBSD.ORG  Thu Mar 24 15:40:03 2011
Return-Path: <owner-freebsd-amd64@FreeBSD.ORG>
Delivered-To: freebsd-amd64@hub.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 99259106564A
	for <freebsd-amd64@hub.freebsd.org>;
	Thu, 24 Mar 2011 15:40:03 +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 84C5F8FC0C
	for <freebsd-amd64@hub.freebsd.org>;
	Thu, 24 Mar 2011 15:40:03 +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 p2OFe3oE096326
	for <freebsd-amd64@freefall.freebsd.org>; Thu, 24 Mar 2011 15:40:03 GMT
	(envelope-from gnats@freefall.freebsd.org)
Received: (from gnats@localhost)
	by freefall.freebsd.org (8.14.4/8.14.4/Submit) id p2OFe3CC096324;
	Thu, 24 Mar 2011 15:40:03 GMT (envelope-from gnats)
Date: Thu, 24 Mar 2011 15:40:03 GMT
Message-Id: <201103241540.p2OFe3CC096324@freefall.freebsd.org>
To: freebsd-amd64@FreeBSD.org
From: Mark Linimon <linimon@lonesome.com>
X-Mailman-Approved-At: Thu, 24 Mar 2011 15:59:08 +0000
Cc: 
Subject: Re: amd64/155903: FreeBSD32 emulation patch to support i386 X11
 Server
X-BeenThere: freebsd-amd64@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: Mark Linimon <linimon@lonesome.com>
List-Id: Porting FreeBSD to the AMD64 platform <freebsd-amd64.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-amd64>,
	<mailto:freebsd-amd64-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-amd64>
List-Post: <mailto:freebsd-amd64@freebsd.org>
List-Help: <mailto:freebsd-amd64-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-amd64>,
	<mailto:freebsd-amd64-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 24 Mar 2011 15:40:03 -0000

The following reply was made to PR amd64/155903; it has been noted by GNATS.

From: Mark Linimon <linimon@lonesome.com>
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: amd64/155903: FreeBSD32 emulation patch to support i386 X11
 Server
Date: Thu, 24 Mar 2011 10:34:30 -0500

 ----- Forwarded message from Kostik Belousov <kostikbel@gmail.com> -----
 
 Date: Thu, 24 Mar 2011 13:27:07 +0200
 From: Kostik Belousov <kostikbel@gmail.com>
 To: John Wehle <john@feith.com>
 Cc: amd64@freebsd.org
 Subject: Re: amd64/155903: FreeBSD32 emulation patch to support i386 X11
 	Server
 User-Agent: Mutt/1.4.2.3i
 
 All of this looks interesting.
 
 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.
 
 Then, I propose to add the compat definitions of MEMRANGE_GET32, SET32
 and PCIOCGETCONF_32.
 
 Then, we could move the copyin_map/copyout_map. Also, we could fix
 the sz == 0 case.
 
 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.
 
 BTW, would you do the shims for other pciconf ioctls, while there ?
 
 ----- End forwarded message -----