- Enable PBCC based rate 22Mbit/s
- Finish incomplete acx_watchdog()
- Add acx_media_change(), which re-initialize NIC and returns 0 if
  ieee80211_media_change() returns ENETRESET.  Use acx_media_change()
  as change_callback for ifmedia instead of ieee80211_media_change().
  This unbreaks 'ifconfig acx0 list sta' in HOSTAP operation mode.
- Nuke no longer needed debug messages

Add PBCC modulation support in netproto/802_11:
- Since ieee80211com.ic_caps is already crowded, add ieee80211com.ic_caps_ext
  to extend it.  Currently only IEEE80211_CEXT_PBCC, which should be set by
  the drivers capable of PBCC modulation, is defined for this field.
- Adapt SIOCG80211:IEEE80211_IOC_DRIVER_CAPS ioctl handling to expose the
  content of ieee80211com.ic_caps_ext.  Old semantic of this ioctl is not
  changed, given ieee80211req is zeroed out before the ioctl call.
- Add debug information about peer node's rate set (after negociation)
- Set PBCC bit in Capability ie base on the value of
  (ieee80211com.ic_caps_ext & IEEE80211_CEXT_PBCC).  This is only done for 2GHz
  channels.
- Teach ifconfig(8) to print ieee80211.ic_caps_ext upon "list caps" command.

Properly set ifmedia_entry.ifm_data when nge(4) tries to camouflage an
isolation media entry, and restore it once media isolation is done.

Reported-by: Steve O'Hara-Smith <steve@sohara.org>
Tested-by: Steve O'Hara-Smith <steve@sohara.org>
Reviewed-by: dillon@

Don't use unprocessed (e.g. negociated) peer node's rate set to fill
Supported rates ie of Association Request, instead, use own supported
rate set (i.e. ieee80211com.ic_sup_rates).

In the orginal way, following senario is broken:
- STA is 11b
- AP is 11g
STA will send out sorted AP's rate set in Association Request.  Obviously
STA will not be able to receive at any data rates > 11Mbit/s.  So this
breaks IEEE Std 802.11b subclause 10.3.3.1.2:
"... The STA must be able to receive at each of the data rates listed
 in the set. ...".

According to IEEE Std 802.11b-1999 subclause 7.3.1.4:
"... STAs shall set the Short Preamble subfield to 1 in transmitted
 Association Request and Reassociation Request MMPDUs when the MIB
 attribute dot11ShortPreambleOptionImplemented is true. ..."

For us, 'dot11ShortPreambleOptionImplemented' is
(ieee80211com.ic_caps & IEEE80211_C_SHPREAMBLE)

M_NOWAIT -> MB_DONTWAIT

Sync zoneinfo database with tzdata2006p from elsie.

australasia:    8.3 -> 8.4
zone.tab:       8.5 -> 8.6

From Paul Eggert's comments:

Changes that affect near-future time stamps:

  * Western Australia starts DST on December 3.  (Thanks to
    Jesper Norgaard Welen for the initial warning, and to Christopher Hunt
    for letting us know it was official.)

    This patch migrates the WA DST idiosyncracies into Rule lines,
    where they probably belonged anyway.  I realize the official
    legislation specifies dates rather than "Mar lastSun", but we
    commonly reverse-engineer the underlying rules in other situations
    and might as well do so consistently here.

Changes affecting location data only:

  * The coordinates for Jersey and Podgorica were off.
    (Thanks to Andy McDonald for noting this.)

Wrong #undef

The Problem [1]:
In em_process_receive_interrupts(), if em_get_buf() fails, which
may be a common case since it calls m_getcl() with MB_DONTWAIT,
then the rest of the RX ring are _not_ processed at all.  The RX
ring may contain many, or even worse is full of, ready RX descriptors.
RDT will not be updated too, in this case.  So hardware will take
this situation as there is _no_ free space in RX ring, and this
probably leads to following catastrophe:
too many RX engine overruns -> RX engine stalls -> interrupts stall [2]

The Fix:
If em_get_buf() fails then
- The current RX descriptor still will be reaped, but the frame,
  which belongs to the current RX descriptor, is not delivered to
  the upper layer.
- RX ring processing keeps going as if there is no failure happens.

Thank dillon@ to help diagnose the problem and give various hints [3]

Thank Mike Tancsa <mike@sentex.net> to provide enough information
to locate the problem.

Reviewed-by: dillon@
Reported-by: Mike Tancsa <mike@sentex.net>
Tested-by: Mike Tancsa <mike@sentex.net> (82571EB copper dual port)
           me (82540EM)

#
# [1] This problem unveiled itself when Mike tried to bench packet
#     forwarding performance.  The information he had provided that
#     is critical to locate the problem is:
#     1) Both polling(4) and normal mode (interrupt based) do not work.
#        This means that interrupt processing possibly not the root of
#        the problem (pointed out by dillon@)
#     2) One line in the output of sysctl hw.em0.debug_info=1:
#        ...
#        em0: Std mbuf cluster failed = 2
#        ...
#        This statistics is the value of adapter->mbuf_cluster_failed,
#        which is updated only when m_getcl() in em_get_buf() fails
#
# [2] This is just a guess from the output of 'vmstat -i' provided by Mike
#     Before his benching:
#     interrupt                   total       rate
#     ...
#     em0                            28          0
#     ...
#     After his benching (i.e. em0 choked):
#     interrupt                   total       rate
#     ...
#     em0                            62          0
#     ...
#     See, only 34 interrupts came o_O
#
# [3] There is still prossibility that RX engine gets confused by
#     the driver.  It is described by dillon@:
#     (I didn't find the URL in the archive, so it is pasted here)
#     "...
#      Clearly when that case occurs ALL the receive frames will be full.
#      Lets look at a degenerate case:
#
#      [0 ...................... N-1]
#      * RDH set to 0
#      * RDT set to N-1
#      * N frames come in  RDH is set to N-1 (??)
#      * We process N frames
#      * The frame at RING[N-1] is cleaned up
#      * i = N
#      * We set RDT to i-1 == N-1.  It's the same value it was set to before
#        we processed all N frames.  The receive engine will think that the
#        ring is still full when it is empty.
#
#      I Think what we need to do here is set RDT to N-2 (mod N of course) in
#      the case where we have processed ALL N frames.  I'll bet the firmware
#      is getting confused in the overrun case because we are setting RDT to
#      the same value it was set at before.
#      ..."
#

A u_short cannot be greater than 0xffff, so remove those checks.

Use LIST_FOREACH()

Submitted-by: Gary Stanley <gary@velocity-servers.net>
Obtained-from: OpenBSD

- Factor out front part of ieee80211_set_shortpreamble() into
  ieee80211_update_shpreamble().  So that the former only turns
  on/off short preamble and informs driver about the changes,
  while the latter makes the decision whether short preamble
  should be turned on or off.
- Replace rest of the places where short preamble is turned on/off
  manually with ieee80211_set_shortpreamble()
- For associated STAs, update short preamble state only when related
  fields of ERP information or capability information in received
  beacons are changed

Add an error if the patch file doesn't actually exist.

Dragonfly-bug: <http://bugs.dragonflybsd.org/issue343>
Submitted-by: y0netan1

Add the directory created by the comma syntax to CLEANDIRS.

Dragonfly-bug: <http://bugs.dragonflybsd.org/issue384>
Submitted-by: corecode

Correct short preamble support.

Problems:
- The assumption that "Short Preamble" field in "Capability information"
  will not be changed, is not correct at least for a *BSD based HostAP
- "Barker Preamble mode" in "EPR information" is ignored

Fixes:
- Add a new function ieee80211_set_shortpreamble() (named after its relative
  ieee80211_set_shortslot()):
  1) For 11a mode, always turn on short preamble, though it is not used under
     11a mode
  2) For 11g mode (ERP), turn on short preamble iff all of the following
     conditions are met:
     o  "Short Preamble" field in "Capability information" is 1
     o  "Barker Preamble mode" in "ERP information" is 0 [*]
     o  ieee80211.ic_caps has IEEE80211_C_PREAMBLE set
  3) For the reset of the mode (11b?), turn on short preamble iff all of the
     following conditions are met:
     o  "Short Preamble" field in "Capability information" is 1
     o  ieee80211.ic_caps has IEEE80211_C_PREAMBLE set
- Add ieee80211com.ic_upadate_preamble.  This field could be set by drivers
  which want to be informed about the preamble changes

#
# [*]
# IEEE Std 802.11g-2003 subclause 7.3.2.13 page 10:
# ...
# ...
# ERP APs and ERP STAs shall use long preambles when transmitting Clause 15,
# Clause 18, and Clause 19 frames after transmission or reception of an ERP
# Information Element with a Barker_Preamble_Mode value of 1 in an MMPDU to
# or from the BSS that the ERP AP or ERP STA has joined or started, regardless
# of the value of the short preamble capability bit from the same received or
# transmitted MMPDU that contained the ERP Information Element ...
# ...
# ...
#

Move conditional preprocessing out from the IEEE80211_DPRINTF macro
invocation.  Per C99 6.10.3 paragraph 11 preprocessing directives
appearing as macro arguments yield undefined behavior.

Obtained-from: FreeBSD (dss@freebsd.org)

Ensure outbound data packets in hostap mode are delivered only to stations
that are associated by making ieee80211_find_txnode return NULL when a
unicast frame is to be delivered to an unassociated station.

Obtained-from: FreeBSD (sam@freebsd.org)

Add per-sta ucast/mcast stats

Obtained-from: FreeBSD (sam@freebsd.org)

Nuke macros that are moved to ieee80211.h

- Add some missing stats to the global ieee80211_stats
- Move accounting work for data frame rx into ieee80211_deliver_data(),
  so L2 update frames are counted
- Add per-sta stats for rx ucast/mcast frames

Strip TABs on empty lines, while I'm here.

Obtained-from: FreeBSD (sam@freebsd.org)

# NOTE: mbuf.m_pkthdr.rcvif still need to be set before calling
#       ieee80211_deliver_data()

- Add IEEE80211_IOC_BMISSTHRESHOLD for managing the beacon miss threshold
- Teach ifconfig(8) to get/set beacon miss threshold

Obtained-from: FreeBSD (sam@freebsd.org)
# It is submitted to FreeBSD by Henrik Brix Andersen <henrik@brixandersen.dk>

- Add noise floor in ieee80211_nodestats; return it as zero until we can
  update the api's so the driver can provide noise floor data
- Add a bandaid so IEEE80211_IOC_STA_STATS works for sta mode; when all
  nodes are in the station table this will no longer be needed
- Fix braino in IEEE80211_IOC_STA_INFO implementation; was supposed to
  take a MAC address and return info for that sta or all stations if
  ff:ff:ff:ff:ff was supplied -- but somehow this didn't get implemented;
  implement the intended semantics

Obtained-from: FreeBSD (sam@freebsd.org)

# NOTE: The RSSI statistics in ieee80211_nodestats is still unsigned

If power save mode changes, call ieee80211com.ic_reset instead of forcefully
changing (by returning ENETRESET) 802.11 state machine, so that driver has
the chance to intercept this change and possibly avoid changing 802.11 state
machine.

Obtained-from: FreeBSD (sam@freebsd.org)

Implement set(IEEE80211_IOC_STA_STATS) for hostapd; for
now just make it clear station statistics (could read
a stat block and assign to caller can do partial changes)

Obtained-from: FreeBSD (sam@freebsd.org)

- Tighten invariant on loops used to parse ie's; this ensures we never
  touch data outside the packet (previously we might touch 1 byte); it
  also has the happy side effect of working around broken orinoco/agere
  firmware that sends malformed association response frames
- Correct ie length check; need to include fixed part of ie

Obtained-from: FreeBSD (sam@freebsd.org)

Use spinlocks instead of tokens to interlock the objcache depot.
The cpu-localized critical path is still lockless.

Fictitious VM pages must remain structurally stable after free.

Add minimal umodem(4) manual page.

Taken-from: FreeBSD

Update for OpenSSH-4.5p1, and add some upgrade notes.

Remove files no longer in vendor's distribution.

Update build for OpenSSH-4.5p1.

Merge from vendor branch OPENSSH:
Import OpenSSH 4.5p1.

Import OpenSSH 4.5p1.

Rework rt2661 TX interrupts processing

The Clues: [1]
"TX DMA Done Interrupt":
A frame has been DMAed to ASIC, probably stored in an ASIC internal
TX FIFO.  The TX descriptor and the resource in host memory are useless
to ASIC

"TX Done Interrupt":
The transfering of a frame, which probably is from ASIC internal TX
FIFO, is done by ASIC, and the information concerning this transfering
is available

The Problems:
- The original driver reduces TX ring occupation counter on "TX Done
  Interrupt", though TX descriptors and related resources are already
  reaped on "TX DMA Done Interrupt"
- For some (maybe all ;-) ASIC, "TX Done Interrupt" comes less frequent
  than "TX DMA Done Interrupt", probably due to the "ASIC internal TX
  FIFO", or just because it is not intended to be used in the original
  driver's way ;-)

So useless TX descriptors slowly occupy TX ring and result in an almost
full TX ring at the most of the time if new frames keep coming at high
rate (e.g. during a long time scp(1))

The Fixes:
- Move rate control context out of rt2661_tx_data, so it is not 1:1 mapping
  with TX descriptor.  This degenerates rt2661_tx_data to rt2661_rx_data,
  so merge them and rename the new structure rt2661_data
- Add a rate control context queue, which temporarily holds rate control
  contexts for each data transfering
- Allocate the rate control context on fly during rt2661_tx_data().  Since
  this context is allocated using M_NOWAIT, it is possible that the
  alloctation fails.  if this happens, later rate control information will
  not be gathered (I think this is acceptable :-) and the reference to the
  ieee80211_node is freed immediately at the end of rt2661_tx_data().

After above three changes reaping of TX descriptors and related resources no
longer needs to be synchronized with rate control information gathering.

- Change the meaning of rt2661_tx_desc.qid, it now indicates whether rate
  control information needs to be gathered (it is not necessary at all for
  control/management frames)
- On "TX DMA Done Interrupt" instead of on "TX Done Interrupt", if some TX
  descriptors and related resources are reaped, reduce TX ring occupation
  counter, and kick ifnet.if_start

As of this commit, my Edimax EW-7608PG can tranfer data stably [2] without
being choked half way.

#
# [1] These clues are infered from a comment concering TX descriptor
#     structure in Linux RT61 driver
# [2] Though data tranfering is quite stable, its performance is still
#     far from expectation.  In infrastructured mode, 3 meters away from
#     the AP, I only got 6Mbits/s - 7Mbits/s, but under the same conditions
#     receiving performance is quite OK (~20Mbits/s)
#

We aren't using the asm files so store this in the Attic.

Remove files no longer used.

Update READMEs for latest import.

The ":R" operator is never needed after ":S/./_/g", which replaces
dots with underscores, removing any "suffixes" from the string.

Upgrade to OpenSSL 0.9.8d

Merge from vendor branch OPENSSL:
Import OpenSSL 0.9.8d.

Import OpenSSL 0.9.8d.

Document pentium-m support

Reported-by: TGEN

Add support for pentium-m CPUs.

Fix typo.

Reported-by: Robin Carey <robin_carey5@yahoo.co.uk>

ifconfig manpage upgrade:

* Bring in all relevant changes from FreeBSD.

* Add tentative/-tentative and deprecated/-deprecated documentation from
  NetBSD.

* Document the autoconf/-autoconf, powersavemode, compress, normal and
  noicmp parameters.

* Add ifmedia(4) reference.

* mdoc(7) cleanup

Reviewed-and-helped-by: sephe

Nuke redundant declaration of 'nfs_diskless_valid'

Don't append extra _H in forward headers' guardian, since
sys/cpu/i386/misc/atomic.c depends on *exact* _MACHINE_ATMOIC_H_ to
create necessary atomic_*_*() symbols for modules to use.

- Add kern.fwd.mk to generate forward headers for missing machine/*.h
  It avoids unnecessary regeneration of forward headers by default, thus
  quickkernel is speedy again :)
- Nuke no longer needed generate_forwarding_headers() in config(8)
- Add forward headers generation to the dependency list of "beforedepend"
  in kmod.mk.  This unbreaks module building which does not happen along
  with kernel building.

The original patch was submitted by y0netan1@, changes made to the original
patch are OKed by him too.

Reviewed-by: dillon@ corecode@

Add stge(4).

an, awi, cue and kue don't use the miibus interface.

- Cleanup #includes
- Add missing '\n'

Don't turn on TX_BUSY (rt2560/rt2661) or TX_CIPHER_BUSY (rt2560) before TX
descriptor is fully setup.

Correct layout of RT2661_TXRX_CSR4

Obtained-from: Linux RT61 driver

#
# Result of:
# ping -c 100 -i 0.1 -s 1300 192.168.3.2
#
# Before this commit
# --- 192.168.3.2 ping statistics ---
# 100 packets transmitted, 37 packets received, 63% packet loss
# round-trip min/avg/max/stddev = 1.578/1.844/3.113/0.310 ms
#
# After this commit
# --- 192.168.3.2 ping statistics ---
# 100 packets transmitted, 100 packets received, 0% packet loss
# round-trip min/avg/max/stddev = 2.043/2.894/4.514/0.529 ms
#
# Maybe "63% packet loss" should be taken as "this card works well" :-P
#

Properly setup Link/Act LED for rt2x61 chips.

LED EEPROM configures and LED MCU data/command layout are obtained from
Linux RT61 driver (the reference driver?).

# This makes my Edimax EW-7608PG(rt2661d) LED shining as expected.

Reflect reality.

Taken-from: FreeBSD

Fix references.

Fix .Xr abuse.

unresolve the vnode associated with the namecache entry for a mount point
before trying to set a new vnode.  This avoids a panic if you are CD'd into
a mount point before the mount occurs.

Disable use of wide char functions for mkmagic.nx, a host tool.  This
unbreaks buildworld for pre-citrus machines (1.2-RELEASE).

Reported-by: Thomas E. Spanjaard

Change to ${.CURDIR} before generating miidevs.h

This unbreaks miidevs.h regeneration if sys/dev/netif/mii_layer has an
object directory.

Thank joerg@ to shed light on the obscure awk error.

Add manpage for stge(4)

Obtained-from: NetBSD (with changes)
Reviewed-by: swildner@

- Add stge(4) for Sundance/Tamarack TC9021 Gigabit Ethernet chip.
  It supports following cards:
    o  Antares Microsystems Gigabit Ethernet
    o  ASUS NX1101 Gigabit Ethernet
    o  D-Link DGE-550T Gigabit Ethernet
    o  IC Plus IP1000A Gigabit Ethernet
    o  Sundance ST-2021 Gigabit Ethernet
    o  Sundance ST-2023 Gigabit Ethernet
    o  Sundance TC9021 Gigabit Ethernet
    o  Tamarack TC9021 Gigabit Ethernet
- Add PHY module for IC Plus IP1000A integrated PHY, which may be used
  by some on-board stge(4)
- Hook stge(4) into GENERIC and LINT

Obtained-from: FreeBSD (yongari@freebsd.org)

regen

Add ids for Marvell 88e3082 and IC PLUS IP1000A

Make sure BBP is ready before writing to it.

- Correct register name
- Add comment abount some registers

Taken-from: Linux rt2500 driver

Clean up #includes

Merge from vendor branch FILE:
Use the vendor-supplied man pages for file.1 and magic.5 instead of the
stale ones we have now.

Reviewed-by: vbd

Use the vendor-supplied man pages for file.1 and magic.5 instead of the
stale ones we have now.

Reviewed-by: vbd

Use the vendor-supplied man pages for file.1 and magic.5 instead of the
stale ones we have now.

Reviewed-by: vbd

Don't break buildworld for upgrading users:  "i386" is no valid platorm
name anymore, so we automatically convert it to "pc32".
Remove checks for build/installworld, as sys.mk deals with fixing now.

Discussed-with: pavalos
DragonFly-issue: http://bugs.dragonflybsd.org/issue379 (among others)

Fix for the latest developments in the kernel architecture reorganization.

Update re(4) manpage to reflect recent re(4) changes.

Reviewed-by: swildner@

Sync re(4) with FreeBSD:
- Add support for RealTek 8169SC/8110SC and RTL8101E devices.  The latter
  is a PCIe 10/100 chip.
- Add support for RealTek RTL8168(B?)
- Fix EEPROM reading code
- Disable diagnostic code in re_attach() by default.  It is almost useless
  and has caused much trouble.
- Manually padding small IP datagrams to work arround hardware checksum
  offload bug [1].  Enable IP/TCP/UDP checksum offload after this fix.
- Work arround hardware TX bug in some PCIe re(4) devices:
  The TX command, which is issued when there is transmission in progress,
  will get lost [2].
  So at the end of re_txeof(), if there are still packets sitting in the
  TX ring, we kick the TX engine again.
- Add a sysctl hw.reX.tx_moderation to turn on/off TX moderation.  It is
  on by default.
- Move softc related structs from if_rereg.h into newly created if_revar.h

Thank Bill Paul (wpaul@freebsd.org) and many other people for their work
on this driver.

#
# [1] Detailed description of this bug is at:
#     FreeBSD dev/re/if_re.c rev1.70 by wpaul@freebsd.org
#
# [2] Detailed description of this bug is at:
#     FreeBSD dev/re/if_re.c rev1.71 by wpaul@freebsd.org
#

update manpage according recent sk(4) changes

Reviewed-by: swildner@

- busdma(9)fy
- Use a per-controller serializer instead of per-driver serializer
- Supports more chips
  Obtained-from: OpenBSD
- Reduce interrupt moderation timer, thus decrease response latency
  Obtained-from: NetBSD via OpenBSD
- Fix watchdog timeout errors seen on a few systems:
  "The SK-NET GENESIS document says reading the SK_ISSR register should stop
   generating further interrupts, but it seems there is the possibility
   of losing interrupts between reading the SK_ISSR register and determining
   which interrupts have been reported. To cope with this situation we
   continuously read the SK_ISSR register until there are no interrupts.

   However, it seems that the above workaround does not fix all cases.
   Add workaround code that tries to protect against false alarms from
   the watchdog handler by trying to reclaim pending Tx descriptors
   before resetting the hardware.

   This should fix occasional watchdog timeouts seen with this driver."
  Obtained-from: FreeBSD via OpenBSD
- Move softc related structs from if_skreg.h into newly added if_skvar.h

Thanks for all of the people for their working on this driver.

regen

- GALILEO -> MARVELL
- Add some Yukon/YukonII PCI ids

Add some more PCI IDs for the SATA300 controllers of newer Intel boards.

Sync the uplcom(4) driver with FreeBSD.

Submitted-by: Johannes Hofmann <Johannes.Hofmann@gmx.de>
Reviewed-by: dillon
I did some minor fixes and synced the manpage, too.

Regenerate

Add USB serial adapters/cable.

Submitted-by: Johannes Hofmann <Johannes.Hofmann@gmx.de>

Upon further review, VERSION isn't even used in libmagic.3.

Avoid duplicated ifname in debug message.

Obtained-from: FreeBSD (sam@freebsd.org)

Our errno is in section 2, so patch the vendor provided man page.

Thanks again to swildner for the manpage markup review.

Add Belarus locales (identical with FreeBSD's).

Submitted-by: Yury Tarasievich <yury.tarasievich@gmail.com>
The following modifications were done by me:

* Fix a comment in be_BY.CP1131.src (silences a preprocessor warning).

* Add a definition for the panic key to the keymap files (as all our
  keymaps have).

* Use FreeBSD's rev. 1.2 of share/mklocale/be_BY.CP1131.src (comments
  correction).

Rename boot_i386.8 to boot_pc32.8 by repo-copy in order to enable the
creation of the boot.8 MLINK again.

Remove boot_i386.8 with 'make upgrade'.

Add a check for installworld (similar to the buildworld check) to ensure
that MACHINE is set to 'pc32' before installing world.

Reported-by: Thomas Schlesinger, Rumko and several other people

Add a libmagic.3 manual page.

Thank you to swildner for the implementation of pulling VERSION from
config.h.

Merge from vendor branch FILE:
Add a libmagic.3 manual page.

Thank you to swildner for the implementation of pulling VERSION from
config.h.

Add a libmagic.3 manual page.

Thank you to swildner for the implementation of pulling VERSION from
config.h.

Fix warning.

Remove files not in the new version.

Upgrade to file-4.18.

Merge from vendor branch FILE:
Upgrade to file-4.18.

Upgrade to file-4.18.

Typo from previous commit (forgot the trailing */).

Revert the previous commit in libmagic to make both libmagic and file(1)
use the config.h in lib/libmagic so we don't have multiple copies of the
same file in the src tree.  Having file(1) rely on files in libmagic is
slightly more logical than the previous behavior.

Revert the previous commit in libmagic to make both libmagic and file(1)
use the config.h in lib/libmagic so we don't have multiple copies of the
same file in the src tree.  Having file(1) rely on files in libmagic is
slightly more logical than the previous behavior.

Fix another typo.

Noticed-by: Trevor Kendall <trevorjkendall@gmail.com>