ipc/ipcdev.git
20 months agoNotifyDriverShm: Update local variable types match usage
Sam Nelson [Sat, 5 Oct 2019 13:15:36 +0000 (09:15 -0400)]
NotifyDriverShm: Update local variable types match usage

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoHeapMultiBufMP: Add NULL check for robustness
Sam Nelson [Sat, 5 Oct 2019 13:14:23 +0000 (09:14 -0400)]
HeapMultiBufMP: Add NULL check for robustness

Add cast to match types

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoHeapMemMP: Add NULL check for robustness
Sam Nelson [Sat, 5 Oct 2019 13:13:12 +0000 (09:13 -0400)]
HeapMemMP: Add NULL check for robustness

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoHeapBufMP: Add NULL check for robustness
Sam Nelson [Sat, 5 Oct 2019 13:12:50 +0000 (09:12 -0400)]
HeapBufMP: Add NULL check for robustness

Also removes unnecessary Int32 cast

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoGatePersonN: Update stack variable to match usage
Sam Nelson [Sat, 5 Oct 2019 13:11:15 +0000 (09:11 -0400)]
GatePersonN: Update stack variable to match usage

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoGateHWSpinlock: Update stack variable to match usage
Sam Nelson [Sat, 5 Oct 2019 13:10:39 +0000 (09:10 -0400)]
GateHWSpinlock: Update stack variable to match usage

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoGateHWSem: Update stack variable to match usage
Sam Nelson [Sat, 5 Oct 2019 13:09:38 +0000 (09:09 -0400)]
GateHWSem: Update stack variable to match usage

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoNotify: Add NULL check for robustness
Sam Nelson [Sat, 5 Oct 2019 13:08:38 +0000 (09:08 -0400)]
Notify: Add NULL check for robustness

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agolistMP: Add NULL check for robustness
Sam Nelson [Sat, 5 Oct 2019 13:08:09 +0000 (09:08 -0400)]
listMP: Add NULL check for robustness

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoIpc: Fix mismatched types for comparison
Sam Nelson [Sat, 5 Oct 2019 13:07:33 +0000 (09:07 -0400)]
Ipc: Fix mismatched types for comparison

Also adds null checks for robustness

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoGateMP: Cast to common type for comparison
Sam Nelson [Sat, 5 Oct 2019 03:46:58 +0000 (23:46 -0400)]
GateMP: Cast to common type for comparison

Also adds NULL check

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoIpcPowerDsp_dra7xx: Add NULL check to avoid NULL pointer access
Sam Nelson [Sat, 5 Oct 2019 03:31:02 +0000 (23:31 -0400)]
IpcPowerDsp_dra7xx: Add NULL check to avoid NULL pointer access

Also make curInit volatile so that multiple threads can touch the same
variable

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoIpcPower: Add NULL check to avoid NULL pointer access
Sam Nelson [Sat, 5 Oct 2019 03:29:24 +0000 (23:29 -0400)]
IpcPower: Add NULL check to avoid NULL pointer access

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoTransportRpmsg: Add NULL check to avoid NULL pointer access
Sam Nelson [Sat, 5 Oct 2019 03:28:46 +0000 (23:28 -0400)]
TransportRpmsg: Add NULL check to avoid NULL pointer access

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoVirtQueue: Avoid mismatch of types in comparison
Sam Nelson [Sat, 5 Oct 2019 03:27:53 +0000 (23:27 -0400)]
VirtQueue: Avoid mismatch of types in comparison

Convert to common unsigned int
Also add error check where needed

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
20 months agoWatchdog: Add NULL check to avoid NULL pointer access
Sam Nelson [Sat, 5 Oct 2019 03:26:19 +0000 (23:26 -0400)]
Watchdog: Add NULL check to avoid NULL pointer access

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
21 months agoam65xx: Update core type for sciclient 3.50.04.07
Sam Nelson [Fri, 30 Aug 2019 02:03:16 +0000 (02:03 +0000)]
am65xx: Update core type for sciclient

The sciclient has library for mcu1_0, mcu1_1 and mpu1_0 separately.
Adding configuration to specify the right core type.
( Note this is supported in only new version of sciclient library)

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
22 months agobuild: Descoping C64x and C66 big endian 3.50.04.06
Mahesh Radhakrishnan [Thu, 1 Aug 2019 19:03:18 +0000 (19:03 +0000)]
build: Descoping C64x and C66 big endian

The C64x and c66 big endian platform are no longer supported and hence
getting removed from the build files
.
Signed-off-by: Sam Nelson <sam.nelson@ti.com>
22 months agotests: Update memory map to match update in Linux dts
Sam Nelson [Wed, 31 Jul 2019 22:45:35 +0000 (22:45 +0000)]
tests: Update memory map to match update in Linux dts

Updated memory region for each core to be 16 MB each
R5F0: 0xA0000000-0xA0FFFFFF: 16 MB
R5F1: 0xA1000000-0xA1FFFFFF: 16 MB
Also re-arranged memory sections to keep the resource table on the top
at offset 0x100000.
Updated resource table addresses and MPU settings accordingly.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
Reviewed-by: Suman Anna <s-anna@ti.com>
2 years agoRemoteproc: OMAPL138: Use FW_RSC_ADDR_ANY for vrings in resource table
Suman Anna [Tue, 26 Mar 2019 23:00:56 +0000 (18:00 -0500)]
Remoteproc: OMAPL138: Use FW_RSC_ADDR_ANY for vrings in resource table

The memory for vrings are allocated dynamically by the Linux kernel-side,
and the corresponding resource table vring da fields are updated with the
allocated vring physical addresses. The current resource table uses fixed
addresses, and these happen to match the allocated addresses with the
current DSP device DMA/CMA pools. Newer Linux kernels > 4.19 still do
dynamic allocation, but only update these fields if they are read as
FW_RSC_ADDR_ANY. The OMAPL138 platforms do not have an MMU and so must
use the actual allocated addresses, so update the current resource table
to use FW_RSC_ADDR_ANY for the vring da fields. The previously defined
macros are also cleaned up.

The patch can only be used after the vdev synchronization logic commit
("omapl138: VirtQueue: Add vdev status synchronization logic").

Signed-off-by: Suman Anna <s-anna@ti.com>
2 years agoomapl138: VirtQueue: Add vdev status synchronization logic
Suman Anna [Tue, 26 Mar 2019 23:00:55 +0000 (18:00 -0500)]
omapl138: VirtQueue: Add vdev status synchronization logic

The VirtQueue objects are currently initialized using the vring
addresses from the corresponding vring entries in the resource
table. The memory for vrings is always allocated on the Linux
kernel at runtime, and the resource table entry is updated with
the corresponding allocated address.

This allocation always used to happen during the vdev resource
handling previously on the Linux kernel side, but this behavior
has changed in newer Linux kernels > 4.19. The memory is allocated
now after all the resources are handled, and the resource table
is updated only when the virtio_rpmsg_bus driver is probed, and
only if a FW_RSC_ADDR_ANY is passed in the resource table. This
driver can be installed after the remoteproc is up, which results
in the RTOS code using a stale value.

Resolve this by adding a synchronization logic for the vdev
status (set during virtio_rpmsg_bus probe on kernel-side). The
synchronization is added during the creation of the first VirtQueue
object. This can probably be done in module startup as well since
the VirtQueue code on OMAPL138 is a XDC module, but the design
follows the non-XDC usage as on DRA7xx/AM57xx platforms.

Signed-off-by: Suman Anna <s-anna@ti.com>
2 years agoomapl138: VirtQueue: Fixup mapVAtoPA() and mapPAtoVA()
Suman Anna [Tue, 26 Mar 2019 23:00:54 +0000 (18:00 -0500)]
omapl138: VirtQueue: Fixup mapVAtoPA() and mapPAtoVA()

The mapVAtoPA() and mapPAtoVA() functions used in the OMAPL138
VirtQueue code uses a hard-coded address translating logic that
caters to only a specific set of images. The DSPs on OMAPL138
do not have MMUs, so the addresses seen in the vring control
structures should be identical between the processor view and
the actual DDR address. The only case where this might not be
true is if the MPAX is being used to perform some address
translations, but this is not the case today.

So, fixup these functions to provide a one-to-one exact translation.
This fixes up issues with latest upstream kernels (v5.1+) where
the virtio buffers are no longer allocated from the same remoteproc
device DMA pool, and so can be allocated anywhere in DDR memory
(from the default Linux DMA/CMA pool).

Signed-off-by: Suman Anna <s-anna@ti.com>
2 years agoRemoteproc: OMAPL138: Remove unused macros from resource table
Suman Anna [Tue, 26 Mar 2019 23:00:53 +0000 (18:00 -0500)]
Remoteproc: OMAPL138: Remove unused macros from resource table

There are couple of macros that are defined in the OMAPL138 resource
table file, but are not used anywhere. These are left-overs from very
old code, so clean them up.

Signed-off-by: Suman Anna <s-anna@ti.com>
2 years agorsc_types: Add the definition of FW_RSC_ADDR_ANY
Suman Anna [Tue, 26 Mar 2019 23:00:52 +0000 (18:00 -0500)]
rsc_types: Add the definition of FW_RSC_ADDR_ANY

Add the definition for a FW_RSC_ADDR_ANY macro. This macro
should be used for da in various resource types if that
particular memory needs to be dynamically allocated on the
kernel. This value is now actively used in all the upstream
kernels post the v4.19 release.

Signed-off-by: Suman Anna <s-anna@ti.com>
2 years agorsc_types: Use the correct integer type in fw_rsc_vdev
Suman Anna [Tue, 26 Mar 2019 23:00:51 +0000 (18:00 -0500)]
rsc_types: Use the correct integer type in fw_rsc_vdev

The Linux kernel uses an unsigned 8-bit integer type for the
status, num_of_vrings and reserved fields of the fw_rsc_vdev
structure. These are slightly mis-represented as a 8-bit Char
type within the IPC code. Fix this by using the equivalent
BIOS-side variable type - UInt8 in the rsc_types.h header
file.

This does fix couple of warnings about comparision between
mixed types.

Signed-off-by: Suman Anna <s-anna@ti.com>
2 years agoResource: Fix compile warning in Resource_getTraceBufSize()
Suman Anna [Tue, 26 Mar 2019 23:00:50 +0000 (18:00 -0500)]
Resource: Fix compile warning in Resource_getTraceBufSize()

The function Resource_getTraceBufSize() should return a value
of type SizeT, but is currently returning a value of NULL upon
failures. This generates the following compile warning,
"warning: return value type does not match the function type".

Fix this properly by simply returning a value of 0.

Signed-off-by: Suman Anna <s-anna@ti.com>
2 years agoLinux: Update user AF_RPMSG define for 4.18+ kernels ipc-3.50-next 3.50.03.05
Suman Anna [Fri, 21 Dec 2018 23:47:53 +0000 (17:47 -0600)]
Linux: Update user AF_RPMSG define for 4.18+ kernels

IPC's Linux user space support contains a copy of a few
kernel-defined structs and definitions including the socket
address family used for rpmsg-based sockets - AF_RPMSG. This
driver is currently out-of-tree and so it's definition is
always 'one more' than the last address family defined by
the mainline kernel's socket.h.

A new address family was introduced in 4.18 (AF_XDP), and as
a result, the AF_RPMSG macro needs to be set to 45 post 4.18
kernel. The user space header has been updated to reflect on
the kernel version and appropriately assign the value of
AF_RPMSG.

Do note that a specific value of AF_RPMSG can always be provided
through the products.mak when KERNEL_INSTALL_DIR is not set.

Signed-off-by: Suman Anna <s-anna@ti.com>
2 years agotrace: Update to remove direct dependence on SysMin 3.50.03.04
Sam Nelson [Thu, 29 Nov 2018 19:40:03 +0000 (14:40 -0500)]
trace: Update to remove direct dependence on SysMin

The size of trace buffer here instead obtaining from the Sysmin
parameter from headerfile, is now obtained directly from the resource
table
Also fixes backward compatibility issues with applications using
xdc/runtime/SysMin.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoam65xx: Update to remove reference to sciclient internal header file
Sam Nelson [Mon, 12 Nov 2018 21:52:33 +0000 (16:52 -0500)]
am65xx: Update to remove reference to sciclient internal header file

Note that this reference to
ti/drv/sciclient/include/V0/tisci_resasg_types.h was added due to an
issue with the sciclient in terms of API.
Now that the issue is resolved sciclient, the reference to this header
file can  be removed.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoc66x: Update compiler options to reduce code size
Gaurav Mitra [Mon, 29 Oct 2018 15:00:41 +0000 (11:00 -0400)]
c66x: Update compiler options to reduce code size

Reduce IPC unit test code size for C66 binaries when using compiler
version 8.3.x by using optimization -mf=2 or --opt_for_speed=2

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotests: R5F Memory map cleanup
Sam Nelson [Wed, 24 Oct 2018 15:52:27 +0000 (11:52 -0400)]
tests: R5F Memory map cleanup

Added section to capture unused sections
Added align to avoid issues with unaligned load

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoam65xx: Fix issue for R5F0 to R5F1 communication
Sam Nelson [Thu, 25 Oct 2018 17:08:54 +0000 (13:08 -0400)]
am65xx: Fix issue for R5F0 to R5F1 communication

This code was originally designed to use local interrupt mechanisms
between R5F-0 and R5F-1 cores. But the implementation now assumes
mailbox interrupts to be directly used. So removing the code here which
bypasses enabling and disable of mailbox interrupts

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotests: Remove misleading memory comments 3.50.03.03
Sam Nelson [Mon, 22 Oct 2018 21:11:47 +0000 (17:11 -0400)]
tests: Remove misleading memory comments

The memory section itself is self explanatory.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoipc: Remove compiler warnings
Sam Nelson [Sun, 21 Oct 2018 20:52:52 +0000 (16:52 -0400)]
ipc: Remove compiler warnings

Removes warnings due to Assert_isTrue id set to NULL
Also removes warnings due to int to pointer conversion

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoam65xx: Fix Compilation warnings on int to pointer conversion
Sam Nelson [Sun, 21 Oct 2018 20:42:47 +0000 (16:42 -0400)]
am65xx: Fix Compilation warnings on int to pointer conversion

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotests: R5F Split mode support
Sam Nelson [Sun, 21 Oct 2018 20:41:25 +0000 (16:41 -0400)]
tests: R5F Split mode support

Add code to build separate test images for R5F0, R5F1.
R5F0 images can be used AS-IS with R5F cores in Lock step mode

Note the DDR memory is partitioned according to the Linux device tree
nodes for r5f-0 and r5f-1

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoam65x: Update resource table for split R5F cores
Sam Nelson [Mon, 22 Oct 2018 19:43:14 +0000 (15:43 -0400)]
am65x: Update resource table for split R5F cores

Note the memory is partitioned between the two split cores and the
sections should match what is the Linux device tree nodes for R5F0 &
R5F1

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoRemoteproc: AM65xx: Remove all RSC_CARVEOUT entries
Suman Anna [Thu, 18 Oct 2018 21:56:12 +0000 (16:56 -0500)]
Remoteproc: AM65xx: Remove all RSC_CARVEOUT entries

The R5F cores on AM65xx SoCs do not have an MMU, and so use
direct mapping of CPU memory to DDR memory. The Region Address
Translator (RAT) sub-module can provide linear address translations
but is not currently supported. The RSC_CARVEOUT entries are
currently being used to allocate memory on the Linux side for
various firmware image sections, but this was a temporary
workaround to leverage existing infrastructure on the Linux kernel
side.

Remove all the RSC_CARVEOUT entries and rely on the newly added
support on the Linux kernel side for fixed memory regions for
loading firmware images. The firmware image linker map sections
should fit the addresses defined through the reserved memory nodes
associated with the remoteproc device on the Linux kernel side.

NOTE:
The RSC_DEVMEM resource entry is retained for now until the
address translation logic in AM65xx VirtQueue code is fixed up,
since this resource type is not supported for remoteprocs without
any MMUs.

Signed-off-by: Suman Anna <s-anna@ti.com>
2 years agoam65xx: Update to use static resources from System firmware
Sam Nelson [Sat, 20 Oct 2018 02:43:00 +0000 (22:43 -0400)]
am65xx: Update to use static resources from System firmware

Instead of hard coding the interrupt resource, use the ones statically
configured as part of the board config

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotests: Update MPU configuration for non-shared memory
Sam Nelson [Sun, 21 Oct 2018 12:50:37 +0000 (08:50 -0400)]
tests: Update MPU configuration for non-shared memory

Making the code/data memory shareable makes the memory non-cacheable and
performance is affected.
Modified MPU configuration, so that only the shared region is marked as
shareable and other regions used for code/data are mareked as non-
shareable.
This results in better performance.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agorsc_table: Update to fix backward compatibilty with xdc SysMin
Sam Nelson [Thu, 13 Sep 2018 18:20:03 +0000 (14:20 -0400)]
rsc_table: Update to fix backward compatibilty with xdc SysMin

With this, application can use either ti.trace.SysMin or
xdc.runtime.SysMin.
Old applications using xdc.runtime.SysMin can still work.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoam65xx: Integrate sciclient for interrupt routing through system-fw 3.50.01.01
Sam Nelson [Fri, 17 Aug 2018 15:17:48 +0000 (11:17 -0400)]
am65xx: Integrate sciclient for interrupt routing through system-fw

- Adds dependency on sciclient part of the PDK package
- Add new module ti.sdo.ip.family.am65xx.NotifySciClient which
configures interrupt routes by communicating with system core using
sciclient module.
- removes old code which configured directly the interrupt routes to cpu

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotests: Add idle function to test code 3.50.01.00C
Sam Nelson [Fri, 17 Aug 2018 16:05:16 +0000 (12:05 -0400)]
tests: Add idle function to test code

This is to plug in function to handle idle state

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoam65xx: Add Power module for idle routine
Sam Nelson [Fri, 17 Aug 2018 16:03:22 +0000 (12:03 -0400)]
am65xx: Add Power module for idle routine

Idle routine is needed to put cpu in safe state for the host processor
to reset.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoping_rpmsg: OMAPL138: Add program section for tracebuf
Sam Nelson [Wed, 22 Aug 2018 14:22:54 +0000 (10:22 -0400)]
ping_rpmsg: OMAPL138: Add program section for tracebuf

This is needed when you use ti.trace.SysMin

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agorsc_table: am65xx: Update trace buffer size to 0x8000 3.50.01.00B
Sam Nelson [Tue, 21 Aug 2018 21:34:10 +0000 (17:34 -0400)]
rsc_table: am65xx: Update trace buffer size to 0x8000

Given availability of DDR, expanding to better size

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agorsc_table: Remove unused carveout definitions
Sam Nelson [Tue, 21 Aug 2018 20:09:26 +0000 (16:09 -0400)]
rsc_table: Remove unused carveout definitions

This is leftover from previous code. Not used anymore.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoResource: Handle AM65xx case correctly
Sam Nelson [Mon, 20 Aug 2018 20:06:32 +0000 (16:06 -0400)]
Resource: Handle AM65xx case correctly

The previous modification handled addressed the backward compatibility
concern, but reverted fix for am65xx

Signed-off-by: Sam Nelson <sam.nelson@ti.com>]
2 years agorpmsg: Fix to handle Keystone 2 correctly
Sam Nelson [Tue, 21 Aug 2018 18:54:19 +0000 (14:54 -0400)]
rpmsg: Fix to handle Keystone 2 correctly

xdc_Fxn is used only by OMAPL138, TCI6638

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotracebuf: Update trace buffer size to 0x2000 3.50.01.00A
Sam Nelson [Mon, 20 Aug 2018 20:56:07 +0000 (16:56 -0400)]
tracebuf: Update trace buffer size to 0x2000

Given tracebuf is circular buffer 0x2000 should be sufficient.
Need to reduce number of buffers for K2E to fix compilation issues

Also remove unused carveout definitions

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoResource: Fix backward compatibility issue
Sam Nelson [Mon, 20 Aug 2018 20:06:32 +0000 (16:06 -0400)]
Resource: Fix backward compatibility issue

Need to still handle case where no load segment is defined and the
.resource table need to be created a noninit section

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agorpmsg: Fix compilation warning
Sam Nelson [Fri, 17 Aug 2018 15:41:23 +0000 (11:41 -0400)]
rpmsg: Fix compilation warning

xdc_Fxn is used only by OMAPL138

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotransports: Add TransportRpmsg.c to package for AM65XX 3.50.01.00
Sam Nelson [Sat, 11 Aug 2018 02:22:00 +0000 (22:22 -0400)]
transports: Add TransportRpmsg.c to package for AM65XX

Without the fix , TransportRpmsg.c not included in package for AM65XX

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
---
Change from previous verison
Update typo in commit title
---

2 years agoqnx: Add 64 bit defines
Sam Nelson [Sun, 5 Aug 2018 22:13:22 +0000 (18:13 -0400)]
qnx: Add 64 bit defines

This is needed for MessageQ API updates to handle processors with 64 bit
support

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotests: Update tracebuf sections for K2 and omapl138
Sam Nelson [Fri, 3 Aug 2018 21:15:40 +0000 (17:15 -0400)]
tests: Update tracebuf sections for K2 and omapl138

- It is needed now that ti.trace.SysMin is used

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoTEMP: smp: Disable SMP for now for A53
Sam Nelson [Thu, 2 Aug 2018 02:15:52 +0000 (22:15 -0400)]
TEMP: smp: Disable SMP for now for A53

BIOS currently does not support SMP for A53
Disabling SMP configuration for now

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agodaemon: Update debug print with more clear information
Sam Nelson [Mon, 30 Jul 2018 20:34:35 +0000 (16:34 -0400)]
daemon: Update debug print with more clear information

Helps to debug if the AF_RPMSG is not the expected value

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoBuild.xs: Cleanup item identified
Sam Nelson [Wed, 9 Aug 2017 13:34:13 +0000 (09:34 -0400)]
Build.xs: Cleanup item identified

Added TODO to mark change needed

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoTEMP: Tests: ping_rpmsg: Override checkFrequency flags
Suman Anna [Wed, 30 May 2018 01:18:59 +0000 (20:18 -0500)]
TEMP: Tests: ping_rpmsg: Override checkFrequency flags

This is a work around to avoid issues with dmtimer.
TODO: Need to check with updated build target.

Signed-off-by: Suman Anna <s-anna@ti.com>
Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoremoteproc: Use ti.trace.SysMin instead of xdc.runtime.SysMin
Suman Anna [Wed, 4 Apr 2018 19:59:59 +0000 (14:59 -0500)]
remoteproc: Use ti.trace.SysMin instead of xdc.runtime.SysMin

ti.trace.SysMin is needed to have the trace handled as a circular buffer
Updating this for omapl138 and K2 platforms.

Signed-off-by: Suman Anna <s-anna@ti.com>
Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoSysMin: Remove error check against CPU memory map
Suman Anna [Wed, 4 Apr 2018 19:55:17 +0000 (14:55 -0500)]
SysMin: Remove error check against CPU memory map

Moving forward new platforms does not capture the cpu memory map, so
remove this error check for AM65X only

Signed-off-by: Suman Anna <s-anna@ti.com>
Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoTMP: listMP: Add Cache operation for v8A
Sam Nelson [Tue, 27 Mar 2018 12:43:34 +0000 (08:43 -0400)]
TMP: listMP: Add Cache operation for v8A

Ideally with the coherence working between A53 and R5F, there is no
cache operations required from A53 side. But the coherence operations
requires the ISC registers configured for R5F transactions to be non-
secure.
Adding this as temporary workaround.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoBuild.xs: Remove -mapcs-frame for A53F
Sam Nelson [Tue, 24 Oct 2017 14:43:51 +0000 (10:43 -0400)]
Build.xs: Remove -mapcs-frame for A53F

This option is deprecated in gcc for A53F target

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoBuild.xs: Update GCC compile option mapcs-frame
Sam Nelson [Wed, 4 Oct 2017 23:59:46 +0000 (19:59 -0400)]
Build.xs: Update GCC compile option mapcs-frame

mapcs is deprecated with new version of gcc tools but identical to
mapcs-frame

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotests: Cleanup for observed compiler warnings
Sam Nelson [Sat, 28 Jul 2018 04:17:15 +0000 (00:17 -0400)]
tests: Cleanup for observed compiler warnings

Removed typecast forcing pointer to UInt. With 64 bit pointer need to
print with appropriate formatting
- added type casts where needed

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agodaemon: Add error check form read and write calls.
Sam Nelson [Sat, 28 Jul 2018 04:13:13 +0000 (00:13 -0400)]
daemon: Add error check form read and write calls.

Added error check of return values from read and write functions.
This also clears compiler warnings

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agotransport: Add error check for read and write function calls
Sam Nelson [Fri, 7 Jul 2017 21:31:43 +0000 (17:31 -0400)]
transport: Add error check for read and write function calls

Added error check of return values from read and write functions.
This also clears compiler warnings

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoBuild.xs: remove unused utilsSources
Sam Nelson [Fri, 30 Jun 2017 15:48:07 +0000 (11:48 -0400)]
Build.xs: remove unused utilsSources

Sources for utils are already captured in packages/ti/sdo/utils/Build.xs

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoMessageQ: Update MesssageQ header for 64 bit addresses
Sam Nelson [Sat, 13 Jan 2018 15:11:31 +0000 (10:11 -0500)]
MessageQ: Update MesssageQ header for 64 bit addresses

The reserved fields are expected to hold the next and previous pointers
which needs to work with 64 bit pointers.
Redefined to be Bits64 to accomodate this.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoautotools: Update autotools generated files with K3 AM65XX support
Sam Nelson [Wed, 28 Jun 2017 19:56:32 +0000 (15:56 -0400)]
autotools: Update autotools generated files with K3 AM65XX support

- autoreconf used to generate makefiles

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoK3: Add support for K3 AM65XX device
Sam Nelson [Fri, 23 Jun 2017 21:17:19 +0000 (17:17 -0400)]
K3: Add support for K3 AM65XX device

- Adds support for A53 and R5F cores

- Adds IPC VirtQueue, Interrupt & Notity drivers for AM65XX platform.

- gates: Add V7R and v8A support

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoResource: Update to handle resource table area using linker symbol
Sam Nelson [Wed, 11 Apr 2018 20:49:30 +0000 (16:49 -0400)]
Resource: Update to handle resource table area using linker symbol

- Modified to  place memory only if loadSegment is defined
- For new platforms, SYSBIOS will not handle platform memory map.
So the linker command file is external. Only way to get the
start address is using the external linker symbol.
- Added new variable loadSymbol to allow setting the tableptr using
symbol

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoSharedRegion: Update to use linker symbols instead of platform map
Sam Nelson [Thu, 5 Oct 2017 01:07:58 +0000 (21:07 -0400)]
SharedRegion: Update to use linker symbols instead of platform map

The Keystone3 platform in sysbios does not support memory maps. The
memory map needs to be done using the custom linker command file.
And hence changed Shared Region module to optionally accept
linker symbols for base and length of shared region.

platform.dataMemory is no longer supported. Need to use linker.cmd
to place sections

Update types to generic types: Previous code assumed UInt32 for Pointer

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agoStd.h: Add definition of 64 bit types
Sam Nelson [Wed, 4 Apr 2018 19:24:14 +0000 (15:24 -0400)]
Std.h: Add definition of 64 bit types

This is needed for supporting new devices with 64 bit pointers

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
2 years agolad: Add Timeout waiting for Response FIFO
Angela Stegmaier [Tue, 5 Jun 2018 19:57:01 +0000 (14:57 -0500)]
lad: Add Timeout waiting for Response FIFO

Add a timeout to the logic which opens the client response FIFO
in the LAD daemon during LAD_CONNECT. There are some corner cases
that might result in this hanging forever waiting for the response
FIFO, in which case, all other LAD communication will be blocked
forever.

Such cases might be:
- Client application crashes after sending LAD_connect request
- Communication with LAD through the LADCMDS fifo takes too long
  and the Client times out waiting for the response FIFO (5 seconds).
  If this happens, Client will never try to open the FIFO again due
  to failing when doing the stat() on the FIFO.

With this patch, if the FIFO connection is not established within the
timeout, the FIFO will be unlinked and the original state restored so
that the connection can be attempted again.

The timeout is chosen as 1 second so as not to block all other LAD
communication for a long period of time. 1 second should (hopefully) be
sufficient time for the Client and LAD to both open the FIFO.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoResource: Rename the IpcMemory name in Resource.xs 3.47 ipc-3.47-next 3.47.02.00
Suman Anna [Tue, 1 May 2018 22:33:27 +0000 (17:33 -0500)]
Resource: Rename the IpcMemory name in Resource.xs

The Resource module is derived from the IpcMemory module within
the sysbios-rpmsg codebase originally, but the Resource.xs file
is still using the IpcMemory name and so the config processing of
this module will print erroneous names on any misconfiguration.
Rename the variable IpcMemory to Resource like is normally done
with .xs files.

Signed-off-by: Suman Anna <s-anna@ti.com>
3 years agoipc: static analysis issues on NULL pointer access
Sam Nelson [Mon, 23 Apr 2018 18:04:01 +0000 (14:04 -0400)]
ipc: static analysis issues on NULL pointer access

Added null check to avoid direct crash and return error or assert.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
3 years agoAndroid: Remove use of LOCAL_MODULE_PATH to allow Multilib build
Andrew F. Davis [Fri, 20 Apr 2018 17:25:20 +0000 (12:25 -0500)]
Android: Remove use of LOCAL_MODULE_PATH to allow Multilib build

LOCAL_MODULE_PATH is not multilib safe, the recommended alternative
is LOCAL_MODULE_RELATIVE_PATH, but we don't need that either as
using LOCAL_VENDOR_MODULE will get our output deployed to the
correct spot without needing to hard-code paths.

Signed-off-by: Andrew F. Davis <afd@ti.com>
3 years agotests: DRA7XX: Fix clock configuration in gatempapp.cfg
Angela Stegmaier [Thu, 19 Apr 2018 18:27:52 +0000 (13:27 -0500)]
tests: DRA7XX: Fix clock configuration in gatempapp.cfg

The gatempapp.cfg was not selecting the proper clock
config for DSP1. This patch fixes the gatempapp.cfg
file to use the same common Dsp_vayu.cfg and Dsp_vayu_power.cfg
files to do the common configurations including clock
configuration.

Without this, the watchdog will not be proper and will
result in unexpected watchdog timings for the messageq_fault
tests for DSP1.

Reported-by: Suman Anna <s-anna@ti.com>
Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
Tested-by: Suman Anna <s-anna@ti.com>
3 years agoLinux: Cleanup on Failure in Ipc_start
Angela Stegmaier [Wed, 7 Mar 2018 19:41:08 +0000 (13:41 -0600)]
Linux: Cleanup on Failure in Ipc_start

Add the appropriate cleanup calls when Ipc_start fails, so
that Ipc_start can be called again in a clean state. This
allows retry of Ipc_start in case of failure due to various
conditions including if reponses are delayed from the remote
core for some reason.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoLinux: GateMP: Fix Error Handling When defaultGate Not Found Due to Timeout
Angela Stegmaier [Wed, 7 Mar 2018 19:39:54 +0000 (13:39 -0600)]
Linux: GateMP: Fix Error Handling When defaultGate Not Found Due to Timeout

In a corner case where remote NameServer requests take too long and
time out, it is possible that remoteSystemGate may be de-referenced
in GateMP_start when it wasn't allocated during GateMP_attach.

This patch improves error checking in GateMP_start to avoid the potential
NULL pointer de-reference. Before trying to access the remoteSystemGates
pointer during GateMP_start, we should first make sure that the pointer is
valid. Also check to make sure that the gate number does not exceed the
number of allocated gates.

If the corner case has been hit where the GateMP_attach failed but GateMP_open
later passes during GateMP_start, GateMP_start will now return an error,
resulting in Ipc_start returning an error. The application can then call
Ipc_start again.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoipc-bios.mak: Update to allow use of top level Rule.mak 3.47.01.00 3.47.01.00_eng
Sam Nelson [Sun, 7 Jan 2018 04:09:03 +0000 (23:09 -0500)]
ipc-bios.mak: Update to allow use of top level Rule.mak

Products.mak sets variables to point to external products.
Allowing RULES_MAKE variable to be overriden, helps in building in a SDK
context.
This is to help with build with SDKs bundling IPC.
Should not affect stand alone IPC builds

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
3 years agoGateMP: Add cache operation to make sure the SR0 contents are synced up
Sam Nelson [Fri, 15 Dec 2017 18:36:40 +0000 (13:36 -0500)]
GateMP: Add cache operation to make sure the SR0 contents are synced up

Before opening the shared region, make sure the cache is
invalidated. Even though the memory was not accessed before, because of
speculative execution the memory may be in the cache line already.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
3 years agoListMP: Add barrier to ensure write backs complete
Sam Nelson [Tue, 14 Nov 2017 04:03:39 +0000 (23:03 -0500)]
ListMP: Add barrier to ensure write backs complete

The call to barrier function is needed to ensure the intended changes
are already written back to memory before leaving the gate to avoid
issues.

Signed-off-by: Sam Nelson <sam.nelson@ti.com>
3 years agoAndroid: Fix compilation warnings
Angela Stegmaier [Wed, 1 Nov 2017 16:52:54 +0000 (11:52 -0500)]
Android: Fix compilation warnings

Fix compilation warnings seen with Android build.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoLinux: Update user AF_RPMSG define for 4.11+ kernels
Suman Anna [Wed, 18 Oct 2017 21:11:13 +0000 (16:11 -0500)]
Linux: Update user AF_RPMSG define for 4.11+ kernels

IPC's Linux user space support contains a copy of a few
kernel-defined structs and definitions including the socket
address family used for rpmsg-based sockets - AF_RPMSG. This
driver is currently out-of-tree and so it's definition is
always 'one more' than the last address family defined by
the mainline kernel's socket.h.

A new address family was introduced in 4.11 (AF_SMC), and as
a result, the AF_RPMSG macro needs to be set to 44 post 4.11
kernel. The user space header has been updated to reflect on
the kernel version and appropriately assign the value of
AF_RPMSG.

Do note that a specific value of AF_RPMSG can always be provided
through the products.mak when KERNEL_INSTALL_DIR is not set.

Signed-off-by: Suman Anna <s-anna@ti.com>
3 years agoLinux: LAD_client: Fix Race Condition with Command FIFO
Angela Stegmaier [Fri, 6 Oct 2017 18:38:07 +0000 (13:38 -0500)]
Linux: LAD_client: Fix Race Condition with Command FIFO

With the current implementation, there is a small window
where the FIFO is closed by the client for writing while it is
still open by the LAD daemon for reading. In this small window,
if the LAD daemon attempts to do a read on the FIFO, it will get
EOF and the LAD daemon will close it's own FIFO and re-open it.

If the client then is able to open the FIFO again after
the LAD daemon has received EOF from the read call, but before
the LAD daemon has closed the FIFO, then there is now a situation
where the client has a handle to a stale FIFO. If all these
things happen, then when the client tries to write/flush to the
FIFO, a SIGPIPE signal will happen, resulting in the client
crash.

To avoid this race condition, we must close the window where
the FIFO is not open for writing by the client while it is already
open for reading by the daemon.

This patch closes the window by modifiying the client code to
delay closing it's non-blocking FIFO handle (used for handshaking)
until after it has successfully opened the regular FIFO.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoAndroid: Makefile: move IPC libraries/binaries to vendor partition 3.47.00.00 3.47.00.00_eng
Vishal Mahaveer [Thu, 14 Sep 2017 21:25:52 +0000 (16:25 -0500)]
Android: Makefile: move IPC libraries/binaries to vendor partition

Updating Android makefile to generate IPC userspace binaries and
libraries in /vendor location instead of /system.

This change requires the lad_dra7xx service definition in init.*.rc
files to be updated accordingly

-service lad_dra7xx /system/bin/lad_dra7xx -g
+service lad_dra7xx /vendor/bin/lad_dra7xx -g

Signed-off-by: Vishal Mahaveer <vishalm@ti.com>
3 years agoVayu: NotifySetup: Incorrect Index in NotifySetup_unplugHwi
Angela Stegmaier [Wed, 13 Sep 2017 21:42:59 +0000 (16:42 -0500)]
Vayu: NotifySetup: Incorrect Index in NotifySetup_unplugHwi

When checking the reference count before deleting the Hwi,
the code was always checking against index 0, instead of
the mbxIdx. This lead to the Hwi being deleted, even when
there were still references. Then, even if NotifySetup_plugHwi
was called again, the reference count was greater than 1,
and the Hwi would never be plugged again.

This issue manifests when Ipc_detach is called followed by
Ipc_attach.

This patch fixes the issue by checking against the appropriate
refcount for the mbxIdx.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoAndroid: update version.h include path for AF_RPMSG handling
Vishal Mahaveer [Wed, 13 Sep 2017 21:32:22 +0000 (16:32 -0500)]
Android: update version.h include path for AF_RPMSG handling

Update include path of version.h for current Android setup.
Previously (before O) we used to update the socket.h bionic header with
AF_RPMSG definitions, so the KERNEL_VERSION check was not exercized.

Now the kernel-headers of dra7xx in Android repo is updated to
reflect the correct Kernel version and can be used to check
the kernel version.

For older Android versions that do not have version.h in
kernel-headers, AF_RPMSG can still be taken from the socket.h
bionic header if it is updated to include AF_RPMSG.

Signed-off-by: Vishal Mahaveer <vishalm@ti.com>
Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoLinux: TransportRpmsg: Fix Mutex Deadlock Issue
Angela Stegmaier [Thu, 3 Aug 2017 21:16:01 +0000 (16:16 -0500)]
Linux: TransportRpmsg: Fix Mutex Deadlock Issue

In an error recovery situation, it is possible that a
mutex deadlock could occur when MessageQ_delete happened
at the same time that an fd was being signaled about the
remote core error in the same process. This error could
occur if there are multiple MessageQs in the same process
communicating with the remote core that is doing recovery.

When MessageQ_delete grabs the lock, it then tries to
communicate with rpmsgThreadFxn, sending a message and
then waiting for a response. If at the same time, the thread
is processing a shutdown notification for an fd, it will
also try to grab the mutex, but will be blocked because
MessageQ_delete is holding the mutex. The rpmsgThreadFxn
will not continue until it gets the mutex, leading to the
deadlock.

The fix is to change the mutex_lock to a trylock when
handling shutdown notifications in rpmsgThreadFxn. If the
thread fails to acquire the lock, it continues processing
commands and will try again later to process the shutdown
request at a time when the lock is no longer being held.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoMmRpc: Fix Compilation Warnings
Angela Stegmaier [Wed, 9 Aug 2017 21:56:32 +0000 (16:56 -0500)]
MmRpc: Fix Compilation Warnings

Fix compilation warnings introduced with commit
6736bb2b9b060cf37d3e6728e21509b24ec12da8,
"MmRpc: Add Input Paramter Checking to APIs".

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoTransportRpmsg: Fix Check for Invalid Socket
Angela Stegmaier [Thu, 29 Sep 2016 21:14:55 +0000 (16:14 -0500)]
TransportRpmsg: Fix Check for Invalid Socket

A check for sock in TransportRpmsg was incorrectly checking
for sock != 0, when it should have been checking against
INVALIDSOCKET (the value uninitialized socket are set to).

This patch fixes the check.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoLinux: LAD_client: Avoid hang in fopen during connect
Angela Stegmaier [Wed, 26 Jul 2017 14:57:00 +0000 (09:57 -0500)]
Linux: LAD_client: Avoid hang in fopen during connect

If the application is started and calls Ipc_start before
the LAD is started and created/opened the fifo, then
the application can block in the fopen trying to connect
to the command FIFO.

This patch makes the sequence more robust by first checking
if the fifo exists, then trying to open it as non-blocking,
thereby getting an immediate success or error instead of
blocking until it has been opened for reading by LAD. Once
the non-blocking open returns successfully, the regular
connect sequence can continue. If it returns with a failure,
it will sleep and then try again for a period of time,
eventually returning a timeout failure if it is unable
to connect. This should prevent the application from hanging
if LAD is not started before it calls Ipc_start.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoMmRpc: Add Input Paramter Checking to APIs
Angela Stegmaier [Wed, 12 Jul 2017 21:33:09 +0000 (16:33 -0500)]
MmRpc: Add Input Paramter Checking to APIs

This patch adds input paramter checking to the MmRpc_* APIs,
specifically checking for NULL pointers, in order to prevent
a crash due to NULL-pointer de-reference in the case that the
user sends a bad (NULL) pointer.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoQNX: NameServer_daemon: Initialize mutex_attr 3.46 ipc-3.46-next 3.46.02.04 3.46.02.04_eng
Angela Stegmaier [Fri, 16 Jun 2017 19:41:21 +0000 (14:41 -0500)]
QNX: NameServer_daemon: Initialize mutex_attr

Initialize mutex_attr before using it. Otherwise, it could
contain some uninitialized values.

This patch is ported from commit id
7b3a739876e46d3194861a2d93077ff95954affb

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoQNX: Tests: Add NameServer and GateMP to Fault Test
Angela Stegmaier [Mon, 5 Jun 2017 19:42:25 +0000 (14:42 -0500)]
QNX: Tests: Add NameServer and GateMP to Fault Test

Add NameServer and GatemP to fault recovery test in
order to test error recovery of these modules.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoQNX: NameServer Daemon: Check that Handles passed from User are Valid
Angela Stegmaier [Mon, 5 Jun 2017 21:01:18 +0000 (16:01 -0500)]
QNX: NameServer Daemon: Check that Handles passed from User are Valid

Check that the handle being passed from the user is a valid
handle that the daemon has previously allocated. This prevents
the resource manager from crashing by accessing a stale handle
that the user has passed (for instance in some error recovery
scenarios).

This patch adds the checking for function calls which are accessible
through a devctl call.

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoQNX: Tests: Add error/cleanup tests to NameServer test
Angela Stegmaier [Mon, 27 Mar 2017 17:03:54 +0000 (12:03 -0500)]
QNX: Tests: Add error/cleanup tests to NameServer test

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>
3 years agoQNX: NameServer_delete in NameServer daemon leaks memory
Angela Stegmaier [Fri, 24 Mar 2017 21:57:37 +0000 (16:57 -0500)]
QNX: NameServer_delete in NameServer daemon leaks memory

Remove an invalid assertion that the name list must be empty.
This is not true and the instance might still be in use by
another client (object is reference counted). Add a while loop
to delete each entry on the name list. Destroy the object's
gate (was not being done). Remove the macro to destruct a
list object (not needed).

[Ported from commits 8a52a6a5fa20ac028158a3ad346da46555ce45b7
    "SDOCM00115434 NameServer_delete in LAD daemon leaks memory"
    and 6c4799151327f2c83b9f253ea25c52bef4e10803
    "Remove NameServer instance from module list in NameServer_delete()"]

Signed-off-by: Angela Stegmaier <angelabaker@ti.com>