I am using a Semaphore to ensure that two tasks are not accessing the same SPI port at the same time. But I am not sure if it is okay to it in this way.
TaskOne() // priority 0
{
for (;
{
xSemaphoreTake( xSemaphoreSPI0, portMAX_DELAY );
Presuming that the semphore is only given once successfully taken then your code will be ok. An alternative is to have one task that controls the SPI port, and any other task that wants to send or receive send a message to the SPI controlling task with the info to be sent/received.
That did not read very well, try again. An alternative is to have one task that controls the SPI port. If another task want to send data on the SPI port it sends the data to the task in charge of the SPI rather than directly to the SPI peripheral itself. The message can be a structure, with for example the message destination, length and data bytes. The message can be send to the controlling task on a queue to ensure serial access is maintained.