Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-27 Thread Alexei Starovoitov
On Wed, Aug 27, 2014 at 12:18 PM, Stephen Hemminger wrote: > Something in man page format similar to FreeBSD man page: > http://www.freebsd.org/cgi/man.cgi?bpf(4) > > would be more readable and reviewable. Ok. will chop it into smallest diff possible and will add a doc for syscall only. I guess

Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-27 Thread Daniel Borkmann
On 08/27/2014 09:18 PM, Stephen Hemminger wrote: Something in man page format similar to FreeBSD man page: http://www.freebsd.org/cgi/man.cgi?bpf(4) would be more readable and reviewable. I think at some point, we could perhaps do a section 7 page with a general overview of the engine and wh

Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-27 Thread Stephen Hemminger
Something in man page format similar to FreeBSD man page: http://www.freebsd.org/cgi/man.cgi?bpf(4) would be more readable and reviewable. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http:

Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-27 Thread Andy Lutomirski
On Tue, Aug 26, 2014 at 9:57 PM, Alexei Starovoitov wrote: > On Tue, Aug 26, 2014 at 9:49 PM, Andy Lutomirski wrote: >> On Tue, Aug 26, 2014 at 9:35 PM, Alexei Starovoitov >> wrote: >>> On Tue, Aug 26, 2014 at 8:56 PM, Andy Lutomirski >>> wrote: On Aug 26, 2014 7:29 PM, "Alexei Starovoit

Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-27 Thread Steven Stewart-Gallus
> I'm personally not reviewing such a large patch series, sorry. > > You need to submit smaller sets if you want to get reasonable > review of your changes and ideas. Hello. As well, this clogs up the mailing boxes of other people who have no interest in the patch set. Thank you, Steven Stewart-

Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-26 Thread David Miller
From: Alexei Starovoitov Date: Tue, 26 Aug 2014 19:29:14 -0700 > posting whole thing again as RFC to get feedback on syscall only. > If syscall bpf(int cmd, union bpf_attr *attr, unsigned int size) is ok, I'm personally not reviewing such a large patch series, sorry. You need to submit smaller

Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-26 Thread Alexei Starovoitov
On Tue, Aug 26, 2014 at 9:49 PM, Andy Lutomirski wrote: > On Tue, Aug 26, 2014 at 9:35 PM, Alexei Starovoitov wrote: >> On Tue, Aug 26, 2014 at 8:56 PM, Andy Lutomirski wrote: >>> On Aug 26, 2014 7:29 PM, "Alexei Starovoitov" wrote: Hi Ingo, David, posting whole thing again

Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-26 Thread Andy Lutomirski
On Tue, Aug 26, 2014 at 9:35 PM, Alexei Starovoitov wrote: > On Tue, Aug 26, 2014 at 8:56 PM, Andy Lutomirski wrote: >> On Aug 26, 2014 7:29 PM, "Alexei Starovoitov" wrote: >>> >>> Hi Ingo, David, >>> >>> posting whole thing again as RFC to get feedback on syscall only. >>> If syscall bpf(int cm

Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-26 Thread Alexei Starovoitov
On Tue, Aug 26, 2014 at 8:56 PM, Andy Lutomirski wrote: > On Aug 26, 2014 7:29 PM, "Alexei Starovoitov" wrote: >> >> Hi Ingo, David, >> >> posting whole thing again as RFC to get feedback on syscall only. >> If syscall bpf(int cmd, union bpf_attr *attr, unsigned int size) is ok, >> I'll split the

Re: [PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-26 Thread Andy Lutomirski
On Aug 26, 2014 7:29 PM, "Alexei Starovoitov" wrote: > > Hi Ingo, David, > > posting whole thing again as RFC to get feedback on syscall only. > If syscall bpf(int cmd, union bpf_attr *attr, unsigned int size) is ok, > I'll split them into small chunks as requested and will repost without RFC. IM

[PATCH RFC v7 net-next 00/28] BPF syscall

2014-08-26 Thread Alexei Starovoitov
Hi Ingo, David, posting whole thing again as RFC to get feedback on syscall only. If syscall bpf(int cmd, union bpf_attr *attr, unsigned int size) is ok, I'll split them into small chunks as requested and will repost without RFC. Right now please only review syscall API patch 0003 introduces sys_