Skip to content

Commit b8291cf

Browse files
peterdelevoryasdavem330
authored andcommitted
net/ncsi: Add NC-SI 1.2 Get MC MAC Address command
This change adds support for the NC-SI 1.2 Get MC MAC Address command, specified here: https://www.dmtf.org/sites/default/files/standards/documents/DSP0222_1.2.0.pdf It serves the exact same function as the existing OEM Get MAC Address commands, so if a channel reports that it supports NC-SI 1.2, we prefer to use the standard command rather than the OEM command. Verified with an invalid MAC address and 2 valid ones: [ 55.137072] ftgmac100 1e690000.ftgmac eth0: NCSI: Received 3 provisioned MAC addresses [ 55.137614] ftgmac100 1e690000.ftgmac eth0: NCSI: MAC address 0: 00:00:00:00:00:00 [ 55.138026] ftgmac100 1e690000.ftgmac eth0: NCSI: MAC address 1: fa:ce:b0:0c:20:22 [ 55.138528] ftgmac100 1e690000.ftgmac eth0: NCSI: MAC address 2: fa:ce:b0:0c:20:23 [ 55.139241] ftgmac100 1e690000.ftgmac eth0: NCSI: Unable to assign 00:00:00:00:00:00 to device [ 55.140098] ftgmac100 1e690000.ftgmac eth0: NCSI: Set MAC address to fa:ce:b0:0c:20:22 Signed-off-by: Peter Delevoryas <peter@pjd.dev> Signed-off-by: Patrick Williams <patrick@stwcx.xyz> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 3084b58 commit b8291cf

File tree

4 files changed

+59
-4
lines changed

4 files changed

+59
-4
lines changed

net/ncsi/ncsi-cmd.c

+2-1
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,8 @@ static struct ncsi_cmd_handler {
270270
{ NCSI_PKT_CMD_GPS, 0, ncsi_cmd_handler_default },
271271
{ NCSI_PKT_CMD_OEM, -1, ncsi_cmd_handler_oem },
272272
{ NCSI_PKT_CMD_PLDM, 0, NULL },
273-
{ NCSI_PKT_CMD_GPUUID, 0, ncsi_cmd_handler_default }
273+
{ NCSI_PKT_CMD_GPUUID, 0, ncsi_cmd_handler_default },
274+
{ NCSI_PKT_CMD_GMCMA, 0, ncsi_cmd_handler_default }
274275
};
275276

276277
static struct ncsi_request *ncsi_alloc_command(struct ncsi_cmd_arg *nca)

net/ncsi/ncsi-manage.c

+7-2
Original file line numberDiff line numberDiff line change
@@ -1038,11 +1038,16 @@ static void ncsi_configure_channel(struct ncsi_dev_priv *ndp)
10381038
case ncsi_dev_state_config_oem_gma:
10391039
nd->state = ncsi_dev_state_config_clear_vids;
10401040

1041-
nca.type = NCSI_PKT_CMD_OEM;
10421041
nca.package = np->id;
10431042
nca.channel = nc->id;
10441043
ndp->pending_req_num = 1;
1045-
ret = ncsi_gma_handler(&nca, nc->version.mf_id);
1044+
if (nc->version.major >= 1 && nc->version.minor >= 2) {
1045+
nca.type = NCSI_PKT_CMD_GMCMA;
1046+
ret = ncsi_xmit_cmd(&nca);
1047+
} else {
1048+
nca.type = NCSI_PKT_CMD_OEM;
1049+
ret = ncsi_gma_handler(&nca, nc->version.mf_id);
1050+
}
10461051
if (ret < 0)
10471052
schedule_work(&ndp->work);
10481053

net/ncsi/ncsi-pkt.h

+10
Original file line numberDiff line numberDiff line change
@@ -338,6 +338,14 @@ struct ncsi_rsp_gpuuid_pkt {
338338
__be32 checksum;
339339
};
340340

341+
/* Get MC MAC Address */
342+
struct ncsi_rsp_gmcma_pkt {
343+
struct ncsi_rsp_pkt_hdr rsp;
344+
unsigned char address_count;
345+
unsigned char reserved[3];
346+
unsigned char addresses[][ETH_ALEN];
347+
};
348+
341349
/* AEN: Link State Change */
342350
struct ncsi_aen_lsc_pkt {
343351
struct ncsi_aen_pkt_hdr aen; /* AEN header */
@@ -398,6 +406,7 @@ struct ncsi_aen_hncdsc_pkt {
398406
#define NCSI_PKT_CMD_GPUUID 0x52 /* Get package UUID */
399407
#define NCSI_PKT_CMD_QPNPR 0x56 /* Query Pending NC PLDM request */
400408
#define NCSI_PKT_CMD_SNPR 0x57 /* Send NC PLDM Reply */
409+
#define NCSI_PKT_CMD_GMCMA 0x58 /* Get MC MAC Address */
401410

402411

403412
/* NCSI packet responses */
@@ -433,6 +442,7 @@ struct ncsi_aen_hncdsc_pkt {
433442
#define NCSI_PKT_RSP_GPUUID (NCSI_PKT_CMD_GPUUID + 0x80)
434443
#define NCSI_PKT_RSP_QPNPR (NCSI_PKT_CMD_QPNPR + 0x80)
435444
#define NCSI_PKT_RSP_SNPR (NCSI_PKT_CMD_SNPR + 0x80)
445+
#define NCSI_PKT_RSP_GMCMA (NCSI_PKT_CMD_GMCMA + 0x80)
436446

437447
/* NCSI response code/reason */
438448
#define NCSI_PKT_RSP_C_COMPLETED 0x0000 /* Command Completed */

net/ncsi/ncsi-rsp.c

+40-1
Original file line numberDiff line numberDiff line change
@@ -1091,6 +1091,44 @@ static int ncsi_rsp_handler_netlink(struct ncsi_request *nr)
10911091
return ret;
10921092
}
10931093

1094+
static int ncsi_rsp_handler_gmcma(struct ncsi_request *nr)
1095+
{
1096+
struct ncsi_dev_priv *ndp = nr->ndp;
1097+
struct net_device *ndev = ndp->ndev.dev;
1098+
struct ncsi_rsp_gmcma_pkt *rsp;
1099+
struct sockaddr saddr;
1100+
int ret = -1;
1101+
int i;
1102+
1103+
rsp = (struct ncsi_rsp_gmcma_pkt *)skb_network_header(nr->rsp);
1104+
saddr.sa_family = ndev->type;
1105+
ndev->priv_flags |= IFF_LIVE_ADDR_CHANGE;
1106+
1107+
netdev_info(ndev, "NCSI: Received %d provisioned MAC addresses\n",
1108+
rsp->address_count);
1109+
for (i = 0; i < rsp->address_count; i++) {
1110+
netdev_info(ndev, "NCSI: MAC address %d: %02x:%02x:%02x:%02x:%02x:%02x\n",
1111+
i, rsp->addresses[i][0], rsp->addresses[i][1],
1112+
rsp->addresses[i][2], rsp->addresses[i][3],
1113+
rsp->addresses[i][4], rsp->addresses[i][5]);
1114+
}
1115+
1116+
for (i = 0; i < rsp->address_count; i++) {
1117+
memcpy(saddr.sa_data, &rsp->addresses[i], ETH_ALEN);
1118+
ret = ndev->netdev_ops->ndo_set_mac_address(ndev, &saddr);
1119+
if (ret < 0) {
1120+
netdev_warn(ndev, "NCSI: Unable to assign %pM to device\n",
1121+
saddr.sa_data);
1122+
continue;
1123+
}
1124+
netdev_warn(ndev, "NCSI: Set MAC address to %pM\n", saddr.sa_data);
1125+
break;
1126+
}
1127+
1128+
ndp->gma_flag = ret == 0;
1129+
return ret;
1130+
}
1131+
10941132
static struct ncsi_rsp_handler {
10951133
unsigned char type;
10961134
int payload;
@@ -1127,7 +1165,8 @@ static struct ncsi_rsp_handler {
11271165
{ NCSI_PKT_RSP_PLDM, -1, ncsi_rsp_handler_pldm },
11281166
{ NCSI_PKT_RSP_GPUUID, 20, ncsi_rsp_handler_gpuuid },
11291167
{ NCSI_PKT_RSP_QPNPR, -1, ncsi_rsp_handler_pldm },
1130-
{ NCSI_PKT_RSP_SNPR, -1, ncsi_rsp_handler_pldm }
1168+
{ NCSI_PKT_RSP_SNPR, -1, ncsi_rsp_handler_pldm },
1169+
{ NCSI_PKT_RSP_GMCMA, -1, ncsi_rsp_handler_gmcma },
11311170
};
11321171

11331172
int ncsi_rcv_rsp(struct sk_buff *skb, struct net_device *dev,

0 commit comments

Comments
 (0)