All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Ihor Solodrai <ihor.solodrai@pm.me>
Cc: dwarves@vger.kernel.org, bpf@vger.kernel.org,
	alan.maguire@oracle.com, eddyz87@gmail.com, andrii@kernel.org,
	mykolal@fb.com, olsajiri@gmail.com
Subject: Re: [PATCH dwarves v4 00/10] pahole: faster reproducible BTF encoding
Date: Thu, 9 Jan 2025 18:21:08 -0300	[thread overview]
Message-ID: <Z4A9xAHiNxoNihYe@x1> (raw)
In-Reply-To: <0gG5e2QvD6I6CuApCVn4O0ph5_k_fnKodGP2wGhm6oxiLKh5A_v1mTeezCjU_oEwQxUwRMN9yycf5xbLfIviTlAg-qm6dxHK-PkgVTFDLQ8=@pm.me>

On Thu, Jan 09, 2025 at 06:38:57PM +0000, Ihor Solodrai wrote:
> On Thursday, January 9th, 2025 at 10:32 AM, Arnaldo Carvalho de Melo <acme@kernel.org> wrote:
> 
> > 
> > 
> > On Tue, Jan 07, 2025 at 07:08:59PM +0000, Ihor Solodrai wrote:
> > 
> > > This is v4 of the series aiming to speed up parallel reproducible BTF
> > > encoding. This version mostly addresses feedback from Jiri Olsa on v3.
> > 
> > 
> > b4 is having trouble with this series, I'm trying to cherry pick
> > things...
> 
> Sorry, I think I messed up some patches with manual changes before submitting.
> I'll resend a clean series in a bit.

Better now:

⬢ [acme@toolbox pahole]$ b4 am -ctsl --cc-trailers 20250109185950.653110-2-ihor.solodrai@pm.me
Grabbing thread from lore.kernel.org/all/20250109185950.653110-2-ihor.solodrai@pm.me/t.mbox.gz
Checking for newer revisions
Grabbing search results from lore.kernel.org
Analyzing 15 messages in the thread
Looking for additional code-review trailers on lore.kernel.org
Checking attestation on all messages, may take a moment...
---
  ✗ [PATCH v4 1/10] btf_encoder: simplify function encoding
    + Acked-by: Eduard Zingerman <eddyz87@gmail.com> (✗ DKIM/gmail.com)
    + Acked-by: Jiri Olsa <jolsa@kernel.org> (✗ DKIM/gmail.com)
    + Link: https://lore.kernel.org/r/20250109185950.653110-2-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ✗ [PATCH v4 2/10] btf_encoder: free encoder->secinfo in btf_encoder__delete
    + Link: https://lore.kernel.org/r/20250109185950.653110-3-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ✗ [PATCH v4 3/10] btf_encoder: separate elf function, saved function representations
    + Link: https://lore.kernel.org/r/20250109185950.653110-4-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ✗ [PATCH v4 4/10] btf_encoder: introduce elf_functions struct type
    + Link: https://lore.kernel.org/r/20250109185950.653110-5-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ✗ [PATCH v4 5/10] btf_encoder: introduce elf_functions_list
    + Link: https://lore.kernel.org/r/20250109185950.653110-6-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ✗ [PATCH v4 6/10] btf_encoder: remove skip_encoding_inconsistent_proto
    + Link: https://lore.kernel.org/r/20250109185950.653110-7-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ✗ [PATCH v4 7/10] dwarf_loader: introduce cu->id
    + Link: https://lore.kernel.org/r/20250109185950.653110-8-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ✗ [PATCH v4 8/10] dwarf_loader: multithreading with a job/worker model
    + Link: https://lore.kernel.org/r/20250109185950.653110-9-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ✗ [PATCH v4 9/10] btf_encoder: clean up global encoders list
    + Link: https://lore.kernel.org/r/20250109185950.653110-10-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ✗ [PATCH v4 10/10] btf_encoder: switch func_states from a list to an array
    + Link: https://lore.kernel.org/r/20250109185950.653110-11-ihor.solodrai@pm.me
    + Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  ---
  ✗ BADSIG: DKIM/pm.me
---
Total patches: 10
---
Cover: ./v4_20250109_ihor_solodrai_pahole_faster_reproducible_btf_encoding.cover
 Link: https://lore.kernel.org/r/20250109185950.653110-1-ihor.solodrai@pm.me
 Base: not specified
       git am ./v4_20250109_ihor_solodrai_pahole_faster_reproducible_btf_encoding.mbx
⬢ [acme@toolbox pahole]$        git am ./v4_20250109_ihor_solodrai_pahole_faster_reproducible_btf_encoding.mbx
Applying: btf_encoder: simplify function encoding
Applying: btf_encoder: free encoder->secinfo in btf_encoder__delete
Applying: btf_encoder: separate elf function, saved function representations
Applying: btf_encoder: introduce elf_functions struct type
Applying: btf_encoder: introduce elf_functions_list
Applying: btf_encoder: remove skip_encoding_inconsistent_proto
Applying: dwarf_loader: introduce cu->id
Applying: dwarf_loader: multithreading with a job/worker model
Applying: btf_encoder: clean up global encoders list
Applying: btf_encoder: switch func_states from a list to an array
⬢ [acme@toolbox pahole]$

      reply	other threads:[~2025-01-09 21:21 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-07 19:08 [PATCH dwarves v4 00/10] pahole: faster reproducible BTF encoding Ihor Solodrai
2025-01-07 19:09 ` [PATCH dwarves v4 01/10] btf_encoder: simplify function encoding Ihor Solodrai
2025-01-07 19:09 ` [PATCH dwarves v4 02/10] btf_encoder: free encoder->secinfo in btf_encoder__delete Ihor Solodrai
2025-01-09 16:54   ` Alan Maguire
2025-01-09 21:36     ` Arnaldo Carvalho de Melo
2025-01-07 19:09 ` [PATCH dwarves v4 03/10] btf_encoder: separate elf function, saved function representations Ihor Solodrai
2025-01-07 19:09 ` [PATCH dwarves v4 04/10] btf_encoder: introduce elf_functions struct type Ihor Solodrai
2025-01-07 19:09 ` [PATCH dwarves v4 05/10] btf_encoder: introduce elf_functions_list Ihor Solodrai
2025-01-07 19:09 ` [PATCH dwarves v4 06/10] btf_encoder: remove skip_encoding_inconsistent_proto Ihor Solodrai
2025-01-07 19:09 ` [PATCH dwarves v4 07/10] dwarf_loader: introduce cu->id Ihor Solodrai
2025-01-07 19:09 ` [PATCH dwarves v4] dwarf_loader: multithreading with a job/worker model Ihor Solodrai
2025-01-07 19:09 ` [PATCH dwarves v4 09/10] btf_encoder: clean up global encoders list Ihor Solodrai
2025-01-07 19:09 ` [PATCH dwarves v4] btf_encoder: switch func_states from a list to an array Ihor Solodrai
2025-01-07 20:35 ` [PATCH dwarves v4 00/10] pahole: faster reproducible BTF encoding Ihor Solodrai
2025-01-09 18:32 ` Arnaldo Carvalho de Melo
2025-01-09 18:38   ` Ihor Solodrai
2025-01-09 21:21     ` Arnaldo Carvalho de Melo [this message]

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=Z4A9xAHiNxoNihYe@x1 \
    --to=acme@kernel.org \
    --cc=alan.maguire@oracle.com \
    --cc=andrii@kernel.org \
    --cc=bpf@vger.kernel.org \
    --cc=dwarves@vger.kernel.org \
    --cc=eddyz87@gmail.com \
    --cc=ihor.solodrai@pm.me \
    --cc=mykolal@fb.com \
    --cc=olsajiri@gmail.com \
    /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.