[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1046/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/938/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1045/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/937/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1044/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread jrushford
Github user jrushford commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
@jpeach and @PSUdaemon - I've made iobuffer_advice a local variable.  
Tested and it is working okay.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
Linux build *failed*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/936/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/933/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1040/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-18 Thread jpeach
Github user jpeach commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
No I think it is trivial to remove the global and we should just do it

> On Oct 18, 2016, at 7:29 AM, John J. Rushford  
wrote:
> 
> @jrushford commented on this pull request.
> 
> In iocore/eventsystem/I_IOBuffer.h:
> 
> > @@ -58,6 +58,7 @@ class VIO;
>  inkcoreapi extern int64_t max_iobuffer_size;
>  extern int64_t default_small_iobuffer_size;
>  extern int64_t default_large_iobuffer_size; // matched to size of OS 
buffers
> +extern int iobuffer_advice;
> @jpeach I'd like to land this as is and write a JIRA to come back to the 
global variable once we've had time to evaluate this. Are you okay with this?
> 
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
> 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-13 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/886/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-13 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/994/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-13 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
FreeBSD build *failed*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/993/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-13 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
Linux build *failed*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/885/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-11 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/975/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-11 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/867/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-11 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/866/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-11 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/974/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-11 Thread jrushford
Github user jrushford commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
[approve ci]


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-11 Thread jrushford
Github user jrushford commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
@PSUdaemon - requested changes have been made and squashed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-11 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/863/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1097: Make the use of madvise() with MADV_DONTDUMP conf...

2016-10-11 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1097
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/971/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---