From owner-freebsd-emulation@FreeBSD.ORG  Sat Aug 26 08:04:17 2006
Return-Path: <owner-freebsd-emulation@FreeBSD.ORG>
X-Original-To: freebsd-emulation@freebsd.org
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 D53AE16A4DA;
	Sat, 26 Aug 2006 08:04:17 +0000 (UTC)
	(envelope-from xdivac02@stud.fit.vutbr.cz)
Received: from eva.fit.vutbr.cz (eva.fit.vutbr.cz [147.229.10.14])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 0020B43D46;
	Sat, 26 Aug 2006 08:04:16 +0000 (GMT)
	(envelope-from xdivac02@stud.fit.vutbr.cz)
Received: from eva.fit.vutbr.cz (localhost [127.0.0.1])
	by eva.fit.vutbr.cz (envelope-from xdivac02@eva.fit.vutbr.cz)
	(8.13.7/8.13.7) with ESMTP id k7Q84BTg056787
	(version=TLSv1/SSLv3 cipher=EDH-RSA-DES-CBC3-SHA bits=168 verify=NO);
	Sat, 26 Aug 2006 10:04:11 +0200 (CEST)
Received: (from xdivac02@localhost)
	by eva.fit.vutbr.cz (8.13.7/8.13.3/Submit) id k7Q84BpL056786;
	Sat, 26 Aug 2006 10:04:11 +0200 (CEST)
Date: Sat, 26 Aug 2006 10:04:11 +0200
From: Divacky Roman <xdivac02@stud.fit.vutbr.cz>
To: Intron is my alias on the Internet <mag@intron.ac>
Message-ID: <20060826080410.GA56721@stud.fit.vutbr.cz>
References: <courier.44EE6A8A.00008655@intron.ac>
	<20060825084755.GA93151@stud.fit.vutbr.cz>
	<200608251028.55915.jhb@freebsd.org>
	<courier.44EFACD6.0000C97F@intron.ac>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <courier.44EFACD6.0000C97F@intron.ac>
User-Agent: Mutt/1.4.2i
X-Scanned-By: MIMEDefang 2.54 on 147.229.10.14
Cc: freebsd-emulation@freebsd.org, freebsd-current@freebsd.org,
	John Baldwin <jhb@freebsd.org>
Subject: Re: Linuxulator: Unbreak Mozilla, Firefox and RealPlayer
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: Sat, 26 Aug 2006 08:04:18 -0000

On Sat, Aug 26, 2006 at 10:07:17AM +0800, Intron is my alias on the Internet wrote:
> John Baldwin wrote:
> 
> >On Friday 25 August 2006 04:47, Divacky Roman wrote:
> >
> >Umm, if you want to reparent a proc you should use the proc_reparent()
> >function instead of just hacking on p_pptr.  You also need to hold
> >the proctree_lock when modifying p_pptr anyway.
> >
> >-- 
> >John Baldwin
> >_______________________________________________
> >freebsd-current@freebsd.org mailing list
> >http://lists.freebsd.org/mailman/listinfo/freebsd-current
> >To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"
> 
> Thank you for your reminder. I have updated my patch:
> 
> http://ftp.intron.ac/tmp/linux_machdep.c.1.53-2.diff

this is wrong

1) you dont PROC_LOCK(p2)

2) you have to lock proctree_lock instead of allproc_lock

are you satisfied with this patch?

www.stud.fit.vutbr.cz/~xdivac02/linux-fix.patch