CVS log for src/sys/dev/ic/mfi.c

Up to [cvs.NetBSD.org] / src / sys / dev / ic

Request diff between arbitrary revisions

Keyword substitution: kv
Default branch: MAIN

s/mangement/management/ in comments.

fix various typos in comment, documentation and log messages.

Use PRIxBUSADDR for bus_addr_t in debug printf.

Kernel i386/ALL compiles again.

Pull up following revision(s) via patch (requested by msaitoh in ticket #1454):
	sys/dev/ic/mfi.c: revisions 1.63, 1.66-77
	sys/dev/ic/mfireg.h: revisions 1.11-1.20 via patch
	sys/dev/pci/mfi_pci.c: revision 1.21
	sys/dev/pci/mfii.c: revisions 1.6-1.7, 1.10-1.15
	share/man/man4/mfi.4: revision 1.13

Improve mfi(4) and mfii(4):
- Set 'ld_sync' to NULL as part of 'again', to prevent use-after-free.
- Add some code for the SKINNY variant to make Dell PERC H310 work.
- Print the percentage correctly when the background initialization is
  running.
- Clear mailbox to not to pass garbage data.
- Use union mbox instead of unit8_t xxx[] to avoid unaligned access.
- Set stripe size for BIOCVOL to show the size correctly in bioctl.
- Add support for iBBU-09 to show BBU voltage, current and temperature
  correctly.
- Fix typos in comments.
- Sprinkle static.
- Improve debug printf()s.
- KNF. Remove extra semicolon. Whitespace fixes.

Pull up following revision(s) via patch (requested by msaitoh in ticket #1454):
	sys/dev/ic/mfi.c: revisions 1.63, 1.66-77
	sys/dev/ic/mfireg.h: revisions 1.11-1.20 via patch
	sys/dev/pci/mfi_pci.c: revision 1.21
	sys/dev/pci/mfii.c: revisions 1.6-1.7, 1.10-1.15
	share/man/man4/mfi.4: revision 1.13

Improve mfi(4) and mfii(4):
- Set 'ld_sync' to NULL as part of 'again', to prevent use-after-free.
- Add some code for the SKINNY variant to make Dell PERC H310 work.
- Print the percentage correctly when the background initialization is
  running.
- Clear mailbox to not to pass garbage data.
- Use union mbox instead of unit8_t xxx[] to avoid unaligned access.
- Set stripe size for BIOCVOL to show the size correctly in bioctl.
- Add support for iBBU-09 to show BBU voltage, current and temperature
  correctly.
- Fix typos in comments.
- Sprinkle static.
- Improve debug printf()s.
- KNF. Remove extra semicolon. Whitespace fixes.

Improve DNPRINTF. No functional change.

Don't use MFI_PD_LIST_SIZE. Same as OpenBSD. No functional change.

mfi(4): Set stripe size correctly for BIOCVOL.

Before:
Volume Status       Size         Device/Label    Level Stripe
=============================================================
     0 Online       5.5T                 mfi0   RAID 5    N/A  65535 seconds

After:
Volume Status       Size         Device/Label    Level Stripe
=============================================================
     0 Online       5.5T                 mfi0   RAID 5    64K  65535 seconds

Use union mbox instead of unit8_t[] to avoid unaligned access.

 - Same as other OSes.
 - mfii.c already use this.
 - Found by kUBSan.

KNF. Whitespace. No functional change.

Clear mailbox to not to pass garbage data.

Use mlp_bgi for MFI_LD_PROG_BGI. Same as OpenBSD.

s/contrller/controller/ in comment.

Add some code for the SKINNY variant to make Dell PERC H310 work.

fix various typos in comments and log messages.

s/decriptor/descriptor/ in comment.

s/allign/align/

Merge thorpej-cfargs2.

Adapt to CFARGS().

Merge thorpej-cfargs branch:

Simplify and make extensible the config_search() / config_found() /
config_attach() interfaces: rather than having different variants for
which arguments you want pass along, just have a single call that
takes a variadic list of tag-value arguments.

Adjust all call sites:
- Simplify wherever possible; don't pass along arguments that aren't
  actually needed.
- Don't be explicit about what interface attribute is attaching if
  the device only has one.  (More simplification.)
- Add a config_probe() function to be used in indirect configuiration
  situations, making is visibly easier to see when indirect config is
  in play, and allowing for future change in semantics.  (As of now,
  this is just a wrapper around config_match(), but that is an
  implementation detail.)

Remove unnecessary or redundant interface attributes where they're not
needed.

There are currently 5 "cfargs" defined:
- CFARG_SUBMATCH (submatch function for direct config)
- CFARG_SEARCH (search function for indirect config)
- CFARG_IATTR (interface attribte)
- CFARG_LOCATORS (locators array)
- CFARG_DEVHANDLE (devhandle_t - wraps OFW, ACPI, etc. handles)

...and a sentinel value CFARG_EOL.

Add some extra sanity checking to ensure that interface attributes
aren't ambiguous.

Use CFARG_DEVHANDLE in MI FDT, OFW, and ACPI code, and macppc and shark
ports to associate those device handles with device_t instance.  This
will trickle trough to more places over time (need back-end for pre-OFW
Sun OBP; any others?).

No need to pass interface or locators to config_found() in mfi_rescan().

Audit CFARG_IATTR in config_found() calls, and remove it in situations
where the interface attribute is not ambiguous.

Mechanical conversion of config_found_sm_loc() -> config_found().
CFARG_IATTR usage needs to be audited.

Mostly merge changes from HEAD upto 20200411

Merge changes from current as of 20200406

Set 'ld_sync' to NULL as part of 'again', to prevent use-after-free.

in many device attach paths, allocate memory with M_WAITOK instead of M_NOWAIT
and remove code to handle failures that can no longer happen.

sys/dev/ic/mfi.c: Add missing break in switch

(The entire switch is guarded by MFI_DEBUG and is known not to build.)
Reported by Oskar.

Sync with HEAD

Pull up following revision(s) (requested by bouyer in ticket #1129):

	sys/arch/amd64/conf/XEN3_DOM0: revision 1.159
	sys/arch/amd64/conf/GENERIC: revision 1.508
	sys/arch/i386/conf/ALL: revision 1.458
	sys/dev/ic/mfi.c: revision 1.60
	sys/dev/pci/mpiireg.h: revision 1.1
	distrib/sets/lists/man/mi: revision 1.1628
	sys/dev/pci/mfii.c: revision 1.1,1.2 (adapted)
	sys/arch/i386/conf/GENERIC: revision 1.1194
	sys/dev/pci/mpii.c: revision 1.13
	sys/dev/ic/mfireg.h: revision 1.9
	share/man/man4/mfii.4: revision 1.1
	share/man/man4/Makefile: revision 1.673 (patch)
	sys/dev/pci/files.pci: revision 1.410
	share/man/man4/mfii.4: revision 1.2
	sys/arch/amd64/conf/ALL: revision 1.108
	sys/arch/i386/conf/XEN3PAE_DOM0: revision 1.8 (patch, in XEN3_DOM0)

Add some definitions from OpenBSD, needed by the upcoming mfii driver.
No functionnal change.

 -

Move registers definitions to a separate file, needed for the upcomning mpii
driver. No functionnal change.

 -

Add mpii(4), a driver for LSI Megaraid Fusion controllers.
Ported from OpenBSD. This driver is MP-safe.

Note that the earlier fusion controllers (Megaraid 2208, codenamed Thunderbold)
are also supported by mfi(4). mpii will take precedence if both drivers
are enabled.

Tested on a
mfii0 at pci6 dev 0 function 0: "PERC H740P Adapter ", firmware 50.3.0-1512, 819
2MB cache
mfii0: interrupting at ioapic2 pin 2
scsibus0 at mfii0: 64 targets, 8 luns per target
scsibus0: waiting 2 seconds for devices to settle...
sd0 at scsibus0 target 0 lun 0: <DELL, PERC H740P Adp, 5.03> disk fixed
sd0: fabricating a geometry
sd0: 99 GB, 102399 cyl, 64 head, 32 sec, 512 bytes/sect x 209714688 sectors
sd0: tagged queueing
sd1 at scsibus0 target 1 lun 0: <DELL, PERC H740P Adp, 5.03> disk fixed
sd1: fabricating a geometry
sd1: 22254 GB, 22788608 cyl, 64 head, 32 sec, 512 bytes/sect x 46671069696 sectors
sd1: fabricating a geometry

It supports bioctl(8) ioctls, as well as sensors for the BBU and logical
drives.

Sponsored by LIP6.

 -

Add my name in copyright list

 -

add a man page for the new mpii, mostly from OpenBSD.

 -

Sort SEE ALSO. Fix date. Whitespace fixes.

Sync with HEAD, resolve a couple of conflicts

Add some definitions from OpenBSD, needed by the upcoming mfii driver.
No functionnal change.

Sync with HEAD

Resolve a couple of conflicts (result of the uimin/uimax changes)

Rename min/max -> uimin/uimax for better honesty.

These functions are defined on unsigned int.  The generic name
min/max should not silently truncate to 32 bits on 64-bit systems.
This is purely a name change -- no functional change intended.

HOWEVER!  Some subsystems have

	#define min(a, b)	((a) < (b) ? (a) : (b))
	#define max(a, b)	((a) > (b) ? (a) : (b))

even though our standard name for that is MIN/MAX.  Although these
may invite multiple evaluation bugs, these do _not_ cause integer
truncation.

To avoid `fixing' these cases, I first changed the name in libkern,
and then compile-tested every file where min/max occurred in order to
confirm that it failed -- and thus confirm that nothing shadowed
min/max -- before changing it.

I have left a handful of bootloaders that are too annoying to
compile-test, and some dead code:

cobalt ews4800mips hp300 hppa ia64 luna68k vax
acorn32/if_ie.c (not included in any kernels)
macppc/if_gm.c (superseded by gem(4))

It should be easy to fix the fallout once identified -- this way of
doing things fails safe, and the goal here, after all, is to _avoid_
silent integer truncations, not introduce them.

Maybe one day we can reintroduce min/max as type-generic things that
never silently truncate.  But we should avoid doing that for a while,
so that existing code has a chance to be detected by the compiler for
conversion to uimin/uimax without changing the semantics until we can
properly audit it all.  (Who knows, maybe in some cases integer
truncation is actually intended!)

update from HEAD

Kill some more extern struct cfdriver declarations.

Down with externs in .c!

Pull up following revision(s) (requested by maxv in ticket #761):
	sys/arch/sgimips/dev/scn.c: revision 1.8
	sys/arch/sgimips/mace/macekbc.c: revision 1.8
	sys/dev/hpc/hpcfb.c: revision 1.59
	sys/dev/i2c/i2c.c: revision 1.47
	sys/dev/ic/mfi.c: revision 1.57
	sys/dev/if_ndis/if_ndis_pci.c: revision 1.21
	sys/dev/sysmon/sysmon_power.c: revisions 1.50-1.52
	sys/dev/usb/umass_isdata.c: revision 1.31
fix double free, found by Brainy.
--
Free cmd on error if we allocated it.
Found by Brainy, reported by maxv@.
--
don't leak rl (but there are other leaks), found by Brainy.
--
Free ped if we can't hand it to the power daemon.
Found by Brainy, reported by maxv@.
--
don't forget to free the dictionary.
--
another missing free dict.
--
fix leak, found by Brainy.
--
don't malloc a tiny, fixed size buffer to scribble into, then not use it
and never free it either
found by Brainy
--
malloc() -> kmem_alloc() for private data, also kmem_free() them if we
don't finish attaching for whatever reason
found by Brainy

Sync with HEAD

fix double free, found by Brainy.

Dedup the conversion of bioc_disk and bioc_vol to envsys_data_t

Pull up following revision(s) (requested by msaitoh in ticket #48):
	sys/dev/ic/mfi.c: revision 1.54-1.55
Fix mfi(4) panic on boot on some mfi(4) chips.

over-initialize for the benefit of gcc

revert previous; code checking tool is incorrect. Variables are not being
used in the non tbolt case.

Rebase to HEAD as of a few days ago.

Rebase.

Add d_discard to all struct cdevsw instances I could find.

All have been set to "nodiscard"; some should get a real implementation.

sync with head.

for a reference, the tree before this commit was tagged
as yamt-pagecache-tag8.

this commit was splitted into small chunks to avoid
a limitation of cvs.  ("Protocol error: too many arguments")

sync with head

Change (mostly mechanically) every cdevsw/bdevsw I can find to use
designated initializers.

I have not built every extant kernel so I have probably broken at
least one build; however I've also found and fixed some wrong
cdevsw/bdevsw entries so even if so I think we come out ahead.

More alignment spellos

- remove unused variables
- move variables inside ifdef sections
- ifdef notdef unused code
- use __USE for debugging variables

Fix the non-thunderbolt case. XXX: Is that right even?
http://m00nbsd.net/ae123a9bae03f7dde5c6d654412daf5a.html

sync with (a bit old) head

Resync to 2012-11-19 00:00:00 UTC

Pull up following revision(s) (requested by bouyer in ticket #674):
	sys/dev/ic/mfi.c: revision 1.48
MFI_STATE_BOOT_MESSAGE_PENDING also needs to be handed for non-TBOLT
controllers, from FreeBSD. tested by Hugo Silva against a GEN2 controller.

MFI_STATE_BOOT_MESSAGE_PENDING also needs to be handed for non-TBOLT
controllers, from FreeBSD. tested by Hugo Silva against a GEN2 controller.

sync with head

Pull up following revision(s) (requested by bouyer in ticket #629):
	sys/compat/linux/common/linux_ioctl.c: revision 1.57
	sys/dev/ic/mfi.c: revision 1.47
	sys/conf/majors: revision 1.62
	etc/etc.i386/MAKEDEV.conf: revision 1.24
	sys/dev/ic/mfivar.h: revision 1.20
	etc/MAKEDEV.tmpl: revision 1.159
	sys/dev/ic/mfiio.h: revision 1.1
	etc/etc.amd64/MAKEDEV.conf: revision 1.19
	sys/dev/ic/mfireg.h: revision 1.8
Add a pass-through ioctl for mfi(4), allowing userland to send raw commands
to the controller. This is compatible with the linux and FreeBSD
implementations.
Add the needed conversion for mfi ioctls in COMPAT_LINUX
Allocate a character major number, and create /dev/mfi0 by default
on amd64 and i386.
This allows (along with a hand-created /emul/linux/proc/devices file)
to run the MegaCLI linux binary provided by LSI.

Pull up following revision(s) (requested by bouyer in ticket #628):
	sys/dev/ic/mfi.c: revision 1.40
	sys/dev/ic/mfi.c: revision 1.41
	sys/dev/ic/mfi.c: revision 1.42
	sys/dev/ic/mfi.c: revision 1.43
	sys/dev/ic/mfi.c: revision 1.44
	sys/dev/ic/mfi.c: revision 1.45
	sys/dev/ic/mfi.c: revision 1.46
	sys/dev/pci/pcidevs: revision 1.1128
	sys/dev/pci/mfi_pci.c: revision 1.15
	sys/dev/pci/mfi_pci.c: revision 1.16
	sys/dev/ic/mfi.c: revision 1.39
	sys/dev/ic/mfivar.h: revision 1.16
	sys/dev/ic/mfireg.h: revision 1.6
	sys/dev/ic/mfivar.h: revision 1.17
	sys/dev/ic/mfireg.h: revision 1.7
	sys/dev/ic/mfivar.h: revision 1.18
	sys/dev/ic/mfivar.h: revision 1.19
Add some support for 64bit DMA but stick to 32bit DMA for now.
From OpenBSD mfi.c rev 1.119.
Add LSI MegaRAID SAS2208
MFI_IOP_SKINNY is an enum not a single bit value, so it can't be used as
a bit flag. As sc_flags is really used to hold an enum mfi_iop value,
change it to enum mfi_iop and rename to sc_ioptype. While there init it
in mfi_attach() instead of mfi_pci_attach().
sg64.len is 32bits, so use htole32()
Really init sc_ioptype in mfi_attach()
Add support newer LSI RAID controllers based on the SAS2208 chip,
codenamed "ThunderBolt". Add tagged queuing support for all adapters
supported by mfi(4).
Tested with a MegaRAID SAS 9265-8i adapter, and an older Dell PERC 5/i.
Avoid "unused variable" warning for non-DIAGNOSTIC kernels.
Pointed out by Havard Eidnes
After discussion on tech-kern@, fix performance issue related to
cache flush commands from WAPBL, by skipping the cache flush if the
BBU is present and considered good. Users which still want the write back
cache with a non-working BBU can set vfs.wapbl.flush_disk_cache to 0.
- add commands to monitor the BBU state. Add a boolean BBU sensor
  to monitor the BBU state via sysmon_envsys(9).
- if the BBU is considered good, turn SCSI_SYNCHRONIZE_CACHE_10 and
  SCSI_SYNCHRONIZE_CACHE_16 commands from upper layer into NOOPs.
  While there, handle SCSI_SYNCHRONIZE_CACHE_16 in addition to
  SCSI_SYNCHRONIZE_CACHE_10.
- Add a shutdown pmf(9) handler, which flushes the cache and shutdown the
  firmware
- on detach, also flush cache and shutdown firmware.
- on attach, print the firmware-provided name, and the BBU state
Tested on a LSI MegaRAID SAS 9265-8i and a PERC 5/i Integrated
Make MFI_DEBUG build on i386.
Report BBU state changes with aprint_normal(), it seems that sysmon_envsys()
doens't report changes for ENVSYS_INDICATOR as it does for ENVSYS_DRIVE.

Add a pass-through ioctl for mfi(4), allowing userland to send raw commands
to the controller. This is compatible with the linux and FreeBSD
implementations.
Add the needed conversion for mfi ioctls in COMPAT_LINUX
Allocate a character major number, and create /dev/mfi0 by default
on amd64 and i386.
This allows (along with a hand-created /emul/linux/proc/devices file)
to run the MegaCLI linux binary provided by LSI.

Make MFI_DEBUG build on i386.
Report BBU state changes with aprint_normal(), it seems that sysmon_envsys()
doens't report changes for ENVSYS_INDICATOR as it does for ENVSYS_DRIVE.

After discussion on tech-kern@, fix performance issue related to
cache flush commands from WAPBL, by skipping the cache flush if the
BBU is present and considered good. Users which still want the write back
cache with a non-working BBU can set vfs.wapbl.flush_disk_cache to 0.
- add commands to monitor the BBU state. Add a boolean BBU sensor
  to monitor the BBU state via sysmon_envsys(9).
- if the BBU is considered good, turn SCSI_SYNCHRONIZE_CACHE_10 and
  SCSI_SYNCHRONIZE_CACHE_16 commands from upper layer into NOOPs.
  While there, handle SCSI_SYNCHRONIZE_CACHE_16 in addition to
  SCSI_SYNCHRONIZE_CACHE_10.
- Add a shutdown pmf(9) handler, which flushes the cache and shutdown the
  firmware
- on detach, also flush cache and shutdown firmware.
- on attach, print the firmware-provided name, and the BBU state
Tested on a LSI MegaRAID SAS 9265-8i and a PERC 5/i Integrated

Avoid "unused variable" warning for non-DIAGNOSTIC kernels.
Pointed out by Havard Eidnes

Add support newer LSI RAID controllers based on the SAS2208 chip,
codenamed "ThunderBolt". Add tagged queuing support for all adapters
supported by mfi(4).
Tested with a MegaRAID SAS 9265-8i adapter, and an older Dell PERC 5/i.

Really init sc_ioptype in mfi_attach()

sg64.len is 32bits, so use htole32()

MFI_IOP_SKINNY is an enum not a single bit value, so it can't be used as
a bit flag. As sc_flags is really used to hold an enum mfi_iop value,
change it to enum mfi_iop and rename to sc_ioptype. While there init it
in mfi_attach() instead of mfi_pci_attach().

Add some support for 64bit DMA but stick to 32bit DMA for now.
From OpenBSD mfi.c rev 1.119.

sync with head

sync to latest -current.

Pull up following revision(s) (requested by sborrill in ticket #138):
	sys/dev/pci/mfi_pci.c: revision 1.13
	sys/dev/ic/mfi.c: revision 1.38
	sys/dev/ic/mfivar.h: revision 1.15
	sys/dev/ic/mfireg.h: revision 1.5
Add support for skinny variants (e.g. IBM ServeRAID M1015). Based on OpenBSD
changes with some improvements. Tested on IBM x3550M3 with RAID0 and RAID1
volumes including bioctl(8) operation.

Pull up following revision(s) (requested by sborrill in ticket #136):
	sys/dev/ic/mfi.c: revision 1.37
From OpenBSD. Fixes a deadlock during autoconf.
scrub more fields in the ccb when returning them to the free list after
theyve been used, in particular the mfi header flags which has a bit
that specifies if a command should be completed via the interrupt path.
if we use a ccb during boot we set that bit, but it isnt necessarily
cleared by things that use it later on. this means a ccb we expected to
complete via an interrupt never actually generates an interrupt or appears
in the reply queue. this obviously stalls the io.

Add support for skinny variants (e.g. IBM ServeRAID M1015). Based on OpenBSD
changes with some improvements. Tested on IBM x3550M3 with RAID0 and RAID1
volumes including bioctl(8) operation.

From OpenBSD. Fixes a deadlock during autoconf.

scrub more fields in the ccb when returning them to the free list after
theyve been used, in particular the mfi header flags which has a bit
that specifies if a command should be completed via the interrupt path.

if we use a ccb during boot we set that bit, but it isnt necessarily
cleared by things that use it later on. this means a ccb we expected to
complete via an interrupt never actually generates an interrupt or appears
in the reply queue. this obviously stalls the io.

Catchup with rmind-uvmplock merge.

Initialize current value for ENVSYS_DRIVE sensors

Initializes sensors states before registering.

sync with head.

Sync with HEAD.

sync to netbsd-5

Apply patch (requested by msaitoh in ticket #1326):
Add support for MFI gen2 devices.  Mention newer devices, RAID 6 and
RAID60 in mfi.4.

Remove setting of edata->monitor since that member no longer exists.

sync with head

Add newer cars supports. Tested on MegaRAID SAS 9260-8i.
 - Add MFI gen2 support from OpenBSD.
 - Add entry for MegaRAID SAS 9260-8i

 Fix a bug that the system which has no logical volume panics in shutdown
hook. Without any logical volume, sysmon_envsys_register() fails. On such
case, sc->sc_sme must be NULL for the detach. Reviewed by dyoung and bouyer.

Pull up the following revisions(s) (requested by bouyer in ticket #1257):
	sys/dev/ic/mfi.c:	revision 1.31

bio(4) is MP-safe but mfi(4) is not. So get the kernel_lock at
mfi_ioctl() entry and release it on exit.

Pull up the following revisions(s) (requested by bouyer in ticket #1257):
	sys/dev/ic/mfi.c:	revision 1.31

bio(4) is MP-safe but mfi(4) is not. So get the kernel_lock at
mfi_ioctl() entry and release it on exit.

Pull up the following revisions(s) (requested by bouyer in ticket #1256):
	sys/dev/ic/mfi.c:	revision 1.21

Don't release CCB twice. Found by code inspection.

bio(4) is MP-safe but mfi(4) is not. So get the kernel_lock at
mfi_ioctl() entry and release it on exit.

sync with head

Delete whitespace at ends of lines.  No functional change intended.

Pull up following revision(s) (requested by bouyer in ticket #931):
	sys/dev/ic/mfi.c: revision 1.28 via patch
mfi.c still uses the spl() synchronisation scheme and so needs the kernel lock.
The sysmon subsystem is marked MPSAFE and so runs without the kernel lock.
So get the kernel lock in mfi_sensor_refresh()  before calling mfi_ioctl_vol().
This fixes command list corruption seen on heavy I/O load on the mfi driver(4).

Pull up following revision(s) (requested by bouyer in ticket #931):
	sys/dev/ic/mfi.c: revision 1.28 via patch
mfi.c still uses the spl() synchronisation scheme and so needs the kernel lock.
The sysmon subsystem is marked MPSAFE and so runs without the kernel lock.
So get the kernel lock in mfi_sensor_refresh() before calling mfi_ioctl_vol().
This fixes command list corruption seen on heavy I/O load on the mfi driver(4).

Oups, remove extra }

mfi.c still uses the spl() synchronisation scheme and so needs the kernel lock.
The sysmon subsystem is marked MPSAFE and so runs without the kernel lock.
So get the kernel lock in mfi_sensor_refresh()  before calling
mfi_ioctl_vol().
This fixes command list corruption seen on heavy I/O load on the mfi
driver(4).

Sync with HEAD.

sync with head.

Let us detach & re-attach children of mfi0.  Detach mfi0 at shutdown.

Detachment may fail after freeing some but not all resources, so
take care not to re-release any resource during detachment.

Tested on a Dell PowerEdge 1950.

device_t/softc split.  Tested and shown to work on a Dell PowerEdge
1950.

Try to detach ancestors (sd0 at scsibus0, scsibus0 at mfi0), first,
to avoid dangling references to envsys(4) sensors and such if
detaching the ancestors should fail.

Add a rudimentary detachment hook for mfi(4).

sync with head

Sync with HEAD.

Commit is split, to avoid a "too many arguments" protocol error.

struct device * -> device_t, no functional changes intended.

sync with head.

Sync with HEAD.

bzero -> memset

Sync with HEAD.

Don't release CCB twice. Found by code inspection.

Sync with HEAD.

Sync with HEAD.

remove extra semicolons.

Update haad-dm branch to haad-dm-base2.

Change mfi_mgmt() to take a mfi_ccb and scsipi_xfer as argument, as
mfi_scsi_io does. Do not tsleep in mfi_mgmt waiting for completion;
let the caller handle it. If mfi_mgmt() is called with a non-NULL scsipi_xfer
have mfi_mgmt_done() wake it up, else wake up the mfi_ccb.
mfi_poll()/mfi_post() is up to the caller.

mfi_scsipi_request(): handle SCSI_SYNCHRONIZE_CACHE_10 as we do for other
commmands: have mfi_mgmt() fill in the ccb, queue of poll the ccb and
return.

Introduce mfi_mgmt_internal() which behaves like mfi_mgmt() did (tsleep
waiting for completion). Use it for internal and ioctl management
requests.

Fix kern/39297 from Greg Oster (mfi calls tsleep() from mfi_intr()),
tested by Mark Davies.

Sync with wrstuden-revivesa-base-2.

Sync with HEAD.

Sync with head.

status is unsigned.

Sync with HEAD.

sync with head.

revert rev. 1.16 and redo the fix as proposed by oster per private mail

Fix error path:
Don't try to free non-allocated memory. Only free allocated memory.
This fixes a crash reported by Brian A. Seklecki on port-xen:
http://mail-index.netbsd.org/port-xen/2008/04/10/msg003555.html

Fix tested by Brian A. Seklecki:
http://mail-index.netbsd.org/port-xen/2008/04/21/msg003604.html

use aprint_*_dev and device_xname

Sync with HEAD.

Add back the bus_dmamap_sync() calls that were in the old
mfi_despatch_cmd() func after merging changes in rev 1.12.

Merge rev 1.69 and 1.64 from openbsd, fixing two bugs:

Fix subtle x ? y : z bug when calling mfi_create_sgl. This worked because
during runtime the bug condition does not happen.

When determining the size of the volumes and disks we want how many
bytes each of them has, not the number of kilobytes. this solves the
"why am i dividing by 2" question. the answer is "you dont, you multiply
by 512".

The first item fixes a panic with Dell Perc 5/i reported by oster@.
Changes tested on Dell Perc 5/i by oster@.

sync with head.

sync with HEAD

sync with head.

Apply some KNF to this driver and do some random changes as well:

- Change malloc + memset(blah, 0, ...)  to malloc with M_ZERO.
- The adapt_ioctl assigned was returning ENOTTY, so there's no point
  on having it.
- Static'ify.

I believe there are some paths on this driver where the splbio()s
are not needed, but can't test to verify...

Added support for the mfi(4) controllers with powerpc IOPs such as
LSI SAS1078 or Dell PERC 6, from OpenBSD.

Tested by Akira Kato on current-users@.

sync with HEAD

sync with head

Sync with HEAD.

Sync with HEAD.

Sync with HEAD

Extend the envsys2 API (one more time, sorry) as defined in:

http://mail-index.netbsd.org/tech-kern/2007/11/09/0001.html

sysmon_envsys_create() and sysmon_envsys_destroy() were added to
create/destroy sysmon_envsys objects (and its TAILQ/LIST for sensors/events).

sysmon_envsys_sensor_attach() and sysmon_envsys_sensor_detach() were
added to attach/detach sensors to a specified sysmon_envsys device.

The events framework is now per device and configurable via the
ENVSYS_SETDICTIONARY ioctl or /etc/envsys.conf and envstat(8).

Update all users and documentation to reflect these changes.

sync with HEAD

sync with head.

Sync with HEAD.

Follow the merge of pmap.c on i386 and amd64 and move
pmap_init_tmp_pgtbl into arch/x86/x86/pmap.c. Modify the ACPI wakeup
code to restore CR4 before jumping back into kernel space as the large
page option might cover that.

Sync with HEAD.

Sync with head.

machine/{bus,cpu,intr}.h -> sys/{bus,cpu,intr}.h

Pull up following revisions via patch (requested by bouyer in ticket #1838):
	distrib/sets/lists/man/mi: revision 1.997
	sbin/bioctl/strtonum.c: revision 1.1
	sys/dev/Makefile: revision 1.25
	sys/arch/amd64/conf/GENERIC: revision 1.139
	sbin/bioctl/strtonum.h: revision 1.1
	sys/dev/bio.c: revision 1.1
	sbin/bioctl/bioctl.c: revision 1.1
	share/man/man4/bio.4: revision 1.1
	sbin/bioctl/bioctl.8: revision 1.1
	sys/sys/envsys.h: revision 1.11
	sbin/bioctl/bioctl.8: revision 1.3
	sbin/bioctl/bioctl.8: revision 1.4
	sys/arch/i386/conf/XEN2_DOM0: revision 1.25
	distrib/sets/lists/base/mi: revision 1.704
	sys/conf/majors: revision 1.34
	share/man/man4/Makefile: revision 1.426
	etc/MAKEDEV.tmpl: revision 1.86
	sys/arch/i386/conf/GENERIC: revision 1.825
	distrib/sets/lists/comp/mi: revision 1.1022
	sys/conf/files: revision 1.839
	usr.sbin/envstat/envstat.c: revision 1.24
	sbin/Makefile: revision 1.105
	sys/dev/ic/mfi.c: revision 1.4
	sys/dev/biovar.h: revision 1.1
	sys/dev/ic/mfivar.h: revision 1.4
	sbin/bioctl/Makefile: revision 1.1
Fix typo.
Add bio(4) and associated bioctl(8) from OpenBSD, a driver control block
device controllers, and more specifically raid controllers.
Add a new sensor type, ENVSYS_DRIVE, to report drive status. From OpenBSD.
Add bio and sysmon support to mfi(4). This allow userland to query
status for drives and logical volumes attached to a mfi(4) controller. While
there fix some debug printfs in mfi so they compile.
Add bio(4) to amd64 and i386 GENERIC.
Add bio(4) and associated bioctl(8) from OpenBSD, a driver control block
device controllers, and more specifically raid controllers.
Add a new sensor type, ENVSYS_DRIVE, to report drive status. From OpenBSD.
Add bio and sysmon support to mfi(4). This allow userland to query
status for drives and logical volumes attached to a mfi(4) controller. While
there fix some debug printfs in mfi so they compile.
Add bio(4) to amd64 and i386 GENERIC.
Fix Dd argument (use full month names).
Use more markup.
Comment out references to safte(4) and softraid(4), which don't exist in
NetBSD.
Remove trailing whitespace.
Use macros instead of characters for HTML output (replace ">", "<"
with \*[Gt], \*[Lt]).
Sort sections.
Create /dev/bio

Sync with head.

Sync with HEAD.

Sync with HEAD.

s/ENVSYS_FMONDRVSTATE/ENVSYS_FMONSTCHANGED/. Noticed by Manuel Bouyer@.

sync with head.

Sync with head.

Sync with head.

mfi_sensor_gtredata:

- There's no need to set edata->units again, it was set already before.
- Remove the last ENVSYS_SVALID assignment that overrides previous
  assignments (found by mhitch@).

There's no need to set ENVSYS_FMONNOTSUPP in sensor's flags anymore,
userland limits for this type of sensor is disabled by sysmon_envsys(9).

Imported envsys 2, a brief description of the new features:
(Part 2: drivers)

* Support for detachable sensors.
* Cleaned up the API for simplicity and efficiency.
* Ability to send capacity/critical/warning events to powerd(8).
* Adapted all the code to the new locking order.
* Compatibility with the old envsys API: the ENVSYS_GTREINFO
  and ENVSYS_GTREDATA ioctl(2)s are supported.
* Added support for a 'dictionary based communication channel' between
  sysmon_power(9) and powerd(8), that means there is no 32 bytes event
  size restriction anymore.
* Binary compatibility with old envstat(8) and powerd(8) via COMPAT_40.
* All drivers with the n^2 gtredata bug were fixed, PR kern/36226.

Tested by:

blymn: smsc(4).
bouyer: ipmi(4), mfi(4).
kefren: ug(4).
njoly: viaenv(4), adt7463.c.
riz: owtemp(4).
xtraeme: acpiacad(4), acpibat(4), acpitz(4), aiboost(4), it(4), lm(4).

Sync with head.

Pull up following revision(s) (requested by bouyer in ticket #603):
	distrib/sets/lists/base/mi: revision 1.704
	distrib/sets/lists/comp/mi: revision 1.1022
	distrib/sets/lists/man/mi: revision 1.997
	doc/CHANGES: revision 1.839
	sbin/Makefile: patch
	sbin/bioctl/Makefile: revision 1.1
	sbin/bioctl/bioctl.8: revision 1.1
	sbin/bioctl/bioctl.c: revision 1.1
	sbin/bioctl/strtonum.c: revision 1.1
	sbin/bioctl/strtonum.h: revision 1.1
	share/man/man4/Makefile: revision 1.426
	share/man/man4/bio.4: revision 1.1
	sys/arch/amd64/conf/GENERIC: revision 1.139
	sys/arch/i386/conf/GENERIC: revision 1.825
	sys/arch/i386/conf/XEN2_DOM0: revision 1.25
	sys/conf/files: revision 1.839
	sys/conf/majors: patch
	sys/dev/Makefile: revision 1.25
	sys/dev/bio.c: patch
	sys/dev/biovar.h: patch
	sys/dev/ic/mfi.c: revision 1.4-1.5
	sys/dev/ic/mfivar.h: revision 1.4
	sys/sys/envsys.h: revision 1.11
	usr.sbin/envstat/envstat.c: revision 1.24
Add bio(4) and associated bioctl(8) from OpenBSD, a driver control block
device controllers, and more specifically raid controllers.
Add a new sensor type, ENVSYS_DRIVE, to report drive status. From OpenBSD.
Add bio and sysmon support to mfi(4). This allow userland to query
status for drives and logical volumes attached to a mfi(4) controller. While
there fix some debug printfs in mfi so they compile.
Add bio(4) to amd64 and i386 GENERIC.

note bio(4), envsys(4) DRIVE, and mfi(4) support for both.

tred->sensor is a u_int.

sync with head.

tred->sensor is a u_int.

Add bio(4) and associated bioctl(8) from OpenBSD, a driver control block
device controllers, and more specifically raid controllers.
Add a new sensor type, ENVSYS_DRIVE, to report drive status. From OpenBSD.
Add bio and sysmon support to mfi(4). This allow userland to query
status for drives and logical volumes attached to a mfi(4) controller. While
there fix some debug printfs in mfi so they compile.
Add bio(4) to amd64 and i386 GENERIC.

Sync with HEAD.

Kill caddr_t; there will be some MI fallout, but it will be fixed shortly.

Sync with head.

sync with head.

Pull up following revision(s) (requested by bouyer in ticket #1609):
	sys/arch/i386/conf/INSTALL: revision 1.298
	sys/arch/i386/conf/GENERIC: revision 1.802
	sys/dev/pci/files.pci: revision 1.274
	sys/arch/amd64/conf/GENERIC: revision 1.122
	sys/dev/pci/mfi_pci.c: revision 1.1 via patch
	distrib/sets/lists/man/mi: revision 1.958
	sys/arch/amd64/conf/INSTALL: revision 1.61
	sys/dev/ic/mfireg.h: revision 1.1
	share/man/man4/Makefile: revision 1.416 via patch
	sys/dev/ic/mfi.c: revision 1.1
	sys/arch/i386/conf/XEN2_DOM0: revision 1.19
	sys/dev/ic/mfivar.h: revision 1.1
	sys/conf/files: revision 1.821
	share/man/man4/mfi.4: revision 1.1
	share/man/man4/mfi.4: revision 1.2
	sys/dev/pci/pcidevs via patch
Add mfi(4), a driver for LSI Logic & Dell MegaRAID SAS RAID controller.
Ported from OpenBSD, tested on i386 and amd64.
Punctuation issues.

sync with head.

Pull up following revision(s) (requested by bouyer in ticket #286):
	sys/dev/pci/mfi_pci.c: revision 1.2
	sys/dev/ic/mfireg.h: revision 1.2
	sys/dev/ic/mfi.c: revision 1.2
	sys/dev/ic/mfivar.h: revision 1.2
Add missing $NetBSD: $

Pull up following revision(s) (requested by bouyer in ticket #286):
	sys/arch/i386/conf/INSTALL: revision 1.298
	sys/arch/i386/conf/GENERIC: revision 1.802
	sys/dev/pci/files.pci: revision 1.274
	sys/arch/amd64/conf/GENERIC: revision 1.122
	sys/dev/pci/mfi_pci.c: revision 1.1
	distrib/sets/lists/man/mi: revision 1.958
	sys/arch/amd64/conf/INSTALL: revision 1.61
	sys/arch/i386/conf/ALL: revision 1.75
	sys/dev/ic/mfireg.h: revision 1.1
	share/man/man4/Makefile: revision 1.416
	sys/dev/ic/mfi.c: revision 1.1
	sys/arch/i386/conf/XEN2_DOM0: revision 1.19
	sys/dev/ic/mfivar.h: revision 1.1
	sys/conf/files: revision 1.821
	share/man/man4/mfi.4: revision 1.1
Add mfi(4), a driver for LSI Logic & Dell MegaRAID SAS RAID controller.
Ported from OpenBSD, tested on i386 and amd64.

file mfi.c was added on branch newlock2 on 2007-01-12 00:57:36 +0000

file mfi.c was added on branch yamt-lazymbuf on 2006-12-30 20:48:03 +0000

file mfi.c was added on branch netbsd-3 on 2006-12-23 16:43:19 +0000

file mfi.c was added on branch netbsd-4 on 2006-12-21 12:46:21 +0000

Add missing $NetBSD: $

sync with head.

file mfi.c was added on branch yamt-splraiseipl on 2006-12-18 11:42:13 +0000

Add mfi(4), a driver for LSI Logic & Dell MegaRAID SAS RAID controller.
Ported from OpenBSD, tested on i386 and amd64.