From owner-freebsd-mips@FreeBSD.ORG  Mon May 17 21:04:18 2010
Return-Path: <owner-freebsd-mips@FreeBSD.ORG>
Delivered-To: freebsd-mips@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 9D1F11065675;
	Mon, 17 May 2010 21:04:18 +0000 (UTC)
	(envelope-from rrs@lakerest.net)
Received: from lakerest.net (unknown [IPv6:2001:240:585:2:213:d4ff:fef3:2d8d])
	by mx1.freebsd.org (Postfix) with ESMTP id 2DE4F8FC08;
	Mon, 17 May 2010 21:04:18 +0000 (UTC)
Received: from mobile-166-129-180-182.mycingular.net
	(mobile-166-129-180-182.mycingular.net [166.129.180.182] (may
	be forged)) (authenticated bits=0)
	by lakerest.net (8.14.3/8.14.3) with ESMTP id o4HL46PW042786
	(version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NOT);
	Mon, 17 May 2010 17:04:14 -0400 (EDT)
	(envelope-from rrs@lakerest.net)
Message-Id: <895F98F0-ED84-4F68-898B-F88BE014633F@lakerest.net>
From: Randall Stewart <rrs@lakerest.net>
To: Neel Natu <neelnatu@gmail.com>
In-Reply-To: <AANLkTin1CmrTmuwChVnwBaSRsnQO8mcwE5P9fXBtPTvp@mail.gmail.com>
Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes
Content-Transfer-Encoding: 7bit
Mime-Version: 1.0 (Apple Message framework v936)
Date: Mon, 17 May 2010 14:03:34 -0700
References: <AANLkTikjF2SUa5GomVRtus176hjKyUyPAeDF-p1kwCcO@mail.gmail.com>
	<AANLkTin1CmrTmuwChVnwBaSRsnQO8mcwE5P9fXBtPTvp@mail.gmail.com>
X-Mailer: Apple Mail (2.936)
Cc: freebsd-mips@freebsd.org
Subject: Re: Minor fixup for the MIPS SMP platform code.
X-BeenThere: freebsd-mips@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Porting FreeBSD to MIPS <freebsd-mips.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-mips>,
	<mailto:freebsd-mips-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-mips>
List-Post: <mailto:freebsd-mips@freebsd.org>
List-Help: <mailto:freebsd-mips-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-mips>,
	<mailto:freebsd-mips-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 17 May 2010 21:04:18 -0000

Neel:

I will make sure I get this fixed when I commit
it tonight..

Thanks for your comments ;-)

R
On May 17, 2010, at 8:38 AM, Neel Natu wrote:

> Hi JC,
>
> Your patch looks good. Thanks for cleaning up the platform-specific
> code from mp_machdep.c.
>
> There are a couple of style things you might want to take care of:
>
> 1. In hwfunc.h there is an extra space after the '*' in the
> declaration of platform_smp_topo()
>
> 2. In platform_smp_topo() for XLR:
>    - Values in return statements should be enclosed in parantheses.
>    - space around binary operator '/'
>
> best
> Neel
>
> On Mon, May 17, 2010 at 7:33 AM, C. Jayachandran
> <c.jayachandran@gmail.com> wrote:
>> The attached patch (also at
>> http://sites.google.com/site/cjayachandran/files) has two changes:
>> - moves the intr masking code to platform_init_ap, so that we can  
>> avoid the
>> #ifdef for XLR
>> - adds a platform_smp_topo() function so that platforms can define  
>> their
>> topology
>>
>> I've also attempted to fixup sys/mips/sibyte to reflect these  
>> changes, but
>> it might need further work.
>> sys/mips/cavium also might need similar changes for SMP.
>>
>> Please review and apply if the changes are okay.
>>
>> Thanks,
>> JC.
>>
>>
>

------------------------------
Randall Stewart
803-317-4952 (cell)
803-345-0391(direct)