]> granicus.if.org Git - zfs/commit
Fix missing 'zpool events'
authorBrian Behlendorf <behlendorf1@llnl.gov>
Mon, 4 Oct 2010 23:21:04 +0000 (16:21 -0700)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Tue, 12 Oct 2010 21:55:03 +0000 (14:55 -0700)
commitbaa40d45cbb336765b2f89d934cd9ea690e4f7c9
tree3beb455efbc97b3c014772c121b1dc9fe3a76e11
parenta69052be7f9a4008e2b09578e9db5fdebc186111
Fix missing 'zpool events'

It turns out that 'zpool events' over 1024 bytes in size where being
silently dropped.  This was discovered while writing the zfault.sh
tests to validate common failure modes.

This could occur because the zfs interface for passing an arbitrary
size nvlist_t over an ioctl() is to provide a buffer for the packed
nvlist which is usually big enough.  In this case 1024 byte is the
default.  If the kernel determines the buffer is to small it returns
ENOMEM and the minimum required size of the nvlist_t.  This was
working properly but in the case of 'zpool events' the event stream
was advanced dispite the error.  Thus the retry with the bigger
buffer would succeed but it would skip over the previous event.

The fix is to pass this size to zfs_zevent_next() and determine
before removing the event from the list if it will fit.  This was
preferable to checking after the event was returned because this
avoids the need to rewind the stream.
include/sys/fm/util.h
module/zfs/fm.c
module/zfs/zfs_ioctl.c