tacit/minix
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Removing redundant logic and correcting debug directive

Browse Source 
Two debug directives were disabled in what appears to be a mistake - updated code to correct entries.
Removed logic that was always true based on switch case.
This commit is contained in:
Leo Custodio 2021-02-05 20:10:26 -06:00 committed by GitHub
parent 4db99f4012
commit 0184887276
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
minix/kernel/proc.c
View File

@ -506,7 +506,7 @@ static int do_sync_ipc(struct proc * caller_ptr, /* who made the call */
{ {
if (call_nr != RECEIVE) if (call_nr != RECEIVE)
{ {
#if 0 #if DEBUG_ENABLE_IPC_WARNINGS
printf("sys_call: %s by %d with bad endpoint %d\n", printf("sys_call: %s by %d with bad endpoint %d\n",
callname, callname,
proc_nr(caller_ptr), src_dst_e); proc_nr(caller_ptr), src_dst_e);
@ -519,7 +519,7 @@ static int do_sync_ipc(struct proc * caller_ptr, /* who made the call */
{ {
/* Require a valid source and/or destination process. */ /* Require a valid source and/or destination process. */
if(!isokendpt(src_dst_e, &src_dst_p)) { if(!isokendpt(src_dst_e, &src_dst_p)) {
#if 0 #if DEBUG_ENABLE_IPC_WARNINGS
printf("sys_call: %s by %d with bad endpoint %d\n", printf("sys_call: %s by %d with bad endpoint %d\n",
callname, callname,
proc_nr(caller_ptr), src_dst_e); proc_nr(caller_ptr), src_dst_e);
@ -576,10 +576,8 @@ static int do_sync_ipc(struct proc * caller_ptr, /* who made the call */
break; /* done, or SEND failed */ break; /* done, or SEND failed */
/* fall through for SENDREC */ /* fall through for SENDREC */
case RECEIVE: case RECEIVE:
if (call_nr == RECEIVE) { caller_ptr->p_misc_flags &= ~MF_REPLY_PEND;
caller_ptr->p_misc_flags &= ~MF_REPLY_PEND; IPC_STATUS_CLEAR(caller_ptr); /* clear IPC status code */
IPC_STATUS_CLEAR(caller_ptr); /* clear IPC status code */
}
result = mini_receive(caller_ptr, src_dst_e, m_ptr, 0); result = mini_receive(caller_ptr, src_dst_e, m_ptr, 0);
break; break;
case NOTIFY: case NOTIFY: