Improve the wording of the "Argument List Processing" section (where
all the sh options, also used with "set", are listed) in response to
a discussion on icb conveyed to me by Darrin B. Jewell.
A few improvements to the description of the "set" built-in as well.

Bump Dd to cover all of this month's changes (so far).

(kre)

Add commented out FONT_DROID_SANS_MONO10x20 option. This enables 80x24
terminal on WVGA models, which is slightly easier on the eyes ;)

(rin)

- Accept anti-aliased fonts.
- Clear garbage from screen when attach.

(rin)

Accept anti-aliased fonts; they are supported by rotated screen found on
hpcarm/WZERO3 and PXA270-based zaurus.

(rin)

Add Droid_Sans_Mono_10x20, generated by ttf2wsfont(1).

Having 10x20 in addition to 9x18 and 12x22 may seem redundant, but
this font enables nice 80x24 and 80x30 terminals on WVGA (800x480)
and SVGA (800x600) displays, respectively :)

(rin)

Sort Droid Sans Mono family fonts by size.

(rin)

Add support to rotate anti-aliased fonts.

wsfont_rotate_{cw,ccw}_internal() have been cleaned up and unified into
new static function wsfont_rotate_internal().

(rin)

Mention that fchmod conforms to XPG 4.2. Bump date.

(wiz)

wsfont_rotate(): Add missing NULL check for wsfont_find0().

(rin)

ahcisata(4): Introduce AHCI_QUIRK_EXTRA_DELAY quirk for devices that
need extra delays as done by AHCISATA_EXTRA_DELAY option.

Enable this quirk for "C600/X79 AHCI". Also add commented out quirk
entries for "Bay Trail SATA (AHCI)" and "Mobile AHCI SATA Controller",
for which non-reproducible failures worked around by extra delays have
been reported.

500 ms of delays inserted by these option/quirk may be too much. Add
AHCISATA_EXTRA_DELAY_MS option to adjust number of delays in ms, like:

----
options AHCISATA_EXTRA_DELAY_MS=200
----

Thanks prlw1@ and jun@ for testing!

(rin)

tests/indent: refine and extend tests

(rillig)

indent: reduce casts to unsigned char for character classification

No functional change.

(rillig)

indent: keep inbuf.save_com_s and inbuf.save_com_e in sync

No functional change.

(rillig)

indent: fix included headers

(rillig)

indent: clean up io.c

No functional change.

(rillig)

indent: replace ps.procname with ps.is_function_definition

Omly the first character of ps.procname was ever read, and it was only
compared to '\0'.  Using a bool for this means simpler code, less
memory and fewer wasted CPU cycles due to the removed strncpy.

No functional change.

(rillig)

indent: fix formatting of function definitions (since 2019-04-04)

In the definition of a function with a pointer return type, the
formatting depended on the name of the function.  Function names
matching [A-Za-z+] were formatted correctly, those containing [$0-9_]
weren't.

(rillig)

indent: merge duplicate code into is_identifier_part

No functional change.

(rillig)

tests/indent: update comment for the fix from the previous commit

(rillig)

indent: fix lost function name (since 2019-04-04)

When indent searched for an identifier followed by a '(', to see whether
the identifier is a function name, it didn't care that the input buffer
could be resized due to a long line, which had made the pointer 'tp'
invalid.  Fix this by stopping the search at the end of the line.  A
better approach would be to have an unlimited lookahead buffer for
situations like these.  The code that deals with character input has
already been extracted to io.c, so it's possible to implement that now.

While here, fix another access to undefined memory, after the loop.

There is still the issue of overwriting procname[0] with a blank, which
results in inconsistent formatting depending on the function name,
probably another case of accessing undefined memory, although the
results have been reproducible, but that may have been pure luck.

The formatted code looks clearly broken, but that's still better than
losing a token and destroying the whole file.

(rillig)

indent: unexport inbuf

No functional change.

(rillig)

indent: use character input API from pr_comment.c

No functional change.

(rillig)

indent: remove all references to inbuf from indent.c

No functional change.

(rillig)

indent: move character input handling from indent.c to io.c

No functional change.

(rillig)

indent: move character input from indent.c to io.c

No functional change.

(rillig)

indent: use character input API from the tokenizer

No functional change.

(rillig)

indent: move character input handling from lexi.c to io.c

No functional change.

(rillig)

indent: replace direct access to the input buffer

This is a preparation for abstracting away all the low-level details of
handling the input.  The goal is to fix the current bugs regarding line
number counting, out of bounds memory access, and generally unreadable
code.

No functional change.

(rillig)

indent: add debug logging for input buffer handling

(rillig)

indent: rename input buffer variables

From reading the names 'save_com' and 'sc_end', it was not obvious
enough that these two variables are the limits of the same buffer, the
names were just too unrelated.

No functional change.

(rillig)

indent: group variables for input handling

No functional change.

(rillig)

PR port-evbarm/50416

Redo the previous change.  The  "snps,..." properties are on the ethernet
node and the "reset-..." properties are on the phy node.  Handle this by
creating a separate reset routine for each case.  Idea from Jared.

(jdc)

indent: prevent use-after-free bug

Triggered by the following artificial program:

---- snip ----
int *
f
(                                                            void)
{
}
---- snap ----

(rillig)

tests/indent: demonstrate disappearing function name

Since 2019-04-04, as usual.

(rillig)

tests/indent: demonstrate another bug imported from FreeBSD

Side note: the newly added test crashes at least the following versions
of indent with a segmentation fault on x86_64:

2000.10.11.14.46.04
2000.10.14.18.07.10
2000.10.17.02.16.44
2000.10.19.14.48.53
2000.10.19.16.31.26
2001.03.16.06.23.20
2001.03.23.20.12.41
2001.06.16.21.47.42
2001.07.20.13.43.50
2001.08.20.12.00.55
2001.12.01.19.27.33
2002.05.26.22.53.38
2003.02.25.10.35.44

(rillig)

tests/indent: add skeletons for testing tokens and parser symbols

The constants that were previously defined in indent_codes.h were a wild
mixture of tokens from the lexer and symbols on the parser stack.  They
were split into separate types starting at indent.h 1.49 from 2021-10-25
and finishing at 1.73 from 2021-10-31.

To match the tests with the new token names, the old tests need to be
migrated to the newly added tests.  This will take some time so first
add the skeletons and migrate them in smaller steps, cleaning them up
and extending them on the way.

(rillig)

distrib/sets: sort mi file

(rillig)

tests/indent: reorder code in test driver

No functional change.

(rillig)

indent: replace old license discussion with a brief history section

(rillig)

Fix crash because of NULL pointer reference

(manu)

Do not pass BIOS geometry when booting using EFI

Recent Mac return garbage data that will crash the code handling it,
and EFI boot does not need it anyway.

(manu)

Add a test case for backtrace(3) across a signal handler.

(thorpej)

Decorate the MIPS signal trampoline with the appropriate .cfi
directives to allow exception unwind / backtrace across a signal
handler.

(thorpej)

- Teach the LLVM-derived unwinder about the DWARF pseudo-registers defined
  by GCC for MIPS to hold the MDHI and MDLO registers, as well as the return
  address where the signal trampoline will resume.  XXX Same treatment is
  needed for MIPS64, but not done as part of this commit.
- In the MIPS validFloatVectorRegister(), compare against the internal
  register numbers, not the DWARF register numbers.

(thorpej)

gicv3: add support for mbi-alias property

The mbi-alias property, if present, contains the base address of a GICD
alias frame that contains only SET/CLRSPI registers. Use this instead of
the GICD register frame if present.

(jmcneill)

PR port-evbarm/50416

Also check the "reset_gpios", "reset-assert-us", and "reset-deassert-us"
properties for ethernet MAC reset information.  Makes the built-in ethernet
work on Odroid C1 and Odroid C2 (with mesongxbb_pinctrl.c >= r1.3).

Based on a hint from Jared.  C2 tested by Martin.

(jdc)

Add missing GPIOZ direction / input / output register defs.

(jmcneill)

s/dataseet/datasheet/ in comment.

(msaitoh)

Adjust new libcurses test filenames to the ones atually installed.
Possibly the intent was that the names used here were correct, and
the error is where they're installed - if so, that can be corrected later.

Hopefully this will fix the remaining current build issue.

(kre)

sh.1: fix typos

(rillig)

Move 'i' into DEBUG only code (now there be three).
Hopefully unbreaks !DEBUG builds.

(kre)

lint: clean up lint2

No functional change.

(rillig)

lint: clean up initialization of lint2 symbol table

No functional change.

(rillig)

echo: fix lint error due to strict bool mode

No functional change.

(rillig)

Add more check files for libcurses tests.

(blymn)

Add more testing for special character handling in ins_wstr family.

(blymn)

lint: fix check for function calls in strict bool mode

Previously, if a function call occurred in the controlling expression,
its return type could be any scalar, not just bool.  This was against
the goal of strict bool mode, which makes bool a separate and
incompabile type to all other types.  For example, it would allow
controlling expressions like 'strcmp(a, b)' without the usual '!= 0',
but only if at least one of 'a' and 'b' came from a macro definition
from a system header.

The fix is that the decision of whether the type of the controlling
expression may be scalar is no longer based on the operand types but on
the main operator of the controlling expression.

(rillig)

Fix handling of the tab special character.

(blymn)

games/caesar: fix error handling for I/O

The '!= 0' is needed for an upcoming fix to lint's strict bool mode.

The additional text in the error messages is necessary because errx only
outputs the given string, with no details from errno.  Since ferror does
not set errno, no such details are available, so just output a generic
message, which is still better than just "caesar: <stdout>".

(rillig)

lint: remove outdated comment

The bug has been fixed in cgram.y 1.369 a few minutes ago.

(rillig)

lint: finish renaming of fcon to lex_floating_constant

(rillig)

lint: fix wrong error message in strict bool mode

This fix only affects the last token before a macro, as well as the last
token in the expanded macro.

(rillig)

lint: rename attron to in_gcc_attribute

No functional change.

(rillig)

Make pwd (both /bin/pwd and the /bin/sh built-in version) check for
write errors on stdout, and indicate an error if that happens.

(kre)

PR bin/56491

Make "hash" exit(!=0) (ie: exit(1)) if it writes an error message to
stderr as required by POSIX (it was writing "not found" errors, yet
still doing exit(0)).

Whether, when doing "hash foobar", and "foobar" is not found as a command
(not a built-in, not a function, and not found via a PATH search), that
should be considered an error differs between shells.  All of the ksh
descendant shells say "no", write no error message in this case, and
exit(0) if no other errors occur.  Other shells (essentially all) do
consider it an error, write a message to stderr, and exit(1) when this happens.

POSIX isn't clear, the bug report:
    https://austingroupbugs.net/view.php?id=1460
which is not yet resolved, suggests that the outcome will be that
this is to be unspecified.  Given the diversity, there might be no
other choice.

Have a foot in both camps - default to the "other shell" behaviour,
but add a -e option (no errors ... applies only to these "not found"
errors) to generate the ksh behaviour.  Without other errors (like an
unknown option, etc) "hash -e anyname" will always exit(0).

See the PR for details on how it all works now, or read the updated man page.

While here, when hash is in its other mode (reporting what is in the
table) check for I/O errors on stdout, and exit(1) (with an error
message!) if any occurred.  This does not apply to output generated
by the -v option when command names are given (that output is incidental).

In sh.1 document all of this.  Also add documentation for a bunch of
other options the hash command has had for years, but which were never
documented.  And while there, clean up some other sections I noticed
needed improving (either formatting or content or both).

(kre)

Detect write errors to stdout, and exit(1) from some built-in
commands which (primarily) are used just to generate output
(or with a particular option combination do so).

(kre)

Fix value of ${LINENO} in "for" commands.

This affects (as best I can tell) only uses of ${LINENO} in PS4
when -x is enabled (and perhaps only when the list contains no
expansions).  "for" like "case" (which was already handled) is
special in that it generates trace output before actually executing
any kind of simple command.

(kre)

Fix a test that has been (unnoticed) failing ever since printf(1) was
changed to exit(1) when it detects a write error to stdout.

Running printf with stdout closed is guaranteed to generate such a
condition.

Until the previous commit, while the test case was actually failing
(stderr was expected to be empty. and was not) this was unnoticed.
We don't want the output (obviously), we also don't want the error
message, so just direct the latter to /dev/null.

(kre)

Correct a systematic atf_check usage error.  One must not pipe into
atf_check and simply expect it to work - the shell is permitted to,
and our shell currently does, run all commands in a pipeline in subshell
environments - when atf_check attempts to exit to indicate failure, it
only exits from that subshell, and the rest of the test continues, usually
to indicate success

Instead, when it is necessary (or just convenient) to pipe into atf_check
check the exit status of the pipeline (if atf_check is not last, which it
would usually be, then we would need the pipefail option set - there are
currently no such cases), and explicitly fail if atf_check did not exit(0).

(kre)

xorg: sprinkle a little bit of USE_FORT around

(nia)

Trailing whitespace

(skrll)

tests/lint: explain the missing error message in strict bool mode

There are 2 separate bugs that lead to the missing error message.

The first bug is that when parsing a name, the information about whether
that name comes from a system header is not properly recorded for the
last token before switching from or to a system header.

The second bug is that for determining whether the type of a function
call expression is relaxed, the types of the function call arguments are
taken into account, even though the type only depends on the function
itself.  The same idea applies to the operators '<<' and '>>', as their
result type only depends on the left operand, at least since C90.

(rillig)

Use 50Hz ticks so that time works accurately on hosts with 100Hz ticks.

Note: The 20Kc CPU isn't handled correctly by QEMU, use the 5Kc CPU for
a 64-bit guest for accurate time.

(simonb)

Use CPU frequencies defined in QEMU for mipssim - 6 MHz for 64-bit
guests and 12 MHz for 32-bit guests.

(simonb)

Use the register define MIPS_HWR_ULR instead of a magic number.

(simonb)

Use the architecture documented name ULR for the RDHWR user local
register.

(simonb)

Add some comments for the RDHWR register numbers.

(simonb)

Only need one #define for MIPS_HWR_CPUNUM.

(simonb)

Remove mb_map.

(msaitoh)

Added a kernel option to set SYNC bit of LACP
while the lagg interface is in STANDBY state

(yamaguchi)

Set SYNC bit of LACPDU when the interface is on STANDBY state

(yamaguchi)

Add handling for backspace special character (\b)

(blymn)

Explain the state of ASU and ACOMPAT.

(christos)

More cleanups and typo corrections.

(kre)

Clean up.

(wiz)

Sort SEE ALSO.

(wiz)

Add man pages for posix_spawn_file_actions_add{f,}chdir from piyush

(christos)

Add missing test

(christos)

Remove extra _, thanks RVP

(christos)

Sprinkle const (and fix build)

(skrll)

introduced APIs to configure VLAN TAG to ethernet devices

(yamaguchi)

Add new check files for curses atf.

(blymn)

Update the ins_(n)wstr test family to split out the check files
for each call separately and update check files to reflect changed
behaviour due to a fix to related calls.

(blymn)

Fix for PR lib/55433

Correct the behaviour for ins_wstr:
* Properly check the string will fit by processing any special
  characters present when preforming the check.
* Simplify the routine by removing code that duplicates the code in
  _cursesi_addwchar and just call _cursesi_addwchar.

(blymn)

tpm@acpi: Require only one locality's worth of register space.

We don't actually use the registers for the other localities, and
some older TPMs only have the first locality exposed via ACPI.

(riastradh)

fix few typos in comments and log message.

(andvar)

Fix some regressions that were introduced.

One must make sure that you release the i2c bus if you acquire.

One must not try to release the i2c bus if you failed to acquire it.

One can't leave too early from the single-shot refresh without
checking the status register as there may have been a reset
and releasing the i2c bus.

(brad)

arm: Fix CPU startup synchronization.

- Use load-acquire instead of (wrong) membar_consumer then load in
  cpu_boot_secondary_processors and cpu_hatched_p.

  => (Could use load then membar_consumer instead but load-acquire is
    shorter.)

- Issue  dmb ish  before setting or clearing the bit in
  cpu_set_hatched and cpu_clr_mbox to effect a store-release.

  => (Could use membar_exit, which is semantically weaker than  dmb ish
    but on arm is just implemented as  dmb ish.)

  => (Could use stlr except we don't have atomic_ops(9) to do that.)

This way, everything before cpu_set_hatched or cpu_clr_mbox is
guaranteed to happen before everything after
cpu_boot_secondary_processors, which was previously not guaranteed.

(riastradh)

Make dtc on riscv

(skrll)

Trailing whitespace

(skrll)

tests/lint: demonstrate missing errors for strict bool mode

Seen in bin/echo/echo.c.

(rillig)

UPDATING: note dts update causing things to be renumbered

(nia)

revert bPreferedVersion rename. It's named like the by spec (from jmcneill).

(andvar)

Use BITS(3) some more.  Same nvme.o before and after.

(skrll)

firmware ver1 doesn't seem to work with linkstat interrupt, so in this case also set it to polling mode.

- Make the version acquisition (aq_fw_reset) timing earlier.
- Unmap registers when terminating with attach after failing to set interrupt

(ryo)

s/bPreferedVersion/bPreferredVersion/ in struct property definition.

(andvar)

Better display in monochrome.

(nat)

tests/make: extend test for undefined variable in doubly indirect ':='

Just to prevent a half-baked fix to the current behavior that would
concatenate the modifiers of the two expressions, leading to
${LATER:value=sysv:tl} in this case.  That expression would be
interpreted as having only a single modifier that would replace the
suffix 'value' with 'sysv:tl'.  This is because the SysV modifier
':from=to' spans until the end of the expression.

(rillig)

tests/make: test double indirection in ':=' with undefined variable

Reported by Simon J Gerraty via private mail.

This edge case has been handled in essentially the same way since at
least 2000-05-30, probably several years earlier as well.  The test file
is:

---- snip ----
.undef LATER
.undef later
INDIRECT:= ${LATER:S,value,replaced,}
indirect:= ${INDIRECT:C,S,s,}
# expect+1: Unknown modifier "s,value,replaced,"
.if ${indirect} != ""
.  error
.else
.  warning XXX Neither branch should be taken.
.endif
LATER= uppercase-value
later= lowercase-value
# expect+1: Unknown modifier "s,value,replaced,"
.if ${indirect} != "uppercase-replaced"
.  warning XXX Neither branch should be taken.
.else
.  error
.endif

all:
@:;
---- snap ----

The output from 'make -r -f later.mk' is:

make-2000.05.30.02.32.21
| make: Unknown modifier 's'
|
| "later.mk", line 9: Need an operator
| make: Unknown modifier 's'
|
| "later.mk", line 15: Need an operator
| Fatal errors encountered -- cannot continue
| exit status 1
make-2000.12.30.16.38.22

The pathnames in the error message gets absolute:

make-2001.01.23.02.48.05
| make: Unknown modifier 's'
|
| ".../later.mk", line 9: Need an operator
| make: Unknown modifier 's'
|
| ".../later.mk", line 15: Need an operator
| Fatal errors encountered -- cannot continue
| exit status 1
make-2001.01.23.02.48.05

All error messages get 'make:' as a common prefix:

make-2001.02.23.21.11.38
| make: Unknown modifier 's'
|
| make: ".../later.mk" line 9: Need an operator
| make: Unknown modifier 's'
|
| make: ".../later.mk" line 15: Need an operator
| make: Fatal errors encountered -- cannot continue
| exit status 1
make-2001.05.29.17.37.52

The 'stopped in' gets added:

make-2001.06.12.23.36.18
| make: Unknown modifier 's'
|
| make: ".../later.mk" line 9: Need an operator
| make: Unknown modifier 's'
|
| make: ".../later.mk" line 15: Need an operator
| make: Fatal errors encountered -- cannot continue
|
| make: stopped in ...
| exit status 1
make-2002.02.21.22.21.34

The empty lines between the error messages get removed.

make-2002.03.21.11.42.21
| make: Unknown modifier 's'
| make: ".../later.mk" line 9: Need an operator
| make: Unknown modifier 's'
| make: ".../later.mk" line 15: Need an operator
| make: Fatal errors encountered -- cannot continue
|
| make: stopped in ...
| exit status 1
make-2009.10.15.02.27.44

The error message for unknown directives gets more helpful:

make-2009.11.19.06.48.37
| make: Unknown modifier 's'
| make: ".../later.mk" line 9: Unknown directive
| make: Unknown modifier 's'
| make: ".../later.mk" line 15: Unknown directive
| make: Fatal errors encountered -- cannot continue
|
| make: stopped in ...
| exit status 1
make-2010.02.22.19.20.33

The directives '.error', '.warning' and '.info' get added:

make-2010.04.29.23.12.21
| make: Unknown modifier 's'
| make: ".../later.mk" line 9: warning: XXX Neither branch should be taken.
| make: Unknown modifier 's'
| make: ".../later.mk" line 15: warning: XXX Neither branch should be taken.
| exit status 0
make-2020.12.20.19.37.23

The error message about an unknown variable modifier gets line number
information:

make-2020.12.20.19.47.34
| make: ".../later.mk" line 6: Unknown modifier 's'
| make: ".../later.mk" line 9: warning: XXX Neither branch should be taken.
| make: ".../later.mk" line 14: Unknown modifier 's'
| make: ".../later.mk" line 15: warning: XXX Neither branch should be taken.
| make: Fatal errors encountered -- cannot continue
| make: stopped in ...
| exit status 1
make-2021.02.23.15.07.58

The error message about an unknown variable modifier gets more context
than only a single letter:

make-2021.02.23.15.19.41
| make: ".../later.mk" line 6: Unknown modifier "s,value,replaced,"
| make: ".../later.mk" line 9: warning: XXX Neither branch should be taken.
| make: ".../later.mk" line 14: Unknown modifier "s,value,replaced,"
| make: ".../later.mk" line 15: warning: XXX Neither branch should be taken.
| make: Fatal errors encountered -- cannot continue
| make: stopped in ...
| exit status 1

(rillig)

fix aarch64 build

(jakllsch)

fix the remaining vestiges of __sigaction14; now the tsan signal tests work.

(christos)

Write back and invalidate cache before starting secondary CPUs.

(jmcneill)

If lookup_fastforward() loses an intermediate searchdir, has to roll
back and retry it must use the initial searchdir from *searchdir_ret
for lookup_parsepath().

(hannken)

Brad wants the __did_not_work code back :-)

(christos)

Add support for RK3288 temperature sensors.

(jmcneill)

Release interrupt lock while allocating ressources in trigger
functions.

Also be more verbose about errors.

(mlelstv)

Hook in riscv dtb build

(skrll)

Correct a comment

(skrll)

Update notes and import riscv dts

(skrll)

Set ACTLR.SMP=1 on Cortex-A17

(jmcneill)

Add rkpwm, rkspi

(jmcneill)

Add pwm and spi clocks

(jmcneill)

Fix tyop in a comment.

(simonb)

Match rockchip,rk3288-i2c

(jmcneill)

Fix width of aclk_cpu_pre divider field

(jmcneill)

add rkv1crypto

(jmcneill)

rk3288: add watchdog and rng clocks

(jmcneill)

Support 'boot -1' on arm

(jmcneill)

Fix register accesses to PMU registers. Unlike the GRF ones, a RMW cycle
is required to update settings here.

(jmcneill)

simplify the code and remove unused (it is in CVS anyway). Try to merge
duplicate code. Follow KNF. Brad, please test!

(christos)

evbarm: Add support for Rockchip RK3288 SoCs.

(jmcneill)

arm: rockchip: Add support for RK3288 SoC.

The Rockchip RK3288 is a quad core Cortex-A17 SoC.

(jmcneill)

gtmr: Add support for arm,cpu-registers-not-fw-configured property.

On armv7, arm,cpu-registers-not-fw-configured means that firmware hasn't
bothered to configure any generic timer registers and we need to
initialize cntfrq ourselves.

(jmcneill)

dw_apb_uart: Honour reg-io-width property

(jmcneill)

com: Add support for 32-bit IO accesses.

(jmcneill)

regen

(jmcneill)

Build rockchip dtb files for armv7.

(jmcneill)

fix license

(macallan)

crude beginnings of a driver for the Southland Media / Quantum 3D MGX
basic acceleration mostly works in 24bit colour
needs native hw cursor code
windowmaker produces visible artifacts, but it's good enough to run amiwm and
a few xterms

(macallan)

Fix the serial number handling of the HTU21D chip and probably others.
Increase the number of read attempts as the HTU21D and probably others
do not respond as fast the actual SI70xx chip can.

(brad)

Explain cd9660 and udf exceptions on the rule that the default sectorsize is
512.

(reinoud)

Trailing whitespace

(skrll)

Fix the conditional for failed interrupt establishment

(skrll)

KNF

(skrll)

Trailing whitespace

(skrll)

Trailing whitespace

(skrll)

Print a big warning about trying to run on early ThunderX parts

(skrll)

Configure vlan to an added interface after setting ifnet::if_lagg

The configuration uses ioctl of the interface, and the ioctl
for port (lagg_port_ioctl) needs ifnet:::if_lagg setting.

(yamaguchi)

lagg: Add vid to vlanid_list in ethercom

(yamaguchi)

Fix the wrong check of interface type

- lp->lp_iftype: original ifnet::if_type
- lp->lp_ifp->if_type: current ifnet::if_type
    - always IFT_IEEE8023ADLAG

(yamaguchi)