From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexei Starovoitov Subject: Re: [PATCH net-next 4/6] tools/lib/bpf: expose bpf_program__set_type() Date: Thu, 30 Mar 2017 19:37:41 -0700 Message-ID: <1e3156bf-7334-53fe-1069-7d7bd9dc09fb@fb.com> References: <20170331013157.3298003-1-ast@fb.com> <20170331013157.3298003-5-ast@fb.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Cc: Daniel Borkmann , Martin KaFai Lau , , To: "Wangnan (F)" , "David S . Miller" Return-path: Received: from mx0a-00082601.pphosted.com ([67.231.145.42]:58337 "EHLO mx0a-00082601.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750820AbdCaCiU (ORCPT ); Thu, 30 Mar 2017 22:38:20 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On 3/30/17 7:33 PM, Wangnan (F) wrote: >> +void bpf_program__set_type(struct bpf_program *prog, enum >> bpf_prog_type type); >> > > This makes libbpf.h depend on uapi/linux/bpf.h (because of enum > bpf_prog_type), which is not always available. > > What about defining another enum inside libbpf.h? how about just including bpf.h? or making it 'int' instead of enum?