Re: FreeBSD_HEAD_amd64_gcc4.9 - Build #1120 - Still Failing

2016-03-12 Thread Dimitry Andric
On 10 Mar 2016, at 03:38, Bryan Drewery  wrote:
> 
> On 3/9/2016 1:48 AM, jenkins-ad...@freebsd.org wrote:
>> FreeBSD_HEAD_amd64_gcc4.9 - Build #1120 - Still Failing:
>> 
>> Build information: 
>> https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc4.9/1120/
>> Full change log: 
>> https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc4.9/1120/changes
>> Full build log: 
>> https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc4.9/1120/console
...
> This seems to be from the clang upgrade.

It's now fixed, after r296687:
https://jenkins.freebsd.org/job/FreeBSD_HEAD_amd64_gcc4.9/1123/

Btw, this job should really be upgraded to use gcc 5.3.0, like the
devel/amd64-xtoolchain-gcc port provides.

-Dimitry



signature.asc
Description: Message signed with OpenPGP using GPGMail


Re: FreeBSD_HEAD_amd64_gcc4.9 - Build #1120 - Still Failing

2016-03-09 Thread Bryan Drewery
On 3/9/2016 1:48 AM, jenkins-ad...@freebsd.org wrote:
> FreeBSD_HEAD_amd64_gcc4.9 - Build #1120 - Still Failing:
> 
> Build information: 
> https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc4.9/1120/
> Full change log: 
> https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc4.9/1120/changes
> Full build log: 
> https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc4.9/1120/console
> 
> Change summaries:
> 
> 296557 by ae:
> Add support for IPPROTO_IPV6 socket layer for getsockopt/setsockopt calls.
> Also add mapping for several options from RFC 3493 and 3542.
> 
> Reviewed by:  dchagin
> Tested by:Joe Love 
> MFC after:2 weeks
> 
> 296556 by bdrewery:
> Follow-up r295827: Don't enable meta stats when recursing PROGS.
> 
> Sponsored by: EMC / Isilon Storage Division
> 
> 296555 by bdrewery:
> DIRDEPS_BUILD+PROGS: Fix staging not respecting (BINDIR|PROGNAME)[._]${PROG}.
> 
> Observed in tests/sys/kern.
> 
> Sponsored by: EMC / Isilon Storage Division
> 
> 296554 by bdrewery:
> Remove things set already by bsd.progs.mk.
> 
> MFC after:2 weeks
> Sponsored by: EMC / Isilon Storage Division
> 
> 296553 by bdrewery:
> PROGS: Track child meta files.
> 
> This will allow Makefile.depend to properly capture all dependencies.
> It is not 100% optimal but works.  Other options would be to use *.meta
> here which would include too much or to keep a Makefile.depend per PROG
> and include it from the main Makefile.depend which would not be
> straight forward.
> 
> Sponsored by: EMC / Isilon Storage Division
> 
> 296552 by np:
> cxgbe(4): Rename regwin_lock to reg_lock.  It is used to protect access
> to indirect registers only.
> 
> 296550 by bdrewery:
> DIRDEPS_BUILD: Update clang dependencies after r296417.
> 
> Sponsored by: EMC / Isilon Storage Division
> 
> 296549 by bdrewery:
> Don't ever create object directories here with MK_AUTO_OBJ.
> 
> Sponsored by: EMC / Isilon Storage Division
> 
> 296548 by dumbbell:
> drm/i915: Update to match Linux 3.8.13
> 
> This update brings initial support for Haswell GPUs.
> 
> Tested by:Many users of FreeBSD, PC-BSD and HardenedBSD
> Relnotes: yes
> Sponsored by: The FreeBSD Foundation
> Differential Revision:https://reviews.freebsd.org/D5554
> 
> 296546 by dchagin:
> Better english.
> 
> Submitted by: Kevin P. Neal
> MFC after:1 week
> 
> 296544 by np:
> cxgbe(4): Reshuffle and rototill t4_hw.c, solely to reduce diffs with
> the internal shared code.
> 
> Obtained from:Chelsio Communications
> 
> 296543 by dchagin:
> Put a commit message from r296502 about Linux alarm() system call
> behaviour to the source.
> 
> Suggested by: emaste@
> 
> MFC after:1 week
> 
> 296542 by dchagin:
> Load linux64 module for amd64 if Linux abi enabled.
> 
> Reviewed by:  emaste@
> MFC after:1 week
> Differential Revision:https://reviews.freebsd.org/D5567
> 
> 296541 by mav:
> MFV r296540: 4448 zfs diff misprints unicode characters
> 
> Reviewed by: Igor Kozhukhov 
> Reviewed by: Toomas Soome 
> Approved by: Matthew Ahrens 
> Author: Joshua M. Clulow 
> 
> illumos/illumos-gate@b211eb9181f99c20acbf4c528f94cb44b4ca8c31
> 
> 296539 by mav:
> MFV r296538: 6544 incorrect comment in libzfs.h about offline status
> 
> Reviewed by: Matthew Ahrens 
> Approved by: Dan McDonald 
> Author: Gerhard Roethlin 
> 
> illumos/illumos-gate@cb605c4d8ab24b5a900b8b4ca85db65c22d05fad
> 
> 296537 by mav:
> MFV r296536: 6551 cmd/zpool: cleanup gcc warnings
> 
> Reviewed by: Matthew Ahrens 
> Reviewed by: Andy Stormont 
> Approved by: Robert Mustacchi 
> 
> illumos/illumos-gate@b327cd3f3b4dab4f29e7140159b1e01ed2ceef2a
> 
> 296535 by mav:
> MFV r296534: 6550 cmd/zfs: cleanup gcc warnings
> 
> Reviewed by: Matthew Ahrens 
> Reviewed by: Andy Stormont 
> Approved by: Dan McDonald 
> Author: Igor Kozhukhov 
> 
> illumos/illumos-gate@c16bcc4577f389573eff411c7b7e040294078c3b
> 
> 296533 by mav:
> MFV r296532: 6637 replacing "dontclose" with "should_close"
> 
> Reviewed by: Matthew Ahrens 
> Reviewed by: Prakash Surya 
> Approved by: Robert Mustacchi 
> Author: David Schwartz 
> 
> illumos/illumos-gate@d189620258b3c9b0e2f7e2104840be2eee7c68e5
> 
> 296530 by mav:
> MFV r296529:
> 6672 arc_reclaim_thread() should use gethrtime() instead of ddi_get_lbolt()
> 6673 want a macro to convert seconds to nanoseconds and vice-versa
> 
> Reviewed by: Matthew Ahrens 
> Reviewed by: Prakash Surya 
> Reviewed by: Josef 'Jeff' Sipek 
> Reviewed by: Robert Mustacchi 
> Approved

FreeBSD_HEAD_amd64_gcc4.9 - Build #1120 - Still Failing

2016-03-09 Thread jenkins-admin
FreeBSD_HEAD_amd64_gcc4.9 - Build #1120 - Still Failing:

Build information: 
https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc4.9/1120/
Full change log: 
https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc4.9/1120/changes
Full build log: 
https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_amd64_gcc4.9/1120/console

Change summaries:

296557 by ae:
Add support for IPPROTO_IPV6 socket layer for getsockopt/setsockopt calls.
Also add mapping for several options from RFC 3493 and 3542.

Reviewed by:dchagin
Tested by:  Joe Love 
MFC after:  2 weeks

296556 by bdrewery:
Follow-up r295827: Don't enable meta stats when recursing PROGS.

Sponsored by:   EMC / Isilon Storage Division

296555 by bdrewery:
DIRDEPS_BUILD+PROGS: Fix staging not respecting (BINDIR|PROGNAME)[._]${PROG}.

Observed in tests/sys/kern.

Sponsored by:   EMC / Isilon Storage Division

296554 by bdrewery:
Remove things set already by bsd.progs.mk.

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

296553 by bdrewery:
PROGS: Track child meta files.

This will allow Makefile.depend to properly capture all dependencies.
It is not 100% optimal but works.  Other options would be to use *.meta
here which would include too much or to keep a Makefile.depend per PROG
and include it from the main Makefile.depend which would not be
straight forward.

Sponsored by:   EMC / Isilon Storage Division

296552 by np:
cxgbe(4): Rename regwin_lock to reg_lock.  It is used to protect access
to indirect registers only.

296550 by bdrewery:
DIRDEPS_BUILD: Update clang dependencies after r296417.

Sponsored by:   EMC / Isilon Storage Division

296549 by bdrewery:
Don't ever create object directories here with MK_AUTO_OBJ.

Sponsored by:   EMC / Isilon Storage Division

296548 by dumbbell:
drm/i915: Update to match Linux 3.8.13

This update brings initial support for Haswell GPUs.

Tested by:  Many users of FreeBSD, PC-BSD and HardenedBSD
Relnotes:   yes
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D5554

296546 by dchagin:
Better english.

Submitted by:   Kevin P. Neal
MFC after:  1 week

296544 by np:
cxgbe(4): Reshuffle and rototill t4_hw.c, solely to reduce diffs with
the internal shared code.

Obtained from:  Chelsio Communications

296543 by dchagin:
Put a commit message from r296502 about Linux alarm() system call
behaviour to the source.

Suggested by:   emaste@

MFC after:  1 week

296542 by dchagin:
Load linux64 module for amd64 if Linux abi enabled.

Reviewed by:emaste@
MFC after:  1 week
Differential Revision:  https://reviews.freebsd.org/D5567

296541 by mav:
MFV r296540: 4448 zfs diff misprints unicode characters

Reviewed by: Igor Kozhukhov 
Reviewed by: Toomas Soome 
Approved by: Matthew Ahrens 
Author: Joshua M. Clulow 

illumos/illumos-gate@b211eb9181f99c20acbf4c528f94cb44b4ca8c31

296539 by mav:
MFV r296538: 6544 incorrect comment in libzfs.h about offline status

Reviewed by: Matthew Ahrens 
Approved by: Dan McDonald 
Author: Gerhard Roethlin 

illumos/illumos-gate@cb605c4d8ab24b5a900b8b4ca85db65c22d05fad

296537 by mav:
MFV r296536: 6551 cmd/zpool: cleanup gcc warnings

Reviewed by: Matthew Ahrens 
Reviewed by: Andy Stormont 
Approved by: Robert Mustacchi 

illumos/illumos-gate@b327cd3f3b4dab4f29e7140159b1e01ed2ceef2a

296535 by mav:
MFV r296534: 6550 cmd/zfs: cleanup gcc warnings

Reviewed by: Matthew Ahrens 
Reviewed by: Andy Stormont 
Approved by: Dan McDonald 
Author: Igor Kozhukhov 

illumos/illumos-gate@c16bcc4577f389573eff411c7b7e040294078c3b

296533 by mav:
MFV r296532: 6637 replacing "dontclose" with "should_close"

Reviewed by: Matthew Ahrens 
Reviewed by: Prakash Surya 
Approved by: Robert Mustacchi 
Author: David Schwartz 

illumos/illumos-gate@d189620258b3c9b0e2f7e2104840be2eee7c68e5

296530 by mav:
MFV r296529:
6672 arc_reclaim_thread() should use gethrtime() instead of ddi_get_lbolt()
6673 want a macro to convert seconds to nanoseconds and vice-versa

Reviewed by: Matthew Ahrens 
Reviewed by: Prakash Surya 
Reviewed by: Josef 'Jeff' Sipek 
Reviewed by: Robert Mustacchi 
Approved by: Dan McDonald 
Author: Eli Rosenthal 

illumos/illumos-gate@a8f6344fa0921599e1f4511e41b5f9a25c38c0f9

296528 by mav:
MFV r296527:  6659 nvlist_free(NULL) is a no-op

Reviewed by: Toomas Soome 
Reviewed by: Marcel Telka 
Approved by: Robert Mustacchi 
Author: Josef 'Jeff' Sipek 

illumos/illumos-gate@aab83bb83be7342f6cfccaed8d5fe0b2f404855d

296526 by bdrewery:
Record command exit status in the typescript file when running simple commands.

Also capitalize 'command:'.

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

296525 by bdrewery:
Just exit in the child if execve(2) fails.

No functional change.

This is mostly addressing a false-positive from the clang static
analyzer due to it thinking that done() was being called with freed
memory, however th