svn commit: r268055 - head/sys/sys

Hans Petter Selasky hps at selasky.org
Mon Jun 30 18:43:37 UTC 2014


On 06/30/14 20:37, John Baldwin wrote:
> Reindenting the whitespace made this diff harder to read.  Why hasn't this
> been a problem before on powerpc64?

Hi,

This has not been a problem before, because producers of DATA_SET()'s 
were using syntax along these lines:

static int test2;
static int test3;

DATA_SET(test, test2);
DATA_SET(test, test3);

Now if you change this simple code to:

static int test2;
int test3;

DATA_SET(test, test2);
DATA_SET(test, test3);

It breaks on powerpc64. Should be very easy to reproduce.

The problem with SYSCTL's is that some are global and some are not. 
Before an indirect pointer was hiding this problem from appearing.

Do you see? Or do you want me to explain more.

--HPS


More information about the svn-src-all mailing list