Age | Commit message (Collapse) | Author |
|
~R01356/linux-devel:bugfix/QUSDPA-704-segfault-in-dma_mem-allocator.c to master
* commit 'e98f36e86bdfbeb1728ab0de4f2be367d6283963':
fsl_usdpaa: Remove TLB entries when unmapping memory
|
|
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>
|
|
Eliminate memory leaks on error paths
Signed-off-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com>
Change-Id: I36d1a35f9d48188a9dad043c78e90d787997ea19
|
|
Check for null pointers to prevent null pointer dereferencing
Signed-off-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com>
Change-Id: Icc267f51d5ad85187c6ee5d03aa87f8649e124c9
|
|
Check for memory allocation failures to protect from
null pointer dereferencing
Signed-off-by: Mahammad Ismayilzada <mahammad.ismayilzada@freescale.com>
|
|
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>
|
|
For JIRA issue QLINUX-3837
Signed-off-by: Hemant Agrawal <Hemant@freescale.com>
Signed-off-by: Haiying Wang <Haiying.wang@freescale.com>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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
|
|
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>
|
|
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>
|
|
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>
|
|
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
|
|
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
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>
|
|
Merge remote-tracking branch 'dpaa/dpa-offload-devel'
|
|
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>
|