summaryrefslogtreecommitdiff
path: root/drivers/staging
AgeCommit message (Collapse)Author
2015-11-09Merge pull request #65 in SDK/linux-devel from ↵Yin Honghua-R63875
~R01356/linux-devel:bugfix/QUSDPA-704-segfault-in-dma_mem-allocator.c to master * commit 'e98f36e86bdfbeb1728ab0de4f2be367d6283963': fsl_usdpaa: Remove TLB entries when unmapping memory
2015-11-03fsl_usdpaa: Remove TLB entries when unmapping memoryRoy Pledge
TLB1 entries must be removed when unmapping a DMA map from a process so they don't point to the wrong physical memory if a new map reuses the same virtual address Signed-off-by: Roy Pledge <Roy.Pledge@freescale.com>
2015-11-02fsl-qbman: Handle memory leaksMahammad Ismayilzada
Eliminate memory leaks on error paths Signed-off-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com> Change-Id: I36d1a35f9d48188a9dad043c78e90d787997ea19
2015-11-02fsl-pme: Check for null pointersMahammad Ismayilzada
Check for null pointers to prevent null pointer dereferencing Signed-off-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com> Change-Id: Icc267f51d5ad85187c6ee5d03aa87f8649e124c9
2015-11-02fsl-dce: Handle memory allocation failuresMahammad Ismayilzada
Check for memory allocation failures to protect from null pointer dereferencing Signed-off-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com>
2015-11-02fsl-dce: Correct error paths and add new labelsMahammad Ismayilzada
Correct the order of existing goto labels and add new ones to handle error paths properly for inflation process Signed-off-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com>
2015-11-02fsl_qbman: null check for lni in qman_ceemt_sp_releaseHaiying Wang
For JIRA issue QLINUX-3837 Signed-off-by: Hemant Agrawal <Hemant@freescale.com> Signed-off-by: Haiying Wang <Haiying.wang@freescale.com>
2015-10-13fsl_qbman: enable sending cscn to dcp for old qman revisionHaiying Wang
Set CSCN_TARG with correct DCP portal in qman_create_cgr_to_dcp() funciton for the qman revision < qman_3.0 JIRA issue# QLINUX-3814 Signed-off-by: Haiying Wang <Haiying.wang@freescale.com>
2015-06-10fsl_qbman: hardcode mps field to 60Haiying Wang
In LNI shaper setup, setting mps to 60 to round up the frame length to 60 for shaper calculations, for any dequeued frame length less than 60 bytes. Signed-off-by: Haiying Wang <Haiying.Wang@freescale.com> Change-Id: I88013d2ee39b3620a4e97f3366a87664ec0ea9dc Reviewed-on: http://git.am.freescale.net:8181/37437 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-06-05fsl_usdpaa: Ensure map name is initialized and clearedRoy Pledge
Make sure that map name is null terminated when a memory map is split and is cleared when a map is destroyed Signed-off-by: Roy Pledge <Roy.Pledge@freescale.com> Change-Id: If779b54817b9c2d49d6e18106b333a51ca2b2dcf Reviewed-on: http://git.am.freescale.net:8181/37436 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-06-04fsl_usdpaa: Fix mem_create behaviorAhmed Mansour
Remove check added in previous patch to disallow size zero to be passed from user space in dma_mem_create(). Size zero is deemed valid if the memory region is already created and a second user wishes to get a mapping to the existent memory. Corrected values copied back to the user to include the length of the memory and the flags. This is important to reflect a memory size correction when the user passes size zero. The user can check the new size using dma_mem_params() Added a warning message if the user attempts to map to an existing area in memory, but specifies a non-zero size that does not match the original memory mapping. In the future this case will trigger an error and the mapping will fail. Currently the behavior is to print a warning message and the kernel passes back to user space the corrected size. Signed-off-by: Ahmed Mansour <Ahmed.Mansour@freescale.com> Change-Id: Ib8535ada6f0fb616986bce3c52eae65f3bf583da Reviewed-on: http://git.am.freescale.net:8181/37365 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-05-25fsl_qbman: Update ceetm with some user friendly APIsHaiying Wang
Signed-off-by: Haiying Wang <Haiying.Wang@freescale.com> Change-Id: Iaaf60a7e20a7cd96698fbcb3f98b5918000872e1 Reviewed-on: http://git.am.freescale.net:8181/36225 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-05-21dce: Typo correctionMahammad Ismayilzada
slab_scr_128b should be checked for NULL. Signed-off-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com> Change-Id: I51f5b7c96fbee9a0f1baf2340ff97b23caed434e Reviewed-on: http://git.am.freescale.net:8181/36649 Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-05-21qbman: Fix null pointer issueMahammad Ismayilzada
bufs passed as NULL and dereferenced. Replaced with local variable. Signed-off-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com> Change-Id: I39aecee95b086567a21487ee2b751126aa0777f1 Reviewed-on: http://git.am.freescale.net:8181/36648 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-05-21fsl_usdpaa: Protect against truncationAhmed Mansour
is_power_of_2 is a Linux function that takes unsigned long. it was used by our new function is_power_of_4 which takes u64. Trucation is possible because unsigned long is 32 bits wide in 32 bit applications. This patch removes the dependancy on is_power_of_2 to solve this issue Signed-off-by: Ahmed Mansour <Ahmed.Mansour@freescale.com> Change-Id: I5939e7e6457e07e9355ba54fef80a7bc71c3a64b Reviewed-on: http://git.am.freescale.net:8181/36609 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-05-21pme: Protect against possible NULL dereferenceAhmed Mansour
Add BUG_ON to prevent NULL dereference if ctx->flags indicate flow mode (none direct mode) then a call back token must be specified otherwise the call back token can be passed NULL Signed-off-by: Ahmed Mansour <Ahmed.Mansour@freescale.com> Change-Id: I20f95b2fd77699eb32ef54515666127ebc6b1743 Reviewed-on: http://git.am.freescale.net:8181/36607 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-05-21qbman: Fix qman_ceetm_cscn_dcp_get()Ahmed Mansour
Corrections to struct qm_mcr_ceetm_ccgr_query to match block guide Corrections to qman_ceetm_cscn_dcp_get() bit manipulation to match bit order in block guide Signed-off-by: Ahmed Mansour <Ahmed.Mansour@freescale.com> Change-Id: I78d730263495358c29be766933aceae9d5b36326 Reviewed-on: http://git.am.freescale.net:8181/36407 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-05-18dpa_offload: Avoid illegal access of IPSec instance after releaseMarian Chereji
The function "dpa_ipsec_free" is calling "free_resources" which is releasing the dpa_ipsec instance by executing kfree(dpa_ipsec); Once this call ends, "dpa_ipsec_free" is accessing again the "instance" pointer to call "mark_unused_gbl_dpa_ipsec". The problem is that the "instance" pointer is inconsistent at his point (was released by the call to "free_resources"). To fix this, the call to "mark_unused_gbl_dpa_ipsec" was simply removed from "dpa_ipsec_free" because the function "free_resources" is already doing that. Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Change-Id: Ibabc69253a8a21c7bb158bc371193a6b9ace01ac Reviewed-on: http://git.am.freescale.net:8181/36404 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Pinghua An <pinghua.an@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-05-13dpa_offload: Restore ONIC interface buffer poolMarian Chereji
The ONIC interface only works when the offline port OH2 is using buffer pool #17 which was prepared for it. This prevents the unification of the DTS example files for the "ipsec_offload" and "dpa_offload" demo applications, because the NF offloading demo application requests the buffer pool #16 on the OH2 offline port as NF IP Forwarding does not support VSPs. Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Change-Id: I3e2996f6777e4b16aee1a3cf489f89b60a37f325 Reviewed-on: http://git.am.freescale.net:8181/36070 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Nicolae-Sebastian Grigore <sebastian.grigore@freescale.com> Reviewed-by: Pinghua An <pinghua.an@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-04-21dpa_offload: Resync B4860QDS device tree files with NFAPI applicationsMarian Chereji
Example device tree files for the B4860QDS platform needed to be synchronized and aligned due to differences in the Virtual Storage Profile utilization between traditional DPAA offloading applications and the DPAA NF offloading applications. Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Change-Id: Ia6f7443b19876c1e9882419ef1e7b6b9729242e1 Reviewed-on: http://git.am.freescale.net:8181/35181 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Nicolae-Sebastian Grigore <sebastian.grigore@freescale.com> Reviewed-by: Radu-Andrei Bulie <Radu.Bulie@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-04-21qbman: Agressively clean QMan portals before useRoy Pledge
Make sure both the message ring and dequeue ring of a QMan portal are clean before attempting to use them Signed-off-by: Roy Pledge <Roy.Pledge@freescale.com> Change-Id: I8b4736625acfb25954cd3fa60e0761f86c2d9620 Reviewed-on: http://git.am.freescale.net:8181/31755 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Jeffrey Ladouceur <Jeffrey.Ladouceur@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/35331 Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-04-02dpa_offload: Add DTS file for NF DPAA offloadingMarian Chereji
Added the B4860QDS platform DTS file example for NF DPAA offloading demo application. Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Change-Id: Ib546a6b2984735f933df90cce1be376bd60245d5 Reviewed-on: http://git.am.freescale.net:8181/33632 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-03-31Revert "dpa_offload: Add DTS file for NF DPAA offloading"Honghua Yin
This reverts commit fa008b7d96cc30a7d711641d1513e226badd3de9. Change-Id: If2569f9e57d697ef1cceb865326c52eeaa5b03ad Reviewed-on: http://git.am.freescale.net:8181/33914 Tested-by: Honghua Yin <Hong-Hua.Yin@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-03-31dpa_offload: Add DTS file for NF DPAA offloadingMarian Chereji
Added the B4860QDS platform DTS file example for NF DPAA offloading demo application. Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Change-Id: Ib546a6b2984735f933df90cce1be376bd60245d5 Reviewed-on: http://git.am.freescale.net:8181/33632 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-03-26qbman: Add Exception handling - static analysisAhmed Mansour
Catch exceptions and return from functions safely in qbman. Exception handling holes were found in static analysis review Signed-off-by: Ahmed Mansour <Ahmed.Mansour@freescale.com> Change-Id: Ie1e2feadffd2c5c968164d0c569af147ab7929fb Reviewed-on: http://git.am.freescale.net:8181/25568 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-03-23dpa_offload: Fix statistics info for multipolicy IPSec outbound SAsMarian Chereji
The dpa_ipsec driver was only providing statistics for the first policy of the outbound SAs. It is now correctly suming up the statistics for all the outbound SA's policies. Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/28731 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Bogdan Constantin Popescu <bogdan.c.popescu@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com> Change-Id: Ide98e16c6a6c684b6d51fa190388c7a95b1a8490 Reviewed-on: http://git.am.freescale.net:8181/33188
2015-03-20fsl: Use ccflags-y instead of deprecated EXTRA_CFLAGSWang Dongsheng
This allows users to pass in additional compiler flags through the environment variable EXTRA_CFLAGS, e.g. make EXTRA_CFLAGS=-Wno-error Signed-off-by: Wang Dongsheng <dongsheng.wang@freescale.com> Change-Id: I36513506ee1671c31bb076e9b73c2e8e2442cf4f Reviewed-on: http://git.am.freescale.net:8181/33146 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-03-16dpa_offload: Fix SEC era info acquisitionMarian Chereji
The dpa_ipsec driver was unreliable is acquiring the SEC era for toggling the ESP padding removal setting of the SEC. The SEC version was used which is not a good criteria to follow in order to assume the ESP padding removal feature is available. The implementation was updated and the driver is now searching for the first device tree node which has a "fsl,sec-era" property and is using that information for further configuration of the SEC. Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Change-Id: I80ffe931995b8aac38d42a0e013d760fbb7a5cd6 Reviewed-on: http://git.am.freescale.net:8181/29887 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Horia Ioan Geanta Neag <horia.geanta@freescale.com> Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
2015-02-25fsl_qman: add qman_delete_cgr_safe()Madalin Bucur
Add qman_delete_cgr_safe() that can be called from any CPU. This in turn schedules qman_delete_cgr() on the proper CPU. Signed-off-by: Madalin Bucur <madalin.bucur@freescale.com> Change-Id: I762e83108533a4e537a534e90073df26a6b7b09c Reviewed-on: http://git.am.freescale.net:8181/28532 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Marian Cristian Rotariu <marian.rotariu@freescale.com>
2015-02-13Merge branch 'rtmerge'Scott Wood
Signed-off-by: Scott Wood <scottwood@freescale.com> Conflicts: arch/arm/kvm/mmu.c arch/arm/mm/proc-v7-3level.S arch/powerpc/kernel/vdso32/getcpu.S drivers/crypto/caam/error.c drivers/crypto/caam/sg_sw_sec4.h drivers/usb/host/ehci-fsl.c
2015-02-13Reset to 3.12.37Scott Wood
2014-12-11dpa_offload: Update P2041RDB shared interfaces DTB fileMarian Chereji
Resync P2041RDB DPAA offloading shared interfaces DTB file with the latest changes in the usdpaa shared interfaces DTB file. Some shared interface parameters were missing. Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Change-Id: Ib91ea04fd482353f48e012971e6196f7201e9fac Reviewed-on: http://git.am.freescale.net:8181/25435 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Bogdan Constantin Popescu <bogdan.c.popescu@freescale.com> Reviewed-by: Matthew Weigel <Matthew.Weigel@freescale.com>
2014-12-11fsl_qman: use do_div in prescaler calculationHaiying Wang
Signed-off-by: Haiying Wang <Haiying.Wang@freescale.com> Change-Id: I103b9031feaa464197d816d1a11a6e8204219f9b Reviewed-on: http://git.am.freescale.net:8181/25397 Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Matthew Weigel <Matthew.Weigel@freescale.com>
2014-12-11Reserve proper length in processes VMA when mapping USDPAA memoryRoy Pledge
USDPAA memory mapping were not properly reserving the entire length of the memory area when perfoming mapping of not power of 4 regions. This causes issues if the choosen address overlapped with an existing memory area. Signed-off-by: Roy Pledge <Roy.Pledge@freescale.com> Change-Id: I078d12279a43ddf3fc7c5de4b9e4a932d588832e Reviewed-on: http://git.am.freescale.net:8181/25329 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Matthew Weigel <Matthew.Weigel@freescale.com>
2014-12-11qbman: Add bman version check when reading idle registerJeffrey Ladouceur
The STATE_IDLE register was added to BMan hw version 2.1.0 Anything earlier than this will return undefined value. This is required in order to not abort suspend phase due to false information Change-Id: I055b37571fc7d500018e8a736cbfffc8e5920f3f Signed-off-by: Jeffrey Ladouceur <Jeffrey.Ladouceur@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/24834 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Matthew Weigel <Matthew.Weigel@freescale.com>
2014-12-11dpa_offload: sync with CAAM changesAlex Porosanu
The CAAM driver function caam_jr_strstatus() has changed its usage, and now it performs the interpretation and printing of the error itself. This patch syncs dpa_offload's sources with the updated version. Change-Id: I5b7b04386dbc03c11e3277adb77af985cc6418ca Signed-off-by: Alex Porosanu <alexandru.porosanu@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/24679 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Cristian Stoica <cristian.stoica@freescale.com> Reviewed-by: Marian-Cornel Chereji <marian.chereji@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com>
2014-12-11Disable CPU migration during USDPAA CleanupRoy Pledge
CPU specific variables are used during USDPAA cleanup. Disabling CPU migration is needed to avoid the cleanup process from being moved to another CPU (especially when using the RT preemption models) Change-Id: If0133117fcec82f69e6123cb4da3167363b0dad3 Signed-off-by: Roy Pledge <Roy.Pledge@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/24105 Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com>
2014-12-11qbman: Corrections based on static analysisAhmed Mansour
ioctl_dma_map. String length check Added check to ensure the string copied from user space is within the length allowed for by the buffer and is null terminated. qm_shutdown_fq. Remove deadcode Removed the case statement label for QM_MCR_NP_STATE_OOS since that condition is checked for earlier in the code and in that case the function exits immediately. ioctl_dma_lock. Remove NULL dereference A pointer is made null and then there is an attempt to dereference one of the members of the struct that it points to. This condition is never reached normally, but in error conditions this path of execution may be possible. Added return -EFAULT to exit function safely and unlocked locked resources usdpaa_get_unmapped_area. Arithmetic overflow protection Added a check to ensure that memory length passed in does not result in an invalid size to be passed because of a possible arithmetic overflow that can occur in the subtraction in the macro USDPAA_MEM_ROUNDUP. qman_ceetm_ccg_claim. If condition fix Removed a check against an unsigned int value that checks if it is below zero (negative). The unsigned value cannot be negative and the compare will always fail in reality. qman_create_portal. Add comment to clarify flow Clarified why no check is made to ensure that qm_mr_current() returned successfully. The addition of a redundant check was rejected because it impairs the reader's understanding of the flow by implying the possibility of a physically impossible path. dpa_alloc_new. Fix memory leak in error conditions Previously allocated memory which is passed by reference in success (list_add_tail) is not freed in error conditions that occur in other parts of the function. Added code to free allocated memory in error cases. qman_create_portal. Fix memory leak in error path Added cleanup code that deletes malloced memory in case of function failure after memory is allocated. ioctl_dma_map. Input bound checks on values from USDPAA Check the struct usdpaa_ioctl_dma_map, input, to ensure that values copied from user space are valid input. qman_init. Check init_pcfg return Previously unused assignment is now being used. The return of init_pcfg() was not checked for success. Now we check for success and print an error message in case of failure. There is a similar issue in bman_driver. bman_init. Check init_pcfg return Added check to ensure init_pcfg() is returning successfully. Print an error message and return if in error case. There is a similar issue in qman_driver. Signed-off-by: Ahmed Mansour <Ahmed.Mansour@freescale.com> Change-Id: I230678948c3e8e43aa5738045244c5dac0b5f5f5 Reviewed-on: http://git.am.freescale.net:8181/21381 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Jeffrey Ladouceur <Jeffrey.Ladouceur@freescale.com> Reviewed-by: Matthew Weigel <Matthew.Weigel@freescale.com>
2014-12-11Set affinity for USDPAA IRQsRoy Pledge
Set the affinity of the USDPAA IRQ to match the affinity of the calling process Signed-off-by: Roy Pledge <Roy.Pledge@freescale.com> Change-Id: I077a48ae5a77fe35a77dfae65c298fa6fa2aecda Reviewed-on: http://git.am.freescale.net:8181/23675 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Jeffrey Ladouceur <Jeffrey.Ladouceur@freescale.com> Reviewed-by: Matthew Weigel <Matthew.Weigel@freescale.com>
2014-12-11dpa_offload: use SEC API for retrieving JRAlex Porosanu
SEC driver offers a function for geting the handle to an available JR (caam_jr_alloc(...) This is mainly needed for split-key calculation per SA. There's no need to search for the JR in the device tree. This patch removes the unneeded code and replaces it with the proper SEC API call. Change-Id: Ibc8b4b216d5699fe5ec7d3d49b8a3560ab79ede0 Signed-off-by: Alex Porosanu <alexandru.porosanu@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/23860 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Marian-Cornel Chereji <marian.chereji@freescale.com> Reviewed-by: Horia Ioan Geanta Neag <horia.geanta@freescale.com> Reviewed-by: Matthew Weigel <Matthew.Weigel@freescale.com>
2014-12-11dpa_offload: Relocate dpa_classifier entry in index management listMarian Chereji
For HASH tables sometimes when a user modifies the key of an entry the entry needs to be moved to a different bucket. The index management list is sorted so that indexes are always ascending. When tempering with the position of an entry in the internal Cc nodes for a HASH table, the position of the entry in the index management list must also be updated. Change-Id: If4b48282c407ed248dfd389cc0c8ae856fb7db75 Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/23700 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Radu-Andrei Bulie <Radu.Bulie@freescale.com> Reviewed-by: Matthew Weigel <Matthew.Weigel@freescale.com>
2014-12-11dce: validate input to test moduleJeffrey Ladouceur
The dce_simple_perf_tester module needs to validate input parameter. Signed-off-by: Jeffrey Ladouceur <Jeffrey.Ladouceur@freescale.com> Change-Id: Ie0dddfb8e23532895e6ec2276f201e3b555fed62 Reviewed-on: http://git.am.freescale.net:8181/22952 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Roy Pledge <roy.pledge@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com>
2014-12-11dpa_offload: Fix memory leak in copy_key_descriptor_compatcpyAnca Jeanina Floarea
In case either the byte or the mask of a key are provided NULL, the memory allocated for the lookup key descriptor needs to be release. Signed-off-by: Anca Jeanina Floarea <anca.floarea@freescale.com> Change-Id: Ibb9b7ec28a93a87c82ab318b763866cf3d9f493a Reviewed-on: http://git.am.freescale.net:8181/23131 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Marian-Cornel Chereji <marian.chereji@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com>
2014-12-11dpa_offload: Avoid dereferencing a NULL pointer in case of corrupted user paramsMarian Chereji
In the dpa_classifier function "try_compatible_node" there are some places where the header manipulation operation flags are compared to the configuration of an existing header manipulation node. The control path however doesn't suspend in case an inconsistency is found in the flags and, in some rare cases when users provide corrupt parameters, there is a danger to dereference a NULL hm_node pointer. Change-Id: I355862c146a228e5878272156adaf92a76391ce6 Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/23111 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Anca Jeanina Floarea <AncaJeanina.Floarea@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com>
2014-12-11dpa_offload: Avoid double free of replicator group params in case of errorMarian Chereji
In dpa_classifier function "dpa_classif_mcast_create_group" the replicator group params could be freed twice if the function "FM_PCD_FrmReplicSetGroup" fails, for instance. The pointer is now reset at first free so that the second free operation ignores it. Change-Id: I8869db05157f68964bd2a131f6e184f78e64cf8c Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/23108 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Radu-Andrei Bulie <Radu.Bulie@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com>
2014-12-11fsl_qbman: correct the lni shaper disable functionHaiying Wang
LNI shaper is disabled by setting all 1's to both CR/ER Signed-off-by: Haiying Wang <Haiying.Wang@freescale.com> Change-Id: I7b680ecbcfcfa41860dddcca34044116719cda08 Reviewed-on: http://git.am.freescale.net:8181/22075 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com>
2014-12-11fsl_qbman: change the ceetm shaping speed bps to be u64 typedHaiying Wang
The original u32 define limited the ceetm shaping speed. Signed-off-by: Haiying Wang <Haiying.Wang@freescale.com> Change-Id: I02fe21c541e5c78151b181b8ab75cc84d019661d Reviewed-on: http://git.am.freescale.net:8181/22074 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com>
2014-12-11qbman: Add check that suspend will succeedJeffrey Ladouceur
Ensure that idle line is asserted before leaving suspend function. If not, fail the request as system could hang waiting for idle line to be asserted which might never happen. Signed-off-by: Jeffrey Ladouceur <Jeffrey.Ladouceur@freescale.com> Change-Id: I95390751160b8825d58d9c04a6884b37e91424b4 Reviewed-on: http://git.am.freescale.net:8181/21956 Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com> Reviewed-by: Geoff Thorpe <Geoff.Thorpe@freescale.com> Reviewed-by: Haiying Wang <Haiying.Wang@freescale.com> Reviewed-by: Richard Schmitt <richard.schmitt@freescale.com>
2014-12-11dpa-offload Linux Kernel Driver Updates for SDK 1.7Rich Schmitt
Merge remote-tracking branch 'dpaa/dpa-offload-devel'
2014-12-11dpa_offload: Fill in proper VSP parameters when policer is enabledMarian Chereji
There are 2 different VSP profile parameters in the FMD API that need to be filled in depending whether the policer is enabled or disabled. The dpa_classifier was filling in the enqueue VSP profile all the time, but this was used only when the next action was "enqueue". When the next action is "policer" the policer VSP profile must be filled in. Change-Id: I09e2a6fad7216e7ce7b3c1190a4441b7a9825705 Signed-off-by: Marian Chereji <marian.chereji@freescale.com> Reviewed-on: http://git.am.freescale.net:8181/22064 Reviewed-by: Francois Massot-Pellet <Francois.Massot-Pellet@freescale.com>