Project

General

Profile

Bug #4039

zfs_rename()/zfs_link() needs stronger test for XDEV

Added by Marcel Telka over 7 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
High
Assignee:
Marcel Telka
Category:
zfs - Zettabyte File System
Start date:
2013-08-12
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
needs-triage
Gerrit CR:

Description

The zfs vnops entry points in zfs_link() and zfs_rename() are already doing the checks for the cross-device operations, but these checks are not complete. They do not count the v_vfsp tweaks done deep in the zfs code here:

http://src.illumos.org/source/xref/illumos-gate/usr/src/uts/common/fs/zfs/zfs_ctldir.c#925

The issue was reported by Pascal Bouchareine <>. Thanks.

Related issues

Related to illumos gate - Bug #4638: Panic in ZFS via rfs3_setattr()/rfs3_write(): dirtying snapshot!ResolvedMarcel Telka2014-02-27

Actions
#1

Updated by Marcel Telka over 7 years ago

  • Status changed from In Progress to Pending RTI
#2

Updated by Dan McDonald over 7 years ago

  • Status changed from Pending RTI to Resolved
  • % Done changed from 0 to 100

commit 18e649782fcf9da52d755b2074b4b9382426c818
Author: Marcel Telka <>
Date: Tue Aug 13 07:32:14 2013 +0200

4039 zfs_rename()/zfs_link() needs stronger test for XDEV
    Reviewed by: Gordon Ross &lt;&gt;
    Reviewed by: Kevin Crowe &lt;&gt;
    Reviewed by: Saso Kiselkov &lt;&gt;
    Reviewed by: Matthew Ahrens &lt;&gt;
    Approved by: Dan McDonald &lt;&gt;

:100644 100644 0e7219e... 19dd2ec... M usr/src/uts/common/fs/zfs/zfs_vnops.c

Also available in: Atom PDF