Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [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