123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511 |
- // SPDX-License-Identifier: GPL-2.0-only
- /****************************************************************************
- * Driver for Solarflare network controllers and boards
- * Copyright 2019 Solarflare Communications Inc.
- * Copyright 2020-2022 Xilinx Inc.
- *
- * This program is free software; you can redistribute it and/or modify it
- * under the terms of the GNU General Public License version 2 as published
- * by the Free Software Foundation, incorporated herein by reference.
- */
- #include "mae.h"
- #include "mcdi.h"
- #include "mcdi_pcol_mae.h"
- int efx_mae_allocate_mport(struct efx_nic *efx, u32 *id, u32 *label)
- {
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_MPORT_ALLOC_ALIAS_OUT_LEN);
- MCDI_DECLARE_BUF(inbuf, MC_CMD_MAE_MPORT_ALLOC_ALIAS_IN_LEN);
- size_t outlen;
- int rc;
- if (WARN_ON_ONCE(!id))
- return -EINVAL;
- if (WARN_ON_ONCE(!label))
- return -EINVAL;
- MCDI_SET_DWORD(inbuf, MAE_MPORT_ALLOC_ALIAS_IN_TYPE,
- MC_CMD_MAE_MPORT_ALLOC_ALIAS_IN_MPORT_TYPE_ALIAS);
- MCDI_SET_DWORD(inbuf, MAE_MPORT_ALLOC_ALIAS_IN_DELIVER_MPORT,
- MAE_MPORT_SELECTOR_ASSIGNED);
- rc = efx_mcdi_rpc(efx, MC_CMD_MAE_MPORT_ALLOC, inbuf, sizeof(inbuf),
- outbuf, sizeof(outbuf), &outlen);
- if (rc)
- return rc;
- if (outlen < sizeof(outbuf))
- return -EIO;
- *id = MCDI_DWORD(outbuf, MAE_MPORT_ALLOC_ALIAS_OUT_MPORT_ID);
- *label = MCDI_DWORD(outbuf, MAE_MPORT_ALLOC_ALIAS_OUT_LABEL);
- return 0;
- }
- int efx_mae_free_mport(struct efx_nic *efx, u32 id)
- {
- MCDI_DECLARE_BUF(inbuf, MC_CMD_MAE_MPORT_FREE_IN_LEN);
- BUILD_BUG_ON(MC_CMD_MAE_MPORT_FREE_OUT_LEN);
- MCDI_SET_DWORD(inbuf, MAE_MPORT_FREE_IN_MPORT_ID, id);
- return efx_mcdi_rpc(efx, MC_CMD_MAE_MPORT_FREE, inbuf, sizeof(inbuf),
- NULL, 0, NULL);
- }
- void efx_mae_mport_wire(struct efx_nic *efx, u32 *out)
- {
- efx_dword_t mport;
- EFX_POPULATE_DWORD_2(mport,
- MAE_MPORT_SELECTOR_TYPE, MAE_MPORT_SELECTOR_TYPE_PPORT,
- MAE_MPORT_SELECTOR_PPORT_ID, efx->port_num);
- *out = EFX_DWORD_VAL(mport);
- }
- void efx_mae_mport_uplink(struct efx_nic *efx __always_unused, u32 *out)
- {
- efx_dword_t mport;
- EFX_POPULATE_DWORD_3(mport,
- MAE_MPORT_SELECTOR_TYPE, MAE_MPORT_SELECTOR_TYPE_FUNC,
- MAE_MPORT_SELECTOR_FUNC_PF_ID, MAE_MPORT_SELECTOR_FUNC_PF_ID_CALLER,
- MAE_MPORT_SELECTOR_FUNC_VF_ID, MAE_MPORT_SELECTOR_FUNC_VF_ID_NULL);
- *out = EFX_DWORD_VAL(mport);
- }
- void efx_mae_mport_vf(struct efx_nic *efx __always_unused, u32 vf_id, u32 *out)
- {
- efx_dword_t mport;
- EFX_POPULATE_DWORD_3(mport,
- MAE_MPORT_SELECTOR_TYPE, MAE_MPORT_SELECTOR_TYPE_FUNC,
- MAE_MPORT_SELECTOR_FUNC_PF_ID, MAE_MPORT_SELECTOR_FUNC_PF_ID_CALLER,
- MAE_MPORT_SELECTOR_FUNC_VF_ID, vf_id);
- *out = EFX_DWORD_VAL(mport);
- }
- /* Constructs an mport selector from an mport ID, because they're not the same */
- void efx_mae_mport_mport(struct efx_nic *efx __always_unused, u32 mport_id, u32 *out)
- {
- efx_dword_t mport;
- EFX_POPULATE_DWORD_2(mport,
- MAE_MPORT_SELECTOR_TYPE, MAE_MPORT_SELECTOR_TYPE_MPORT_ID,
- MAE_MPORT_SELECTOR_MPORT_ID, mport_id);
- *out = EFX_DWORD_VAL(mport);
- }
- /* id is really only 24 bits wide */
- int efx_mae_lookup_mport(struct efx_nic *efx, u32 selector, u32 *id)
- {
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_MPORT_LOOKUP_OUT_LEN);
- MCDI_DECLARE_BUF(inbuf, MC_CMD_MAE_MPORT_LOOKUP_IN_LEN);
- size_t outlen;
- int rc;
- MCDI_SET_DWORD(inbuf, MAE_MPORT_LOOKUP_IN_MPORT_SELECTOR, selector);
- rc = efx_mcdi_rpc(efx, MC_CMD_MAE_MPORT_LOOKUP, inbuf, sizeof(inbuf),
- outbuf, sizeof(outbuf), &outlen);
- if (rc)
- return rc;
- if (outlen < sizeof(outbuf))
- return -EIO;
- *id = MCDI_DWORD(outbuf, MAE_MPORT_LOOKUP_OUT_MPORT_ID);
- return 0;
- }
- static int efx_mae_get_basic_caps(struct efx_nic *efx, struct mae_caps *caps)
- {
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_GET_CAPS_OUT_LEN);
- size_t outlen;
- int rc;
- BUILD_BUG_ON(MC_CMD_MAE_GET_CAPS_IN_LEN);
- rc = efx_mcdi_rpc(efx, MC_CMD_MAE_GET_CAPS, NULL, 0, outbuf,
- sizeof(outbuf), &outlen);
- if (rc)
- return rc;
- if (outlen < sizeof(outbuf))
- return -EIO;
- caps->match_field_count = MCDI_DWORD(outbuf, MAE_GET_CAPS_OUT_MATCH_FIELD_COUNT);
- caps->action_prios = MCDI_DWORD(outbuf, MAE_GET_CAPS_OUT_ACTION_PRIOS);
- return 0;
- }
- static int efx_mae_get_rule_fields(struct efx_nic *efx, u32 cmd,
- u8 *field_support)
- {
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_GET_AR_CAPS_OUT_LEN(MAE_NUM_FIELDS));
- MCDI_DECLARE_STRUCT_PTR(caps);
- unsigned int count;
- size_t outlen;
- int rc, i;
- BUILD_BUG_ON(MC_CMD_MAE_GET_AR_CAPS_IN_LEN);
- rc = efx_mcdi_rpc(efx, cmd, NULL, 0, outbuf, sizeof(outbuf), &outlen);
- if (rc)
- return rc;
- count = MCDI_DWORD(outbuf, MAE_GET_AR_CAPS_OUT_COUNT);
- memset(field_support, MAE_FIELD_UNSUPPORTED, MAE_NUM_FIELDS);
- caps = _MCDI_DWORD(outbuf, MAE_GET_AR_CAPS_OUT_FIELD_FLAGS);
- /* We're only interested in the support status enum, not any other
- * flags, so just extract that from each entry.
- */
- for (i = 0; i < count; i++)
- if (i * sizeof(*outbuf) + MC_CMD_MAE_GET_AR_CAPS_OUT_FIELD_FLAGS_OFST < outlen)
- field_support[i] = EFX_DWORD_FIELD(caps[i], MAE_FIELD_FLAGS_SUPPORT_STATUS);
- return 0;
- }
- int efx_mae_get_caps(struct efx_nic *efx, struct mae_caps *caps)
- {
- int rc;
- rc = efx_mae_get_basic_caps(efx, caps);
- if (rc)
- return rc;
- return efx_mae_get_rule_fields(efx, MC_CMD_MAE_GET_AR_CAPS,
- caps->action_rule_fields);
- }
- /* Bit twiddling:
- * Prefix: 1...110...0
- * ~: 0...001...1
- * + 1: 0...010...0 is power of two
- * so (~x) & ((~x) + 1) == 0. Converse holds also.
- */
- #define is_prefix_byte(_x) !(((_x) ^ 0xff) & (((_x) ^ 0xff) + 1))
- enum mask_type { MASK_ONES, MASK_ZEROES, MASK_PREFIX, MASK_OTHER };
- static const char *mask_type_name(enum mask_type typ)
- {
- switch (typ) {
- case MASK_ONES:
- return "all-1s";
- case MASK_ZEROES:
- return "all-0s";
- case MASK_PREFIX:
- return "prefix";
- case MASK_OTHER:
- return "arbitrary";
- default: /* can't happen */
- return "unknown";
- }
- }
- /* Checks a (big-endian) bytestring is a bit prefix */
- static enum mask_type classify_mask(const u8 *mask, size_t len)
- {
- bool zeroes = true; /* All bits seen so far are zeroes */
- bool ones = true; /* All bits seen so far are ones */
- bool prefix = true; /* Valid prefix so far */
- size_t i;
- for (i = 0; i < len; i++) {
- if (ones) {
- if (!is_prefix_byte(mask[i]))
- prefix = false;
- } else if (mask[i]) {
- prefix = false;
- }
- if (mask[i] != 0xff)
- ones = false;
- if (mask[i])
- zeroes = false;
- }
- if (ones)
- return MASK_ONES;
- if (zeroes)
- return MASK_ZEROES;
- if (prefix)
- return MASK_PREFIX;
- return MASK_OTHER;
- }
- static int efx_mae_match_check_cap_typ(u8 support, enum mask_type typ)
- {
- switch (support) {
- case MAE_FIELD_UNSUPPORTED:
- case MAE_FIELD_SUPPORTED_MATCH_NEVER:
- if (typ == MASK_ZEROES)
- return 0;
- return -EOPNOTSUPP;
- case MAE_FIELD_SUPPORTED_MATCH_OPTIONAL:
- if (typ == MASK_ZEROES)
- return 0;
- fallthrough;
- case MAE_FIELD_SUPPORTED_MATCH_ALWAYS:
- if (typ == MASK_ONES)
- return 0;
- return -EINVAL;
- case MAE_FIELD_SUPPORTED_MATCH_PREFIX:
- if (typ == MASK_OTHER)
- return -EOPNOTSUPP;
- return 0;
- case MAE_FIELD_SUPPORTED_MATCH_MASK:
- return 0;
- default:
- return -EIO;
- }
- }
- int efx_mae_match_check_caps(struct efx_nic *efx,
- const struct efx_tc_match_fields *mask,
- struct netlink_ext_ack *extack)
- {
- const u8 *supported_fields = efx->tc->caps->action_rule_fields;
- __be32 ingress_port = cpu_to_be32(mask->ingress_port);
- enum mask_type ingress_port_mask_type;
- int rc;
- /* Check for _PREFIX assumes big-endian, so we need to convert */
- ingress_port_mask_type = classify_mask((const u8 *)&ingress_port,
- sizeof(ingress_port));
- rc = efx_mae_match_check_cap_typ(supported_fields[MAE_FIELD_INGRESS_PORT],
- ingress_port_mask_type);
- if (rc) {
- efx_tc_err(efx, "No support for %s mask in field ingress_port\n",
- mask_type_name(ingress_port_mask_type));
- NL_SET_ERR_MSG_MOD(extack, "Unsupported mask type for ingress_port");
- return rc;
- }
- return 0;
- }
- static bool efx_mae_asl_id(u32 id)
- {
- return !!(id & BIT(31));
- }
- int efx_mae_alloc_action_set(struct efx_nic *efx, struct efx_tc_action_set *act)
- {
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_ACTION_SET_ALLOC_OUT_LEN);
- MCDI_DECLARE_BUF(inbuf, MC_CMD_MAE_ACTION_SET_ALLOC_IN_LEN);
- size_t outlen;
- int rc;
- MCDI_SET_DWORD(inbuf, MAE_ACTION_SET_ALLOC_IN_SRC_MAC_ID,
- MC_CMD_MAE_MAC_ADDR_ALLOC_OUT_MAC_ID_NULL);
- MCDI_SET_DWORD(inbuf, MAE_ACTION_SET_ALLOC_IN_DST_MAC_ID,
- MC_CMD_MAE_MAC_ADDR_ALLOC_OUT_MAC_ID_NULL);
- MCDI_SET_DWORD(inbuf, MAE_ACTION_SET_ALLOC_IN_COUNTER_ID,
- MC_CMD_MAE_COUNTER_ALLOC_OUT_COUNTER_ID_NULL);
- MCDI_SET_DWORD(inbuf, MAE_ACTION_SET_ALLOC_IN_COUNTER_LIST_ID,
- MC_CMD_MAE_COUNTER_LIST_ALLOC_OUT_COUNTER_LIST_ID_NULL);
- MCDI_SET_DWORD(inbuf, MAE_ACTION_SET_ALLOC_IN_ENCAP_HEADER_ID,
- MC_CMD_MAE_ENCAP_HEADER_ALLOC_OUT_ENCAP_HEADER_ID_NULL);
- if (act->deliver)
- MCDI_SET_DWORD(inbuf, MAE_ACTION_SET_ALLOC_IN_DELIVER,
- act->dest_mport);
- BUILD_BUG_ON(MAE_MPORT_SELECTOR_NULL);
- rc = efx_mcdi_rpc(efx, MC_CMD_MAE_ACTION_SET_ALLOC, inbuf, sizeof(inbuf),
- outbuf, sizeof(outbuf), &outlen);
- if (rc)
- return rc;
- if (outlen < sizeof(outbuf))
- return -EIO;
- act->fw_id = MCDI_DWORD(outbuf, MAE_ACTION_SET_ALLOC_OUT_AS_ID);
- /* We rely on the high bit of AS IDs always being clear.
- * The firmware API guarantees this, but let's check it ourselves.
- */
- if (WARN_ON_ONCE(efx_mae_asl_id(act->fw_id))) {
- efx_mae_free_action_set(efx, act->fw_id);
- return -EIO;
- }
- return 0;
- }
- int efx_mae_free_action_set(struct efx_nic *efx, u32 fw_id)
- {
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_ACTION_SET_FREE_OUT_LEN(1));
- MCDI_DECLARE_BUF(inbuf, MC_CMD_MAE_ACTION_SET_FREE_IN_LEN(1));
- size_t outlen;
- int rc;
- MCDI_SET_DWORD(inbuf, MAE_ACTION_SET_FREE_IN_AS_ID, fw_id);
- rc = efx_mcdi_rpc(efx, MC_CMD_MAE_ACTION_SET_FREE, inbuf, sizeof(inbuf),
- outbuf, sizeof(outbuf), &outlen);
- if (rc)
- return rc;
- if (outlen < sizeof(outbuf))
- return -EIO;
- /* FW freed a different ID than we asked for, should never happen.
- * Warn because it means we've now got a different idea to the FW of
- * what action-sets exist, which could cause mayhem later.
- */
- if (WARN_ON(MCDI_DWORD(outbuf, MAE_ACTION_SET_FREE_OUT_FREED_AS_ID) != fw_id))
- return -EIO;
- return 0;
- }
- int efx_mae_alloc_action_set_list(struct efx_nic *efx,
- struct efx_tc_action_set_list *acts)
- {
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_ACTION_SET_LIST_ALLOC_OUT_LEN);
- struct efx_tc_action_set *act;
- size_t inlen, outlen, i = 0;
- efx_dword_t *inbuf;
- int rc;
- list_for_each_entry(act, &acts->list, list)
- i++;
- if (i == 0)
- return -EINVAL;
- if (i == 1) {
- /* Don't wrap an ASL around a single AS, just use the AS_ID
- * directly. ASLs are a more limited resource.
- */
- act = list_first_entry(&acts->list, struct efx_tc_action_set, list);
- acts->fw_id = act->fw_id;
- return 0;
- }
- if (i > MC_CMD_MAE_ACTION_SET_LIST_ALLOC_IN_AS_IDS_MAXNUM_MCDI2)
- return -EOPNOTSUPP; /* Too many actions */
- inlen = MC_CMD_MAE_ACTION_SET_LIST_ALLOC_IN_LEN(i);
- inbuf = kzalloc(inlen, GFP_KERNEL);
- if (!inbuf)
- return -ENOMEM;
- i = 0;
- list_for_each_entry(act, &acts->list, list) {
- MCDI_SET_ARRAY_DWORD(inbuf, MAE_ACTION_SET_LIST_ALLOC_IN_AS_IDS,
- i, act->fw_id);
- i++;
- }
- MCDI_SET_DWORD(inbuf, MAE_ACTION_SET_LIST_ALLOC_IN_COUNT, i);
- rc = efx_mcdi_rpc(efx, MC_CMD_MAE_ACTION_SET_LIST_ALLOC, inbuf, inlen,
- outbuf, sizeof(outbuf), &outlen);
- if (rc)
- goto out_free;
- if (outlen < sizeof(outbuf)) {
- rc = -EIO;
- goto out_free;
- }
- acts->fw_id = MCDI_DWORD(outbuf, MAE_ACTION_SET_LIST_ALLOC_OUT_ASL_ID);
- /* We rely on the high bit of ASL IDs always being set.
- * The firmware API guarantees this, but let's check it ourselves.
- */
- if (WARN_ON_ONCE(!efx_mae_asl_id(acts->fw_id))) {
- efx_mae_free_action_set_list(efx, acts);
- rc = -EIO;
- }
- out_free:
- kfree(inbuf);
- return rc;
- }
- int efx_mae_free_action_set_list(struct efx_nic *efx,
- struct efx_tc_action_set_list *acts)
- {
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_ACTION_SET_LIST_FREE_OUT_LEN(1));
- MCDI_DECLARE_BUF(inbuf, MC_CMD_MAE_ACTION_SET_LIST_FREE_IN_LEN(1));
- size_t outlen;
- int rc;
- /* If this is just an AS_ID with no ASL wrapper, then there is
- * nothing for us to free. (The AS will be freed later.)
- */
- if (efx_mae_asl_id(acts->fw_id)) {
- MCDI_SET_DWORD(inbuf, MAE_ACTION_SET_LIST_FREE_IN_ASL_ID,
- acts->fw_id);
- rc = efx_mcdi_rpc(efx, MC_CMD_MAE_ACTION_SET_LIST_FREE, inbuf,
- sizeof(inbuf), outbuf, sizeof(outbuf), &outlen);
- if (rc)
- return rc;
- if (outlen < sizeof(outbuf))
- return -EIO;
- /* FW freed a different ID than we asked for, should never happen.
- * Warn because it means we've now got a different idea to the FW of
- * what action-set-lists exist, which could cause mayhem later.
- */
- if (WARN_ON(MCDI_DWORD(outbuf, MAE_ACTION_SET_LIST_FREE_OUT_FREED_ASL_ID) != acts->fw_id))
- return -EIO;
- }
- /* We're probably about to free @acts, but let's just make sure its
- * fw_id is blatted so that it won't look valid if it leaks out.
- */
- acts->fw_id = MC_CMD_MAE_ACTION_SET_LIST_ALLOC_OUT_ACTION_SET_LIST_ID_NULL;
- return 0;
- }
- static int efx_mae_populate_match_criteria(MCDI_DECLARE_STRUCT_PTR(match_crit),
- const struct efx_tc_match *match)
- {
- if (match->mask.ingress_port) {
- if (~match->mask.ingress_port)
- return -EOPNOTSUPP;
- MCDI_STRUCT_SET_DWORD(match_crit,
- MAE_FIELD_MASK_VALUE_PAIRS_V2_INGRESS_MPORT_SELECTOR,
- match->value.ingress_port);
- }
- MCDI_STRUCT_SET_DWORD(match_crit, MAE_FIELD_MASK_VALUE_PAIRS_V2_INGRESS_MPORT_SELECTOR_MASK,
- match->mask.ingress_port);
- MCDI_STRUCT_SET_BYTE(match_crit, MAE_FIELD_MASK_VALUE_PAIRS_V2_RECIRC_ID,
- match->value.recirc_id);
- MCDI_STRUCT_SET_BYTE(match_crit, MAE_FIELD_MASK_VALUE_PAIRS_V2_RECIRC_ID_MASK,
- match->mask.recirc_id);
- return 0;
- }
- int efx_mae_insert_rule(struct efx_nic *efx, const struct efx_tc_match *match,
- u32 prio, u32 acts_id, u32 *id)
- {
- MCDI_DECLARE_BUF(inbuf, MC_CMD_MAE_ACTION_RULE_INSERT_IN_LEN(MAE_FIELD_MASK_VALUE_PAIRS_V2_LEN));
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_ACTION_RULE_INSERT_OUT_LEN);
- MCDI_DECLARE_STRUCT_PTR(match_crit);
- MCDI_DECLARE_STRUCT_PTR(response);
- size_t outlen;
- int rc;
- if (!id)
- return -EINVAL;
- match_crit = _MCDI_DWORD(inbuf, MAE_ACTION_RULE_INSERT_IN_MATCH_CRITERIA);
- response = _MCDI_DWORD(inbuf, MAE_ACTION_RULE_INSERT_IN_RESPONSE);
- if (efx_mae_asl_id(acts_id)) {
- MCDI_STRUCT_SET_DWORD(response, MAE_ACTION_RULE_RESPONSE_ASL_ID, acts_id);
- MCDI_STRUCT_SET_DWORD(response, MAE_ACTION_RULE_RESPONSE_AS_ID,
- MC_CMD_MAE_ACTION_SET_ALLOC_OUT_ACTION_SET_ID_NULL);
- } else {
- /* We only had one AS, so we didn't wrap it in an ASL */
- MCDI_STRUCT_SET_DWORD(response, MAE_ACTION_RULE_RESPONSE_ASL_ID,
- MC_CMD_MAE_ACTION_SET_LIST_ALLOC_OUT_ACTION_SET_LIST_ID_NULL);
- MCDI_STRUCT_SET_DWORD(response, MAE_ACTION_RULE_RESPONSE_AS_ID, acts_id);
- }
- MCDI_SET_DWORD(inbuf, MAE_ACTION_RULE_INSERT_IN_PRIO, prio);
- rc = efx_mae_populate_match_criteria(match_crit, match);
- if (rc)
- return rc;
- rc = efx_mcdi_rpc(efx, MC_CMD_MAE_ACTION_RULE_INSERT, inbuf, sizeof(inbuf),
- outbuf, sizeof(outbuf), &outlen);
- if (rc)
- return rc;
- if (outlen < sizeof(outbuf))
- return -EIO;
- *id = MCDI_DWORD(outbuf, MAE_ACTION_RULE_INSERT_OUT_AR_ID);
- return 0;
- }
- int efx_mae_delete_rule(struct efx_nic *efx, u32 id)
- {
- MCDI_DECLARE_BUF(outbuf, MC_CMD_MAE_ACTION_RULE_DELETE_OUT_LEN(1));
- MCDI_DECLARE_BUF(inbuf, MC_CMD_MAE_ACTION_RULE_DELETE_IN_LEN(1));
- size_t outlen;
- int rc;
- MCDI_SET_DWORD(inbuf, MAE_ACTION_RULE_DELETE_IN_AR_ID, id);
- rc = efx_mcdi_rpc(efx, MC_CMD_MAE_ACTION_RULE_DELETE, inbuf, sizeof(inbuf),
- outbuf, sizeof(outbuf), &outlen);
- if (rc)
- return rc;
- if (outlen < sizeof(outbuf))
- return -EIO;
- /* FW freed a different ID than we asked for, should also never happen.
- * Warn because it means we've now got a different idea to the FW of
- * what rules exist, which could cause mayhem later.
- */
- if (WARN_ON(MCDI_DWORD(outbuf, MAE_ACTION_RULE_DELETE_OUT_DELETED_AR_ID) != id))
- return -EIO;
- return 0;
- }
|