From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [PATCH net-next] tc: bpf: generalize pedit action Date: Fri, 27 Mar 2015 07:38:19 +0100 Message-ID: <20150327063819.GC2073@nanopsycho.orion> References: <1427424837-7757-1-git-send-email-ast@plumgrid.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1427424837-7757-1-git-send-email-ast-uqk4Ao+rVK5Wk0Htik3J/w@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Alexei Starovoitov Cc: "David S. Miller" , Daniel Borkmann , Jamal Hadi Salim , linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-api@vger.kernel.org Fri, Mar 27, 2015 at 03:53:57AM CET, ast-uqk4Ao+rVK5Wk0Htik3J/w@public.gmane.org wrote: >existing TC action 'pedit' can munge any bits of the packet. >Generalize it for use in bpf programs attached as cls_bpf and act_bpf via >bpf_skb_store_bytes() helper function. > >Signed-off-by: Alexei Starovoitov This looks fine to me. Great stuff. Reviewed-by: Jiri Pirko