Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755616AbaJNU2S (ORCPT ); Tue, 14 Oct 2014 16:28:18 -0400 Received: from mx1.redhat.com ([209.132.183.28]:4850 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755440AbaJNU2Q (ORCPT ); Tue, 14 Oct 2014 16:28:16 -0400 Message-ID: <543D8741.5040107@redhat.com> Date: Tue, 14 Oct 2014 22:27:45 +0200 From: Daniel Borkmann User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 MIME-Version: 1.0 To: Alexei Starovoitov CC: "David S. Miller" , Ingo Molnar , Linus Torvalds , Andy Lutomirski , Steven Rostedt , Hannes Frederic Sowa , Chema Gonzalez , Eric Dumazet , Peter Zijlstra , "H. Peter Anvin" , Andrew Morton , Kees Cook , Linux API , Network Development , LKML Subject: Re: [PATCH v9 net-next 2/4] net: filter: split filter.h and expose eBPF to user space References: <1409714246-31054-1-git-send-email-ast@plumgrid.com> <1409714246-31054-3-git-send-email-ast@plumgrid.com> <540737DF.1010801@redhat.com> <543C0A2D.5040908@redhat.com> <543CD206.709@redhat.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/14/2014 10:43 AM, Alexei Starovoitov wrote: > On Tue, Oct 14, 2014 at 12:34 AM, Daniel Borkmann wrote: >> On 10/13/2014 11:49 PM, Alexei Starovoitov wrote: >>> >>> On Mon, Oct 13, 2014 at 10:21 AM, Daniel Borkmann >>> wrote: >>>> >>>> On 09/03/2014 05:46 PM, Daniel Borkmann wrote: >>>> ... >>>>> >>>>> Ok, given you post the remaining two RFCs later on this window as >>>>> you indicate, I have no objections: >>>>> >>>>> Acked-by: Daniel Borkmann >>>> >>>> Ping, Alexei, are you still sending the patch for bpf_common.h or >>>> do you want me to take care of this? >>> >>> It's not forgotten. >>> I'm not sending it only because net-next is closed >>> and it seems to be -next material. >> >> Well, the point was since it's UAPI you're modifying, that it needs >> to be shipped before it first gets exposed to user land ... >> >> I think that should be reason enough ... there's no point in doing >> this at a later point in time. > > Moving common #defines from filter.h into bpf_common.h can > be done at any point in time. For the sake of argument if > there is an app that includes both filter.h and bpf.h, it will > continue to work just fine. Correct, but the argument was that we can _avoid_ this from the very beginning. Thus, user space applications making use of eBPF only need to include , nothing more. Doing this at any later point in time will just lead to the need to include both headers. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/