Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 5 Feb 2019 06:25:35 +0000 (UTC)
From:      "Jayachandran C." <jchandra@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r343764 - head/sys/arm/arm
Message-ID:  <201902050625.x156PZbc074295@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jchandra
Date: Tue Feb  5 06:25:35 2019
New Revision: 343764
URL: https://svnweb.freebsd.org/changeset/base/343764

Log:
  arm, acpi: increase size of memory region arrays
  
  Bump up MAX_HWCNT and MAX_EXCNT to 32 when ACPI is enabled. These are
  the sizes of the hwregions and exregions arrays respectively. ACPI
  firmware typically has more memory regions and the current value of
  16 is not sufficient for some platforms.
  
  This commit fixes a failure seen with AMI firmware on Cavium's Sabre
  ThunderX2 reference platform. This platform needs 21 physical memory
  regions and 18 excluded regions to boot correctly with the current
  firmware release.
  
  Reviewed by:	andrew
  Differential Revision:	https://reviews.freebsd.org/D19073

Modified:
  head/sys/arm/arm/physmem.c

Modified: head/sys/arm/arm/physmem.c
==============================================================================
--- head/sys/arm/arm/physmem.c	Tue Feb  5 04:47:41 2019	(r343763)
+++ head/sys/arm/arm/physmem.c	Tue Feb  5 06:25:35 2019	(r343764)
@@ -29,6 +29,7 @@
 #include <sys/cdefs.h>
 __FBSDID("$FreeBSD$");
 
+#include "opt_acpi.h"
 #include "opt_ddb.h"
 
 /*
@@ -48,8 +49,13 @@ __FBSDID("$FreeBSD$");
  * that can be allocated, or both, depending on the exclusion flags associated
  * with the region.
  */
+#ifdef DEV_ACPI
+#define	MAX_HWCNT	32	/* ACPI needs more regions */
+#define	MAX_EXCNT	32
+#else
 #define	MAX_HWCNT	16
 #define	MAX_EXCNT	16
+#endif
 
 #if defined(__arm__)
 #define	MAX_PHYS_ADDR	0xFFFFFFFFull



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201902050625.x156PZbc074295>