Re: FreeBSD_HEAD_i386 - Build #2215 - Failure

2016-01-26 Thread Justin Hibbits
Sorry, didn't fully tinderbox after a minor change before checking in  
r294883.  Should be fixed by r294885.


- Justin

On Jan 26, 2016, at 9:39 PM, jenkins-ad...@freebsd.org wrote:


FreeBSD_HEAD_i386 - Build #2215 - Failure:

Build information: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/2215/
Full change log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/2215/changes
Full build log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/2215/console

Change summaries:

294884 by cy:
Allow specification of fetch options for ntp leap-seconds fetch.

MFC after:  1 week
X-MFC with: r289421, r293037, r294773

294883 by jhibbits:
Convert rman to use rman_res_t instead of u_long

Summary:
Migrate to using the semi-opaque type rman_res_t to specify rman  
resources.  For

now, this is still compatible with u_long.

This is step one in migrating rman to use uintmax_t for resources  
instead of

u_long.

Going forward, this could feasibly be used to specify architecture- 
specific
definitions of resource ranges, rather than baking a specific  
integer type into

the API.

This change has been broken out to facilitate MFC'ing drivers back  
to 10 without

breaking ABI.

Reviewed By: jhb
Sponsored by:   Alex Perez/Inertial Computing
Differential Revision: https://reviews.freebsd.org/D5075

294882 by luigi:
cleanup and document in some detail the internals of the testing code
for dummynet schedulers

294881 by luigi:
the _Static_assert was not supposed to be in the commit.

294880 by dteske:
Replace awk with more efficient builtins-only algo

294879 by luigi:
bugfix: the scheduler template (dn_schk) for the round robin scheduler
is followed by another structure (rr_schk) whose size must be set
in the schk_datalen field of the descriptor.
Not allocating the memory may cause other memory to be overwritten
(though dn_schk is 192 bytes and rr_schk only 12 so we may be lucky
and end up in the padding after the dn_schk).

This is a merge candidate for stable and 10.3

MFC after:  3 days

294878 by bdrewery:
Revert yacc dependency back to pre-r241298.

Several attempts to fix this logic was done after r241298, which were
all reverted, yet this change was not.

The .h file does not depend on the .c file, so do not impose such a
dependency on it.  They are generated by the same command but do not
depend on each other.  Restore the .ORDER which should handle  
parallel build

issues.  This fixes an actual bug where the .h file is not recreated
when missing [1].  For example:
 cd lib/libc
 make cleanobj
 make nsparser.h
 rm nsparser.h
 make nsparser.h # will not rebuild nsparser.h

I have been trying to track down a build problem where nsparser.h is
missing when nslexer.o is built.  It is possible this is related.

Reported by:bde [1]

https://lists.freebsd.org/pipermail/svn-src-all/2012-October/059481.html

https://lists.freebsd.org/pipermail/svn-src-all/2012-October/060038.html
MFC after:  3 weeks
Sponsored by:   EMC / Isilon Storage Division

294877 by bdrewery:
Replace nslexer.l->nslexer.c custom rule with a -D CFLAG.

This avoids reproducing the lex logic which had dependencies set wrong
and used an intermediate file for modifying the YY_BUF_SIZE.

This has only been possible since flex 2.5.37 was imported in r250873,
which uses #ifndef YY_BUF_SIZE.

MFC after:  2 weeks
Sponsored by:   EMC / Isilon Storage Division

294876 by bdrewery:
nslexer.c does not depend on nsparser.h.

nslexer.o depends on nsparser.h, which is already added by bsd.lib.mk
and .depend.

This reverts r237402.

MFC after:  2 weeks
Sponsored by:   EMC / Isilon Storage Division

294875 by bdrewery:
FAST_DEPEND: Mark some unneeded code for later removal.

Sponsored by:   EMC / Isilon Storage Division

294874 by bdrewery:
FAST_DEPEND: Apply missed nofilemon fix from r294351.

Sponsored by:   EMC / Isilon Storage Division

294873 by bdrewery:
Set a value for _RECURSING_PROGS for debugging.

MFC after:  3 days
Sponsored by:   EMC / Isilon Storage Division

294872 by bdrewery:
Fix DIRDEPS_BUILD after r294752.

DIRDEPS_BUILD does not yet support PROGS having their own dependency
file.

Overriding .MAKE.DEPENDFILE here causes major problems with the meta
mode logic since it creates the Makefile.depend as '.depend' resulting
in infinite loops in make due to dirdeps.mk including .depend  
endlessly.


X-MFC-With: r294752
MFC after:  1 week
Sponsored by:   EMC / Isilon Storage Division



The end of the build log:

[...truncated 59511 lines...]
===> lib/libcom_err (all)
--- all_subdir_libbsm ---
--- bsm_mask.So ---
cc  -fpic -DPIC -g -O2 -pipe -I/usr/src/lib/libbsm/../../contrib/ 
openbsm -I/usr/src/lib/libbsm/../../contrib/openbsm/libbsm   - 
std=gnu99 -fstack-protector-strong -Wsystem-headers -Werror -Wno- 
pointer-sign -Wno-empty-body -Wno-string-plus-int -Wno-unused-const- 
variable -Wno-tautological-compare -Wno-unused-value -Wno- 
parentheses-equality -Wno-unused-functio

FreeBSD_HEAD_i386 - Build #2215 - Failure

2016-01-26 Thread jenkins-admin
FreeBSD_HEAD_i386 - Build #2215 - Failure:

Build information: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/2215/
Full change log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/2215/changes
Full build log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/2215/console

Change summaries:

294884 by cy:
Allow specification of fetch options for ntp leap-seconds fetch.

MFC after:  1 week
X-MFC with: r289421, r293037, r294773

294883 by jhibbits:
Convert rman to use rman_res_t instead of u_long

Summary:
Migrate to using the semi-opaque type rman_res_t to specify rman resources.  For
now, this is still compatible with u_long.

This is step one in migrating rman to use uintmax_t for resources instead of
u_long.

Going forward, this could feasibly be used to specify architecture-specific
definitions of resource ranges, rather than baking a specific integer type into
the API.

This change has been broken out to facilitate MFC'ing drivers back to 10 without
breaking ABI.

Reviewed By: jhb
Sponsored by:   Alex Perez/Inertial Computing
Differential Revision: https://reviews.freebsd.org/D5075

294882 by luigi:
cleanup and document in some detail the internals of the testing code
for dummynet schedulers

294881 by luigi:
the _Static_assert was not supposed to be in the commit.

294880 by dteske:
Replace awk with more efficient builtins-only algo

294879 by luigi:
bugfix: the scheduler template (dn_schk) for the round robin scheduler
is followed by another structure (rr_schk) whose size must be set
in the schk_datalen field of the descriptor.
Not allocating the memory may cause other memory to be overwritten
(though dn_schk is 192 bytes and rr_schk only 12 so we may be lucky
and end up in the padding after the dn_schk).

This is a merge candidate for stable and 10.3

MFC after:  3 days

294878 by bdrewery:
Revert yacc dependency back to pre-r241298.

Several attempts to fix this logic was done after r241298, which were
all reverted, yet this change was not.

The .h file does not depend on the .c file, so do not impose such a
dependency on it.  They are generated by the same command but do not
depend on each other.  Restore the .ORDER which should handle parallel build
issues.  This fixes an actual bug where the .h file is not recreated
when missing [1].  For example:
  cd lib/libc
  make cleanobj
  make nsparser.h
  rm nsparser.h
  make nsparser.h # will not rebuild nsparser.h

I have been trying to track down a build problem where nsparser.h is
missing when nslexer.o is built.  It is possible this is related.

Reported by:bde [1]

https://lists.freebsd.org/pipermail/svn-src-all/2012-October/059481.html

https://lists.freebsd.org/pipermail/svn-src-all/2012-October/060038.html
MFC after:  3 weeks
Sponsored by:   EMC / Isilon Storage Division

294877 by bdrewery:
Replace nslexer.l->nslexer.c custom rule with a -D CFLAG.

This avoids reproducing the lex logic which had dependencies set wrong
and used an intermediate file for modifying the YY_BUF_SIZE.

This has only been possible since flex 2.5.37 was imported in r250873,
which uses #ifndef YY_BUF_SIZE.

MFC after:  2 weeks
Sponsored by:   EMC / Isilon Storage Division

294876 by bdrewery:
nslexer.c does not depend on nsparser.h.

nslexer.o depends on nsparser.h, which is already added by bsd.lib.mk
and .depend.

This reverts r237402.

MFC after:  2 weeks
Sponsored by:   EMC / Isilon Storage Division

294875 by bdrewery:
FAST_DEPEND: Mark some unneeded code for later removal.

Sponsored by:   EMC / Isilon Storage Division

294874 by bdrewery:
FAST_DEPEND: Apply missed nofilemon fix from r294351.

Sponsored by:   EMC / Isilon Storage Division

294873 by bdrewery:
Set a value for _RECURSING_PROGS for debugging.

MFC after:  3 days
Sponsored by:   EMC / Isilon Storage Division

294872 by bdrewery:
Fix DIRDEPS_BUILD after r294752.

DIRDEPS_BUILD does not yet support PROGS having their own dependency
file.

Overriding .MAKE.DEPENDFILE here causes major problems with the meta
mode logic since it creates the Makefile.depend as '.depend' resulting
in infinite loops in make due to dirdeps.mk including .depend endlessly.

X-MFC-With: r294752
MFC after:  1 week
Sponsored by:   EMC / Isilon Storage Division



The end of the build log:

[...truncated 59511 lines...]
===> lib/libcom_err (all)
--- all_subdir_libbsm ---
--- bsm_mask.So ---
cc  -fpic -DPIC -g -O2 -pipe -I/usr/src/lib/libbsm/../../contrib/openbsm 
-I/usr/src/lib/libbsm/../../contrib/openbsm/libbsm   -std=gnu99 
-fstack-protector-strong -Wsystem-headers -Werror -Wno-pointer-sign 
-Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable 
-Wno-tautological-compare -Wno-unused-value -Wno-parentheses-equality 
-Wno-unused-function -Wno-enum-conversion -Wno-unused-local-typedef -Wno-switch 
-Wno-switch-enum -Wno-knr-promoted-parameter -Wno-parentheses  
-Qunused-arguments -c 
/usr/src/lib/libbsm/../../contrib/openbsm/libbsm/bsm_mas