Date: Mon, 17 May 2010 14:55:46 -0700 From: Neel Natu <neelnatu@gmail.com> To: Juli Mallett <jmallett@freebsd.org> Cc: svn-src-head@freebsd.org, Randall Stewart <rrs@freebsd.org>, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r208165 - in head/sys: kern mips/conf mips/include mips/mips mips/rmi mips/rmi/dev/xlr Message-ID: <AANLkTil1s54bQrz4xn7l7T4it8551s79ZC3cTZc52PlZ@mail.gmail.com> In-Reply-To: <AANLkTilXPPhlnYN1KbEw-4DQRVIXgqje3WrrfmzQ63Rv@mail.gmail.com> References: <201005161943.o4GJhnTo096839@svn.freebsd.org> <AANLkTilXPPhlnYN1KbEw-4DQRVIXgqje3WrrfmzQ63Rv@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Randall, On Sun, May 16, 2010 at 12:54 PM, Juli Mallett <jmallett@freebsd.org> wrote= : > On Sun, May 16, 2010 at 12:43, Randall Stewart <rrs@freebsd.org> wrote: >> Author: rrs >> Date: Sun May 16 19:43:48 2010 >> New Revision: 208165 >> URL: http://svn.freebsd.org/changeset/base/208165 >> >> Log: >> =A0This pushes all of JC's patches that I have in place. I >> =A0am now able to run 32 cores ok.. but I still will hang >> =A0on buildworld with a NFS problem. I suspect I am missing >> =A0a patch for the netlogic rge driver. >> >> =A0JC check and see if I am missing anything except your >> =A0core-mask changes > > This isn't a very good commit message. =A0Commits communicate things > going forward. =A0A follow-up E-Mail CCing JC would have been a good way > to chat with him about its completeness. =A0More importantly, though, > there's just no description of the contents. =A0This is particularly > frustrating as it's also not broken up into functional chunks very > well. =A0Given the fact that there are MI changes and pan-MIPS changes > here it would be nice to see a little more care taken. =A0The changes to > the MIPS MP code, for instance, I thought we had discussed and decided > were sub-par and easily made less invasive. =A0I can't comment on the > RMI-specific stuff except to say that it's an unenumerated jumble of > VM, networking and SMP changes. > I agree with Juli. This is especially true with respect to the changes to pmap.c. There needed to be a companion set of changes to exception.S to undo some of the changes to the TLB invalid handler that deal with recursive faults in the TLB miss handler. best Neel
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTil1s54bQrz4xn7l7T4it8551s79ZC3cTZc52PlZ>