From owner-p4-projects  Fri Jul 12 19:21:40 2002
Delivered-To: p4-projects@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 32767)
	id 3277337B401; Fri, 12 Jul 2002 19:21:31 -0700 (PDT)
Delivered-To: perforce@freebsd.org
Received: from mx1.FreeBSD.org (mx1.FreeBSD.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 6323637B401
	for <perforce@freebsd.org>; Fri, 12 Jul 2002 19:21:30 -0700 (PDT)
Received: from mail.speakeasy.net (mail16.speakeasy.net [216.254.0.216])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 046C243E67
	for <perforce@freebsd.org>; Fri, 12 Jul 2002 19:21:30 -0700 (PDT)
	(envelope-from jhb@FreeBSD.org)
Received: (qmail 12125 invoked from network); 13 Jul 2002 02:21:28 -0000
Received: from unknown (HELO server.baldwin.cx) ([216.27.160.63]) (envelope-sender <jhb@FreeBSD.org>)
          by mail16.speakeasy.net (qmail-ldap-1.03) with DES-CBC3-SHA encrypted SMTP
          for <perforce@freebsd.org>; 13 Jul 2002 02:21:28 -0000
Received: from laptop.baldwin.cx (laptop.baldwin.cx [192.168.0.4])
	by server.baldwin.cx (8.11.6/8.11.6) with ESMTP id g6D2LR038003;
	Fri, 12 Jul 2002 22:21:27 -0400 (EDT)
	(envelope-from jhb@FreeBSD.org)
Message-ID: <XFMail.20020710112523.jhb@FreeBSD.org>
X-Mailer: XFMail 1.5.2 on FreeBSD
X-Priority: 3 (Normal)
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
In-Reply-To: <Pine.BSF.4.21.0207091503180.35930-100000@InterJet.elischer.org>
Date: Fri, 12 Jul 2002 22:21:37 -0400 (EDT)
From: John Baldwin <jhb@FreeBSD.org>
To: Julian Elischer <julian@elischer.org>
Subject: Re: PERFORCE change 13995 for review
Cc: Perforce Change Reviews <perforce@freebsd.org>
Sender: owner-p4-projects@FreeBSD.ORG
Precedence: bulk
List-ID: <p4-projects.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20p4-projects>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20p4-projects>
X-Loop: FreeBSD.ORG


On 09-Jul-2002 Julian Elischer wrote:
>  this looks "odd"

Unfortuantely user_ldt_free has a weird rule with sched_lock: it
drops it.  user_ldt_alloc() has a weirder one in that it drops it
if it returns NULL.

> ==== //depot/projects/smpng/sys/i386/i386/machdep.c#40 (text+ko) ====
> 
> @@ -825,8 +825,11 @@
>         struct trapframe *regs = td->td_frame;
>         struct pcb *pcb = td->td_pcb;
>  
> +       mtx_lock_spin(&sched_lock);
>         if (td->td_proc->p_md.md_ldt)
>                 user_ldt_free(td);
> +       else
> +               mtx_unlock_spin(&sched_lock);
>    
>         bzero((char *)regs, sizeof(struct trapframe));
>         regs->tf_eip = entry;
> 
> 
> does user_ldt_free(td) release the sched lock?
> 
> 
> 

-- 

John Baldwin <jhb@FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve!"  -  http://www.FreeBSD.org/

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe p4-projects" in the body of the message