Re: [head tinderbox] failure on sparc64/sun4v

From: Eygene Ryabinkin <rea-fbsd_at_codelabs.ru>
Date: Wed, 3 Jun 2009 09:25:38 +0400
Tue, Jun 02, 2009 at 06:24:45PM -0400, FreeBSD Tinderbox wrote:
> cc -c -O2 -pipe -fno-strict-aliasing  -std=c99  -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign -fformat-extensions -nostdinc  -I. -I/src/sys -I/src/sys/contrib/altq -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common -finline-limit=15000 --param inline-unit-growth=100 --param large-function-growth=1000 -fno-builtin -mcmodel=medany -msoft-float -ffreestanding -fstack-protector -Werror  /src/sys/sun4v/sun4v/machdep.c
> /src/sys/sun4v/sun4v/machdep.c:192: error: size of array '__assert192' is negative
> *** Error code 1
> 
> Stop in /obj/sun4v/src/sys/LINT.

This seems to be related to the recent NETISR changes, namely, the
addition of the pc_netisr member to the struct pcpu:
  http://svn.freebsd.org/viewvc/base/head/sys/sys/pcpu.h?r1=187679&r2=193219&diff_format=u

I am not sure how large (void *) is on sun4v, but it seems to me
that it is 4 bytes long, so PCPU_MD_FIELDS_PAD inside sun4v/include/pcpu.h
should be compensated for this change.  Something like
-----
#ifdef KTR
#define	PCPU_MD_FIELDS_PAD	(3 - (PCPU_NAME_LEN + 7) / 8)
#else
#define	PCPU_MD_FIELDS_PAD	3
#endif
-----
though I am not very sure about KTR's case.
-- 
Eygene
 _                ___       _.--.   #
 \`.|\..----...-'`   `-._.-'_.-'`   #  Remember that it is hard
 /  ' `         ,       __.--'      #  to read the on-line manual
 )/' _/     \   `-_,   /            #  while single-stepping the kernel.
 `-'" `"\_  ,_.-;_.-\_ ',  fsc/as   #
     _.-'_./   {_.'   ; /           #    -- FreeBSD Developers handbook
    {_.-``-'         {_/            #
Received on Wed Jun 03 2009 - 03:25:43 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:49 UTC