Print this page
3742 zfs comments need cleaner, more consistent style
Submitted by: Will Andrews <willa@spectralogic.com>
Submitted by: Alan Somers <alans@spectralogic.com>
Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed by: George Wilson <george.wilson@delphix.com>
Reviewed by: Eric Schrock <eric.schrock@delphix.com>
*** 1004,1016 ****
}
ZFS_OBJ_HOLD_EXIT(zfsvfs, obj);
}
/*
! * zfs_xvattr_set only updates the in-core attributes
! * it is assumed the caller will be doing an sa_bulk_update
! * to push the changes out
*/
void
zfs_xvattr_set(znode_t *zp, xvattr_t *xvap, dmu_tx_t *tx)
{
xoptattr_t *xoap;
--- 1004,1015 ----
}
ZFS_OBJ_HOLD_EXIT(zfsvfs, obj);
}
/*
! * Update in-core attributes. It is assumed the caller will be doing an
! * sa_bulk_update to push the changes out.
*/
void
zfs_xvattr_set(znode_t *zp, xvattr_t *xvap, dmu_tx_t *tx)
{
xoptattr_t *xoap;
*** 1445,1456 ****
* Increase the file length
*
* IN: zp - znode of file to free data in.
* end - new end-of-file
*
! * RETURN: 0 if success
! * error code if failure
*/
static int
zfs_extend(znode_t *zp, uint64_t end)
{
zfsvfs_t *zfsvfs = zp->z_zfsvfs;
--- 1444,1454 ----
* Increase the file length
*
* IN: zp - znode of file to free data in.
* end - new end-of-file
*
! * RETURN: 0 on success, error code on failure
*/
static int
zfs_extend(znode_t *zp, uint64_t end)
{
zfsvfs_t *zfsvfs = zp->z_zfsvfs;
*** 1523,1534 ****
*
* IN: zp - znode of file to free data in.
* off - start of section to free.
* len - length of section to free.
*
! * RETURN: 0 if success
! * error code if failure
*/
static int
zfs_free_range(znode_t *zp, uint64_t off, uint64_t len)
{
zfsvfs_t *zfsvfs = zp->z_zfsvfs;
--- 1521,1531 ----
*
* IN: zp - znode of file to free data in.
* off - start of section to free.
* len - length of section to free.
*
! * RETURN: 0 on success, error code on failure
*/
static int
zfs_free_range(znode_t *zp, uint64_t off, uint64_t len)
{
zfsvfs_t *zfsvfs = zp->z_zfsvfs;
*** 1562,1573 ****
* Truncate a file
*
* IN: zp - znode of file to free data in.
* end - new end-of-file.
*
! * RETURN: 0 if success
! * error code if failure
*/
static int
zfs_trunc(znode_t *zp, uint64_t end)
{
zfsvfs_t *zfsvfs = zp->z_zfsvfs;
--- 1559,1569 ----
* Truncate a file
*
* IN: zp - znode of file to free data in.
* end - new end-of-file.
*
! * RETURN: 0 on success, error code on failure
*/
static int
zfs_trunc(znode_t *zp, uint64_t end)
{
zfsvfs_t *zfsvfs = zp->z_zfsvfs;
*** 1661,1672 ****
* off - start of range
* len - end of range (0 => EOF)
* flag - current file open mode flags.
* log - TRUE if this action should be logged
*
! * RETURN: 0 if success
! * error code if failure
*/
int
zfs_freesp(znode_t *zp, uint64_t off, uint64_t len, int flag, boolean_t log)
{
vnode_t *vp = ZTOV(zp);
--- 1657,1667 ----
* off - start of range
* len - end of range (0 => EOF)
* flag - current file open mode flags.
* log - TRUE if this action should be logged
*
! * RETURN: 0 on success, error code on failure
*/
int
zfs_freesp(znode_t *zp, uint64_t off, uint64_t len, int flag, boolean_t log)
{
vnode_t *vp = ZTOV(zp);