PROCESSOR-SDK-J721S2: IPC issue with QNX 710

Manan Bhatt said:

i am using TI SDKJ721S2 11.0 SDK and back ported QNX 710 SDK.

Hi,

Can you tell us what changes you made to backport to QNX 710?

Thanks.

Hi Praveen , we have downloaded from this link:

www.ti.com/.../J721S2-QNX-SDK

Hi Praveen,

Also as part of 11.0SDK when the QNX Safety qualification in A72 was added wrt to SCIClient, UDMA, IPC,  whether any changes on TIIPC driver that would need some adaption for safety at the Process(user space) or Kernel level. This question I have asked because when we use the R5F and C7x with SDK 11.0 drivers and A72 with 10.1 SDK drivers, the IPC b/w QNX process in A72 and R5F is working fine. Also crash is seen on the A72 only when A72 is trying to communicate to remote cores. 

Manan Bhatt said:

Hi Praveen , we have downloaded from this link:

www.ti.com/.../J721S2-QNX-SDK

This SDK is for SDP 7.1, so it should ideally work without any changes. There is no backport needed.

Ideally, we recommend avoiding the mixing and matching of RTOS and QNX releases. If you are using RTOS SDK 11.0, then you must be using QNX SDK 11.0 for A72.

The above issue is a kernel crash, so we will check with our dev team and see if they have any feedback.

Thanks.

Checked with the dev team, and they have not seen this issue during their 11.0 release validation. They hinted that it could be due to some localized integration issue. Let us know.

Thanks.

Hi Praveen,

After changes the buffsize to fix in rpmsg create, IPC is fine,

case DCMD_TIIPC_RPMSG_CREATE:
{
// Saving local copy of user provided buf to pass back later
void *buf = cargs->args.create.params.buf;
printf("buff size@@@ %d \n",cargs->args.create.params.bufSize);
// Creating local queue buf in resource manager context
cargs->args.create.params.buf = malloc(sizeof(uint8_t)*135424);
//cargs->args.create.params.buf = malloc(cargs->args.create.params.bufSize);

if (cargs->args.create.params.buf != NULL)
{
// printf("DCMD_TIIPC_RPMSG_CREATE insdie if \n");
RPMessage_Handle handle = NULL;

handle = RPMessage_create(&cargs->args.create.params,
&cargs->args.create.ept);
if (handle == NULL)
{
QNX_PR_ERR("%s:%d Error: handle is NULL", __FUNCTION__, __LINE__);
printf("DCMD_TIIPC_RPMSG_CREATE handle is NULL \n");
free(cargs->args.create.params.buf);
cargs->args.create.handle = 0;
cargs->status = (uint32_t)IPC_EFAIL;
}
else
{
ipc_rpmsg_handle_entry *entry = NULL;

entry = malloc(sizeof(ipc_rpmsg_handle_entry));
if (entry != NULL)
{
entry->ept = cargs->args.create.ept;
entry->handle = handle;
entry->rcvid = 0;
cargs->args.create.handle = entry->ept;

QNX_PR_NOTICE("%s.%d: Created handle with endpoint %d", __FUNCTION__, __LINE__, entry->ept);

(void)pthread_mutex_lock(&ipc_ocb->lock);
IpcUtils_Qput(&ipc_ocb->created_handles, &entry->elem);
if (-1 != ipc_ocb->callbackPulseCoid)
{
QNX_PR_NOTICE("%s.%d: Setting callback coid...endPt=%d, coid=%d", __FUNCTION__, __LINE__, entry->ept, ipc_ocb->callbackPulseCoid);
gCallbackCoidPool[entry->ept] = ipc_ocb->callbackPulseCoid;
}
(void)pthread_mutex_unlock(&ipc_ocb->lock);
}
}
}
else
{
printf("DCMD_TIIPC_RPMSG_CREATE insdie else \n");
}
cargs->args.create.params.buf = buf;
break;
}

it is printing negative values at startup so allocate is getting fail.

We will review your latest findings with our development team and will get back to you with their feedback as soon as we can.

Hi Manan,

The feedback we received from our dev team is as follows:

"It sounds like params.bufSize is not provided to RPMessage_create, causing an uninitialized buf size. However, we are missing a check on the buffer size and returning an error with a slog message in the RPMessage_Create() (psdkqa/qnx/resmgr/ipc_qnx_rsmgr/usr/ipc_api.c line 423-465). We'll create a JIRA bug to add a check of the parameter passed and incorporate it into the next release.

FYI, in our IPC test example code that we provide (psdkqa/qnx/examples/ipc/ipc_test_qnxipc_testsetup.c line 213-217), we show how to set up the params structure with buf information filled. "

Thanks.