gh0s7/mckernel
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

eliminate futex_cmpxchg_enabled check (not used and dereffed a NULL pointer)

Browse Source 
Change-Id: I97b0e79acfd51b57eeaa6556eba880d231330f01
This commit is contained in:
Balazs Gerofi
2018-12-24 01:13:41 +09:00
committed by Dominique Martinet
parent 7dd0cbd9a6
commit 557f33a705
1 changed files with 0 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
kernel/futex.c
View File

@@ -86,7 +86,6 @@
unsigned long ihk_mc_get_ns_per_tsc(void); unsigned long ihk_mc_get_ns_per_tsc(void);
int futex_cmpxchg_enabled;
/** /**
* struct futex_q - The hashed futex queue entry, one per waiting task * struct futex_q - The hashed futex queue entry, one per waiting task
@@ -962,25 +961,8 @@ int futex(uint32_t *uaddr, int op, uint32_t val, uint64_t timeout,
int futex_init(void) int futex_init(void)
{ {
int curval;
int i; int i;
/*
* This will fail and we want it. Some arch implementations do
* runtime detection of the futex_atomic_cmpxchg_inatomic()
* functionality. We want to know that before we call in any
* of the complex code paths. Also we want to prevent
* registration of robust lists in that case. NULL is
* guaranteed to fault and we get -EFAULT on functional
* implementation, the non functional ones will return
* -ENOSYS.
*/
curval = cmpxchg_futex_value_locked(NULL, 0, 0);
if (curval == -EFAULT) {
dkprintf("futex_cmpxchg_enabled = 1 ??\n");
futex_cmpxchg_enabled = 1;
}
for (i = 0; i < ARRAY_SIZE(futex_queues); i++) { for (i = 0; i < ARRAY_SIZE(futex_queues); i++) {
plist_head_init(&futex_queues[i].chain, &futex_queues[i].lock); plist_head_init(&futex_queues[i].chain, &futex_queues[i].lock);
ihk_mc_spinlock_init(&futex_queues[i].lock); ihk_mc_spinlock_init(&futex_queues[i].lock);