ifconfig: add -D option to print driver name for interface

Add -D option to add the drivername and unit number to ifconfig output
for normal display, including -a.  Use ifconfig_get_orig_name() from
libifconfig to fetch the name.  Note that this is the original name
for many drivers, but not for some exceptions like epair (which appends
'a' or 'b' to the unit number).  epair interface pairs both display
as "epair0", etc.  Make -v imply -D; might as well be fully verbose.

MFC after:	1 week
Reviewed by:	zlei, kp
Differential Revision:	https://reviews.freebsd.org/D42721
This commit is contained in:
Mike Karels 2023-11-28 13:47:37 -06:00
parent 87826c87c6
commit cd201c0908
4 changed files with 30 additions and 3 deletions

View file

@ -34,7 +34,7 @@
.Sh SYNOPSIS
.Nm
.Op Fl j Ar jail
.Op Fl kLmn
.Op Fl DkLmn
.Op Fl f Ar type Ns Cm \&: Ns Ar format
.Ar interface
.Op Cm create
@ -53,7 +53,7 @@
.Nm
.Op Fl j Ar jail
.Fl a
.Op Fl dkLmuv
.Op Fl dDkLmuv
.Op Fl f Ar type Ns Cm \&: Ns Ar format
.Op Fl G Ar groupname
.Op Fl g Ar groupname
@ -102,6 +102,12 @@ with no additional information.
Use of this flag is mutually exclusive with all other flags and commands.
.It Fl d
Display only the interfaces that are down.
.It Fl D
Include the driver name and unit number of the interface in the output.
This is normally the original name of the interface,
even if it has been renamed; it may differ from the original name
in some cases, such as
.Xr epair 4 .
.It Fl f Xo
.Ar type Ns Cm \&: Ns Ar format Ns
.Op Cm \&, Ns Ar type Ns Cm \&: Ns Ar format Ar ...

View file

@ -456,7 +456,7 @@ args_parse(struct ifconfig_args *args, int argc, char *argv[])
int c;
/* Parse leading line options */
strlcpy(options, "G:adf:j:klmnuv", sizeof(options));
strlcpy(options, "G:adDf:j:klmnuv", sizeof(options));
for (p = opts; p != NULL; p = p->next)
strlcat(options, p->opt, sizeof(options));
while ((c = getopt(argc, argv, options)) != -1) {
@ -467,6 +467,9 @@ args_parse(struct ifconfig_args *args, int argc, char *argv[])
case 'd': /* restrict scan to "down" interfaces */
args->downonly = true;
break;
case 'D': /* Print driver name */
args->drivername = true;
break;
case 'f':
if (optarg == NULL)
usage();

View file

@ -231,6 +231,7 @@ struct ifconfig_args {
bool supmedia; /* Supported media */
bool printkeys; /* Print security keys */
bool allfamilies; /* Print all families */
bool drivername; /* Print driver name */
int verbose; /* verbosity level */
int argc;
char **argv;

View file

@ -388,6 +388,7 @@ status_nl(if_ctx *ctx, struct iface *iface)
{
if_link_t *link = &iface->link;
struct ifconfig_args *args = ctx->args;
char *drivername = NULL;
printf("%s: ", link->ifla_ifname);
@ -432,6 +433,22 @@ status_nl(if_ctx *ctx, struct iface *iface)
args->afp->af_other_status(ctx);
print_ifstatus(ctx);
if (args->drivername || args->verbose) {
if (ifconfig_get_orig_name(lifh, link->ifla_ifname,
&drivername) != 0) {
if (ifconfig_err_errtype(lifh) == OTHER)
fprintf(stderr, "get original name: %s\n",
strerror(ifconfig_err_errno(lifh)));
else
fprintf(stderr,
"get original name: error type %d\n",
ifconfig_err_errtype(lifh));
exit_code = 1;
}
if (drivername != NULL)
printf("\tdrivername: %s\n", drivername);
free(drivername);
}
if (args->verbose > 0)
sfp_status(ctx);
}