src/include, branch zfs-0.6.3

Allow building without ACLs

2014-05-30T19:01:57+00:00

Some kernel definitions were buried inside the #if... #endif logic for
ACLs.  When ACLs are not available these definitions get lost causing
the build to fail.

Signed-off-by: Chris Wedgwood 
Signed-off-by: Brian Behlendorf 
Closes #2349

Allow for lock-free reading zfsdev_state_list.

2014-05-19T18:45:11+00:00

Restructure the zfsdev_state_list to allow for lock-free reading by
converting to a simple singly-linked list from which items are never
deleted and over which only forward iterations are performed.  It depends
on, among other things, the atomicity of accessing the zs_minor integer
and zs_next pointer.

This fixes a lock inversion in which the zfsdev_state_lock is used by
both the sync task (txg_sync) and indirectly by any user program which
uses /dev/zfs; the zfsdev_release method uses the same lock and then
blocks on the sync task.

The most typical failure scenerio occurs when the sync task is cleaning
up a user hold while various concurrent "zfs" commands are in progress.

Neither Illumos nor Solaris are affected by this issue because they use
DDI interface which provides lock-free reading of device state via the
ddi_get_soft_state() function.

Signed-off-by: Tim Chase 
Signed-off-by: Chunwei Chen 
Signed-off-by: Brian Behlendorf 
Closes #2301

Use a dedicated taskq for vdev_file

2014-05-14T23:20:21+00:00

Originally, vdev_file used system_taskq. This would cause a deadlock,
especially on system with few CPUs. The reason is that the prefetcher
threads, which are on system_taskq, will sometimes be blocked waiting
for I/O to finish. If the prefetcher threads consume all the tasks in
system_taskq, the I/O cannot be served and thus results in a deadlock.

We fix this by creating a dedicated vdev_file_taskq for vdev_file I/O.

Signed-off-by: Chunwei Chen 
Signed-off-by: Brian Behlendorf 
Closes #2270

Check the dataset type more rigorously when fetching properties.

2014-05-06T17:41:46+00:00

When fetching property values of snapshots, a check against the head
dataset type must be performed.  Previously, this additional check was
performed only when fetching "version", "normalize", "utf8only" or "case".

This caused the ZPL properties "acltype", "exec", "devices", "nbmand",
"setuid" and "xattr" to be erroneously displayed with meaningless values
for snapshots of volumes.  It also did not allow for the display of
"volsize" of a snapshot of a volume.

This patch adds the headcheck flag paramater to zfs_prop_valid_for_type()
and zprop_valid_for_type() to indicate the check is being done
against a head dataset's type in order that properties valid only for
snapshots are handled correctly.  This allows the the head check in
get_numeric_property() to be performed when fetching a property for
a snapshot.

Signed-off-by: Tim Chase 
Signed-off-by: Brian Behlendorf 
Closes #2265

libspl: Implement LWP rwlock interface

2014-05-01T22:53:52+00:00

This implements a subset of the LWP rwlock interface by wrapping the
equivalent POSIX thread interface. It is a superset of the features
needed by ztest.

The missing bits are {,_}rw_read_held() and {,_}rw_write_held().

Signed-off-by: Richard Yao 
Signed-off-by: Brian Behlendorf 
Issue #1970

Implement File Attribute Support

2014-05-01T17:11:18+00:00

We add support for lsattr and chattr to resolve a regression caused
by 88c283952f0bfeab54612f9ce666601d83c4244f that broke Python's
xattr.list(). That changet broke Gentoo Portage's FEATURES=xattr,
which depended on Python's xattr.list().

Only attributes common to both Solaris and Linux are supported. These
are 'a', 'd' and 'i' in Linux's lsattr and chattr commands. File
attributes exclusive to Solaris are present in the ZFS code, but cannot
be accessed or modified through this method.  That was the case prior to
this patch. The resolution of issue zfsonlinux/zfs#229 should implement
some method to permit access and modification of Solaris-specific
attributes.

References:
  https://bugs.gentoo.org/show_bug.cgi?id=483516

Original-patch-by: Brian Behlendorf 
Signed-off-by: Richard Yao 
Signed-off-by: Brian Behlendorf 
Closes #1691

Use ddi_time_after and friends to compare time

2014-04-14T20:27:56+00:00

Also, make sure we use clock_t for ddi_get_lbolt to prevent type conversion
from screwing things.

Signed-off-by: Chunwei Chen 
Signed-off-by: Brian Behlendorf 
Closes #2142

Linux 3.14 compat: rq_for_each_segment in dmu_req_copy

2014-04-10T21:28:51+00:00

rq_for_each_segment changed from taking bio_vec * to taking bio_vec.
We provide rq_for_each_segment4 which takes both.

Signed-off-by: Chunwei Chen 
Signed-off-by: Richard Yao 
Signed-off-by: Brian Behlendorf 
Issue #2124

Linux 3.14 compat: Immutable biovec changes in vdev_disk.c

2014-04-10T21:28:38+00:00

bi_sector, bi_size and bi_idx are moved from bio to bio->bi_iter.
This patch creates BIO_BI_*(bio) macros to hide the differences.

Signed-off-by: Chunwei Chen 
Signed-off-by: Richard Yao 
Signed-off-by: Brian Behlendorf 
Issue #2124

Linux 3.14 compat: posix_acl_{create,chmod}

2014-04-10T21:27:03+00:00

posix_acl_{create,chmod} is changed to __posix_acl_{create_chmod}

Signed-off-by: Chunwei Chen 
Signed-off-by: Richard Yao 
Signed-off-by: Brian Behlendorf 
Issue #2124