----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20781/#review42774 -----------------------------------------------------------
Ship it! src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76672> s/libnl fixes the bug/the bug is fixed in libnl/ src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76674> Can you add a comment on what "mirred" does here? src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76673> const. src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76676> ditto. src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76677> What is a "u32" filter? And why is the matched packet rule is only applied to them? src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76679> new line. src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76683> s/get/Clses/getFilters/ ? src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76682> s/getCls/getFilter/ src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76685> s/check/exists_/ src/linux/routing/filter/internal.hpp <https://reviews.apache.org/r/20781/#comment76688> Can we print the old and new priorities here? src/linux/routing/queueing/internal.hpp <https://reviews.apache.org/r/20781/#comment76702> s/queue/queueing/ ? src/linux/routing/queueing/internal.hpp <https://reviews.apache.org/r/20781/#comment76703> s/queue/queueing/ ? - Vinod Kone On May 9, 2014, 10:27 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20781/ > ----------------------------------------------------------- > > (Updated May 9, 2014, 10:27 p.m.) > > > Review request for mesos, Benjamin Hindman, Chi Zhang, Ian Downes, Vinod > Kone, and Cong Wang. > > > Bugs: MESOS-1228 > https://issues.apache.org/jira/browse/MESOS-1228 > > > Repository: mesos-git > > > Description > ------- > > Added internal APIs for filtering. This patch is used by the following > patches (ICMP, ARP, IP filters). All kinds of filters will use the same > internal APIs. > > In this patch, I also exposed APIs for queueing disciplines (qdisc) and > queueing classes. Currently, we only use the ingress queueing discipline for > packets redirection. In the future, we may wanna introduce different queueing > disciplines and classes for network isolation and shape control. The internal > APIs make the extension very easy. > > > Diffs > ----- > > src/Makefile.am d2e006d > src/linux/routing/filter/action.hpp PRE-CREATION > src/linux/routing/filter/filter.hpp PRE-CREATION > src/linux/routing/filter/internal.hpp PRE-CREATION > src/linux/routing/filter/priority.hpp PRE-CREATION > src/linux/routing/queueing/handle.hpp PRE-CREATION > src/linux/routing/queueing/handle.cpp PRE-CREATION > src/linux/routing/queueing/ingress.hpp PRE-CREATION > src/linux/routing/queueing/ingress.cpp PRE-CREATION > src/linux/routing/queueing/internal.hpp PRE-CREATION > > Diff: https://reviews.apache.org/r/20781/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
