Re: [PATCH bpf-next] libbpf 1.0: deprecate bpf_map__is_offload_neutral()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jan 4, 2022 at 4:06 PM Christy Lee <christylee@xxxxxx> wrote:
>
> Deprecate bpf_map__is_offload_neutral(). It’s most probably broken
> already. PERF_EVENT_ARRAY isn’t the only map that’s not suitable
> for hardware offloading. Applications can directly check map type
> instead.
>
> [0] Closes: https://github.com/libbpf/libbpf/issues/306
>
> Signed-off-by: Christy Lee <christylee@xxxxxx>
> ---

Applied to bpf-next, thanks!

>  tools/bpf/bpftool/prog.c | 2 +-
>  tools/lib/bpf/libbpf.h   | 1 +
>  2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/bpf/bpftool/prog.c b/tools/bpf/bpftool/prog.c
> index f874896c4154..2a21d50516bc 100644
> --- a/tools/bpf/bpftool/prog.c
> +++ b/tools/bpf/bpftool/prog.c
> @@ -1655,7 +1655,7 @@ static int load_with_options(int argc, char **argv, bool first_prog_only)
>         j = 0;
>         idx = 0;
>         bpf_object__for_each_map(map, obj) {
> -               if (!bpf_map__is_offload_neutral(map))
> +               if (bpf_map__type(map) != BPF_MAP_TYPE_PERF_EVENT_ARRAY)
>                         bpf_map__set_ifindex(map, ifindex);
>
>                 if (j < old_map_fds && idx == map_replace[j].idx) {
> diff --git a/tools/lib/bpf/libbpf.h b/tools/lib/bpf/libbpf.h
> index 85dfef88b3d2..ec4309cb9771 100644
> --- a/tools/lib/bpf/libbpf.h
> +++ b/tools/lib/bpf/libbpf.h
> @@ -744,6 +744,7 @@ LIBBPF_API void *bpf_map__priv(const struct bpf_map *map);
>  LIBBPF_API int bpf_map__set_initial_value(struct bpf_map *map,
>                                           const void *data, size_t size);
>  LIBBPF_API const void *bpf_map__initial_value(struct bpf_map *map, size_t *psize);
> +LIBBPF_DEPRECATED_SINCE(0, 8, "use bpf_map__type() instead")
>  LIBBPF_API bool bpf_map__is_offload_neutral(const struct bpf_map *map);
>
>  /**
> --
> 2.30.2
>




[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux