Re: Review Request 34426: Extend the Traffic Control queueing models to allow dynamically allocated Handles, extend API to allow searching by handle (or parent).
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34426/ --- (Updated May 20, 2015, 5:55 p.m.) Review request for mesos, Chi Zhang, Ian Downes, Jie Yu, and Cong Wang. Bugs: MESOS-2665 https://issues.apache.org/jira/browse/MESOS-2665 Repository: mesos Description --- Extend the Traffic Control queueing models to allow dynamically allocated Handles, extend API to allow searching by handle (or parent). Diffs (updated) - src/linux/routing/queueing/fq_codel.hpp 4f67ab7d64afea96a07dfcf36769a9c667749a00 src/linux/routing/queueing/fq_codel.cpp 02ad8df7814c0e549a9ca9aef39777684e6abdcb src/linux/routing/queueing/ingress.hpp b323a7f6daed828327d6d9e9740df81582e0ba2b src/linux/routing/queueing/ingress.cpp 47c73376097d70819defdee31a6d1e446df6b8ba src/linux/routing/queueing/internal.hpp 7c6c4d3d960b9a4bf44dcf482212317522353d69 src/slave/containerizer/isolators/network/port_mapping.cpp 49e983edab598e2ac487bb488fdd12840a9e7dfc src/tests/port_mapping_tests.cpp b8c2db6d0a02f79d38a21c227575299880980502 src/tests/routing_tests.cpp 6bf5e63aecf2dee36fb8cf3575c0bd2625d29dfa Diff: https://reviews.apache.org/r/34426/diff/ Testing --- make check Thanks, Paul Brett
Re: Review Request 34426: Extend the Traffic Control queueing models to allow dynamically allocated Handles, extend API to allow searching by handle (or parent).
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34426/#review84445 --- src/linux/routing/queueing/fq_codel.hpp https://reviews.apache.org/r/34426/#comment135709 It doesn't return false, it returns a Handle. src/linux/routing/queueing/fq_codel.hpp https://reviews.apache.org/r/34426/#comment135711 Please reorder the parameters, move 'parent' before 'handle', this reads better. src/linux/routing/queueing/fq_codel.cpp https://reviews.apache.org/r/34426/#comment135712 Ditto src/linux/routing/queueing/ingress.hpp https://reviews.apache.org/r/34426/#comment135708 The comment needs to update, since now the return value is Handle. src/linux/routing/queueing/ingress.cpp https://reviews.apache.org/r/34426/#comment135715 Please print the link name in error message. src/linux/routing/queueing/internal.hpp https://reviews.apache.org/r/34426/#comment135717 Ditto src/linux/routing/queueing/internal.hpp https://reviews.apache.org/r/34426/#comment135710 Ditto src/slave/containerizer/isolators/network/port_mapping.cpp https://reviews.apache.org/r/34426/#comment135713 s/the filter handles/the ingress qdisc handle/ ? src/slave/containerizer/isolators/network/port_mapping.cpp https://reviews.apache.org/r/34426/#comment135718 Ditto src/tests/port_mapping_tests.cpp https://reviews.apache.org/r/34426/#comment135725 s/eth0IngressQdisc/eth0IngressHandle/ src/tests/port_mapping_tests.cpp https://reviews.apache.org/r/34426/#comment135726 Ditto src/tests/routing_tests.cpp https://reviews.apache.org/r/34426/#comment135719 vethHandle is a bad name, it reads like a handle of veth. Please rename it to 'egressHandle'. - Cong Wang On May 19, 2015, 7:55 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34426/ --- (Updated May 19, 2015, 7:55 p.m.) Review request for mesos, Chi Zhang, Ian Downes, Jie Yu, and Cong Wang. Bugs: MESOS-2665 https://issues.apache.org/jira/browse/MESOS-2665 Repository: mesos Description --- Extend the Traffic Control queueing models to allow dynamically allocated Handles, extend API to allow searching by handle (or parent). Diffs - src/linux/routing/queueing/fq_codel.hpp 4f67ab7d64afea96a07dfcf36769a9c667749a00 src/linux/routing/queueing/fq_codel.cpp 02ad8df7814c0e549a9ca9aef39777684e6abdcb src/linux/routing/queueing/ingress.hpp b323a7f6daed828327d6d9e9740df81582e0ba2b src/linux/routing/queueing/ingress.cpp 47c73376097d70819defdee31a6d1e446df6b8ba src/linux/routing/queueing/internal.hpp 7c6c4d3d960b9a4bf44dcf482212317522353d69 src/slave/containerizer/isolators/network/port_mapping.cpp 49e983edab598e2ac487bb488fdd12840a9e7dfc src/tests/port_mapping_tests.cpp b8c2db6d0a02f79d38a21c227575299880980502 src/tests/routing_tests.cpp 6bf5e63aecf2dee36fb8cf3575c0bd2625d29dfa Diff: https://reviews.apache.org/r/34426/diff/ Testing --- make check Thanks, Paul Brett
Review Request 34426: Extend the Traffic Control queueing models to allow dynamically allocated Handles, extend API to allow searching by handle (or parent).
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34426/ --- Review request for mesos, Chi Zhang, Ian Downes, Jie Yu, and Cong Wang. Bugs: MESOS-2665 https://issues.apache.org/jira/browse/MESOS-2665 Repository: mesos Description --- Extend the Traffic Control queueing models to allow dynamically allocated Handles, extend API to allow searching by handle (or parent). Diffs - src/linux/routing/queueing/fq_codel.hpp 4f67ab7d64afea96a07dfcf36769a9c667749a00 src/linux/routing/queueing/fq_codel.cpp 02ad8df7814c0e549a9ca9aef39777684e6abdcb src/linux/routing/queueing/ingress.hpp b323a7f6daed828327d6d9e9740df81582e0ba2b src/linux/routing/queueing/ingress.cpp 47c73376097d70819defdee31a6d1e446df6b8ba src/linux/routing/queueing/internal.hpp 7c6c4d3d960b9a4bf44dcf482212317522353d69 src/slave/containerizer/isolators/network/port_mapping.cpp 49e983edab598e2ac487bb488fdd12840a9e7dfc src/tests/port_mapping_tests.cpp b8c2db6d0a02f79d38a21c227575299880980502 src/tests/routing_tests.cpp 6bf5e63aecf2dee36fb8cf3575c0bd2625d29dfa Diff: https://reviews.apache.org/r/34426/diff/ Testing --- make check Thanks, Paul Brett
Re: Review Request 34426: Extend the Traffic Control queueing models to allow dynamically allocated Handles, extend API to allow searching by handle (or parent).
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34426/ --- (Updated May 19, 2015, 7:55 p.m.) Review request for mesos, Chi Zhang, Ian Downes, Jie Yu, and Cong Wang. Bugs: MESOS-2665 https://issues.apache.org/jira/browse/MESOS-2665 Repository: mesos Description --- Extend the Traffic Control queueing models to allow dynamically allocated Handles, extend API to allow searching by handle (or parent). Diffs (updated) - src/linux/routing/queueing/fq_codel.hpp 4f67ab7d64afea96a07dfcf36769a9c667749a00 src/linux/routing/queueing/fq_codel.cpp 02ad8df7814c0e549a9ca9aef39777684e6abdcb src/linux/routing/queueing/ingress.hpp b323a7f6daed828327d6d9e9740df81582e0ba2b src/linux/routing/queueing/ingress.cpp 47c73376097d70819defdee31a6d1e446df6b8ba src/linux/routing/queueing/internal.hpp 7c6c4d3d960b9a4bf44dcf482212317522353d69 src/slave/containerizer/isolators/network/port_mapping.cpp 49e983edab598e2ac487bb488fdd12840a9e7dfc src/tests/port_mapping_tests.cpp b8c2db6d0a02f79d38a21c227575299880980502 src/tests/routing_tests.cpp 6bf5e63aecf2dee36fb8cf3575c0bd2625d29dfa Diff: https://reviews.apache.org/r/34426/diff/ Testing --- make check Thanks, Paul Brett