From owner-freebsd-emulation@FreeBSD.ORG Tue Feb 8 04:50:11 2011 Return-Path: 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 E55B2106564A for ; Tue, 8 Feb 2011 04:50:11 +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 D611A8FC08 for ; Tue, 8 Feb 2011 04:50:11 +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 p184oBvg065082 for ; Tue, 8 Feb 2011 04:50:11 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.4/8.14.4/Submit) id p184oB8R065081; Tue, 8 Feb 2011 04:50:11 GMT (envelope-from gnats) Date: Tue, 8 Feb 2011 04:50:11 GMT Message-Id: <201102080450.p184oB8R065081@freefall.freebsd.org> To: freebsd-emulation@FreeBSD.org From: John Wehle Cc: Subject: Re: kern/149168: [linux] [patch] Linux sendmsg / recvmsg / etc fixes for pulseaudio X-BeenThere: freebsd-emulation@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: John Wehle List-Id: Development of Emulators of other operating systems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 08 Feb 2011 04:50:12 -0000 The following reply was made to PR kern/149168; it has been noted by GNATS. From: John Wehle To: avg@FreeBSD.org Cc: rdivacky@FreeBSD.org, bug-followup@FreeBSD.org Subject: Re: kern/149168: [linux] [patch] Linux sendmsg / recvmsg / etc fixes for pulseaudio Date: Tue, 8 Feb 2011 00:28:29 -0500 (EST) > 1. What is a reason for "msg_controllen skipped" ? Is this a proper > solution or some hacking remnants? It's in the spirit of /* msg_flags skipped */. Since the BSD and LINUX control messages are potentially different sizes (e.g. the cred structure used by SCM_CREDS is different between the two operating system) you can't just copy msg_controllen. linux_recvmsg fills in the proper value after it has parse / converted the control messages. For consistency it would probably be nice if this was also reflected in linux_to_bsd_msghdr / linux_sendmsg. I'm happy to address this issue, though it existed prior to my patch so I don't know if you want me to just bundle the necessary changes into an updated version of my patch for kern/149168 or if you want it addressed as in separate patch. > 2. amd64 counterparts of changes in i386 arch code are missing. Those > should be trivial to add. Yep ... the original patch probably should have had a comment to the effect: a) The amd64 counterparts of the changes in the i386 arch code are not supplied since I don't have a way of testing amd64 changes and I try to avoid supplying untested code. b) The changes should be trivial to implement by someone who has an amd64 environment. -- John ------------------------------------------------------------------------- | Feith Systems | Voice: 1-215-646-8000 | Email: john@feith.com | | John Wehle | Fax: 1-215-540-5495 | | -------------------------------------------------------------------------