All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shahab Vahedi via buildroot <buildroot@buildroot.org>
To: Arnout Vandecappelle <arnout@mind.be>
Cc: "linux-snps-arc@lists.infradead.org"
	<linux-snps-arc@lists.infradead.org>,
	Alexey Brodkin <Alexey.Brodkin@synopsys.com>,
	Shahab Vahedi <Shahab.Vahedi@synopsys.com>,
	"buildroot@buildroot.org" <buildroot@buildroot.org>
Subject: Re: [Buildroot] [PATCH v4 1/1] package/bpftool: revert bpf_cookie patch to allow building
Date: Thu, 9 Feb 2023 12:23:57 +0000	[thread overview]
Message-ID: <b98c7a16-ea21-c398-4045-0edbf46caa40@synopsys.com> (raw)
In-Reply-To: <fce2f167-bd7a-8b93-41ec-c7c339ec070f@mind.be>

Hi Arnout,

On 2/8/23 17:36, Arnout Vandecappelle wrote:
> 
> It's not clear to me why you did a revert of the cookie feature rather than
> using those patches. I think the first two or three should be sufficient to
> fix your issue?

Ideally, the correct way would be applying that pending series. But since
there were comments that some of the patches might need reworking [1], I did
not feel confident to add them. Therefore, I tried to keep my solution minimal
and solve it by eliminating the problem. I know, it is not appealing.

> For the time being, I marked this as Changes Requested. If the upstream
> patches are not sufficient, please send a mail upstream (as a reply to that
> series) explaining the problem (again), and re-send this patch to
> Buildroot. If it does fix things, please send a mail upstream to ask to
> move the series forward (explaining the issue you have), and send a patch
> to Buildroot that adds the needed patches.

They (re)confirmed the issue to me in the past [2] and are aware that its
solution is missing from all the releases onward (see the first point of
"Known bugs" in the release page [3]).

[1]
https://lore.kernel.org/bpf/20220421003152.339542-3-alobakin@pm.me/T/#rb9ce86e9f25b2d097b88d87af8ffdfa401d63c68

[2]
https://lore.kernel.org/bpf/b1a604e1-be09-ac0b-ff22-b194ae9ce886@isovalent.com/

[3]
https://github.com/libbpf/bpftool/releases

--
Shahab


_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

WARNING: multiple messages have this Message-ID (diff)
From: Shahab Vahedi <Shahab.Vahedi@synopsys.com>
To: Arnout Vandecappelle <arnout@mind.be>
Cc: "buildroot@buildroot.org" <buildroot@buildroot.org>,
	"linux-snps-arc@lists.infradead.org"
	<linux-snps-arc@lists.infradead.org>,
	Alexey Brodkin <Alexey.Brodkin@synopsys.com>,
	Shahab Vahedi <Shahab.Vahedi@synopsys.com>
Subject: Re: [Buildroot] [PATCH v4 1/1] package/bpftool: revert bpf_cookie patch to allow building
Date: Thu, 9 Feb 2023 12:23:57 +0000	[thread overview]
Message-ID: <b98c7a16-ea21-c398-4045-0edbf46caa40@synopsys.com> (raw)
In-Reply-To: <fce2f167-bd7a-8b93-41ec-c7c339ec070f@mind.be>

Hi Arnout,

On 2/8/23 17:36, Arnout Vandecappelle wrote:
> 
> It's not clear to me why you did a revert of the cookie feature rather than
> using those patches. I think the first two or three should be sufficient to
> fix your issue?

Ideally, the correct way would be applying that pending series. But since
there were comments that some of the patches might need reworking [1], I did
not feel confident to add them. Therefore, I tried to keep my solution minimal
and solve it by eliminating the problem. I know, it is not appealing.

> For the time being, I marked this as Changes Requested. If the upstream
> patches are not sufficient, please send a mail upstream (as a reply to that
> series) explaining the problem (again), and re-send this patch to
> Buildroot. If it does fix things, please send a mail upstream to ask to
> move the series forward (explaining the issue you have), and send a patch
> to Buildroot that adds the needed patches.

They (re)confirmed the issue to me in the past [2] and are aware that its
solution is missing from all the releases onward (see the first point of
"Known bugs" in the release page [3]).

[1]
https://lore.kernel.org/bpf/20220421003152.339542-3-alobakin@pm.me/T/#rb9ce86e9f25b2d097b88d87af8ffdfa401d63c68

[2]
https://lore.kernel.org/bpf/b1a604e1-be09-ac0b-ff22-b194ae9ce886@isovalent.com/

[3]
https://github.com/libbpf/bpftool/releases

--
Shahab


_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc

  reply	other threads:[~2023-02-09 12:48 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-13 21:30 [Buildroot] [PATCH] package/bpftool: revert bpf_cookie patch to allow building Shahab Vahedi via buildroot
2022-06-14  8:26 ` [Buildroot] [PATCH v2 1/1] " Shahab Vahedi via buildroot
2022-06-14  8:26   ` Shahab Vahedi
2022-06-14  9:31   ` [Buildroot] [PATCH v3 " Shahab Vahedi via buildroot
2022-06-14  9:31     ` Shahab Vahedi
2022-06-14 17:14     ` [Buildroot] " Arnout Vandecappelle
2022-06-14 17:14       ` Arnout Vandecappelle
2022-06-14 17:27       ` [Buildroot] " Arnout Vandecappelle
2022-06-14 17:27         ` Arnout Vandecappelle
2022-06-15 11:03         ` [Buildroot] " Shahab Vahedi via buildroot
2022-06-15 11:03           ` Shahab Vahedi
2022-06-15 11:10       ` [Buildroot] " Shahab Vahedi via buildroot
2022-06-15 11:10         ` Shahab Vahedi
2022-06-15 23:27         ` [Buildroot] " James Hilliard
2022-06-15 23:27           ` James Hilliard
2022-06-16  8:11           ` Shahab Vahedi via buildroot
2022-06-16  8:11             ` Shahab Vahedi
2022-06-19 15:20             ` Arnout Vandecappelle
2022-06-19 15:20               ` Arnout Vandecappelle
2022-06-19 23:19               ` James Hilliard
2022-06-19 23:19                 ` James Hilliard
2022-06-20  6:45                 ` Arnout Vandecappelle
2022-06-20  6:45                   ` Arnout Vandecappelle
2022-06-20  9:17                   ` James Hilliard
2022-06-20  9:17                     ` James Hilliard
2022-06-20 18:27                     ` Arnout Vandecappelle
2022-06-20 18:27                       ` Arnout Vandecappelle
2022-08-09  9:46                       ` James Hilliard
2022-08-09  9:46                         ` James Hilliard
2022-06-15 10:57 ` [Buildroot] [PATCH v4 " Shahab Vahedi via buildroot
2022-06-15 10:57   ` Shahab Vahedi
2022-06-24 14:21   ` Shahab Vahedi via buildroot
2022-06-24 14:21     ` Shahab Vahedi
2023-02-08 16:36   ` Arnout Vandecappelle
2023-02-08 16:36     ` Arnout Vandecappelle
2023-02-09 12:23     ` Shahab Vahedi via buildroot [this message]
2023-02-09 12:23       ` Shahab Vahedi
2023-02-09 20:27       ` Arnout Vandecappelle
2023-02-09 20:27         ` Arnout Vandecappelle
2023-02-10  9:17         ` Shahab Vahedi via buildroot
2023-02-10  9:17           ` Shahab Vahedi
2023-02-10 13:12           ` Arnout Vandecappelle
2023-02-10 13:12             ` Arnout Vandecappelle

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b98c7a16-ea21-c398-4045-0edbf46caa40@synopsys.com \
    --to=buildroot@buildroot.org \
    --cc=Alexey.Brodkin@synopsys.com \
    --cc=Shahab.Vahedi@synopsys.com \
    --cc=arnout@mind.be \
    --cc=linux-snps-arc@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.