diff options
author | Jim Quigley <jim.quigley@arm.com> | 2020-09-21 13:13:11 +0100 |
---|---|---|
committer | tarek-arm <56722698+tarek-arm@users.noreply.github.com> | 2020-10-07 09:41:29 +0100 |
commit | d5a2a0f751c82acdcb0dde5a6e9f91611bb789f5 (patch) | |
tree | c4eef9a5e8acfcf173e4bdbad872bbfa13f4810a | |
parent | 6cd5a676a2dc04407b6ef4aeacb253fead3bfab1 (diff) |
fwk: Some checks in event handling should be for DEBUG builds only
We can remove some of the data checking in the event handler for
release builds, saving a few cycles.
Change-Id: Ie77a35de5cacdc9f13fa690e75a9d5d12999d4e8
Signed-off-by: Jim Quigley <jim.quigley@arm.com>
-rw-r--r-- | framework/src/fwk_thread.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/framework/src/fwk_thread.c b/framework/src/fwk_thread.c index d62a64bf..e3bed452 100644 --- a/framework/src/fwk_thread.c +++ b/framework/src/fwk_thread.c @@ -314,6 +314,7 @@ int fwk_thread_put_event(struct fwk_event *event) unsigned int interrupt; enum thread_interrupt_states intr_state; +#ifdef BUILD_MODE_DEBUG if (!ctx.initialized) { status = FWK_E_INIT; goto error; @@ -321,6 +322,7 @@ int fwk_thread_put_event(struct fwk_event *event) if (event == NULL) goto error; +#endif status = fwk_interrupt_get_current(&interrupt); if (status != FWK_SUCCESS) @@ -335,6 +337,7 @@ int fwk_thread_put_event(struct fwk_event *event) goto error; } +#ifdef BUILD_MODE_DEBUG status = FWK_E_PARAM; if (event->is_notification) { if (!fwk_module_is_valid_notification_id(event->id)) @@ -359,6 +362,7 @@ int fwk_thread_put_event(struct fwk_event *event) goto error; } } +#endif return put_event(event, intr_state); @@ -376,9 +380,10 @@ int fwk_thread_put_event_and_wait(struct fwk_event *event, struct fwk_event response_event; struct fwk_event *next_event; struct fwk_event *allocated_event; - unsigned int interrupt; int status = FWK_E_PARAM; enum wait_states wait_state = WAITING_FOR_EVENT; +#ifdef BUILD_MODE_DEBUG + unsigned int interrupt; if (!ctx.initialized) { status = FWK_E_INIT; @@ -395,6 +400,7 @@ int fwk_thread_put_event_and_wait(struct fwk_event *event, status = FWK_E_STATE; goto error; } +#endif if (ctx.current_event != NULL) event->source_id = ctx.current_event->target_id; |