* [PATCH v1] gdb/aarch64: fix typo gcs_availabe
@ 2026-06-05 16:48 Matthieu Longo
2026-06-07 2:29 ` Thiago Jung Bauermann
0 siblings, 1 reply; 3+ messages in thread
From: Matthieu Longo @ 2026-06-05 16:48 UTC (permalink / raw)
To: gdb-patches; +Cc: Alan Hayward, Luis Machado, Luis Machado, Matthieu Longo
---
gdb/aarch64-linux-tdep.c | 6 +++---
gdb/arch/aarch64.h | 1 +
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/gdb/aarch64-linux-tdep.c b/gdb/aarch64-linux-tdep.c
index ae21b029276..f11eccc1bc1 100644
--- a/gdb/aarch64-linux-tdep.c
+++ b/gdb/aarch64-linux-tdep.c
@@ -261,7 +261,7 @@ struct aarch64_linux_sigframe
bool zt_available = false;
/* True if we have a GCS entry in the signal context, false otherwise. */
- bool gcs_availabe = false;
+ bool gcs_available = false;
/* The Guarded Control Stack Pointer Register. */
uint64_t gcspr;
/* Flags indicating which GCS features are enabled for the thread. */
@@ -584,7 +584,7 @@ aarch64_linux_read_signal_frame_info (const frame_info_ptr &this_frame,
signal_frame.gcs_features_enabled
= extract_unsigned_integer (buf, byte_order);
- signal_frame.gcs_availabe = true;
+ signal_frame.gcs_available = true;
section += size;
break;
}
@@ -787,7 +787,7 @@ aarch64_linux_sigframe_init (const struct tramp_frame *self,
/* Restore the GCS registers, if the target supports it and if there is
an entry for them. */
- if (signal_frame.gcs_availabe && tdep->has_gcs_linux ())
+ if (signal_frame.gcs_available && tdep->has_gcs_linux ())
{
/* Restore GCSPR. */
trad_frame_set_reg_value (this_cache, tdep->gcs_reg_base,
diff --git a/gdb/arch/aarch64.h b/gdb/arch/aarch64.h
index c9a89ac4098..cf7318cbe74 100644
--- a/gdb/arch/aarch64.h
+++ b/gdb/arch/aarch64.h
@@ -97,6 +97,7 @@ namespace std
/* SME2 feature. */
h = h << 1 | features.sme2;
+ /* GCS feature. */
h = h << 1 | features.gcs;
h = h << 1 | features.gcs_linux;
--
2.54.0
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v1] gdb/aarch64: fix typo gcs_availabe
2026-06-05 16:48 [PATCH v1] gdb/aarch64: fix typo gcs_availabe Matthieu Longo
@ 2026-06-07 2:29 ` Thiago Jung Bauermann
2026-06-08 13:52 ` Tom de Vries
0 siblings, 1 reply; 3+ messages in thread
From: Thiago Jung Bauermann @ 2026-06-07 2:29 UTC (permalink / raw)
To: Matthieu Longo; +Cc: gdb-patches, Alan Hayward, Luis Machado
Matthieu Longo <matthieu.longo@arm.com> writes:
> ---
> gdb/aarch64-linux-tdep.c | 6 +++---
> gdb/arch/aarch64.h | 1 +
> 2 files changed, 4 insertions(+), 3 deletions(-)
Thank you! This is ok.
Approved-by: Thiago Jung Bauermann <thiago.bauermann@linaro.org>
This doesn't even need an approval, actually. It falls under the obvious
fix rule mentioned in gdb/MAINTAINERS.
--
Thiago
(he/him)
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v1] gdb/aarch64: fix typo gcs_availabe
2026-06-07 2:29 ` Thiago Jung Bauermann
@ 2026-06-08 13:52 ` Tom de Vries
0 siblings, 0 replies; 3+ messages in thread
From: Tom de Vries @ 2026-06-08 13:52 UTC (permalink / raw)
To: Thiago Jung Bauermann, Matthieu Longo
Cc: gdb-patches, Alan Hayward, Luis Machado
On 6/7/26 4:29 AM, Thiago Jung Bauermann wrote:
> Matthieu Longo <matthieu.longo@arm.com> writes:
>
>> ---
>> gdb/aarch64-linux-tdep.c | 6 +++---
>> gdb/arch/aarch64.h | 1 +
>> 2 files changed, 4 insertions(+), 3 deletions(-)
>
> Thank you! This is ok.
>
FWIW, I got curious about whether codespell would detect this typo, and
posted my findings in a PR (
https://sourceware.org/bugzilla/show_bug.cgi?id=34241 ).
Thanks,
- Tom
> Approved-by: Thiago Jung Bauermann <thiago.bauermann@linaro.org>
>
> This doesn't even need an approval, actually. It falls under the obvious
> fix rule mentioned in gdb/MAINTAINERS.
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-06-08 13:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-06-05 16:48 [PATCH v1] gdb/aarch64: fix typo gcs_availabe Matthieu Longo
2026-06-07 2:29 ` Thiago Jung Bauermann
2026-06-08 13:52 ` Tom de Vries
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox