Compare commits
No commits in common. 'c9' and 'c8' have entirely different histories.
@ -1 +1 @@
|
|||||||
8a0ee8a4693faef7232c7370d5c90a7806413edc SOURCES/fcoe-utils-1.0.34-14ef0d2.tar.gz
|
263e7e1c31e5c834126afb0a510a852268bd07af SOURCES/fcoe-utils-1.0.33-848bcc6.tar.gz
|
||||||
|
|||||||
@ -1 +1 @@
|
|||||||
SOURCES/fcoe-utils-1.0.34-14ef0d2.tar.gz
|
SOURCES/fcoe-utils-1.0.33-848bcc6.tar.gz
|
||||||
|
|||||||
@ -0,0 +1,115 @@
|
|||||||
|
From 729135eea0ed39b3dfd57b7ea15f284e67af532f Mon Sep 17 00:00:00 2001
|
||||||
|
From: Chris Leech <cleech@redhat.com>
|
||||||
|
Date: Mon, 18 Jan 2021 13:37:39 -0800
|
||||||
|
Subject: [PATCH 1/1] Revert "fcoemon: Correctly handle options in the service
|
||||||
|
file"
|
||||||
|
|
||||||
|
This reverts commit b06c3df0f0b06c1ad37b3bbf8bd602b24318aae6.
|
||||||
|
---
|
||||||
|
doc/fcoemon.txt | 6 +++---
|
||||||
|
etc/systemd/fcoe.service | 4 ++--
|
||||||
|
fcoemon.c | 24 +++++++++---------------
|
||||||
|
3 files changed, 14 insertions(+), 20 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/doc/fcoemon.txt b/doc/fcoemon.txt
|
||||||
|
index ba70478d141..3bfb3581a1b 100644
|
||||||
|
--- a/doc/fcoemon.txt
|
||||||
|
+++ b/doc/fcoemon.txt
|
||||||
|
@@ -53,13 +53,13 @@ OPTIONS
|
||||||
|
-------
|
||||||
|
*-f*, *--foreground*::
|
||||||
|
Run *fcoemon* in the foreground.
|
||||||
|
-*-d*, *--debug=yes|no*::
|
||||||
|
- Enable or disable debugging messages.
|
||||||
|
+*-d*, *--debug*::
|
||||||
|
+ Enable debugging messages.
|
||||||
|
*-l*, *--legacy*::
|
||||||
|
Force fcoemon to use the legacy /sys/module/libfcoe/parameters/
|
||||||
|
interface. The default is to use the newer /sys/bus/fcoe/ interfaces
|
||||||
|
if they are available.
|
||||||
|
-*-s*, *--syslog=yes|no*::
|
||||||
|
+*-s*, *--syslog*::
|
||||||
|
Use syslogd for logging. The default behavior is to log to stdout
|
||||||
|
and stderr.
|
||||||
|
*-h*, *--help*::
|
||||||
|
diff --git a/etc/systemd/fcoe.service b/etc/systemd/fcoe.service
|
||||||
|
index b1d95671573..5e5c8a2b67b 100644
|
||||||
|
--- a/etc/systemd/fcoe.service
|
||||||
|
+++ b/etc/systemd/fcoe.service
|
||||||
|
@@ -4,9 +4,9 @@ After=syslog.target network.target
|
||||||
|
|
||||||
|
[Service]
|
||||||
|
Type=simple
|
||||||
|
-EnvironmentFile=/etc/fcoe/config
|
||||||
|
+EnvironmentFile=/etc/sysconfig/fcoe
|
||||||
|
ExecStartPre=/sbin/modprobe -qa $SUPPORTED_DRIVERS
|
||||||
|
-ExecStart=/usr/sbin/fcoemon --foreground --debug=$DEBUG --syslog=$SYSLOG
|
||||||
|
+ExecStart=/usr/sbin/fcoemon $FCOEMON_OPTS
|
||||||
|
|
||||||
|
[Install]
|
||||||
|
WantedBy=multi-user.target
|
||||||
|
diff --git a/fcoemon.c b/fcoemon.c
|
||||||
|
index 8c08bc5a032..0cc36fec304 100644
|
||||||
|
--- a/fcoemon.c
|
||||||
|
+++ b/fcoemon.c
|
||||||
|
@@ -335,9 +335,9 @@ static int fcoe_vid_from_ifname(const char *ifname);
|
||||||
|
* Table for getopt_long(3).
|
||||||
|
*/
|
||||||
|
static struct option fcm_options[] = {
|
||||||
|
- {"debug", 1, NULL, 'd'},
|
||||||
|
+ {"debug", 0, NULL, 'd'},
|
||||||
|
{"legacy", 0, NULL, 'l'},
|
||||||
|
- {"syslog", 1, NULL, 's'},
|
||||||
|
+ {"syslog", 0, NULL, 's'},
|
||||||
|
{"exec", 1, NULL, 'e'},
|
||||||
|
{"foreground", 0, NULL, 'f'},
|
||||||
|
{"version", 0, NULL, 'v'},
|
||||||
|
@@ -3271,9 +3271,9 @@ static void fcm_usage(void)
|
||||||
|
{
|
||||||
|
printf("Usage: %s\n"
|
||||||
|
"\t [-f|--foreground]\n"
|
||||||
|
- "\t [-d|--debug=yes|no]\n"
|
||||||
|
+ "\t [-d|--debug]\n"
|
||||||
|
"\t [-l|--legacy]\n"
|
||||||
|
- "\t [-s|--syslog=yes|no]\n"
|
||||||
|
+ "\t [-s|--syslog]\n"
|
||||||
|
"\t [-v|--version]\n"
|
||||||
|
"\t [-h|--help]\n\n", progname);
|
||||||
|
exit(1);
|
||||||
|
@@ -3729,28 +3729,22 @@ int main(int argc, char **argv)
|
||||||
|
sa_log_flags = 0;
|
||||||
|
openlog(sa_log_prefix, LOG_CONS, LOG_DAEMON);
|
||||||
|
|
||||||
|
- while ((c = getopt_long(argc, argv, "fd:hls:v",
|
||||||
|
+ while ((c = getopt_long(argc, argv, "fdhlsv",
|
||||||
|
fcm_options, NULL)) != -1) {
|
||||||
|
switch (c) {
|
||||||
|
case 'f':
|
||||||
|
fcm_fg = 1;
|
||||||
|
break;
|
||||||
|
case 'd':
|
||||||
|
- if (!strncmp(optarg, "yes", 3) ||
|
||||||
|
- !strncmp(optarg, "YES", 3)) {
|
||||||
|
- fcoe_config.debug = 1;
|
||||||
|
- enable_debug_log(1);
|
||||||
|
- }
|
||||||
|
+ fcoe_config.debug = 1;
|
||||||
|
+ enable_debug_log(1);
|
||||||
|
break;
|
||||||
|
case 'l':
|
||||||
|
force_legacy = true;
|
||||||
|
break;
|
||||||
|
case 's':
|
||||||
|
- if (!strncmp(optarg, "yes", 3) ||
|
||||||
|
- !strncmp(optarg, "YES", 3)) {
|
||||||
|
- fcoe_config.use_syslog = 1;
|
||||||
|
- enable_syslog(1);
|
||||||
|
- }
|
||||||
|
+ fcoe_config.use_syslog = 1;
|
||||||
|
+ enable_syslog(1);
|
||||||
|
break;
|
||||||
|
case 'v':
|
||||||
|
printf("%s\n", FCOE_UTILS_VERSION);
|
||||||
|
--
|
||||||
|
2.26.2
|
||||||
|
|
||||||
@ -0,0 +1,78 @@
|
|||||||
|
From 1e6837c8ce063399eeb9580104da33f807e15443 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Chris Leech <cleech@redhat.com>
|
||||||
|
Date: Tue, 23 Mar 2021 11:16:06 -0700
|
||||||
|
Subject: [PATCH 2/5] Revert "Make gcc compiler happy about ifname string
|
||||||
|
truncation."
|
||||||
|
|
||||||
|
This change dropped the "." from between the physical interface name and
|
||||||
|
the vlan number, making fipvlan created vlan names incompatible with
|
||||||
|
fcoeadm commands that ended up calling get_pci_dev_from_netdev in
|
||||||
|
lib/sysfs_hba.c (fcoeadm -i). That requirement should be fixed, but for
|
||||||
|
now lets deal with the fipvlan naming regression.
|
||||||
|
|
||||||
|
safe_makevlan_name isn't doing anything that can't be handled by
|
||||||
|
checking the return from snprintf
|
||||||
|
|
||||||
|
This reverts commit eee875e6526786031ec916274deec92148677c38.
|
||||||
|
|
||||||
|
Signed-off-by: Chris Leech <cleech@redhat.com>
|
||||||
|
---
|
||||||
|
fipvlan.c | 34 +---------------------------------
|
||||||
|
1 file changed, 1 insertion(+), 33 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/fipvlan.c b/fipvlan.c
|
||||||
|
index c8a07339314..fe8d7955cc5 100644
|
||||||
|
--- a/fipvlan.c
|
||||||
|
+++ b/fipvlan.c
|
||||||
|
@@ -595,36 +595,6 @@ static int rtnl_listener_handler(struct nlmsghdr *nh, UNUSED void *arg)
|
||||||
|
return -1;
|
||||||
|
}
|
||||||
|
|
||||||
|
-static int
|
||||||
|
-safe_makevlan_name(char *vlan_name, size_t vsz,
|
||||||
|
- char *ifname, int vlan_num, char *suffix)
|
||||||
|
-{
|
||||||
|
- size_t ifsz = strlen(ifname);
|
||||||
|
- size_t susz = strlen(suffix); /* should never be NULL */
|
||||||
|
- int nusz;
|
||||||
|
- char numbuf[16];
|
||||||
|
- char *cp = vlan_name;
|
||||||
|
-
|
||||||
|
- nusz = snprintf(numbuf, sizeof(numbuf), "%d", vlan_num);
|
||||||
|
-
|
||||||
|
- if ((ifsz + susz + nusz + 2) > vsz) {
|
||||||
|
- FIP_LOG_ERR(EINVAL,
|
||||||
|
- "Cannot make VLAN name from ifname=\"%s\", vlan %d, and suffix=\"%s\"\n",
|
||||||
|
- ifname, vlan_num, suffix);
|
||||||
|
- return -EINVAL;
|
||||||
|
- }
|
||||||
|
- memcpy(cp, ifname, ifsz);
|
||||||
|
- cp += ifsz;
|
||||||
|
- memcpy(cp, numbuf, nusz);
|
||||||
|
- cp += nusz;
|
||||||
|
- if (susz > 0) {
|
||||||
|
- memcpy(cp, suffix, susz);
|
||||||
|
- cp += susz;
|
||||||
|
- }
|
||||||
|
- *cp = '\0';
|
||||||
|
- return 0;
|
||||||
|
-}
|
||||||
|
-
|
||||||
|
static int
|
||||||
|
create_and_start_vlan(struct fcf *fcf, bool vn2vn)
|
||||||
|
{
|
||||||
|
@@ -654,10 +624,8 @@ create_and_start_vlan(struct fcf *fcf, bool vn2vn)
|
||||||
|
real_dev->ifname, fcf->vlan, vlan->ifname);
|
||||||
|
rc = 0;
|
||||||
|
} else {
|
||||||
|
- rc = safe_makevlan_name(vlan_name, sizeof(vlan_name),
|
||||||
|
+ snprintf(vlan_name, IFNAMSIZ, "%s.%d%s",
|
||||||
|
real_dev->ifname, fcf->vlan, config.suffix);
|
||||||
|
- if (rc < 0)
|
||||||
|
- return rc;
|
||||||
|
rc = vlan_create(fcf->ifindex, fcf->vlan, vlan_name);
|
||||||
|
if (rc < 0)
|
||||||
|
printf("Failed to create VLAN device %s\n\t%s\n",
|
||||||
|
--
|
||||||
|
2.26.2
|
||||||
|
|
||||||
@ -0,0 +1,38 @@
|
|||||||
|
From b9885692cb283a674e04528486984fb61f92a190 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Chris Leech <cleech@redhat.com>
|
||||||
|
Date: Tue, 23 Mar 2021 11:21:17 -0700
|
||||||
|
Subject: [PATCH 3/5] fix VLAN device name overflow check
|
||||||
|
|
||||||
|
check snprintf return for truncation
|
||||||
|
|
||||||
|
Signed-off-by: Chris Leech <cleech@redhat.com>
|
||||||
|
---
|
||||||
|
fipvlan.c | 12 ++++++++++--
|
||||||
|
1 file changed, 10 insertions(+), 2 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/fipvlan.c b/fipvlan.c
|
||||||
|
index fe8d7955cc5..3ce913d5eaf 100644
|
||||||
|
--- a/fipvlan.c
|
||||||
|
+++ b/fipvlan.c
|
||||||
|
@@ -624,8 +624,16 @@ create_and_start_vlan(struct fcf *fcf, bool vn2vn)
|
||||||
|
real_dev->ifname, fcf->vlan, vlan->ifname);
|
||||||
|
rc = 0;
|
||||||
|
} else {
|
||||||
|
- snprintf(vlan_name, IFNAMSIZ, "%s.%d%s",
|
||||||
|
- real_dev->ifname, fcf->vlan, config.suffix);
|
||||||
|
+ rc = snprintf(vlan_name, IFNAMSIZ, "%s.%d%s",
|
||||||
|
+ real_dev->ifname, fcf->vlan,
|
||||||
|
+ config.suffix);
|
||||||
|
+ if (rc < 0 || rc >= IFNAMSIZ) {
|
||||||
|
+ printf("Failed to create VLAN device "
|
||||||
|
+ "(name %s.%d%s is too long)\n",
|
||||||
|
+ real_dev->ifname, fcf->vlan,
|
||||||
|
+ config.suffix);
|
||||||
|
+ return -EINVAL;
|
||||||
|
+ }
|
||||||
|
rc = vlan_create(fcf->ifindex, fcf->vlan, vlan_name);
|
||||||
|
if (rc < 0)
|
||||||
|
printf("Failed to create VLAN device %s\n\t%s\n",
|
||||||
|
--
|
||||||
|
2.26.2
|
||||||
|
|
||||||
@ -0,0 +1,35 @@
|
|||||||
|
From 108387a2aa986a8107faa7548f3f9e9c084749d2 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Chris Leech <cleech@redhat.com>
|
||||||
|
Date: Wed, 24 Mar 2021 09:50:51 -0700
|
||||||
|
Subject: [PATCH 4/5] fix regressions caused by safe_makepath change in
|
||||||
|
libopenfcoe.c
|
||||||
|
|
||||||
|
This needs to use strlen(dname) not sizeof(dname) or the directory path
|
||||||
|
is truncated. On a 64-bit arch an 8 byte truncation kind of looks like
|
||||||
|
a valid path '/sys/bus' which is sort of funny unless you're debugging
|
||||||
|
it.
|
||||||
|
|
||||||
|
This caused fcoeadm --fcf to fail to read in FCF info, and fcoeadm
|
||||||
|
--lesb to fail to find the link error status block counters.
|
||||||
|
|
||||||
|
Signed-off-by: Chris Leech <cleech@redhat.com>
|
||||||
|
---
|
||||||
|
libopenfcoe.c | 2 +-
|
||||||
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/libopenfcoe.c b/libopenfcoe.c
|
||||||
|
index 452ee803e63..cd35c868eee 100644
|
||||||
|
--- a/libopenfcoe.c
|
||||||
|
+++ b/libopenfcoe.c
|
||||||
|
@@ -58,7 +58,7 @@ out_err:
|
||||||
|
static char *safe_makepath(char *path, size_t path_sz,
|
||||||
|
char *dname, char *fname)
|
||||||
|
{
|
||||||
|
- size_t dsz = sizeof(dname);
|
||||||
|
+ size_t dsz = strlen(dname);
|
||||||
|
size_t fsz = strlen(fname);
|
||||||
|
char *cp = path;
|
||||||
|
|
||||||
|
--
|
||||||
|
2.26.2
|
||||||
|
|
||||||
@ -0,0 +1,35 @@
|
|||||||
|
From c4fe62dd3d26b30fdcf94c94f74bc0b9f7034c17 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Chris Leech <cleech@redhat.com>
|
||||||
|
Date: Wed, 24 Mar 2021 10:17:52 -0700
|
||||||
|
Subject: [PATCH 5/5] minor fcoeadm output issues
|
||||||
|
|
||||||
|
Signed-off-by: Chris Leech <cleech@redhat.com>
|
||||||
|
---
|
||||||
|
fcoeadm_display.c | 4 ++--
|
||||||
|
1 file changed, 2 insertions(+), 2 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/fcoeadm_display.c b/fcoeadm_display.c
|
||||||
|
index 585ecfae54e..c2ef33b82cf 100644
|
||||||
|
--- a/fcoeadm_display.c
|
||||||
|
+++ b/fcoeadm_display.c
|
||||||
|
@@ -132,7 +132,7 @@ static void show_port_info(struct port_attributes *lp_info)
|
||||||
|
printf(" Port Name: %s\n",
|
||||||
|
lp_info->port_name);
|
||||||
|
|
||||||
|
- printf(" Fabric Name: %s\n",
|
||||||
|
+ printf(" Fabric Name: %s\n",
|
||||||
|
lp_info->fabric_name);
|
||||||
|
|
||||||
|
printf(" Speed: %s\n",
|
||||||
|
@@ -853,7 +853,7 @@ static void print_fcoe_fcf_device(void *ep, UNUSED void *arg)
|
||||||
|
buf = temp;
|
||||||
|
printf(" Connection Mode: %s\n", buf);
|
||||||
|
printf(" Fabric Name: 0x%016" PRIx64 "\n", fcf->fabric_name);
|
||||||
|
- printf(" Switch Name 0x%016" PRIx64 "\n", fcf->switch_name);
|
||||||
|
+ printf(" Switch Name: 0x%016" PRIx64 "\n", fcf->switch_name);
|
||||||
|
mac2str(fcf->mac, mac, MAX_STR_LEN);
|
||||||
|
printf(" MAC Address: %s\n", mac);
|
||||||
|
printf(" FCF Priority: %u\n", fcf->priority);
|
||||||
|
--
|
||||||
|
2.26.2
|
||||||
|
|
||||||
@ -0,0 +1,25 @@
|
|||||||
|
From db650524cb395827cf0d5a43bd07f77396513cd7 Mon Sep 17 00:00:00 2001
|
||||||
|
From: rpm-build <rpm-build>
|
||||||
|
Date: Tue, 7 Aug 2018 21:23:38 -0700
|
||||||
|
Subject: [PATCH 1/1] set default DCB_REQUIRED to 'no'
|
||||||
|
|
||||||
|
---
|
||||||
|
etc/cfg-ethx | 2 +-
|
||||||
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/etc/cfg-ethx b/etc/cfg-ethx
|
||||||
|
index c5aaf36b4625..e2eb3309ccc7 100644
|
||||||
|
--- a/etc/cfg-ethx
|
||||||
|
+++ b/etc/cfg-ethx
|
||||||
|
@@ -8,7 +8,7 @@ FCOE_ENABLE="yes"
|
||||||
|
## Default: no
|
||||||
|
# Indicate if DCB service is required at the Ethernet port
|
||||||
|
# Normally set to "yes"
|
||||||
|
-DCB_REQUIRED="yes"
|
||||||
|
+DCB_REQUIRED="no"
|
||||||
|
|
||||||
|
## Type: yes/no
|
||||||
|
## Default: no
|
||||||
|
--
|
||||||
|
2.14.4
|
||||||
|
|
||||||
@ -0,0 +1,5 @@
|
|||||||
|
# All supported drivers listed here are loaded when service starts
|
||||||
|
SUPPORTED_DRIVERS="libfc bnx2fc qedf"
|
||||||
|
|
||||||
|
# Add --debug to enable debug messages
|
||||||
|
FCOEMON_OPTS="--syslog"
|
||||||
@ -0,0 +1,12 @@
|
|||||||
|
[Unit]
|
||||||
|
Description=Open-FCoE Inititator.
|
||||||
|
After=syslog.target network.target lldpad.service
|
||||||
|
|
||||||
|
[Service]
|
||||||
|
Type=forking
|
||||||
|
EnvironmentFile=/etc/sysconfig/fcoe
|
||||||
|
ExecStartPre=/sbin/modprobe -qa $SUPPORTED_DRIVERS
|
||||||
|
ExecStart=/usr/sbin/fcoemon $FCOEMON_OPTS
|
||||||
|
|
||||||
|
[Install]
|
||||||
|
WantedBy=multi-user.target
|
||||||
@ -0,0 +1,25 @@
|
|||||||
|
Quick Start guide for Open-FCoE
|
||||||
|
===============================
|
||||||
|
|
||||||
|
1. Install fcoe-utils package. This should also install dcbd, libhbaapi and
|
||||||
|
libhbalinux as dependencies.
|
||||||
|
|
||||||
|
2. Rename /etc/fcoe/cfg-ethx so it corresponds with name of your network
|
||||||
|
interface (e.g. /etc/fcoe/cfg-eth0). Copy and rename this file accordingly
|
||||||
|
if you have more interfaces, which should be fcoe-enabled
|
||||||
|
|
||||||
|
3. Modify configuration files to enable FCoE. Set FCOE_ENABLE="yes" and
|
||||||
|
DCB_REQUIRED="yes".
|
||||||
|
|
||||||
|
3. Run 'systemctl enable fcoe.service' to start FCoE per run level. This
|
||||||
|
will setup FCoE to start on reboot.
|
||||||
|
|
||||||
|
4. Run 'systemctl enable lldpad.service' to start LLDP agent per run
|
||||||
|
level. This will setup DCB to start on reboot.
|
||||||
|
|
||||||
|
5. Run 'systemctl start lldpad.service' to start LLDP agent.
|
||||||
|
|
||||||
|
6. Run 'dcbtool sc ethX dcb on; dcbtool sc ethX app:0 e:1;' for each fcoe-enabled
|
||||||
|
interface to setup DCB for FCoE.
|
||||||
|
|
||||||
|
7. Run 'systemctl start fcoe.sertvice' to start FCoE.
|
||||||
Loading…
Reference in new issue