Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 27 Jan 2003 09:08:39 +0800
From:      "David Xu" <davidx@viasoft.com.cn>
To:        <obrien@FreeBSD.org>
Cc:        <cvs-committers@FreeBSD.org>, <cvs-all@FreeBSD.org>
Subject:   Re: cvs commit: src/sys/alpha/alpha trap.c vm_machdep.c src/sys/ddb db_ps.c src/sys/i386/i386 critical.c exception.s genassym.c trap.c vm_machdep.c src/sys/ia64/ia64 trap.c vm_machdep.c src/sys/kern init_main.c kern_clock.c kern_exec.c kern_exit.c kern_for
Message-ID:  <008a01c2c5a0$a0474620$f001a8c0@davidw2k>
References:  <200301261141.h0QBfZxZ001813@repoman.freebsd.org> <20030126215252.GA24818@dragon.nuxi.com>

next in thread | previous in thread | raw e-mail | index | archive | help


----- Original Message ----- 
From: "David O'Brien" <obrien@FreeBSD.org>
To: "David Xu" <davidxu@FreeBSD.org>
Cc: <cvs-committers@FreeBSD.org>; <cvs-all@FreeBSD.org>
Sent: Monday, January 27, 2003 5:52 AM
Subject: Re: cvs commit: src/sys/alpha/alpha trap.c vm_machdep.c src/sys/ddb db_ps.c src/sys/i386/i386 critical.c exception.s genassym.c trap.c vm_machdep.c src/sys/ia64/ia64 trap.c vm_machdep.c src/sys/kern init_main.c kern_clock.c kern_exec.c kern_exit.c kern_fork.c ...


> On Sun, Jan 26, 2003 at 03:41:35AM -0800, David Xu wrote:
> > davidxu     2003/01/26 03:41:35 PST
> > 
> >   Modified files:
> >     sys/alpha/alpha      trap.c vm_machdep.c 
> >     sys/ddb              db_ps.c 
> >     sys/i386/i386        critical.c exception.s genassym.c trap.c 
> >                          vm_machdep.c 
> >     sys/ia64/ia64        trap.c vm_machdep.c 
> >     sys/kern             init_main.c kern_clock.c kern_exec.c 
> >                          kern_exit.c kern_fork.c kern_lock.c 
> >                          kern_resource.c kern_sig.c kern_switch.c 
> >                          kern_thread.c subr_prof.c subr_trap.c 
> >                          subr_witness.c 
> >     sys/powerpc/powerpc  vm_machdep.c 
> >     sys/sparc64/sparc64  trap.c vm_machdep.c 
> >     sys/sys              buf.h lockmgr.h proc.h resourcevar.h 
> >                          systm.h 
> >   Log:
> >   Move UPCALL related data structure out of kse, introduce a new
> >   data structure called kse_upcall to manage UPCALL. All KSE binding
> >   and loaning code are gone.
> 
> Hi David,
> 
> There have been reports from some senior kernel developers that this
> commit breaks sparc64, alpha, ia64, and SMP i386 kernels.  You've also
> broke userland profiling for everything but i386.
> 
Julian has fixed it while I am sleeping.  I am living in another half of the earth.

> Would you please back this commit out and have it farther reviewed.  For
> such extensive changes it should also have been tested on something other
> than just UP i386.  It is bad this happened on a weekend, as that is the
> time of most FreeBSD activity for many of us. :-(
> 

I have patch there if someone has interest to review:
http://davidxu.freebsd.org/~davidxu/kse2.diff
if you think some code should be adjusted, I would like to do.

> thanks,
> -- David

Thanks,
David Xu



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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?008a01c2c5a0$a0474620$f001a8c0>