Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH] sim/erc32: Rename EVENT_MAX -> MAX_EVENTS
@ 2024-03-03 10:22 Orgad Shaneh
  2024-03-08 17:23 ` Tom Tromey
  0 siblings, 1 reply; 4+ messages in thread
From: Orgad Shaneh @ 2024-03-03 10:22 UTC (permalink / raw)
  To: gdb-patches; +Cc: Orgad Shaneh

From: Orgad Shaneh <orgads@gmail.com>

EVENT_MAX is defined as 0x7FFFFFFF (INT_MAX) in winuser.h, so when
building on Windows, the value is overridden and compilation fails
because the array size of evbuf is too large.

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=28476
---
 sim/erc32/func.c | 6 +++---
 sim/erc32/sis.h  | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/sim/erc32/func.c b/sim/erc32/func.c
index 972a636e619..80f416d87d7 100644
--- a/sim/erc32/func.c
+++ b/sim/erc32/func.c
@@ -697,7 +697,7 @@ init_signals(void)
 extern struct disassemble_info dinfo;
 
 struct estate   ebase;
-struct evcell   evbuf[EVENT_MAX];
+struct evcell   evbuf[MAX_EVENTS];
 struct irqcell  irqarr[16];
 
 static int
@@ -864,10 +864,10 @@ init_event(void)
 
     ebase.eq.nxt = NULL;
     ebase.freeq = evbuf;
-    for (i = 0; i < EVENT_MAX; i++) {
+    for (i = 0; i < MAX_EVENTS; i++) {
 	evbuf[i].nxt = &evbuf[i + 1];
     }
-    evbuf[EVENT_MAX - 1].nxt = NULL;
+    evbuf[MAX_EVENTS - 1].nxt = NULL;
 }
 
 void
diff --git a/sim/erc32/sis.h b/sim/erc32/sis.h
index 504d7e69c77..cf44ddfb7fe 100644
--- a/sim/erc32/sis.h
+++ b/sim/erc32/sis.h
@@ -31,7 +31,7 @@
 #define I_ACC_EXC 1
 
 /* Maximum events in event queue */
-#define EVENT_MAX	256
+#define MAX_EVENTS	256
 
 /* Maximum # of floating point queue */
 #define FPUQN	1
-- 
2.40.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] sim/erc32: Rename EVENT_MAX -> MAX_EVENTS
  2024-03-03 10:22 [PATCH] sim/erc32: Rename EVENT_MAX -> MAX_EVENTS Orgad Shaneh
@ 2024-03-08 17:23 ` Tom Tromey
  2024-03-21  6:37   ` Orgad Shaneh
  0 siblings, 1 reply; 4+ messages in thread
From: Tom Tromey @ 2024-03-08 17:23 UTC (permalink / raw)
  To: Orgad Shaneh; +Cc: gdb-patches

>>>>> "Orgad" == Orgad Shaneh <orgads@gmail.com> writes:

Orgad> From: Orgad Shaneh <orgads@gmail.com>
Orgad> EVENT_MAX is defined as 0x7FFFFFFF (INT_MAX) in winuser.h, so when
Orgad> building on Windows, the value is overridden and compilation fails
Orgad> because the array size of evbuf is too large.

Orgad> Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=28476

Thanks.  I think this is ok.
Approved-By: Tom Tromey <tom@tromey.com>

Tom

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] sim/erc32: Rename EVENT_MAX -> MAX_EVENTS
  2024-03-08 17:23 ` Tom Tromey
@ 2024-03-21  6:37   ` Orgad Shaneh
  2024-03-21 16:49     ` Tom Tromey
  0 siblings, 1 reply; 4+ messages in thread
From: Orgad Shaneh @ 2024-03-21  6:37 UTC (permalink / raw)
  To: Tom Tromey; +Cc: gdb-patches

On Fri, Mar 8, 2024 at 7:23 PM Tom Tromey <tom@tromey.com> wrote:
>
> >>>>> "Orgad" == Orgad Shaneh <orgads@gmail.com> writes:
>
> Orgad> From: Orgad Shaneh <orgads@gmail.com>
> Orgad> EVENT_MAX is defined as 0x7FFFFFFF (INT_MAX) in winuser.h, so when
> Orgad> building on Windows, the value is overridden and compilation fails
> Orgad> because the array size of evbuf is too large.
>
> Orgad> Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=28476
>
> Thanks.  I think this is ok.
> Approved-By: Tom Tromey <tom@tromey.com>

Thank you. What's the next step for getting this merged?

- Orgad

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] sim/erc32: Rename EVENT_MAX -> MAX_EVENTS
  2024-03-21  6:37   ` Orgad Shaneh
@ 2024-03-21 16:49     ` Tom Tromey
  0 siblings, 0 replies; 4+ messages in thread
From: Tom Tromey @ 2024-03-21 16:49 UTC (permalink / raw)
  To: Orgad Shaneh; +Cc: Tom Tromey, gdb-patches

>>>>> "Orgad" == Orgad Shaneh <orgads@gmail.com> writes:

>> Thanks.  I think this is ok.
>> Approved-By: Tom Tromey <tom@tromey.com>

Orgad> Thank you. What's the next step for getting this merged?

I've pushed it.

Do you have a copyright assignment in place?  I think this patch is
small/obvious enough without one, but we'll need one at some point.
With an assignment you can also get write-after-approval access.

thanks,
Tom

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-03-21 16:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-03 10:22 [PATCH] sim/erc32: Rename EVENT_MAX -> MAX_EVENTS Orgad Shaneh
2024-03-08 17:23 ` Tom Tromey
2024-03-21  6:37   ` Orgad Shaneh
2024-03-21 16:49     ` Tom Tromey

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox