From owner-freebsd-mips@FreeBSD.ORG Mon May 17 17:38:23 2010 Return-Path: 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 739231065676; Mon, 17 May 2010 17:38:23 +0000 (UTC) (envelope-from c.jayachandran@gmail.com) Received: from mail-pv0-f182.google.com (mail-pv0-f182.google.com [74.125.83.182]) by mx1.freebsd.org (Postfix) with ESMTP id 3AF108FC18; Mon, 17 May 2010 17:38:22 +0000 (UTC) Received: by pvh11 with SMTP id 11so2580592pvh.13 for ; Mon, 17 May 2010 10:38:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:cc:content-type; bh=gOzsas8Kk6IcupSyYV1oTycFHPzvKdN0BBPzDad5mrM=; b=iCVGK+r7Ct21ej4Da4ssi6HxTle/4C+7ye4IkmifolaxRSGq2S+qZfFzoFnY2D/cnL PDg+Ti1txEHLEL8oHYB4dflyWaUgJmH/0R7m/NXBUnBwCExMp1e7vRLupqhiiYJ+CEl4 baTFYb+Nx7TeWv578wnCKdoOeSvqCeXBXj9Fg= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=imSawx696oz6RV5LtemAgfFI7ds1QV8KfL95epozH3DcDlwqDn2m56acrOu1b0RF5g L1cFJD274ku5kQwj/N63MqFrPeJXobJRi/mz7UAc5qQgTVNxYvs9BzqfgwH4KbFasjqe 2iSUrc5EbSht+m89R95gd8MOoLTOSkqrySCyQ= MIME-Version: 1.0 Received: by 10.141.89.8 with SMTP id r8mr3956209rvl.32.1274117902559; Mon, 17 May 2010 10:38:22 -0700 (PDT) Received: by 10.141.32.4 with HTTP; Mon, 17 May 2010 10:38:22 -0700 (PDT) In-Reply-To: References: Date: Mon, 17 May 2010 23:08:22 +0530 Message-ID: From: "C. Jayachandran" To: Neel Natu Content-Type: multipart/mixed; boundary=000e0cd138e0fbebda0486cdb051 X-Content-Filtered-By: Mailman/MimeDel 2.1.5 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 17 May 2010 17:38:23 -0000 --000e0cd138e0fbebda0486cdb051 Content-Type: text/plain; charset=ISO-8859-1 On Mon, May 17, 2010 at 9:08 PM, 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 '/' > > Looks like I've been a bit sloppy here, updated patch attached, with all the comments so far addressed. Thanks, JC. > best > Neel > > On Mon, May 17, 2010 at 7:33 AM, C. Jayachandran > 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. > > > > > -- C. Jayachandran c.jayachandran@gmail.com --000e0cd138e0fbebda0486cdb051 Content-Type: application/octet-stream; name="mips-smp-fixup.patch" Content-Disposition: attachment; filename="mips-smp-fixup.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_g9bk0xqp0 SW5kZXg6IHN5cy9taXBzL3NpYnl0ZS9zYl9tYWNoZGVwLmMKPT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gc3lzL21p cHMvc2lieXRlL3NiX21hY2hkZXAuYwkocmV2aXNpb24gMjA4MTg5KQorKysgc3lzL21pcHMvc2li eXRlL3NiX21hY2hkZXAuYwkod29ya2luZyBjb3B5KQpAQCAtMzQ0LDkgKzM0NCwxNiBAQAogCXJl dHVybiAoNCk7CiB9CiAKK3N0cnVjdCBjcHVfZ3JvdXAgKgorcGxhdGZvcm1fc21wX3RvcG8odm9p ZCkKK3sKKwlyZXR1cm4gKHNtcF90b3BvX25vbmUoKSk7Cit9CisKIHZvaWQKIHBsYXRmb3JtX2lu aXRfYXAoaW50IGNwdWlkKQogeworCWludCBpcGlfaW50X21hc2ssIGNsb2NrX2ludF9tYXNrOwog CiAJS0FTU0VSVChjcHVpZCA9PSAxLCAoIkFQIGhhcyBhbiBpbnZhbGlkIGNwdSBpZCAlZCIsIGNw dWlkKSk7CiAKQEAgLTM1Niw2ICszNjMsMTMgQEAKIAlrc2VnMF9tYXBfY29oZXJlbnQoKTsKIAog CXNiX2ludHJfaW5pdChjcHVpZCk7CisKKwkvKgorCSAqIFVubWFzayB0aGUgY2xvY2sgYW5kIGlw aSBpbnRlcnJ1cHRzLgorCSAqLworCWNsb2NrX2ludF9tYXNrID0gaGFyZF9pbnRfbWFzayg1KTsK KwlpcGlfaW50X21hc2sgPSBoYXJkX2ludF9tYXNrKHBsYXRmb3JtX2lwaV9pbnRybnVtKCkpOwor CXNldF9pbnRyX21hc2soQUxMX0lOVF9NQVNLICYgfihpcGlfaW50X21hc2sgfCBjbG9ja19pbnRf bWFzaykpOwogfQogCiBpbnQKSW5kZXg6IHN5cy9taXBzL2luY2x1ZGUvaHdmdW5jLmgKPT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PQotLS0gc3lzL21pcHMvaW5jbHVkZS9od2Z1bmMuaAkocmV2aXNpb24gMjA4MTg5KQorKysg c3lzL21pcHMvaW5jbHVkZS9od2Z1bmMuaAkod29ya2luZyBjb3B5KQpAQCAtOTMsNSArOTMsMTEg QEAKICAqLwogZXh0ZXJuIGludCBwbGF0Zm9ybV9udW1fcHJvY2Vzc29ycyh2b2lkKTsKIAorLyoK KyAqIFJldHVybiB0aGUgdG9wb2xvZ3kgb2YgcHJvY2Vzc29ycyBvbiB0aGlzIHBsYXRmb3JtCisg Ki8KK3N0cnVjdCBjcHVfZ3JvdXAgKnBsYXRmb3JtX3NtcF90b3BvKHZvaWQpOworCisKICNlbmRp ZgkvKiBTTVAgKi8KICNlbmRpZiAvKiAhX01BQ0hJTkVfSFdGVU5DX0hfICovCkluZGV4OiBzeXMv bWlwcy9ybWkveGxyX21hY2hkZXAuYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBzeXMvbWlwcy9ybWkveGxyX21h Y2hkZXAuYwkocmV2aXNpb24gMjA4MTg5KQorKysgc3lzL21pcHMvcm1pL3hscl9tYWNoZGVwLmMJ KHdvcmtpbmcgY29weSkKQEAgLTUzNCw2ICs1MzQsOCBAQAogCiAJLyogU2V0dXAgaW50ZXJydXB0 cyBmb3Igc2Vjb25kYXJ5IENQVXMgaGVyZSAqLwogCXN0YXQgPSBtaXBzX3JkX3N0YXR1cygpOwor CUtBU1NFUlQoKHN0YXQgJiBNSVBTX1NSX0lOVF9JRSkgPT0gMCwKKwkgICAgKCJJbnRlcnJ1cHRz IGVuYWJsZWQgaW4gJXMhIiwgX19mdW5jX18pKTsKIAlzdGF0IHw9IE1JUFNfU1JfQ09QXzJfQklU IHwgTUlQU19TUl9DT1BfMF9CSVQ7CiAJbWlwc193cl9zdGF0dXMoc3RhdCk7CiAKQEAgLTU3MCw0 ICs1NzIsMTEgQEAKIHsKIAlyZXR1cm4gZmxzKHhscl9ib290MV9pbmZvLmNwdV9vbmxpbmVfbWFw KTsKIH0KKworc3RydWN0IGNwdV9ncm91cCAqCitwbGF0Zm9ybV9zbXBfdG9wbygpCit7CisJcmV0 dXJuIChzbXBfdG9wb18ybGV2ZWwoQ0dfU0hBUkVfTDIsIHBsYXRmb3JtX251bV9wcm9jZXNzb3Jz KCkgLyA0LAorCSAgICBDR19TSEFSRV9MMSwgNCwgQ0dfRkxBR19USFJFQUQpKTsKK30KICNlbmRp ZgpJbmRleDogc3lzL21pcHMvbWlwcy9tcGJvb3QuUwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBzeXMvbWlwcy9t aXBzL21wYm9vdC5TCShyZXZpc2lvbiAyMDgxODkpCisrKyBzeXMvbWlwcy9taXBzL21wYm9vdC5T CSh3b3JraW5nIGNvcHkpCkBAIC03NiwxMCArNzYsOCBAQAogCiAJUFRSX0xBCWdwLCBfQ19MQUJF TChfZ3ApCiAKLSNpZm5kZWYgVEFSR0VUX1hMUl9YTFMKIAlqYWwJcGxhdGZvcm1faW5pdF9hcAog CW1vdmUJYTAsIHMwCi0jZW5kaWYKIAlqYWwJc21wX2luaXRfc2Vjb25kYXJ5CiAJbW92ZQlhMCwg czAKIApJbmRleDogc3lzL21pcHMvbWlwcy9tcF9tYWNoZGVwLmMKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gc3lz L21pcHMvbWlwcy9tcF9tYWNoZGVwLmMJKHJldmlzaW9uIDIwODE4OSkKKysrIHN5cy9taXBzL21p cHMvbXBfbWFjaGRlcC5jCSh3b3JraW5nIGNvcHkpCkBAIC0xOTYsOCArMTk2LDcgQEAKIHN0cnVj dCBjcHVfZ3JvdXAgKgogY3B1X3RvcG8odm9pZCkKIHsKLQotCXJldHVybiAoc21wX3RvcG9fbm9u ZSgpKTsKKwlyZXR1cm4gKHBsYXRmb3JtX3NtcF90b3BvKCkpOwogfQogCiBpbnQKQEAgLTIzOCwx MCArMjM3LDYgQEAKIHZvaWQKIHNtcF9pbml0X3NlY29uZGFyeSh1X2ludDMyX3QgY3B1aWQpCiB7 Ci0jaWZuZGVmIFRBUkdFVF9YTFJfWExTCi0JaW50IGlwaV9pbnRfbWFzaywgY2xvY2tfaW50X21h c2s7Ci0jZW5kaWYKLQogCS8qIFRMQiAqLwogCU1pcHNfU2V0V0lSRUQoMCk7CiAJTWlwc19UTEJG bHVzaChudW1fdGxiZW50cmllcyk7CkBAIC0yOTQsMTggKzI4OSw3IEBACiAJd2hpbGUgKHNtcF9z dGFydGVkID09IDApCiAJCTsgLyogbm90aGluZyAqLwogCi0jaWZuZGVmIFRBUkdFVF9YTFJfWExT CiAJLyoKLQkgKiBVbm1hc2sgdGhlIGNsb2NrIGFuZCBpcGkgaW50ZXJydXB0cy4KLQkgKi8KLQlj bG9ja19pbnRfbWFzayA9IGhhcmRfaW50X21hc2soNSk7Ci0JaXBpX2ludF9tYXNrID0gaGFyZF9p bnRfbWFzayhwbGF0Zm9ybV9pcGlfaW50cm51bSgpKTsKLQlzZXRfaW50cl9tYXNrKEFMTF9JTlRf TUFTSyAmIH4oaXBpX2ludF9tYXNrIHwgY2xvY2tfaW50X21hc2spKTsKLSNlbHNlCi0JcGxhdGZv cm1faW5pdF9hcChjcHVpZCk7Ci0jZW5kaWYKLQotCS8qCiAJICogQm9vdHN0cmFwIHRoZSBjb21w YXJlIHJlZ2lzdGVyLgogCSAqLwogCW1pcHNfd3JfY29tcGFyZShtaXBzX3JkX2NvdW50KCkgKyBj b3VudGVyX2ZyZXEgLyBoeik7Cg== --000e0cd138e0fbebda0486cdb051--