On Mon, Dec 12, 2022 at 06:48:36AM +0000, Jason McIntyre wrote:
> On Fri, Dec 09, 2022 at 09:32:13PM +0000, Jason McIntyre wrote:
> > hi.
> > 
> > two points about the recent ability to use lladdr:
> > 
> > - the example of "bridge0" made sense when bridge was regarded as a
> >   separate entity and not integrated with ifconfig. plus a list of one
> >   example looks rubbish. now that we have a second example (lladdr) and
> >   bridge is not flagged as a special case, i think we can simply the
> >   text and reduce it to two examples

I thought it was an example showing that it works for "dynamic" interfaces,
but that's probably obvious enough without.


> > - i'm not sure about using "lladdr". although we use this term in
> >   ifconfig(8), we explain it. and people may miss it if they are thinking
> >   of mac address. i've attempted to both write the term fully as "link
> >   layer local address" and add a "(MAC)". i suppose you could argue that
> >   people who think of the term as "lladdr" might miss that (!) but i
> >   don;t think that is a real worry.

I think that since `ifconfig` prints out "lladdr" that even folks who
don't know what it means should be able to figure out how to use it.
I am a fan of defining terms though.


> > so here's my cut at tweaking...
> > jmc
> > 
> 
> on the back of afresh's prioroty flip diff, here's a revision of my diff
> after some input from deraadt. it will need adjusting again if we flip
> priority...

This looks OK to me.  I'm waiting to see if kn@ or others have feedback
on the swap and other installer bits before committing, but it seems the
idea of preferring lladdr has been given the go-ahead, so it's just a
matter of time.

I tend think of the interface name as its driver + number, so I did find
the distinction that the name is just the part without the number to be
surprising.  ifconfig(8) does say it is "name unit" with the "name" not
including the number, so this is correct.

 
> jmc
> 
> Index: hostname.if.5
> ===================================================================
> RCS file: /cvs/src/share/man/man5/hostname.if.5,v
> retrieving revision 1.80
> diff -u -p -r1.80 hostname.if.5
> --- hostname.if.5     5 Dec 2022 20:12:00 -0000       1.80
> +++ hostname.if.5     12 Dec 2022 06:48:15 -0000
> @@ -40,13 +40,16 @@
>  The
>  .Nm hostname.*\&
>  files contain information regarding the configuration of each network 
> interface.
> -The interface can be referenced by name or lladdr, such as
> -.Pa hostname.fxp0 ,
> -.Pa hostname.00:00:5e:00:53:af ,
> -or
> -.Pa hostname.bridge0 .
> -One file should exist for each interface that is to be configured,
> -with priority given to configuration by interface name over lladdr.
> +Interfaces are referenced by name and number,
> +such as
> +.Dq hostname.fxp0 .
> +For some machines,
> +autoconfiguration makes this system inconsistent,
> +so interfaces can alternatively be referenced by
> +their link layer address (lladdr),
> +such as
> +.Dq hostname.00:00:5e:00:53:af .
> +Priority is given to configuration by interface name/number over lladdr.
>  A configuration file is not needed for lo0.
>  .Pp
>  The configuration information is expressed in a line-by-line packed format
> 

Reply via email to