Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Carlos O'Donell <carlos@redhat.com>
To: Simon Marchi <simark@simark.ca>,
	Yury Khrustalev <yury.khrustalev@arm.com>,
	Muhammad Kamran <muhammad.kamran@arm.com>
Cc: gdb-patches@sourceware.org,
	Wilco Dijkstra <Wilco.Dijkstra@arm.com>,
	Thiago Jung Bauermann <thiago.bauermann@linaro.org>,
	Adhemerval Zanella Netto <adhemerval.zanella@linaro.org>
Subject: Re: [PATCH 0/1] gdb: Preserve IFUNC marker when finding inferior functions
Date: Thu, 25 Jun 2026 19:08:57 -0400	[thread overview]
Message-ID: <050c1813-98cd-4fdf-a629-11ffa9ce9858@redhat.com> (raw)
In-Reply-To: <9f0013bd-f19a-44be-a90d-788c078bfba6@simark.ca>

On 6/25/26 4:16 PM, Simon Marchi wrote:
> 
> 
> On 2026-06-24 08:56, Yury Khrustalev wrote:
>> On Wed, Jun 24, 2026 at 09:50:01AM +0000, Muhammad Kamran wrote:
>>> This patch fixes a GDB inferior-call issue exposed by malloc being a GNU
>>> IFUNC in glibc on AArch64.
>>>
>>> GDB calls find_function_in_inferior ("malloc") when expression evaluation
>>> needs to allocate memory in the inferior, for example for string literal
>>> arguments.  In the minimal-symbol fallback, GDB created a synthetic ordinary
>>> function pointer from the minimal symbol address.  If the symbol was a GNU
>>> IFUNC, this lost the IFUNC marker, so call_function_by_hand did not
>>> resolve the symbol before calling it.
>>>
>>> The patch uses find_minsym_type_and_address to classify the minimal symbol
>>> and propagates the IFUNC marker to the synthetic function type when needed.
>>> The existing fallback return type is unchanged.
>>
>> Thanks! I confirm that all GDB testsuites that showed regressions now
>> pass with Glibc from master and this patch applied on top of GDB master.
>>
>>>
>>> Should this be considered for backporting to release branches?
>>
>> Yes please, at least GDB 16.x and 17.x I think should be covered.
> 
> The bugfix release of GDB 17 (17.2) has already been released, we have
> historically not done more than one bugfix release of a given release
> branch (just a handful of special cases where we noticed the release was
> completely broken, just after releasing it).  We usually just move on to
> working on the next release (18, in this case).
> 
> As the new co-release manager (along with Andrew), I would be open to
> discuss changing this, to allow having as many bugfix releases on a
> stable branch as needed, at least until the following major version is
> available.  But I have not done a release myself yet, so I'd like to
> wait to see how it's actually like before deciding.

Has gdb ever considered a rolling release branch model like glibc?

In glibc we acknowledged that the major consumers could and would like
a release branch that is basically rolling with fixes, while we do cut
an official X.Y release, after that the release branch rolls forward
with each commit valid and containing an additional fix.

> In the mean time, it's always possible to push a fix to the
> gdb-17-branch (and even the gdb-16-branch) without it being part of a
> release.  Downstream packagers can then take this commit and carry it as
> a local patch.
I like this idea, what I was trying to express above was that the project
could also pinky-swear not to leave the branch in a broken state such
that it is considered a rolling working release until the next point
release.

In glibc we did away with point releases, they were too much work and
downstream was just as happy to "git pull --rebase" from the release
branch.

Granted the release branch should have some rules, like no backporting
of things that break ABI, API, or exiting commands etc. etc.

-- 
Cheers,
Carlos.


  reply	other threads:[~2026-06-25 23:09 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-24  9:50 Muhammad Kamran
2026-06-24  9:50 ` [PATCH 1/1] " Muhammad Kamran
2026-06-24 15:53   ` Andrew Burgess
2026-06-25 15:37     ` Muhammad Kamran
2026-06-24 12:18 ` [PATCH 0/1] " Carlos O'Donell
2026-06-24 12:56 ` Yury Khrustalev
2026-06-25 20:16   ` Simon Marchi
2026-06-25 23:08     ` Carlos O'Donell [this message]
2026-06-25 23:52       ` Simon Marchi
2026-07-16 21:00         ` Carlos O'Donell

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=050c1813-98cd-4fdf-a629-11ffa9ce9858@redhat.com \
    --to=carlos@redhat.com \
    --cc=Wilco.Dijkstra@arm.com \
    --cc=adhemerval.zanella@linaro.org \
    --cc=gdb-patches@sourceware.org \
    --cc=muhammad.kamran@arm.com \
    --cc=simark@simark.ca \
    --cc=thiago.bauermann@linaro.org \
    --cc=yury.khrustalev@arm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox