Re: run-debuginfod-webapi-concurrency.sh
Hi, On Sat, Apr 23, 2022 at 10:56:01PM -0400, Thomas Fitzsimmons wrote: > "Frank Ch. Eigler" writes: > >> But there is another way to prevent the "Server reached connection > >> limit. Closing inbound connection." Pass the MHD_USE_ITC flag to > >> MHD_start_daemon: > > > > Yeah, that looked promising to me too. When I was last working on > > this, that would have been my next thing to try. I can't think of > > a relevant downside, so let's try it. (Add a #ifdef guard around > > that macro, for older libmicrohttpd, like rhel7 methinks.) > > On debian-ppc64, with and without the MHD_USE_ITC patch, I ran the test > 20 times in a shell loop. With MHD_USE_ITC, I got 20 passes, without > it, 9 passes and 11 failures. > > With the patch applied, a full "make check" succeeded. Nice. Thanks for the feedback and testing. I pushed the attached. Cheers, Mark >From 3bcf887340fd47d0d8a3671cc45abe2989d1fd6c Mon Sep 17 00:00:00 2001 From: Mark Wielaard Date: Sun, 24 Apr 2022 12:16:58 +0200 Subject: [PATCH] debuginfod: Use MHD_USE_ITC in MHD_start_daemon flags MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This prevents the "Server reached connection limit. Closing inbound connection." issue we have been seeing in the run-debuginfod-webapi-concurrency.sh testcase. From the manual: If the connection limit is reached, MHD’s behavior depends a bit on other options. If MHD_USE_ITC was given, MHD will stop accepting connections on the listen socket. This will cause the operating system to queue connections (up to the listen() limit) above the connection limit. Those connections will be held until MHD is done processing at least one of the active connections. If MHD_USE_ITC is not set, then MHD will continue to accept() and immediately close() these connections. https://sourceware.org/bugzilla/show_bug.cgi?id=28708 Signed-off-by: Mark Wielaard --- debuginfod/ChangeLog | 4 debuginfod/debuginfod.cxx | 3 +++ tests/ChangeLog| 4 tests/run-debuginfod-webapi-concurrency.sh | 4 +++- 4 files changed, 14 insertions(+), 1 deletion(-) diff --git a/debuginfod/ChangeLog b/debuginfod/ChangeLog index d6f7b282..0f1bca6f 100644 --- a/debuginfod/ChangeLog +++ b/debuginfod/ChangeLog @@ -1,3 +1,7 @@ +2022-04-24 Mark Wielaard + + * debuginfod.cxx (main): Add MHD_USE_ITC to MHD_start_daemon flags. + 2022-04-13 Aaron Merey * debuginfod-client.c (debuginfod_query_server): diff --git a/debuginfod/debuginfod.cxx b/debuginfod/debuginfod.cxx index 9c0217f6..adca8208 100644 --- a/debuginfod/debuginfod.cxx +++ b/debuginfod/debuginfod.cxx @@ -3910,6 +3910,9 @@ main (int argc, char *argv[]) | MHD_USE_EPOLL #endif | MHD_USE_DUAL_STACK +#if MHD_VERSION >= 0x00095200 + | MHD_USE_ITC +#endif | MHD_USE_DEBUG, /* report errors to stderr */ http_port, NULL, NULL, /* default accept policy */ diff --git a/tests/ChangeLog b/tests/ChangeLog index 2286f53f..44b8df88 100644 --- a/tests/ChangeLog +++ b/tests/ChangeLog @@ -1,3 +1,7 @@ +2022-04-24 Mark Wielaard + + * run-debuginfod-webapi-concurrency.sh: Fix PR number in xfail. + 2022-04-23 Mark Wielaard * run-debuginfod-webapi-concurrency.sh: Lower parallel lookups. diff --git a/tests/run-debuginfod-webapi-concurrency.sh b/tests/run-debuginfod-webapi-concurrency.sh index 4928f6d0..47dcadcc 100755 --- a/tests/run-debuginfod-webapi-concurrency.sh +++ b/tests/run-debuginfod-webapi-concurrency.sh @@ -62,6 +62,8 @@ do PID1=0 done -xfail "grep Server.reached.connection vlog$PORT1" # PR18661 +# Note this xfail comes too late, the above wait_ready for +# http_responses_transfer_bytes_count will have failed. +xfail "grep Server.reached.connection vlog$PORT1" # PR28661 exit 0 -- 2.30.2
[Bug debuginfod/28708] run-debuginfod-webapi-concurrency.sh seems to be flaky
https://sourceware.org/bugzilla/show_bug.cgi?id=28708 Mark Wielaard changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED|RESOLVED --- Comment #14 from Mark Wielaard --- commit 3bcf887340fd47d0d8a3671cc45abe2989d1fd6c Author: Mark Wielaard Date: Sun Apr 24 12:16:58 2022 +0200 debuginfod: Use MHD_USE_ITC in MHD_start_daemon flags This prevents the "Server reached connection limit. Closing inbound connection." issue we have been seeing in the run-debuginfod-webapi-concurrency.sh testcase. From the manual: If the connection limit is reached, MHD’s behavior depends a bit on other options. If MHD_USE_ITC was given, MHD will stop accepting connections on the listen socket. This will cause the operating system to queue connections (up to the listen() limit) above the connection limit. Those connections will be held until MHD is done processing at least one of the active connections. If MHD_USE_ITC is not set, then MHD will continue to accept() and immediately close() these connections. https://sourceware.org/bugzilla/show_bug.cgi?id=28708 Signed-off-by: Mark Wielaard -- You are receiving this mail because: You are on the CC list for the bug.
☺ Buildbot (GNU Toolchain): elfutils - build successful (master)
A restored build has been detected on builder elfutils-debian-ppc64 while building elfutils. Full details are available at: https://builder.sourceware.org/buildbot/#builders/63/builds/8 Build state: build successful Revision: 3bcf887340fd47d0d8a3671cc45abe2989d1fd6c Worker: debian-ppc64 Build Reason: (unknown) Blamelist: Mark Wielaard , наб via Elfutils-devel Steps: - 0: worker_preparation ( success ) - 1: set package name ( success ) - 2: git checkout ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/2/logs/stdio - 3: autoreconf ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/3/logs/stdio - 4: configure ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/4/logs/stdio - 5: get version ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/5/logs/stdio - property changes: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/5/logs/property_changes - 6: make ( warnings ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/6/logs/stdio - warnings (3): https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/6/logs/warnings__3_ - 7: make check ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/7/logs/stdio - test-suite.log: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/7/logs/test-suite_log - 8: prep ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/8/logs/stdio - 9: fetch ['tests/*.trs', 'tests/*.log'] ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/9/logs/stdio - 10: fetch ['elfutils-*/_build/sub/tests/*.trs', 'elfut ( skipped ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/10/logs/stdio - 11: fetch ['config.log'] ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/11/logs/stdio - 12: fetch ['elfutils-*/_build/sub/config.log'] ( skipped ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/12/logs/stdio - 13: pass .bunsen.source.gitname ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/13/logs/stdio - 14: pass .bunsen.source.gitrepo ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/14/logs/stdio - 15: upload to bunsen ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/15/logs/stdio - 16: clean up ( success ) Logs: - stdio: https://builder.sourceware.org/buildbot/#builders/63/builds/8/steps/16/logs/stdio
Re: [PATCH] debuginfod: Use the debuginfod-size response header
Hi Aaron, On Fri, Apr 22, 2022 at 06:56:41PM -0400, Aaron Merey via Elfutils-devel wrote: > I've updated the patch below with the changes Mark recommended. > > A couple X-DEBUGINFOD-SIZE tests were added in another patch I recently > posted [1] that also fixes a bug when computing this header's value for > an archived file. > > Aaron > > [1] https://sourceware.org/pipermail/elfutils-devel/2022q2/004936.html > > From b56f1568b832fe1c23ffb711aa0486fbd2c5067f Mon Sep 17 00:00:00 2001 > From: Aaron Merey > Date: Tue, 11 Jan 2022 22:07:55 -0500 > > debuginfod: Use the debuginfod-size response header > > In some cases the content-length header may not be available in order > to pass to a progressfn. If content-length isn't available then attempt > to get the size of the download from the debuginfod-size header instead. > > It should be mentioned that if a compressed file (ex. gzip) is being > transferred, the actual transfer length will be less than debuginfod-size. > In this case debuginfod-size is a best-guess upper bound on the size of > the transfer. Looks good. Pleas commit. Cheers, Mark
[Bug general/28891] Cross compilation for ARM fails when trying to run i386_gendis
https://sourceware.org/bugzilla/show_bug.cgi?id=28891 Mark Wielaard changed: What|Removed |Added CC||mark at klomp dot org --- Comment #1 from Mark Wielaard --- Note that this does work when using release tar balls. Then i386_gendis isn't needed. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug tools/28724] eu-elfclassify --no-stdin option is not effect
https://sourceware.org/bugzilla/show_bug.cgi?id=28724
Mark Wielaard changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|--- |FIXED
CC||mark at klomp dot org
--- Comment #1 from Mark Wielaard ---
Thanks, that was obviously a typo in the source. Nobody noticed before because
--no-stdin is the default. But it did indeed not work when given explicitly.
Fixed with:
commit 6398e94e9ac9a170da088768198d4a2b6e989e19 (HEAD -> master, origin/master,
origin/HEAD)
Author: Mark Wielaard
Date: Sun Apr 24 17:48:39 2022 +0200
elfclassify: Fix --no-stdin flag
The no-stdin option was using the wrong flag, classify_flag_stdin,
instead of classify_flag_no_stdin.
https://sourceware.org/bugzilla/show_bug.cgi?id=28724
Signed-off-by: Mark Wielaard
diff --git a/src/ChangeLog b/src/ChangeLog
index 6ef81862..fd87ce2f 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,7 @@
+2022-04-24 Mark Wielaard
+
+ * elfclassify.c (main): Use classify_flag_no_stdin for no-std in
options.
+
2022-04-19 Mark Wielaard
* readelf.c (get_dyn_ents): Define dyn_mem outside the while loop.
diff --git a/src/elfclassify.c b/src/elfclassify.c
index 2f70b29a..25fe9a65 100644
--- a/src/elfclassify.c
+++ b/src/elfclassify.c
@@ -946,7 +946,7 @@ separated by newlines"), 2 },
{ "stdin0", classify_flag_stdin0, NULL, 0,
N_("Also read file names to process from standard input, \
separated by ASCII NUL bytes"), 2 },
- { "no-stdin", classify_flag_stdin, NULL, 0,
+ { "no-stdin", classify_flag_no_stdin, NULL, 0,
N_("Do not read files from standard input (default)"), 2 },
{ "compressed", 'z', NULL, 0,
N_("Try to open compressed files or embedded (kernel) ELF images"),
--
You are receiving this mail because:
You are on the CC list for the bug.
[Bug debuginfod/28194] confirm cross-building works
https://sourceware.org/bugzilla/show_bug.cgi?id=28194 Mark Wielaard changed: What|Removed |Added CC||mark at klomp dot org Resolution|--- |FIXED Status|NEW |RESOLVED --- Comment #2 from Mark Wielaard --- I can confirm that on x86_64 fedora, with all i386 devel packages installed the following configure/build works and passes all tests: CC="gcc -m32" CXX="g++ -m32" ./configure --host=i686-pc-linux-gnu --enable-debuginfod && make -j8 && make check -j8 Testsuite summary for elfutils 0.186 # TOTAL: 255 # PASS: 249 # SKIP: 6 # XFAIL: 0 # FAIL: 0 # XPASS: 0 # ERROR: 0 -- You are receiving this mail because: You are on the CC list for the bug.
[Bug tools/28723] "eu-size -A" does not display .comment and .note.GNU-stack
https://sourceware.org/bugzilla/show_bug.cgi?id=28723 Mark Wielaard changed: What|Removed |Added CC||mark at klomp dot org --- Comment #1 from Mark Wielaard --- (In reply to panxiaohe from comment #0) > The result of "eu-size -A" does not display section .comment and > .note.GNU-stack > which "size -A" can display. > Please confirm the attachment. > Would that be correct？ eu-size only lists sections which are allocated (have SHF_ALLOC, which is the A flag in eu-readelf -s). This is because eu-size thinks the user is only interested in the "runtime size" of the file (those text/data sections that are loaded into memory). I don't know which sections binutils size lists. -- You are receiving this mail because: You are on the CC list for the bug.
[COMMITTED] libelf: Check for mremap, elf_update needs it for ELF_C_RDWR_MMAP
Add a AC_CHECK_FUNCS configure check for mremap. Some systems like
KFreeBSD and the Hurd don't have it. Also add a configure warning
because without mremap elf_update will often fail when ELF_C_RDWR_MMAP
is used. ELF_C_RDWR_MMAP is an elfutils extension to libelf.
https://sourceware.org/bugzilla/show_bug.cgi?id=27337
Signed-off-by: Mark Wielaard
---
ChangeLog | 4
configure.ac| 5 -
libelf/ChangeLog| 4
libelf/elf_update.c | 2 ++
4 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
index 3357f69b..f0cd28a8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2022-04-24 Mark Wielaard
+
+ * configure.ac (AC_CHECK_FUNCS): Add mremap.
+
2021-03-20 Mark Wielaard
* configure.ac: Remove -m64 on 32bit target comments for
diff --git a/configure.ac b/configure.ac
index 2418d474..33c4b5e5 100644
--- a/configure.ac
+++ b/configure.ac
@@ -435,7 +435,10 @@ AC_CHECK_DECLS([reallocarray],[],[],
[#define _GNU_SOURCE
#include ])
-AC_CHECK_FUNCS([process_vm_readv])
+AC_CHECK_FUNCS([process_vm_readv mremap])
+
+AS_IF([test "x$ac_cv_func_mremap" = "xno"],
+ [AC_MSG_WARN([elf_update needs mremap to support ELF_C_RDWR_MMAP])])
AC_CHECK_HEADERS([error.h])
AC_CHECK_HEADERS([err.h])
diff --git a/libelf/ChangeLog b/libelf/ChangeLog
index 985f795d..00d4ac0f 100644
--- a/libelf/ChangeLog
+++ b/libelf/ChangeLog
@@ -1,3 +1,7 @@
+2022-04-24 Mark Wielaard
+
+ * elf_update.c (write_file): Check HAVE_MREMAP.
+
2022-04-01 Mark Wielaard
* libelfP.h (struct Elf_Data_Chunk): Add an int64_t offset field.
diff --git a/libelf/elf_update.c b/libelf/elf_update.c
index 9b8867ce..97ca9ca9 100644
--- a/libelf/elf_update.c
+++ b/libelf/elf_update.c
@@ -106,8 +106,10 @@ write_file (Elf *elf, int64_t size, int change_bo, size_t
shnum)
if (elf->cmd == ELF_C_RDWR_MMAP
&& (size_t) size > elf->maximum_size)
{
+#ifdef HAVE_MREMAP
if (mremap (elf->map_address, elf->maximum_size,
size, 0) == MAP_FAILED)
+#endif
{
__libelf_seterrno (ELF_E_WRITE_ERROR);
return -1;
--
2.30.2
[Bug general/27337] elfutils ftbfs on KFreeBSD and the Hurd
https://sourceware.org/bugzilla/show_bug.cgi?id=27337 Mark Wielaard changed: What|Removed |Added Status|NEW |RESOLVED CC||mark at klomp dot org Resolution|--- |FIXED --- Comment #1 from Mark Wielaard --- commit 3cf386984c1870c2b39ec3fb47b0deb7b30535c3 Author: Mark Wielaard Date: Sun Apr 24 19:42:29 2022 +0200 libelf: Check for mremap, elf_update needs it for ELF_C_RDWR_MMAP Add a AC_CHECK_FUNCS configure check for mremap. Some systems like KFreeBSD and the Hurd don't have it. Also add a configure warning because without mremap elf_update will often fail when ELF_C_RDWR_MMAP is used. ELF_C_RDWR_MMAP is an elfutils extension to libelf. https://sourceware.org/bugzilla/show_bug.cgi?id=27337 Signed-off-by: Mark Wielaard Note that various tests will FAIL without mremap: FAIL: run-bug1-test.sh FAIL: elfstrtab FAIL: emptyfile FAIL: run-copyadd-sections.sh elf_update: cannot write data to file -- You are receiving this mail because: You are on the CC list for the bug.
[Bug general/24498] 0.179: isn't LTO ready
https://sourceware.org/bugzilla/show_bug.cgi?id=24498 Mark Wielaard changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --- Comment #24 from Mark Wielaard --- (In reply to kloczek from comment #23) > Just retested elfutils with more patches from git and looks like now test > suite is passing successfully: Thanks again for testing. I am closing this bug now. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug debuginfod/28240] debuginfod client cache falsely sticky for root user
https://sourceware.org/bugzilla/show_bug.cgi?id=28240 --- Comment #3 from Mark Wielaard --- We got rid of the zero-permission 000 files with: commit 8b568fdea8e1baea3d68cc38dec587e4c9219276 Author: Aaron Merey Date: Fri Apr 8 19:37:11 2022 -0400 PR29022: 000-permissions files cause problems for backups 000-permission files currently used for negative caching can cause permission problems for some backup software and disk usage checkers. Fix this by using empty files for negative caching instead. Also use each empty file's mtime to determine the time since last download attempt instead of the cache_miss_s file's mtime. https://sourceware.org/bugzilla/show_bug.cgi?id=29022 Tested-by: Milian Wolff Signed-off-by: Aaron Merey And I think that also got rid of this race issue. Any other client that races past us now will either create a new empty file with open (target_cache_path, O_CREAT|O_EXCL, DEFFILEMODE) or puts in a new non-empty file using rename (target_cache_tmppath, target_cache_path). Both of which are atomic. So I think this is resolved now. But would like someone else to double check. These races are tricky. -- You are receiving this mail because: You are on the CC list for the bug.
[Bug debuginfod/28583] FAIL run-debuginfod-response-headers.sh (exit status: 1) with 0.186
https://sourceware.org/bugzilla/show_bug.cgi?id=28583 --- Comment #3 from Mark Wielaard --- (In reply to Martin Liska from comment #2) > Hmm, I restarted the build and now it succeeded. Is this still an issue? -- You are receiving this mail because: You are on the CC list for the bug.
RFC patch: generated AUTHORS
Hi -
The following patch adds a tool AUTHORS.sh to compute a complete
AUTHORS list to cover the entire git history. We do this in systemtap
and it's a nice way of showing off the large list of contributors over
time.
commit 72823322be9a8f0143de21ccfb67a89696a9522f (HEAD -> master2)
Author: Frank Ch. Eigler
Date: Sun Apr 24 19:39:47 2022 -0400
AUTHORS: Use generator script & git mailmap
We can compute a pretty complete list of contributors.
Signed-off-by: Frank Ch. Eigler
diff --git a/.mailmap b/.mailmap
new file mode 100644
index ..fd42169b8a3f
--- /dev/null
+++ b/.mailmap
@@ -0,0 +1,25 @@
+# help out git with erroneous or incomplete email address mappings
+
+Piotr Drąg
+Domingo Becker
+Ulrich Drepper
+Ulrich Drepper
+Hyu_gabaru Ryu_ichi
+kiyoto hashida
+Claudio Rodrigo Pereyra Diaz
+Gladys Guerrero
+Wei Liu
+Michael Münch
+Noah Sanci
+Noriko Mizumoto
+Cornelius Neckenig
+Francesco Tombolini
+Thomas Spura
+Geert Warrink
+Yulia Poyarkova
+Yuri Chornoivan
+Ahelenia Ziemiańska
+Daniel Cabrera
+Thomas Canniot
+Ahelenia Ziemiańska наб via Elfutils-devel
+
diff --git a/AUTHORS b/AUTHORS
index ef3c5430a963..45cb1f8d8e9f 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -1,4 +1,104 @@
-For Now:
-Ulrich Drepper.
-Roland McGrath
+Aaron Merey
+Adam Markey
+Adrian Ratiu
+Ahelenia Ziemiańska
+Akihiko Odaki
+Alexander Cherepanov
+Alexander Kanavin
+Alexander Miller
+Alice Zhang
+Andreas Krebbel
+Andreas Schwab
+Andrei Homescu
+Anthony G. Basile
+Ben Woodard
+Chih-Hung Hsieh
+Claudio Rodrigo Pereyra Diaz
+Colin Cross
+Cornelius Neckenig
+Daniel Cabrera
+David Abdurachmanov
+Di Chen
+Dima Kogan
+Dimitris Glezos
+Dmitry V. Levin
+Dodji Seketeli
+Domingo Becker
+Eduardo Santiago
+Eli Schwartz
+Érico Nogueira
+Érico Rolim
+Filipe Brandenburger
+Florian Weimer
+Francesco Tombolini
+Frank Ch. Eigler
+Geert Warrink
+Gladys Guerrero
+Gustavo Romero
+Hayatsu Shunsuke
+H.J. Lu
+Hyu_gabaru Ryu_ichi
+Jakub Jelinek
+Jan Kratochvil
+Jan Pokorný
+Jason P. Leasure
+Jean Pihet
+Jeff Kenton
+Jim Wilson
+John M Mellor-Crummey
+John Ogness
+Jonathan Lebon
+Jonathon Anderson
+Jose E. Marchesi
+Josh Stone
+Joshua Watt
+Kevin Cernekee
+kiyoto hashida
+Konrad Kleine
+Kurt Roeckx
+Kyle McMartin
+Lei Zhang
+Lubomir Rintel
+Luca Boccassi
+Luiz Angelo Daros de Luca
+Mao Han
+Marek Polacek
+Mark Wielaard
+Martin Liska
+Masatake YAMATO
+Matt Fleming
+Matthias Klose
+Matthias Maennich
+Max Filippov
+Michael Forney
+Michael Münch
+Mike Frysinger
+Milian Wolff
+Namhyung Kim
+Noah Sanci
+Noriko Mizumoto
+Omar Sandoval
Petr Machata
+Pino Toscano
+Piotr Drąg
+Ravi Bangoria
+Richard Henderson
+Roland McGrath
+Rosen Penev
+Ross Burton
+Saleem Abdulrasool
+Sergei Trofimovich
+Srđan Milaković
+Steven Chamberlain
+Thomas Canniot
+Thomas Spura
+Timm Bäder
+Tom Tromey
+Ulf Hermann
+Ulrich Drepper
+Wei Liu
+William Cohen
+Yonghong Song
+Yulia Poyarkova
+Yunlian Jiang
+Yuri Chornoivan
diff --git a/AUTHORS.sh b/AUTHORS.sh
new file mode 100644
index ..ff048a6489e0
--- /dev/null
+++ b/AUTHORS.sh
@@ -0,0 +1,12 @@
+#! /bin/sh
+
+# Create the AUTHORS file, by searching the git history.
+
+# Run as "AUTHORS.sh" to get complete history
+# Run with "AUTHORS.sh commitish..commitish" for history between tags
+
+# shortlog will canonicalize the names using the file .mailmap
+git shortlog -s ${1-} |
+sed -e 's, via Elfutils-devel,,' |
+cut -b8- | # strip the commit counts
+sort | uniq
diff --git a/ChangeLog b/ChangeLog
index f0cd28a8ddbe..5ad93a16eeba 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2022-04-24 Frank Ch. Eigler
+
+ * AUTHORS.sh, .mailmap: New files.
+ * AUTHORS: Regenerated to cover entire history of elfutils.
+
2022-04-24 Mark Wielaard
* configure.ac (AC_CHECK_FUNCS): Add mremap.
