mirror of
https://github.com/apache/nuttx.git
synced 2025-12-06 17:23:49 +08:00
3e29996d97
Change every occurence of up_switch_context to use this_task() as the first parameter. "nxsched_add_readytorun" returns "true" if context switch is required. "nxsched_add_readytorun" typically could only switch the assigned/running task to the one which is passed in as parameter. But this is not strictly guaranteed in SMP; if other CPUs tweak with affinities or priorities, it may be possible that the running task after the call is changed, but is some other task from the readytorun list (and it should be, if there is higher priority one available or the affinity of the added task prevents it to be scheduled in, but the previous head of the readytorun list should run. this_task() is always the correct one to switch to, since it always points to the tcb which was just switched in by the nxsched_add_readytorun. This is also a precursor to re-writing the SMP queue logic to remove pending lists for SMP. Signed-off-by: Jukka Laitinen <jukka.laitinen@tii.ae>
113 lines
3.5 KiB
C
113 lines
3.5 KiB
C
/****************************************************************************
|
|
* sched/mqueue/mq_waitirq.c
|
|
*
|
|
* SPDX-License-Identifier: Apache-2.0
|
|
*
|
|
* Licensed to the Apache Software Foundation (ASF) under one or more
|
|
* contributor license agreements. See the NOTICE file distributed with
|
|
* this work for additional information regarding copyright ownership. The
|
|
* ASF licenses this file to you under the Apache License, Version 2.0 (the
|
|
* "License"); you may not use this file except in compliance with the
|
|
* License. You may obtain a copy of the License at
|
|
*
|
|
* http://www.apache.org/licenses/LICENSE-2.0
|
|
*
|
|
* Unless required by applicable law or agreed to in writing, software
|
|
* distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
* WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
* License for the specific language governing permissions and limitations
|
|
* under the License.
|
|
*
|
|
****************************************************************************/
|
|
|
|
/****************************************************************************
|
|
* Included Files
|
|
****************************************************************************/
|
|
|
|
#include <nuttx/config.h>
|
|
|
|
#include <sched.h>
|
|
#include <assert.h>
|
|
#include <errno.h>
|
|
|
|
#include <nuttx/irq.h>
|
|
#include <nuttx/arch.h>
|
|
#include <nuttx/mqueue.h>
|
|
|
|
#include "mqueue/mqueue.h"
|
|
#include "sched/sched.h"
|
|
|
|
/****************************************************************************
|
|
* Public Functions
|
|
****************************************************************************/
|
|
|
|
/****************************************************************************
|
|
* Name: nxmq_wait_irq
|
|
*
|
|
* Description:
|
|
* This function is called when a signal or a timeout is received by a
|
|
* task that is waiting on a message queue -- either for a queue to
|
|
* becoming not full (on mq_send and friends) or not empty (on mq_receive
|
|
* and friends).
|
|
* Note: this function should used within critical_section
|
|
*
|
|
* Input Parameters:
|
|
* wtcb - A pointer to the TCB of the task that is waiting on a message
|
|
* queue, but has received a signal instead.
|
|
*
|
|
* Returned Value:
|
|
* None
|
|
*
|
|
* Assumptions:
|
|
*
|
|
****************************************************************************/
|
|
|
|
void nxmq_wait_irq(FAR struct tcb_s *wtcb, int errcode)
|
|
{
|
|
FAR struct tcb_s *rtcb = this_task();
|
|
FAR struct mqueue_inode_s *msgq;
|
|
|
|
/* It is possible that an interrupt/context switch beat us to the punch and
|
|
* already changed the task's state. NOTE: The operations within the if
|
|
* are safe because interrupts are always disabled with the waitobj,
|
|
* nwaitnotempty, and nwaitnotfull fields are modified.
|
|
*/
|
|
|
|
/* Get the message queue associated with the waiter from the TCB */
|
|
|
|
msgq = wtcb->waitobj;
|
|
DEBUGASSERT(msgq);
|
|
|
|
/* Decrement the count of waiters and cancel the wait */
|
|
|
|
if (wtcb->task_state == TSTATE_WAIT_MQNOTEMPTY)
|
|
{
|
|
DEBUGASSERT(msgq->cmn.nwaitnotempty > 0);
|
|
msgq->cmn.nwaitnotempty--;
|
|
dq_rem((FAR dq_entry_t *)wtcb, MQ_WNELIST(msgq->cmn));
|
|
}
|
|
else
|
|
{
|
|
DEBUGASSERT(msgq->cmn.nwaitnotfull > 0);
|
|
msgq->cmn.nwaitnotfull--;
|
|
dq_rem((FAR dq_entry_t *)wtcb, MQ_WNFLIST(msgq->cmn));
|
|
}
|
|
|
|
/* Indicate that the wait is over. */
|
|
|
|
wtcb->waitobj = NULL;
|
|
|
|
/* Mark the error value for the thread. */
|
|
|
|
wtcb->errcode = errcode;
|
|
|
|
/* Add the task to ready-to-run task list and
|
|
* perform the context switch if one is needed
|
|
*/
|
|
|
|
if (nxsched_add_readytorun(wtcb))
|
|
{
|
|
up_switch_context(this_task(), rtcb);
|
|
}
|
|
}
|