From owner-svn-src-all@FreeBSD.ORG Wed Dec 17 12:51:23 2014 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 027E73BA; Wed, 17 Dec 2014 12:51:23 +0000 (UTC) Received: from mho-02-ewr.mailhop.org (mho-02-ewr.mailhop.org [204.13.248.72]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id CB70E84D; Wed, 17 Dec 2014 12:51:22 +0000 (UTC) Received: from [73.34.117.227] (helo=ilsoft.org) by mho-02-ewr.mailhop.org with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.72) (envelope-from ) id 1Y1E4R-000Ox6-IF; Wed, 17 Dec 2014 12:51:15 +0000 Received: from revolution.hippie.lan (revolution.hippie.lan [172.22.42.240]) by ilsoft.org (8.14.9/8.14.9) with ESMTP id sBHCpEP8055318; Wed, 17 Dec 2014 05:51:14 -0700 (MST) (envelope-from ian@freebsd.org) X-Mail-Handler: Dyn Standard SMTP by Dyn X-Originating-IP: 73.34.117.227 X-Report-Abuse-To: abuse@dyndns.com (see http://www.dyndns.com/services/sendlabs/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX1+mVjhshGl7AanqaiB2pL3s Message-ID: <1418820674.1015.14.camel@freebsd.org> Subject: Re: svn commit: r275858 - head/sys/mips/beri From: Ian Lepore To: Ruslan Bukin Date: Wed, 17 Dec 2014 05:51:14 -0700 In-Reply-To: <201412170934.sBH9Ytqp071808@svn.freebsd.org> References: <201412170934.sBH9Ytqp071808@svn.freebsd.org> Content-Type: text/plain; charset="us-ascii" X-Mailer: Evolution 3.12.8 FreeBSD GNOME Team Port Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 17 Dec 2014 12:51:23 -0000 On Wed, 2014-12-17 at 09:34 +0000, Ruslan Bukin wrote: > Author: br > Date: Wed Dec 17 09:34:54 2014 > New Revision: 275858 > URL: https://svnweb.freebsd.org/changeset/base/275858 > > Log: > Use memory regions information provided in FDT. > > Reviewed by: brooks > Sponsored by: DARPA, AFRL > > Modified: > head/sys/mips/beri/beri_machdep.c > > Modified: head/sys/mips/beri/beri_machdep.c > ============================================================================== > --- head/sys/mips/beri/beri_machdep.c Wed Dec 17 07:47:25 2014 (r275857) > +++ head/sys/mips/beri/beri_machdep.c Wed Dec 17 09:34:54 2014 (r275858) > @@ -88,6 +88,11 @@ static void > mips_init(void) > { > int i; > +#ifdef FDT > + struct mem_region mr[FDT_MEM_REGIONS]; > + int mr_cnt, val; > + int j; > +#endif > > for (i = 0; i < 10; i++) { > phys_avail[i] = 0; > @@ -102,6 +107,29 @@ mips_init(void) > > physmem = realmem; > > +#ifdef FDT > + if (fdt_get_mem_regions(mr, &mr_cnt, &val) == 0) { > + > + physmem = btoc(val); > + > + KASSERT((phys_avail[0] >= mr[0].mr_start) && \ > + (phys_avail[0] < (mr[0].mr_start + mr[0].mr_size)), > + ("First region is not within FDT memory range")); > + > + /* Limit size of the first region */ > + phys_avail[1] = MIN(mr[0].mr_size, ctob(realmem)); > + dump_avail[1] = phys_avail[1]; > + > + /* Add the rest of regions */ > + for (i = 1, j = 2; i < mr_cnt; i++, j+=2) { > + phys_avail[j] = mr[i].mr_start; > + phys_avail[j+1] = mr[i].mr_size; > + dump_avail[j] = phys_avail[j]; > + dump_avail[j+1] = phys_avail[j+1]; > + } > + } > +#endif > + > init_param1(); > init_param2(physmem); > mips_cpu_init(); > This doesn't look right, the values in the odd-numbered array slots for phys_avail and dump_avail need to be addresses, not sizes. That is, avail[j] = start, avail[j+1] = start+size (note: not start+size-1). -- Ian