From owner-freebsd-emulation@FreeBSD.ORG  Tue Apr 10 23:39:42 2012
Return-Path: <owner-freebsd-emulation@FreeBSD.ORG>
Delivered-To: freebsd-emulation@FreeBSD.org
Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28])
	by hub.freebsd.org (Postfix) with ESMTP id 49E74106564A
	for <freebsd-emulation@FreeBSD.org>;
	Tue, 10 Apr 2012 23:39:42 +0000 (UTC)
	(envelope-from jkim@FreeBSD.org)
From: Jung-uk Kim <jkim@FreeBSD.org>
To: freebsd-emulation@FreeBSD.org
Date: Tue, 10 Apr 2012 19:39:31 -0400
User-Agent: KMail/1.6.2
References: <201204101856.46591.jkim@FreeBSD.org>
In-Reply-To: <201204101856.46591.jkim@FreeBSD.org>
MIME-Version: 1.0
Content-Disposition: inline
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Message-Id: <201204101939.33201.jkim@FreeBSD.org>
Cc: 
Subject: Re: [PATCH] pipe2 for Linuxulator
X-BeenThere: freebsd-emulation@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
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: Tue, 10 Apr 2012 23:39:42 -0000

On Tuesday 10 April 2012 06:56 pm, Jung-uk Kim wrote:
> Th attached patch implements pipe2 syscall for Linuxulator, which
> is quite trivial.  Although it was added in Linux 2.6.27 (thanks,
> netchild!), it is often used by popular Linux applications, e.g.,
> Adobe Flash plugin.  Please note linux_pipe() was moved from MD
> files to sys/compat/linux/linux_file.c because both amd64 and i386
> versions looked the same and I failed to see any reason.  I also
> changed file descriptor argument from l_ulong * to l_int *, which
> seemed more appropriate.  Any objections?

Oops, I missed a patch for sys/kern/kern_descript.c:

Index: sys/kern/kern_descrip.c
===================================================================
--- sys/kern/kern_descrip.c	(revision 234112)
+++ sys/kern/kern_descrip.c	(working copy)
@@ -1681,6 +1681,8 @@ finstall(struct thread *td, struct file *fp, int 
*
 	}
 	fhold(fp);
 	fdp->fd_ofiles[*fd] = fp;
+	if ((flags & O_NONBLOCK) != 0)
+		fp->f_flag |= FNONBLOCK;
 	if ((flags & O_CLOEXEC) != 0)
 		fdp->fd_ofileflags[*fd] |= UF_EXCLOSE;
 	FILEDESC_XUNLOCK(fdp);

> * PS: This patch is also available from here:
>
> http://people.freebsd.org/~jkim/linux_pipe2.diff

This patch is updated with it.

Sorry,

Jung-uk Kim