Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 17 Jun 2009 08:24:44 +0400
From:      Eygene Ryabinkin <rea-fbsd@codelabs.ru>
To:        Marius Strobl <marius@alchemy.franken.de>
Cc:        current@freebsd.org, rwatson@freebsd.org, FreeBSD Tinderbox <tinderbox@freebsd.org>, sparc64@freebsd.org, kmacy@freebsd.org
Subject:   Re: [head tinderbox] failure on sparc64/sun4v
Message-ID:  <wkM2Fp4DnDPq6wI6lCgfyHjyasY@j4OYE6OL8eALCd4BvSxIfwgoxSc>
In-Reply-To: <20090603200013.GB43137@alchemy.franken.de>
References:  <20090602222445.2F6017302F@freebsd-current.sentex.ca> <mqQn8SFFPOY77oNsI7n1tk5O7LE@10Ilc7MfiXA2JVIRVQpZfk7cTQ4> <rPuoszoUcXlrNmrZFDbbbNJuZzs@XX1fo6zQUfC4h0jjRC6IBz3oNH4> <vGGiriRDUTPxnXzdq/aYGNGLA2U@j4OYE6OL8eALCd4BvSxIfwgoxSc> <20090603200013.GB43137@alchemy.franken.de>

next in thread | previous in thread | raw e-mail | index | archive | help
Marius, *, good day.

Wed, Jun 03, 2009 at 10:00:13PM +0200, Marius Strobl wrote:
> On Wed, Jun 03, 2009 at 02:15:55PM +0400, Eygene Ryabinkin wrote:
> > And while I am here: definition for PCPU_NAME_LEN seems to be wrong.
> > It is intended to fit ("CPU %d", cpuid) where cpuid <= MAXCPU.  If this
> > is correct, then (sys/sys/pcpu.h, line 57)
> > 
> >  1. sizeof(__XSTRING(MAXCPU) + 1) is a typo: typeof(__XSTRING(...) + 1)
> >     is 'char *', so sizeof() will return the size of the pointer, not
> >     the size of the string contents.  The proper expression should be
> >     'sizeof(__XSTRING(MAXCPU)) + 1'.
> > 
> >  2. one should not add one, but substract it: sizeof() accounts for the
> >     trailing '\0' and we have two sizeof's, so the size of one '\0'
> >     should be substracted -- this will give the maximal string buffer
> >     length for CPU with its number, no less, no more.
> > 
> > Does the attached patch looks sane?
> > diff --git a/sys/sys/pcpu.h b/sys/sys/pcpu.h
> > index 63c3fa3..98705eb 100644
> > --- a/sys/sys/pcpu.h
> > +++ b/sys/sys/pcpu.h
> > @@ -54,7 +54,7 @@ struct rm_queue {
> >  	struct rm_queue* volatile rmq_prev;
> >  };
> >  
> > -#define	PCPU_NAME_LEN (sizeof("CPU ") + sizeof(__XSTRING(MAXCPU) + 1))
> > +#define	PCPU_NAME_LEN (sizeof("CPU ") + sizeof(__XSTRING(MAXCPU)) - 1)
> >  
> >  
> >  /*
> 
> This looks correct to me. Jeff?

Any updates on this matter?  May be someone else will be able to review
the patch?  Kip, Robert?
-- 
Eygene
 _                ___       _.--.   #
 \`.|\..----...-'`   `-._.-'_.-'`   #  Remember that it is hard
 /  ' `         ,       __.--'      #  to read the on-line manual
 )/' _/     \   `-_,   /            #  while single-stepping the kernel.
 `-'" `"\_  ,_.-;_.-\_ ',  fsc/as   #
     _.-'_./   {_.'   ; /           #    -- FreeBSD Developers handbook
    {_.-``-'         {_/            #



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