> On May 13, 2014, 12:22 a.m., Vinod Kone wrote: > > src/linux/routing/filter/internal.hpp, line 250 > > <https://reviews.apache.org/r/20781/diff/1/?file=569519#file569519line250> > > > > What is a "u32" filter? And why is the matched packet rule is only > > applied to them?
Added comments. > On May 13, 2014, 12:22 a.m., Vinod Kone wrote: > > src/linux/routing/filter/internal.hpp, line 313 > > <https://reviews.apache.org/r/20781/diff/1/?file=569519#file569519line313> > > > > s/get/Clses/getFilters/ ? I have the following rules to avoid confusion. Cls --> libnl stuff Classifier -> our own abstraction - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20781/#review42774 ----------------------------------------------------------- On May 14, 2014, 11:40 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20781/ > ----------------------------------------------------------- > > (Updated May 14, 2014, 11:40 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 12374c4 > 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 > >
