On Wednesday 28 June 2017 09:17 PM, Ferruh Yigit wrote:
> On 6/16/2017 6:41 AM, Shreyansh Jain wrote:
>> Signed-off-by: Hemant Agrawal <hemant.agra...@nxp.com>
>> Signed-off-by: Shreyansh Jain <shreyansh.j...@nxp.com>
>
> <...>
>
>> static int
>> +dpaa_flow_ctrl_set(struct rte_eth_dev *dev,
>> + struct rte_eth_fc_conf *fc_conf)
>> +{
>> + struct dpaa_if *dpaa_intf = dev->data->dev_private;
>> + struct rte_eth_fc_conf *net_fc;
>> +
>> + PMD_INIT_FUNC_TRACE();
>> +
>> + if (!(dpaa_intf->fc_conf)) {
>> + dpaa_intf->fc_conf = rte_zmalloc(NULL,
>> + sizeof(struct rte_eth_fc_conf), MAX_CACHELINE);
>
> Should this be freed in rte_dpaa_remove()
Will fix this in v2.
>
>> + if (!dpaa_intf->fc_conf) {
>> + PMD_DRV_LOG(ERR, "unable to save flow control info");
>> + return -ENOMEM;
>> + }
>> + }
>> + net_fc = dpaa_intf->fc_conf;
>> +
> <...>
>
>
