svn commit: r244526 - head/usr.sbin/mptable

[Garrett Cooper]

On Thu, Dec 20, 2012 at 5:31 PM, Neel Natu <neel at freebsd.org> wrote:
> Author: neel
> Date: Fri Dec 21 01:31:56 2012
> New Revision: 244526
> URL: http://svnweb.freebsd.org/changeset/base/244526
>
> Log:
>   The MPtable specification allows for an 8-bit "BUS ID" and "I/O APIC ID".
>
>   Since the 'busses[]' and 'apics[]' arrays are indexed by these 8-bit IDs
>   make sure that they have enough space to accomodate up to 256 entries.
>
>   Submitted by: Ravi Shamanna
>   Obtained from:        NetApp
>
> Modified:
>   head/usr.sbin/mptable/mptable.c
>
> Modified: head/usr.sbin/mptable/mptable.c
> ==============================================================================
> --- head/usr.sbin/mptable/mptable.c     Fri Dec 21 01:19:48 2012        (r244525)
> +++ head/usr.sbin/mptable/mptable.c     Fri Dec 21 01:31:56 2012        (r244526)
> @@ -270,8 +270,8 @@ static void pnstr( char* s, int c );
>  /* global data */
>  static int     pfd;            /* physical /dev/mem fd */
>
> -static int     busses[16];
> -static int     apics[16];
> +static int     busses[256];
> +static int     apics[256];
>
>  static int     ncpu;
>  static int     nbus;
> @@ -711,7 +711,7 @@ MPConfigTableHeader( u_int32_t pap )
>      printf( "MP Config Base Table Entries:\n\n" );
>
>      /* initialze tables */
> -    for ( x = 0; x < 16; ++x ) {
> +    for ( x = 0; x < 256; ++x ) {

`nitems(busses)` (requires sys/param.h)?

>         busses[ x ] = apics[ x ] = 0xff;
>
>      }

Thanks,
-Garrett