From: ljsebald <ljs...@us...> - 2023-11-16 15:51:20
|
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "A pseudo Operating System for the Dreamcast.". The branch, master has been updated via 9462865fd8e91ea45d2241c3bead0722c55a020d (commit) via de9a02b5d7e9914a5e63ceaf70db3e7103ce3a5e (commit) from a8c46d4bf635a113b6895873389ed8aeee793017 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 9462865fd8e91ea45d2241c3bead0722c55a020d Merge: a8c46d4 de9a02b Author: Lawrence Sebald <ljs...@us...> Date: Thu Nov 16 10:45:32 2023 -0500 Merge pull request #365 from KallistiOS/thread_mode_warning Removed INIT_THD_PREEMPT startup warning commit de9a02b5d7e9914a5e63ceaf70db3e7103ce3a5e Author: Falco Girgis <gyr...@gm...> Date: Thu Nov 16 08:47:20 2023 -0600 Removed INIT_THD_PREEMPT startup warning - Removed warning from init.c that gets printed when the INIT_THD_PREEMPT flag gets used * this flag is always used by default, so the warning is just clutter and is confusing when the flag wasn't explicitly added - Hid extern variable for current thread pointer from the Doxygen in thread.h ----------------------------------------------------------------------- Summary of changes: include/kos/thread.h | 11 +++-------- kernel/arch/dreamcast/kernel/init.c | 5 ----- 2 files changed, 3 insertions(+), 13 deletions(-) diff --git a/include/kos/thread.h b/include/kos/thread.h index c57004f..6b8be23 100644 --- a/include/kos/thread.h +++ b/include/kos/thread.h @@ -271,18 +271,13 @@ typedef struct kthread_attr { @{ */ #define THD_MODE_NONE -1 /**< \brief Threads not running */ -#define THD_MODE_COOP 0 /**< \brief Cooperative mode (deprecated) */ +#define THD_MODE_COOP 0 /**< \brief Cooperative mode \deprecated */ #define THD_MODE_PREEMPT 1 /**< \brief Preemptive threading mode */ /** @} */ -/** \brief The currently executing thread. - - \warning - Do not manipulate this variable directly! - - \sa thd_get_current -*/ +/** \cond The currently executing thread -- Do not manipulate directly! */ extern kthread_t *thd_current; +/** \endcond */ /** \brief Block the current thread. diff --git a/kernel/arch/dreamcast/kernel/init.c b/kernel/arch/dreamcast/kernel/init.c index ee2d6d5..9c7ae7e 100644 --- a/kernel/arch/dreamcast/kernel/init.c +++ b/kernel/arch/dreamcast/kernel/init.c @@ -139,11 +139,6 @@ int __attribute__((weak)) arch_auto_init(void) { timer_ms_enable(); rtc_init(); - /* Threads */ - if(!(__kos_init_flags & INIT_THD_PREEMPT)) - dbglog(DBG_WARNING, "Cooperative threading mode is deprecated. KOS is \ - always in pre-emptive threading mode. \n"); - thd_init(); nmmgr_init(); hooks/post-receive -- A pseudo Operating System for the Dreamcast. |