[GitHub] trafficserver pull request: TS-4211: Make freelist_new cleaner so ...

2016-02-15 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/481#issuecomment-184527976 I just confirmed that this is correct, it will actually be hugepage aligned. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] trafficserver pull request: TS-4211: Make freelist_new cleaner so ...

2016-02-15 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/481#issuecomment-184524565 All good :+1: just set alignment = 0 on declaration and let the complier deal with it if it's always assigned later. --- If your project is set up for it, you can

[GitHub] trafficserver pull request: TS-4211: Make freelist_new cleaner so ...

2016-02-15 Thread bgaff
Github user bgaff commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/481#discussion_r52969591 --- Diff: lib/ts/ink_queue.cc --- @@ -200,19 +200,20 @@ freelist_new(InkFreeList *f) if (TO_PTR(FREELIST_POINTER(item)) == NULL) { uin

[GitHub] trafficserver pull request: TS-4211: Make freelist_new cleaner so ...

2016-02-15 Thread PSUdaemon
GitHub user PSUdaemon opened a pull request: https://github.com/apache/trafficserver/pull/481 TS-4211: Make freelist_new cleaner so we can debug better. You can merge this pull request into a Git repository by running: $ git pull https://github.com/PSUdaemon/trafficserver free

[GitHub] trafficserver pull request: TS-4208: Fix freelist mem dump total a...

2016-02-15 Thread PSUdaemon
Github user PSUdaemon closed the pull request at: https://github.com/apache/trafficserver/pull/479 --- 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 featu

[GitHub] trafficserver pull request: TS-4209: Limit concurrent streams corr...

2016-02-15 Thread masaori335
GitHub user masaori335 opened a pull request: https://github.com/apache/trafficserver/pull/480 TS-4209: Limit concurrent streams correctly [TS-4209](https://issues.apache.org/jira/browse/TS-4209) You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] trafficserver pull request: TS-4208: Fix freelist mem dump total a...

2016-02-15 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/479#issuecomment-184509159 +1 --- 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 enable

[GitHub] trafficserver pull request: TS-4208: Fix freelist mem dump total a...

2016-02-15 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/479#issuecomment-184495815 :+1: looks good to me. --- 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

[GitHub] trafficserver pull request: Freelist cleanup 2

2016-02-15 Thread PSUdaemon
GitHub user PSUdaemon opened a pull request: https://github.com/apache/trafficserver/pull/479 Freelist cleanup 2 You can merge this pull request into a Git repository by running: $ git pull https://github.com/PSUdaemon/trafficserver freelist_cleanup_2 Alternatively you can re

[GitHub] trafficserver pull request: Doc: [TS-4112] By default, TS caches o...

2016-02-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/410 --- 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