RE: Working with ELF already loaded in memory

2021-07-08 Thread Sonal Santan via Elfutils-devel
Hello Mark, > -Original Message- > From: Mark Wielaard > Sent: Thursday, July 8, 2021 8:02 AM > To: Sonal Santan ; elfutils-devel@sourceware.org > Subject: Re: Working with ELF already loaded in memory > > Hi Sinal, > > On Thu, 2021-07-08 at 05:02 +, Sonal Santan via Elfutils-devel

[debuginfod/25978]

2021-07-08 Thread Noah Sanci via Elfutils-devel
Hello, Please find the patch information for pr25978 attached. Noah Sanci From af80431af96b3d1f921df264672829db71b46d20 Mon Sep 17 00:00:00 2001 From: Noah Date: Thu, 10 Jun 2021 10:29:45 -0400 Subject: [PATCH] debuginfod: PR25978 - Created the prefetch fdcache The debuginfod fdcache-prefetch l

Re: [PATCH] libelf: Fix unaligned d_off offsets for input sections with large alignments

2021-07-08 Thread Andrei Homescu
On Thu, Jul 8, 2021 at 2:37 AM Mark Wielaard wrote: > Hi Andrei, > > On Wed, 2021-07-07 at 13:24 -0700, Andrei Homescu wrote: > > I wrote this patch on behalf of someone else, so the copyright line > > should be: > > Copyright (C) 2021 Runsafe Security, Inc. > > OK, changed. > > > > The testcase

Re: PR27711

2021-07-08 Thread Mark Wielaard
Hi Noah, On Thu, 2021-07-08 at 10:48 -0400, Noah Sanci via Elfutils-devel wrote: > Find the patch information for pr27711 attached . Looks good. Pushed. Thanks, Mark

Re: PR: 25978

2021-07-08 Thread Mark Wielaard
Hi, On Thu, 2021-07-08 at 09:12 -0400, Frank Ch. Eigler wrote: > > If they are portions of the full fdcache shouldn't there be a check > > in the code that the specified fdcache_prefetch_fds and > > fdcache_prefetch_mbs aren't larger than fdcache_fds and fdcache_mbs? Or > > maybe they should be gi

Re: Working with ELF already loaded in memory

2021-07-08 Thread Mark Wielaard
Hi Sinal, On Thu, 2021-07-08 at 05:02 +, Sonal Santan via Elfutils-devel wrote: > Going through the libdw it appears that all APIs require either a > file handle or a file name of the ELF object to create a session. > Since we do not have access to the ELF file -- but rather the ELF > file con

PR27711

2021-07-08 Thread Noah Sanci via Elfutils-devel
Hello, Find the patch information for pr27711 attached . Noah Sanci From f126b48bf1131070d80e063bfd296ddb69af8c9a Mon Sep 17 00:00:00 2001 From: Noah Date: Wed, 7 Jul 2021 14:40:10 -0400 Subject: [PATCH] debuginfod: PR27711 - Use -I/-X regexes during groom phase The debuginfod -I/-X regexes ope

Re: [PATCH] PR27531: retry within default retry_limit will be supported. In debuginfod-client.c (debuginfod_query_server), insert a goto statement for jumping back to the beginning of curl handles set

2021-07-08 Thread Mark Wielaard
Hi Alice, On Tue, 2021-07-06 at 16:15 -0400, Alice Zhang via Elfutils-devel wrote: > Also introduced DEBUGINFOD_RETRY_LIMIT_ENV_VAR and default > DEBUGINFOD_RETRY_LIMIT(which is 2). > > Correponding test has been added to tests/run-debuginfod-find.sh > > Signed-off-by: Alice Zhang Nice. But tr

Re: PR: 25978

2021-07-08 Thread Frank Ch. Eigler via Elfutils-devel
Hi - > If they are portions of the full fdcache shouldn't there be a check in > the code that the specified fdcache_prefetch_fds and > fdcache_prefetch_mbs aren't larger than fdcache_fds and fdcache_mbs? Or > maybe they should be given as percentages? We've iterated on a couple of ways of represe

Re: PR: 25978

2021-07-08 Thread Mark Wielaard
Hi Noah, Like the other patch this really needs a commit message. If I understand the code correctly you are introducing another cache just for prefetches separate from the fdcache. But this new cache is a part of larger libarchive_fdcache so access to it is shared with the existing one, it is jus

Re: Buildbot failure in Wildebeest Builder on whole buildset

2021-07-08 Thread Mark Wielaard
On Thu, 2021-07-08 at 09:53 +, build...@builder.wildebeest.org wrote: > The Buildbot has detected a new failure on builder elfutils-centos- > x86_64 while building elfutils. > Full details are available at: > https://builder.wildebeest.org/buildbot/#builders/1/builds/784 > > Buildbot URL:

Buildbot failure in Wildebeest Builder on whole buildset

2021-07-08 Thread buildbot
The Buildbot has detected a new failure on builder elfutils-centos-x86_64 while building elfutils. Full details are available at: https://builder.wildebeest.org/buildbot/#builders/1/builds/784 Buildbot URL: https://builder.wildebeest.org/buildbot/ Worker for this Build: centos-x86_64 Build

Re: [PATCH] libelf: Fix unaligned d_off offsets for input sections with large alignments

2021-07-08 Thread Mark Wielaard
Hi Andrei, On Wed, 2021-07-07 at 13:24 -0700, Andrei Homescu wrote: > I wrote this patch on behalf of someone else, so the copyright line > should be: > Copyright (C) 2021 Runsafe Security, Inc. OK, changed. > > The testcase already fails before the patch and succeeds after, but > > it would be