Review Request 54950: Remove the use of FORTIFY_SOURCE from libprocess.

2016-12-21 Thread Aaron Wood
is applied to MESOS_CPPFLAGS thus failing the whole build. Diffs - 3rdparty/libprocess/Makefile.am c33ae4306 Diff: https://reviews.apache.org/r/54950/diff/ Testing --- Build all of Mesos from source. Thanks, Aaron Wood

Review Request 54949: Remove the use of FORTIFY_SOURCE from Mesos.

2016-12-21 Thread Aaron Wood
is applied to `MESOS_CPPFLAGS` thus failing the whole build. Diffs - src/Makefile.am abcf7eed7 Diff: https://reviews.apache.org/r/54949/diff/ Testing --- Build all of Mesos from source. Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-30 Thread Aaron Wood
/896944ea-9b31-4d62-b1b9-97fb4700a882__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/b32667ce-3e3b-4d2b-b4f8-4c2404a0fc1c__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-30 Thread Aaron Wood
this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52696: Harden stout

2016-11-30 Thread Aaron Wood
-b3bb-5ac60daaf498__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/3baa96cf-be05-4ac0-ad4c-ef571386e8f4__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-30 Thread Aaron Wood
). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-30 Thread Aaron Wood
-9b31-4d62-b1b9-97fb4700a882__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/b32667ce-3e3b-4d2b-b4f8-4c2404a0fc1c__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52696: Harden stout

2016-11-30 Thread Aaron Wood
-5ac60daaf498__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/3baa96cf-be05-4ac0-ad4c-ef571386e8f4__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-29 Thread Aaron Wood
/files/2016/11/02/896944ea-9b31-4d62-b1b9-97fb4700a882__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/b32667ce-3e3b-4d2b-b4f8-4c2404a0fc1c__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52696: Harden stout

2016-11-29 Thread Aaron Wood
/02/046b37a9-5aff-4543-b3bb-5ac60daaf498__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/3baa96cf-be05-4ac0-ad4c-ef571386e8f4__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-29 Thread Aaron Wood
. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-29 Thread Aaron Wood
. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-29 Thread Aaron Wood
% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-22 Thread Aaron Wood
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52645/#review156367 --- On Nov. 9, 2016, 7:37 p.m., Aaron Wood wrote: > >

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-11-18 Thread Aaron Wood
with > > `CHAR_MAX` in the first place...? > > Aaron Wood wrote: > I'm not 100% clear on this but my guess is that it's from a negotiated > max body size between the server and clients within Mesos...? > > James Peach wrote: > AFAICT this is assigning the

Re: Review Request 52886: Fix new sign comparison errors in stout produced by hardened flags

2016-11-18 Thread Aaron Wood
& make bench. Thanks, Aaron Wood

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-11-18 Thread Aaron Wood
worked with both gcc and clang. Ran `make && make check && make bench`. Thanks, Aaron Wood

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-11-18 Thread Aaron Wood
add a .h at the time :) - Aaron --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52647/#review156238 --- On Nov. 7,

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-11-18 Thread Aaron Wood
enerated e-mail. To reply, visit: https://reviews.apache.org/r/52647/#review156238 ------- On Nov. 7, 2016, 4:45 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 52645: Harden Mesos

2016-11-09 Thread Aaron Wood
used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-09 Thread Aaron Wood
, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-09 Thread Aaron Wood
/b32667ce-3e3b-4d2b-b4f8-4c2404a0fc1c__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52696: Harden stout

2016-11-09 Thread Aaron Wood
in 2015 so I figured it was best no to say v2016.09.16. - Aaron --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52696/#review154526 --- On Nov. 9, 2016, 7:05 p.m., Aaron Wood wrote: > > --

Re: Review Request 52696: Harden stout

2016-11-09 Thread Aaron Wood
-ef571386e8f4__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-08 Thread Aaron Wood
> On Nov. 2, 2016, 9:33 a.m., Benjamin Bannier wrote: > > src/Makefile.am, line 120 > > <https://reviews.apache.org/r/52645/diff/7/?file=1550864#file1550864line120> > > > > Not sure we want to remove the existing `-Werror`. > > Aaron Wood wrote: &g

Re: Review Request 52696: Harden stout

2016-11-08 Thread Aaron Wood
/3baa96cf-be05-4ac0-ad4c-ef571386e8f4__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-08 Thread Aaron Wood
/uploaded/files/2016/11/02/b32667ce-3e3b-4d2b-b4f8-4c2404a0fc1c__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-08 Thread Aaron Wood
and without the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52754: Remove unused code which now throws errors with the new hardening flags

2016-11-08 Thread Aaron Wood
://reviews.apache.org/r/52754/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52886: Fix new sign comparison errors in stout produced by hardened flags

2016-11-08 Thread Aaron Wood
/strings_tests.cpp 7dd3301 Diff: https://reviews.apache.org/r/52886/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52754: Remove unused code which now throws errors with the new hardening flags

2016-11-08 Thread Aaron Wood
://reviews.apache.org/r/52754/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52754: Remove unused code which now throws errors with the new hardening flags

2016-11-08 Thread Aaron Wood
://reviews.apache.org/r/52754/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-07 Thread Aaron Wood
slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52696: Harden stout

2016-11-07 Thread Aaron Wood
-5ac60daaf498__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/3baa96cf-be05-4ac0-ad4c-ef571386e8f4__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-07 Thread Aaron Wood
/896944ea-9b31-4d62-b1b9-97fb4700a882__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/b32667ce-3e3b-4d2b-b4f8-4c2404a0fc1c__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-07 Thread Aaron Wood
-b1b9-97fb4700a882__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/b32667ce-3e3b-4d2b-b4f8-4c2404a0fc1c__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52696: Harden stout

2016-11-07 Thread Aaron Wood
-5ac60daaf498__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/3baa96cf-be05-4ac0-ad4c-ef571386e8f4__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-07 Thread Aaron Wood
benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-07 Thread Aaron Wood
> On Nov. 2, 2016, 9:33 a.m., Benjamin Bannier wrote: > > src/Makefile.am, line 120 > > <https://reviews.apache.org/r/52645/diff/7/?file=1550864#file1550864line120> > > > > Not sure we want to remove the existing `-Werror`. > > Aaron Wood wrote: &g

Re: Review Request 52696: Harden stout

2016-11-07 Thread Aaron Wood
r modification done in a separate patch. - Aaron --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52696/#review154526 -------

Re: Review Request 52695: Harden libprocess

2016-11-07 Thread Aaron Wood
- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52695/#review154527 --- On Nov. 2, 2016, 3:14 p.m., Aaron Wood wrote: > > --

Re: Review Request 52645: Harden Mesos

2016-11-07 Thread Aaron Wood
r modification done in a separate patch. - Aaron --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52645/#review154524 -------

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-11-07 Thread Aaron Wood
and clang. Ran `make && make check && make bench`. Thanks, Aaron Wood

Re: Review Request 52886: Fix new sign comparison errors in stout produced by hardened flags

2016-11-07 Thread Aaron Wood
/tests/os_tests.cpp 0b7ee07 3rdparty/stout/tests/strings_tests.cpp 7dd3301 Diff: https://reviews.apache.org/r/52886/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52886: Fix new sign comparison errors in stout produced by hardened flags

2016-11-07 Thread Aaron Wood
To reply, visit: https://reviews.apache.org/r/52886/#review154529 --- On Oct. 27, 2016, 7:32 p.m., Aaron Wood wrote: > > --- > This is an automatica

Re: Review Request 52754: Remove unused code which now throws errors with the new hardening flags

2016-11-07 Thread Aaron Wood
/subprocess_tests.cpp 0dc1c62 Diff: https://reviews.apache.org/r/52754/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-03 Thread Aaron Wood
ly, visit: https://reviews.apache.org/r/52695/#review154527 --- On Nov. 2, 2016, 3:14 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 52645: Harden Mesos

2016-11-02 Thread Aaron Wood
t;overall build process more to fix that issue. - Aaron --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52645/#review154524 -----

Re: Review Request 52696: Harden stout

2016-11-02 Thread Aaron Wood
-5ac60daaf498__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/3baa96cf-be05-4ac0-ad4c-ef571386e8f4__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52696: Harden stout

2016-11-02 Thread Aaron Wood
On Nov. 2, 2016, 3:35 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52696/ > ---

Re: Review Request 52695: Harden libprocess

2016-11-02 Thread Aaron Wood
is is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52695/#review154527 ------- On Nov. 2, 2016, 3:14 p.m., Aaron Wood wrote: > > --- > This is an

Re: Review Request 52695: Harden libprocess

2016-11-02 Thread Aaron Wood
-97fb4700a882__optimized.txt No hardening applied and no --enable-optimized https://reviews.apache.org/media/uploaded/files/2016/11/02/b32667ce-3e3b-4d2b-b4f8-4c2404a0fc1c__unoptimized.txt Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-02 Thread Aaron Wood
r/52695/#review154527 --- On Nov. 2, 2016, 3:14 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52695/ > --

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-11-02 Thread Aaron Wood
What do you think? Same comment as above for your first comment. I agree with what you're saying about the casting here. I'll swap it around. - Aaron --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52647/#review154530 ---

Re: Review Request 52645: Harden Mesos

2016-11-01 Thread Aaron Wood
). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-11-01 Thread Aaron Wood
and without the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-11-01 Thread Aaron Wood
this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52696: Harden stout

2016-11-01 Thread Aaron Wood
). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52886: Fix new sign comparison errors in stout produced by hardened flags

2016-10-27 Thread Aaron Wood
3rdparty/stout/tests/os_tests.cpp 0b7ee07 3rdparty/stout/tests/strings_tests.cpp 7dd3301 Diff: https://reviews.apache.org/r/52886/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52886: Fix new sign comparison errors in stout produced by hardened flags

2016-10-27 Thread Aaron Wood
-------- On Oct. 21, 2016, 6:29 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52886/ > -

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-10-27 Thread Aaron Wood
worked with both gcc and clang. Ran `make && make check && make bench`. Thanks, Aaron Wood

Re: Review Request 52754: Remove unused code which now throws errors with the new hardening flags

2016-10-25 Thread Aaron Wood
/process_tests.cpp 3936f47 3rdparty/libprocess/src/tests/subprocess_tests.cpp c8350cf Diff: https://reviews.apache.org/r/52754/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52754: Remove unused code which now throws errors with the new hardening flags

2016-10-25 Thread Aaron Wood
://reviews.apache.org/r/52754/#review153715 --- On Oct. 21, 2016, 6:31 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52754/ > ---

Re: Review Request 52696: Harden stout

2016-10-25 Thread Aaron Wood
ttps://reviews.apache.org/r/52696/#review153025 --- On Oct. 21, 2016, 6:29 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To reply, v

Re: Review Request 52645: Harden Mesos

2016-10-25 Thread Aaron Wood
`-fPIE` only to shared libs. - Aaron --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52645/#review153713 --- On Oct

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-10-18 Thread Aaron Wood
th` is `ssize_t` (set on line 235) - Aaron --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52647/#review153024 --- On Oct. 14,

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-10-18 Thread Aaron Wood
------ On Oct. 14, 2016, 3:14 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52647/ > -

Re: Review Request 52695: Harden libprocess

2016-10-14 Thread Aaron Wood
. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52696: Harden stout

2016-10-14 Thread Aaron Wood
the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-14 Thread Aaron Wood
optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Review Request 52886: Fix new sign comparison errors in stout produced by hardened flags

2016-10-14 Thread Aaron Wood
://reviews.apache.org/r/52886/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52647: Fix new sign comparison errors in libprocess produced by hardened flags

2016-10-14 Thread Aaron Wood
/subprocess_tests.cpp c8350cf Diff: https://reviews.apache.org/r/52647/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran `make && make check && make bench`. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-12 Thread Aaron Wood
645/#comment221381> Move these into `configure.ac`. - Aaron Wood On Oct. 11, 2016, 10:47 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 52695: Harden libprocess

2016-10-12 Thread Aaron Wood
tps://reviews.apache.org/r/52695/#comment221282> Only use `-fstack-protector-strong` if we have GCC >= 4.9. - Aaron Wood On Oct. 11, 2016, 10:47 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 52696: Harden stout

2016-10-12 Thread Aaron Wood
org/r/52696/#comment221284> Only use `-fstack-protector-strong` if we have GCC >= 4.9. - Aaron Wood On Oct. 11, 2016, 10:47 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 52696: Harden stout

2016-10-11 Thread Aaron Wood
the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-11 Thread Aaron Wood
and without the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-10-11 Thread Aaron Wood
the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Review Request 52754: Remove unused code which now throws errors with the new hardening flags

2016-10-11 Thread Aaron Wood
, tests, and benchmarks worked with both gcc and clang. Ran make && make check && make bench. Thanks, Aaron Wood

Re: Review Request 52647: Fix new sign comparison errors produced by hardened flags

2016-10-11 Thread Aaron Wood
7dd3301 Diff: https://reviews.apache.org/r/52647/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran `make && make check && make bench`. Thanks, Aaron Wood

Re: Review Request 52647: Fix new errors/warnings produced by hardened flags

2016-10-11 Thread Aaron Wood
and another RR for the unused code removals. - Aaron Wood On Oct. 10, 2016, 3:51 p.m., Aaron Wood wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 52645: Harden Mesos

2016-10-11 Thread Aaron Wood
645/#comment220913> http://savannah.gnu.org/patch/?8186 http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=commitdiff;h=39683064bbccb4008f239262cb681a970bf53603 - Aaron Wood On Oct. 10, 2016, 7:50 p.m., Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-11 Thread Aaron Wood
east on OS X due to these warnings being treated as hard errors: ``` clang: warning: argument unused during compilation: '-pthread' clang: warning: argument unused during compilation: '-pie' ``` - Aaron Wood On Oct. 10, 2016, 7:50 p.m., Aaro

Re: Review Request 52696: Harden stout

2016-10-10 Thread Aaron Wood
and without the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52695: Harden libprocess

2016-10-10 Thread Aaron Wood
and without the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-10 Thread Aaron Wood
optimizations and without the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-10 Thread Aaron Wood
with and without optimizations and without the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-10 Thread Aaron Wood
with and without optimizations and without the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-10 Thread Aaron Wood
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52645/#review152008 ------- On Oct. 10, 2016, 3:42 p.m., Aaron Wood wrote: > >

Re: Review Request 52647: Fix new errors/warnings produced by hardened flags

2016-10-10 Thread Aaron Wood
/stout/tests/strings_tests.cpp 7dd3301 Diff: https://reviews.apache.org/r/52647/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran `make && make check && make bench`. Thanks, Aaron Wood

Review Request 52696: Harden stout

2016-10-10 Thread Aaron Wood
% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Review Request 52695: Harden libprocess

2016-10-10 Thread Aaron Wood
with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-10 Thread Aaron Wood
, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-10 Thread Aaron Wood
On Oct. 7, 2016, 10:15 p.m., Aaron Wood wrote: > > (1) Do we need to make the `CXXFLAGS` conditional on being supported by the > > current compiler? Seems like these flags are quite specific to (certain > > versions of?) gcc/clang. > > > > (2) You should split t

Re: Review Request 52647: Fix new errors/warnings produced by hardened flags

2016-10-07 Thread Aaron Wood
/stout/tests/strings_tests.cpp 7dd3301 Diff: https://reviews.apache.org/r/52647/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran `make && make check && make bench`. Thanks, Aaron Wood

Re: Review Request 52645: Harden Mesos

2016-10-07 Thread Aaron Wood
the flags being used with and without optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Review Request 52645: Harden Mesos

2016-10-07 Thread Aaron Wood
optimizations. Overall the performance hit was very small with a 3-8% overhead (optimizations brings this down slightly). Most benchmarks were about 5% (or less) slower. Thanks, Aaron Wood

Review Request 52647: Fix new errors/warnings produced by hardened flags

2016-10-07 Thread Aaron Wood
Diff: https://reviews.apache.org/r/52647/diff/ Testing --- Made sure compilation, tests, and benchmarks worked with both gcc and clang. Ran `make && make check && make bench`. Thanks, Aaron Wood

Re: Review Request 51068: Prevent memory leaks

2016-08-18 Thread Aaron Wood
enerated e-mail. To reply, visit: https://reviews.apache.org/r/51068/#review145930 --- On Aug. 17, 2016, 6:21 p.m., Aaron Wood wrote: > > --- > This is an automatically generated

Re: Review Request 51068: Prevent memory leaks

2016-08-17 Thread Aaron Wood
te the promise after satisfying it. Can > > you add more detail as to why you're making this change? > > Aaron Wood wrote: > Hi Ben, thanks for reviewing this patch! Can I ask you the same thing > that I asked in my most recent comment here? > https://github.com/apache/m

<    1   2   3   >