7bbc7dc431
This patch fixes 'maybe-uninitialized' warnings reported by compiler
when using LTO.
Compiler warning pointing to this error (with LTO enabled):
error: ‘kg_cfg.extracts[0].masks[0].mask’ may be used uninitialized in
this function [-Werror=maybe-uninitialized]
extr->masks[j].mask = cfg->extracts[i].masks[j].mask;
Fixes: 16bbc98a3e ("bus/fslmc: update MC to 10.3.x")
Cc: stable@dpdk.org
Signed-off-by: Andrzej Ostruszka <aostruszka@marvell.com>
78 lines
2.1 KiB
C
78 lines
2.1 KiB
C
/* SPDX-License-Identifier: (BSD-3-Clause OR GPL-2.0)
|
|
*
|
|
* Copyright 2017 NXP
|
|
*
|
|
*/
|
|
#include <fsl_mc_sys.h>
|
|
#include <fsl_mc_cmd.h>
|
|
#include <fsl_dpkg.h>
|
|
|
|
/**
|
|
* dpkg_prepare_key_cfg() - function prepare extract parameters
|
|
* @cfg: defining a full Key Generation profile (rule)
|
|
* @key_cfg_buf: Zeroed 256 bytes of memory before mapping it to DMA
|
|
*
|
|
* This function has to be called before the following functions:
|
|
* - dpni_set_rx_tc_dist()
|
|
* - dpni_set_qos_table()
|
|
* - dpkg_prepare_key_cfg()
|
|
*/
|
|
int
|
|
dpkg_prepare_key_cfg(const struct dpkg_profile_cfg *cfg, uint8_t *key_cfg_buf)
|
|
{
|
|
int i, j;
|
|
struct dpni_ext_set_rx_tc_dist *dpni_ext;
|
|
struct dpni_dist_extract *extr;
|
|
|
|
if (cfg->num_extracts > DPKG_MAX_NUM_OF_EXTRACTS)
|
|
return -EINVAL;
|
|
|
|
dpni_ext = (struct dpni_ext_set_rx_tc_dist *)key_cfg_buf;
|
|
dpni_ext->num_extracts = cfg->num_extracts;
|
|
|
|
for (i = 0; i < cfg->num_extracts; i++) {
|
|
extr = &dpni_ext->extracts[i];
|
|
|
|
switch (cfg->extracts[i].type) {
|
|
case DPKG_EXTRACT_FROM_HDR:
|
|
extr->prot = cfg->extracts[i].extract.from_hdr.prot;
|
|
dpkg_set_field(extr->efh_type, EFH_TYPE,
|
|
cfg->extracts[i].extract.from_hdr.type);
|
|
extr->size = cfg->extracts[i].extract.from_hdr.size;
|
|
extr->offset = cfg->extracts[i].extract.from_hdr.offset;
|
|
extr->field = cpu_to_le32(
|
|
cfg->extracts[i].extract.from_hdr.field);
|
|
extr->hdr_index =
|
|
cfg->extracts[i].extract.from_hdr.hdr_index;
|
|
break;
|
|
case DPKG_EXTRACT_FROM_DATA:
|
|
extr->size = cfg->extracts[i].extract.from_data.size;
|
|
extr->offset =
|
|
cfg->extracts[i].extract.from_data.offset;
|
|
break;
|
|
case DPKG_EXTRACT_FROM_PARSE:
|
|
extr->size = cfg->extracts[i].extract.from_parse.size;
|
|
extr->offset =
|
|
cfg->extracts[i].extract.from_parse.offset;
|
|
break;
|
|
default:
|
|
return -EINVAL;
|
|
}
|
|
|
|
extr->num_of_byte_masks = cfg->extracts[i].num_of_byte_masks;
|
|
dpkg_set_field(extr->extract_type, EXTRACT_TYPE,
|
|
cfg->extracts[i].type);
|
|
|
|
if (extr->num_of_byte_masks > DPKG_NUM_OF_MASKS)
|
|
return -EINVAL;
|
|
|
|
for (j = 0; j < extr->num_of_byte_masks; j++) {
|
|
extr->masks[j].mask = cfg->extracts[i].masks[j].mask;
|
|
extr->masks[j].offset =
|
|
cfg->extracts[i].masks[j].offset;
|
|
}
|
|
}
|
|
|
|
return 0;
|
|
}
|