Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (c) 2000-2006 Silicon Graphics, Inc. |
| 3 | * Copyright (c) 2012-2013 Red Hat, Inc. |
| 4 | * All rights reserved. |
| 5 | * |
| 6 | * This program is free software; you can redistribute it and/or |
| 7 | * modify it under the terms of the GNU General Public License as |
| 8 | * published by the Free Software Foundation. |
| 9 | * |
| 10 | * This program is distributed in the hope that it would be useful, |
| 11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of |
| 12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
| 13 | * GNU General Public License for more details. |
| 14 | * |
| 15 | * You should have received a copy of the GNU General Public License |
| 16 | * along with this program; if not, write the Free Software Foundation, |
| 17 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA |
| 18 | */ |
| 19 | #include "xfs.h" |
Dave Chinner | 239880e | 2013-10-23 10:50:10 +1100 | [diff] [blame] | 20 | #include "xfs_shared.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 21 | #include "xfs_fs.h" |
Dave Chinner | 6ca1c90 | 2013-08-12 20:49:26 +1000 | [diff] [blame] | 22 | #include "xfs_format.h" |
Dave Chinner | 239880e | 2013-10-23 10:50:10 +1100 | [diff] [blame] | 23 | #include "xfs_log_format.h" |
| 24 | #include "xfs_trans_resv.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 25 | #include "xfs_bit.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 26 | #include "xfs_mount.h" |
Dave Chinner | 5706278 | 2013-10-15 09:17:51 +1100 | [diff] [blame] | 27 | #include "xfs_da_format.h" |
Dave Chinner | d6cf130 | 2014-06-06 15:14:11 +1000 | [diff] [blame] | 28 | #include "xfs_da_btree.h" |
Dave Chinner | 2b9ab5a | 2013-08-12 20:49:37 +1000 | [diff] [blame] | 29 | #include "xfs_dir2.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 30 | #include "xfs_inode.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 31 | #include "xfs_ialloc.h" |
| 32 | #include "xfs_alloc.h" |
| 33 | #include "xfs_bmap.h" |
Dave Chinner | a4fbe6a | 2013-10-23 10:51:50 +1100 | [diff] [blame] | 34 | #include "xfs_bmap_btree.h" |
Dave Chinner | 6898811 | 2013-08-12 20:49:42 +1000 | [diff] [blame] | 35 | #include "xfs_bmap_util.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 36 | #include "xfs_error.h" |
| 37 | #include "xfs_quota.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 38 | #include "xfs_trans_space.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 39 | #include "xfs_trace.h" |
| 40 | #include "xfs_symlink.h" |
Dave Chinner | 239880e | 2013-10-23 10:50:10 +1100 | [diff] [blame] | 41 | #include "xfs_trans.h" |
Dave Chinner | 239880e | 2013-10-23 10:50:10 +1100 | [diff] [blame] | 42 | #include "xfs_log.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 43 | |
| 44 | /* ----- Kernel only functions below ----- */ |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 45 | STATIC int |
| 46 | xfs_readlink_bmap( |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 47 | struct xfs_inode *ip, |
| 48 | char *link) |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 49 | { |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 50 | struct xfs_mount *mp = ip->i_mount; |
| 51 | struct xfs_bmbt_irec mval[XFS_SYMLINK_MAPS]; |
| 52 | struct xfs_buf *bp; |
| 53 | xfs_daddr_t d; |
| 54 | char *cur_chunk; |
| 55 | int pathlen = ip->i_d.di_size; |
| 56 | int nmaps = XFS_SYMLINK_MAPS; |
| 57 | int byte_cnt; |
| 58 | int n; |
| 59 | int error = 0; |
| 60 | int fsblocks = 0; |
| 61 | int offset; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 62 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 63 | fsblocks = xfs_symlink_blocks(mp, pathlen); |
| 64 | error = xfs_bmapi_read(ip, 0, fsblocks, mval, &nmaps, 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 65 | if (error) |
| 66 | goto out; |
| 67 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 68 | offset = 0; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 69 | for (n = 0; n < nmaps; n++) { |
| 70 | d = XFS_FSB_TO_DADDR(mp, mval[n].br_startblock); |
| 71 | byte_cnt = XFS_FSB_TO_B(mp, mval[n].br_blockcount); |
| 72 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 73 | bp = xfs_buf_read(mp->m_ddev_targp, d, BTOBB(byte_cnt), 0, |
| 74 | &xfs_symlink_buf_ops); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 75 | if (!bp) |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 76 | return -ENOMEM; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 77 | error = bp->b_error; |
| 78 | if (error) { |
| 79 | xfs_buf_ioerror_alert(bp, __func__); |
| 80 | xfs_buf_relse(bp); |
Dave Chinner | ac75a1f | 2014-03-07 16:19:14 +1100 | [diff] [blame] | 81 | |
| 82 | /* bad CRC means corrupted metadata */ |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 83 | if (error == -EFSBADCRC) |
| 84 | error = -EFSCORRUPTED; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 85 | goto out; |
| 86 | } |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 87 | byte_cnt = XFS_SYMLINK_BUF_SPACE(mp, byte_cnt); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 88 | if (pathlen < byte_cnt) |
| 89 | byte_cnt = pathlen; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 90 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 91 | cur_chunk = bp->b_addr; |
| 92 | if (xfs_sb_version_hascrc(&mp->m_sb)) { |
Eric Sandeen | bda65ef | 2014-04-14 19:05:43 +1000 | [diff] [blame] | 93 | if (!xfs_symlink_hdr_ok(ip->i_ino, offset, |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 94 | byte_cnt, bp)) { |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 95 | error = -EFSCORRUPTED; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 96 | xfs_alert(mp, |
| 97 | "symlink header does not match required off/len/owner (0x%x/Ox%x,0x%llx)", |
| 98 | offset, byte_cnt, ip->i_ino); |
| 99 | xfs_buf_relse(bp); |
| 100 | goto out; |
| 101 | |
| 102 | } |
| 103 | |
| 104 | cur_chunk += sizeof(struct xfs_dsymlink_hdr); |
| 105 | } |
| 106 | |
Eric Sandeen | 2ac56d3 | 2015-06-22 09:42:48 +1000 | [diff] [blame] | 107 | memcpy(link + offset, cur_chunk, byte_cnt); |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 108 | |
| 109 | pathlen -= byte_cnt; |
| 110 | offset += byte_cnt; |
| 111 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 112 | xfs_buf_relse(bp); |
| 113 | } |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 114 | ASSERT(pathlen == 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 115 | |
| 116 | link[ip->i_d.di_size] = '\0'; |
| 117 | error = 0; |
| 118 | |
| 119 | out: |
| 120 | return error; |
| 121 | } |
| 122 | |
| 123 | int |
| 124 | xfs_readlink( |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 125 | struct xfs_inode *ip, |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 126 | char *link) |
| 127 | { |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 128 | struct xfs_mount *mp = ip->i_mount; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 129 | xfs_fsize_t pathlen; |
| 130 | int error = 0; |
| 131 | |
| 132 | trace_xfs_readlink(ip); |
| 133 | |
Christoph Hellwig | 30ee052 | 2016-04-06 07:53:29 +1000 | [diff] [blame] | 134 | ASSERT(!(ip->i_df.if_flags & XFS_IFINLINE)); |
| 135 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 136 | if (XFS_FORCED_SHUTDOWN(mp)) |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 137 | return -EIO; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 138 | |
| 139 | xfs_ilock(ip, XFS_ILOCK_SHARED); |
| 140 | |
| 141 | pathlen = ip->i_d.di_size; |
| 142 | if (!pathlen) |
| 143 | goto out; |
| 144 | |
| 145 | if (pathlen < 0 || pathlen > MAXPATHLEN) { |
| 146 | xfs_alert(mp, "%s: inode (%llu) bad symlink length (%lld)", |
| 147 | __func__, (unsigned long long) ip->i_ino, |
| 148 | (long long) pathlen); |
| 149 | ASSERT(0); |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 150 | error = -EFSCORRUPTED; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 151 | goto out; |
| 152 | } |
| 153 | |
| 154 | |
Christoph Hellwig | 30ee052 | 2016-04-06 07:53:29 +1000 | [diff] [blame] | 155 | error = xfs_readlink_bmap(ip, link); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 156 | |
| 157 | out: |
| 158 | xfs_iunlock(ip, XFS_ILOCK_SHARED); |
| 159 | return error; |
| 160 | } |
| 161 | |
| 162 | int |
| 163 | xfs_symlink( |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 164 | struct xfs_inode *dp, |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 165 | struct xfs_name *link_name, |
| 166 | const char *target_path, |
| 167 | umode_t mode, |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 168 | struct xfs_inode **ipp) |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 169 | { |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 170 | struct xfs_mount *mp = dp->i_mount; |
| 171 | struct xfs_trans *tp = NULL; |
| 172 | struct xfs_inode *ip = NULL; |
| 173 | int error = 0; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 174 | int pathlen; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 175 | struct xfs_bmap_free free_list; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 176 | xfs_fsblock_t first_block; |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 177 | bool unlock_dp_on_error = false; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 178 | xfs_fileoff_t first_fsb; |
| 179 | xfs_filblks_t fs_blocks; |
| 180 | int nmaps; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 181 | struct xfs_bmbt_irec mval[XFS_SYMLINK_MAPS]; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 182 | xfs_daddr_t d; |
| 183 | const char *cur_chunk; |
| 184 | int byte_cnt; |
| 185 | int n; |
| 186 | xfs_buf_t *bp; |
| 187 | prid_t prid; |
Chandra Seetharaman | 113a568 | 2013-06-27 17:25:07 -0500 | [diff] [blame] | 188 | struct xfs_dquot *udqp = NULL; |
| 189 | struct xfs_dquot *gdqp = NULL; |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 190 | struct xfs_dquot *pdqp = NULL; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 191 | uint resblks; |
| 192 | |
| 193 | *ipp = NULL; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 194 | |
| 195 | trace_xfs_symlink(dp, link_name); |
| 196 | |
| 197 | if (XFS_FORCED_SHUTDOWN(mp)) |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 198 | return -EIO; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 199 | |
| 200 | /* |
| 201 | * Check component lengths of the target path name. |
| 202 | */ |
| 203 | pathlen = strlen(target_path); |
| 204 | if (pathlen >= MAXPATHLEN) /* total string too long */ |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 205 | return -ENAMETOOLONG; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 206 | |
| 207 | udqp = gdqp = NULL; |
Zhi Yong Wu | 163467d | 2013-12-18 08:22:39 +0800 | [diff] [blame] | 208 | prid = xfs_get_initial_prid(dp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 209 | |
| 210 | /* |
| 211 | * Make sure that we have allocated dquot(s) on disk. |
| 212 | */ |
Dwight Engen | 7aab1b2 | 2013-08-15 14:08:01 -0400 | [diff] [blame] | 213 | error = xfs_qm_vop_dqalloc(dp, |
| 214 | xfs_kuid_to_uid(current_fsuid()), |
| 215 | xfs_kgid_to_gid(current_fsgid()), prid, |
| 216 | XFS_QMOPT_QUOTALL | XFS_QMOPT_INHERIT, |
| 217 | &udqp, &gdqp, &pdqp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 218 | if (error) |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 219 | return error; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 220 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 221 | /* |
| 222 | * The symlink will fit into the inode data fork? |
| 223 | * There can't be any attributes so we get the whole variable part. |
| 224 | */ |
| 225 | if (pathlen <= XFS_LITINO(mp, dp->i_d.di_version)) |
| 226 | fs_blocks = 0; |
| 227 | else |
Dave Chinner | 321a958 | 2013-05-27 16:38:20 +1000 | [diff] [blame] | 228 | fs_blocks = xfs_symlink_blocks(mp, pathlen); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 229 | resblks = XFS_SYMLINK_SPACE_RES(mp, link_name->len, fs_blocks); |
Christoph Hellwig | 253f491 | 2016-04-06 09:19:55 +1000 | [diff] [blame] | 230 | |
| 231 | error = xfs_trans_alloc(mp, &M_RES(mp)->tr_symlink, resblks, 0, 0, &tp); |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 232 | if (error == -ENOSPC && fs_blocks == 0) { |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 233 | resblks = 0; |
Christoph Hellwig | 253f491 | 2016-04-06 09:19:55 +1000 | [diff] [blame] | 234 | error = xfs_trans_alloc(mp, &M_RES(mp)->tr_symlink, 0, 0, 0, |
| 235 | &tp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 236 | } |
Christoph Hellwig | 4906e21 | 2015-06-04 13:47:56 +1000 | [diff] [blame] | 237 | if (error) |
Christoph Hellwig | 253f491 | 2016-04-06 09:19:55 +1000 | [diff] [blame] | 238 | goto out_release_inode; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 239 | |
Dave Chinner | dbad7c9 | 2015-08-19 10:33:00 +1000 | [diff] [blame] | 240 | xfs_ilock(dp, XFS_IOLOCK_EXCL | XFS_ILOCK_EXCL | |
| 241 | XFS_IOLOCK_PARENT | XFS_ILOCK_PARENT); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 242 | unlock_dp_on_error = true; |
| 243 | |
| 244 | /* |
| 245 | * Check whether the directory allows new symlinks or not. |
| 246 | */ |
| 247 | if (dp->i_d.di_flags & XFS_DIFLAG_NOSYMLINKS) { |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 248 | error = -EPERM; |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 249 | goto out_trans_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 250 | } |
| 251 | |
| 252 | /* |
| 253 | * Reserve disk quota : blocks and inode. |
| 254 | */ |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 255 | error = xfs_trans_reserve_quota(tp, mp, udqp, gdqp, |
| 256 | pdqp, resblks, 1, 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 257 | if (error) |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 258 | goto out_trans_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 259 | |
| 260 | /* |
| 261 | * Check for ability to enter directory entry, if no space reserved. |
| 262 | */ |
Eric Sandeen | 94f3cad | 2014-09-09 11:57:52 +1000 | [diff] [blame] | 263 | if (!resblks) { |
| 264 | error = xfs_dir_canenter(tp, dp, link_name); |
| 265 | if (error) |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 266 | goto out_trans_cancel; |
Eric Sandeen | 94f3cad | 2014-09-09 11:57:52 +1000 | [diff] [blame] | 267 | } |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 268 | /* |
| 269 | * Initialize the bmap freelist prior to calling either |
| 270 | * bmapi or the directory create code. |
| 271 | */ |
| 272 | xfs_bmap_init(&free_list, &first_block); |
| 273 | |
| 274 | /* |
| 275 | * Allocate an inode for the symlink. |
| 276 | */ |
| 277 | error = xfs_dir_ialloc(&tp, dp, S_IFLNK | (mode & ~S_IFMT), 1, 0, |
| 278 | prid, resblks > 0, &ip, NULL); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 279 | if (error) |
| 280 | goto out_trans_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 281 | |
| 282 | /* |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 283 | * Now we join the directory inode to the transaction. We do not do it |
| 284 | * earlier because xfs_dir_ialloc might commit the previous transaction |
| 285 | * (and release all the locks). An error from here on will result in |
| 286 | * the transaction cancel unlocking dp so don't do it explicitly in the |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 287 | * error path. |
| 288 | */ |
Dave Chinner | dbad7c9 | 2015-08-19 10:33:00 +1000 | [diff] [blame] | 289 | xfs_trans_ijoin(tp, dp, XFS_IOLOCK_EXCL | XFS_ILOCK_EXCL); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 290 | unlock_dp_on_error = false; |
| 291 | |
| 292 | /* |
| 293 | * Also attach the dquot(s) to it, if applicable. |
| 294 | */ |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 295 | xfs_qm_vop_create_dqattach(tp, ip, udqp, gdqp, pdqp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 296 | |
| 297 | if (resblks) |
| 298 | resblks -= XFS_IALLOC_SPACE_RES(mp); |
| 299 | /* |
| 300 | * If the symlink will fit into the inode, write it inline. |
| 301 | */ |
| 302 | if (pathlen <= XFS_IFORK_DSIZE(ip)) { |
Christoph Hellwig | 143f4ae | 2016-04-06 07:41:43 +1000 | [diff] [blame] | 303 | xfs_init_local_fork(ip, XFS_DATA_FORK, target_path, pathlen); |
| 304 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 305 | ip->i_d.di_size = pathlen; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 306 | ip->i_d.di_format = XFS_DINODE_FMT_LOCAL; |
| 307 | xfs_trans_log_inode(tp, ip, XFS_ILOG_DDATA | XFS_ILOG_CORE); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 308 | } else { |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 309 | int offset; |
| 310 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 311 | first_fsb = 0; |
| 312 | nmaps = XFS_SYMLINK_MAPS; |
| 313 | |
| 314 | error = xfs_bmapi_write(tp, ip, first_fsb, fs_blocks, |
| 315 | XFS_BMAPI_METADATA, &first_block, resblks, |
| 316 | mval, &nmaps, &free_list); |
| 317 | if (error) |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 318 | goto out_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 319 | |
| 320 | if (resblks) |
| 321 | resblks -= fs_blocks; |
| 322 | ip->i_d.di_size = pathlen; |
| 323 | xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE); |
| 324 | |
| 325 | cur_chunk = target_path; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 326 | offset = 0; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 327 | for (n = 0; n < nmaps; n++) { |
Dave Chinner | 321a958 | 2013-05-27 16:38:20 +1000 | [diff] [blame] | 328 | char *buf; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 329 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 330 | d = XFS_FSB_TO_DADDR(mp, mval[n].br_startblock); |
| 331 | byte_cnt = XFS_FSB_TO_B(mp, mval[n].br_blockcount); |
| 332 | bp = xfs_trans_get_buf(tp, mp->m_ddev_targp, d, |
| 333 | BTOBB(byte_cnt), 0); |
| 334 | if (!bp) { |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 335 | error = -ENOMEM; |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 336 | goto out_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 337 | } |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 338 | bp->b_ops = &xfs_symlink_buf_ops; |
| 339 | |
| 340 | byte_cnt = XFS_SYMLINK_BUF_SPACE(mp, byte_cnt); |
Dave Chinner | 321a958 | 2013-05-27 16:38:20 +1000 | [diff] [blame] | 341 | byte_cnt = min(byte_cnt, pathlen); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 342 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 343 | buf = bp->b_addr; |
| 344 | buf += xfs_symlink_hdr_set(mp, ip->i_ino, offset, |
| 345 | byte_cnt, bp); |
| 346 | |
| 347 | memcpy(buf, cur_chunk, byte_cnt); |
| 348 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 349 | cur_chunk += byte_cnt; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 350 | pathlen -= byte_cnt; |
| 351 | offset += byte_cnt; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 352 | |
Dave Chinner | daf7b79 | 2013-09-02 10:32:00 +1000 | [diff] [blame] | 353 | xfs_trans_buf_set_type(tp, bp, XFS_BLFT_SYMLINK_BUF); |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 354 | xfs_trans_log_buf(tp, bp, 0, (buf + byte_cnt - 1) - |
| 355 | (char *)bp->b_addr); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 356 | } |
Dave Chinner | 321a958 | 2013-05-27 16:38:20 +1000 | [diff] [blame] | 357 | ASSERT(pathlen == 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 358 | } |
| 359 | |
| 360 | /* |
| 361 | * Create the directory entry for the symlink. |
| 362 | */ |
| 363 | error = xfs_dir_createname(tp, dp, link_name, ip->i_ino, |
| 364 | &first_block, &free_list, resblks); |
| 365 | if (error) |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 366 | goto out_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 367 | xfs_trans_ichgtime(tp, dp, XFS_ICHGTIME_MOD | XFS_ICHGTIME_CHG); |
| 368 | xfs_trans_log_inode(tp, dp, XFS_ILOG_CORE); |
| 369 | |
| 370 | /* |
| 371 | * If this is a synchronous mount, make sure that the |
| 372 | * symlink transaction goes to disk before returning to |
| 373 | * the user. |
| 374 | */ |
| 375 | if (mp->m_flags & (XFS_MOUNT_WSYNC|XFS_MOUNT_DIRSYNC)) { |
| 376 | xfs_trans_set_sync(tp); |
| 377 | } |
| 378 | |
Eric Sandeen | f6106ef | 2016-01-11 11:34:01 +1100 | [diff] [blame] | 379 | error = xfs_bmap_finish(&tp, &free_list, NULL); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 380 | if (error) |
| 381 | goto out_bmap_cancel; |
| 382 | |
Christoph Hellwig | 7039331 | 2015-06-04 13:48:08 +1000 | [diff] [blame] | 383 | error = xfs_trans_commit(tp); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 384 | if (error) |
| 385 | goto out_release_inode; |
| 386 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 387 | xfs_qm_dqrele(udqp); |
| 388 | xfs_qm_dqrele(gdqp); |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 389 | xfs_qm_dqrele(pdqp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 390 | |
| 391 | *ipp = ip; |
| 392 | return 0; |
| 393 | |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 394 | out_bmap_cancel: |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 395 | xfs_bmap_cancel(&free_list); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 396 | out_trans_cancel: |
Christoph Hellwig | 4906e21 | 2015-06-04 13:47:56 +1000 | [diff] [blame] | 397 | xfs_trans_cancel(tp); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 398 | out_release_inode: |
| 399 | /* |
| 400 | * Wait until after the current transaction is aborted to finish the |
| 401 | * setup of the inode and release the inode. This prevents recursive |
| 402 | * transactions and deadlocks from xfs_inactive. |
| 403 | */ |
| 404 | if (ip) { |
| 405 | xfs_finish_inode_setup(ip); |
| 406 | IRELE(ip); |
| 407 | } |
| 408 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 409 | xfs_qm_dqrele(udqp); |
| 410 | xfs_qm_dqrele(gdqp); |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 411 | xfs_qm_dqrele(pdqp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 412 | |
| 413 | if (unlock_dp_on_error) |
Dave Chinner | dbad7c9 | 2015-08-19 10:33:00 +1000 | [diff] [blame] | 414 | xfs_iunlock(dp, XFS_IOLOCK_EXCL | XFS_ILOCK_EXCL); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 415 | return error; |
| 416 | } |
| 417 | |
| 418 | /* |
| 419 | * Free a symlink that has blocks associated with it. |
| 420 | */ |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 421 | STATIC int |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 422 | xfs_inactive_symlink_rmt( |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 423 | struct xfs_inode *ip) |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 424 | { |
| 425 | xfs_buf_t *bp; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 426 | int done; |
| 427 | int error; |
| 428 | xfs_fsblock_t first_block; |
| 429 | xfs_bmap_free_t free_list; |
| 430 | int i; |
| 431 | xfs_mount_t *mp; |
| 432 | xfs_bmbt_irec_t mval[XFS_SYMLINK_MAPS]; |
| 433 | int nmaps; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 434 | int size; |
| 435 | xfs_trans_t *tp; |
| 436 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 437 | mp = ip->i_mount; |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 438 | ASSERT(ip->i_df.if_flags & XFS_IFEXTENTS); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 439 | /* |
| 440 | * We're freeing a symlink that has some |
| 441 | * blocks allocated to it. Free the |
| 442 | * blocks here. We know that we've got |
| 443 | * either 1 or 2 extents and that we can |
| 444 | * free them all in one bunmapi call. |
| 445 | */ |
| 446 | ASSERT(ip->i_d.di_nextents > 0 && ip->i_d.di_nextents <= 2); |
| 447 | |
Christoph Hellwig | 253f491 | 2016-04-06 09:19:55 +1000 | [diff] [blame] | 448 | error = xfs_trans_alloc(mp, &M_RES(mp)->tr_itruncate, 0, 0, 0, &tp); |
| 449 | if (error) |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 450 | return error; |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 451 | |
| 452 | xfs_ilock(ip, XFS_ILOCK_EXCL); |
| 453 | xfs_trans_ijoin(tp, ip, 0); |
| 454 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 455 | /* |
| 456 | * Lock the inode, fix the size, and join it to the transaction. |
| 457 | * Hold it so in the normal path, we still have it locked for |
| 458 | * the second transaction. In the error paths we need it |
| 459 | * held so the cancel won't rele it, see below. |
| 460 | */ |
| 461 | size = (int)ip->i_d.di_size; |
| 462 | ip->i_d.di_size = 0; |
| 463 | xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE); |
| 464 | /* |
| 465 | * Find the block(s) so we can inval and unmap them. |
| 466 | */ |
| 467 | done = 0; |
| 468 | xfs_bmap_init(&free_list, &first_block); |
| 469 | nmaps = ARRAY_SIZE(mval); |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 470 | error = xfs_bmapi_read(ip, 0, xfs_symlink_blocks(mp, size), |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 471 | mval, &nmaps, 0); |
| 472 | if (error) |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 473 | goto error_trans_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 474 | /* |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 475 | * Invalidate the block(s). No validation is done. |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 476 | */ |
| 477 | for (i = 0; i < nmaps; i++) { |
| 478 | bp = xfs_trans_get_buf(tp, mp->m_ddev_targp, |
| 479 | XFS_FSB_TO_DADDR(mp, mval[i].br_startblock), |
| 480 | XFS_FSB_TO_BB(mp, mval[i].br_blockcount), 0); |
| 481 | if (!bp) { |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 482 | error = -ENOMEM; |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 483 | goto error_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 484 | } |
| 485 | xfs_trans_binval(tp, bp); |
| 486 | } |
| 487 | /* |
| 488 | * Unmap the dead block(s) to the free_list. |
| 489 | */ |
Dave Chinner | ab7bb61 | 2015-07-29 11:51:01 +1000 | [diff] [blame] | 490 | error = xfs_bunmapi(tp, ip, 0, size, 0, nmaps, |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 491 | &first_block, &free_list, &done); |
| 492 | if (error) |
| 493 | goto error_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 494 | ASSERT(done); |
| 495 | /* |
| 496 | * Commit the first transaction. This logs the EFI and the inode. |
| 497 | */ |
Eric Sandeen | f6106ef | 2016-01-11 11:34:01 +1100 | [diff] [blame] | 498 | error = xfs_bmap_finish(&tp, &free_list, ip); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 499 | if (error) |
| 500 | goto error_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 501 | /* |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 502 | * The first xact was committed, so add the inode to the new one. |
| 503 | * Mark it dirty so it will be logged and moved forward in the log as |
| 504 | * part of every commit. |
| 505 | */ |
| 506 | xfs_trans_ijoin(tp, ip, 0); |
| 507 | xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE); |
| 508 | /* |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 509 | * Commit the transaction containing extent freeing and EFDs. |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 510 | */ |
Christoph Hellwig | 7039331 | 2015-06-04 13:48:08 +1000 | [diff] [blame] | 511 | error = xfs_trans_commit(tp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 512 | if (error) { |
| 513 | ASSERT(XFS_FORCED_SHUTDOWN(mp)); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 514 | goto error_unlock; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 515 | } |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 516 | |
| 517 | /* |
| 518 | * Remove the memory for extent descriptions (just bookkeeping). |
| 519 | */ |
| 520 | if (ip->i_df.if_bytes) |
| 521 | xfs_idata_realloc(ip, -ip->i_df.if_bytes, XFS_DATA_FORK); |
| 522 | ASSERT(ip->i_df.if_bytes == 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 523 | |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 524 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 525 | return 0; |
| 526 | |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 527 | error_bmap_cancel: |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 528 | xfs_bmap_cancel(&free_list); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 529 | error_trans_cancel: |
Christoph Hellwig | 4906e21 | 2015-06-04 13:47:56 +1000 | [diff] [blame] | 530 | xfs_trans_cancel(tp); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 531 | error_unlock: |
| 532 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 533 | return error; |
| 534 | } |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 535 | |
| 536 | /* |
| 537 | * xfs_inactive_symlink - free a symlink |
| 538 | */ |
| 539 | int |
| 540 | xfs_inactive_symlink( |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 541 | struct xfs_inode *ip) |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 542 | { |
| 543 | struct xfs_mount *mp = ip->i_mount; |
| 544 | int pathlen; |
| 545 | |
| 546 | trace_xfs_inactive_symlink(ip); |
| 547 | |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 548 | if (XFS_FORCED_SHUTDOWN(mp)) |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 549 | return -EIO; |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 550 | |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 551 | xfs_ilock(ip, XFS_ILOCK_EXCL); |
| 552 | |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 553 | /* |
| 554 | * Zero length symlinks _can_ exist. |
| 555 | */ |
| 556 | pathlen = (int)ip->i_d.di_size; |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 557 | if (!pathlen) { |
| 558 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 559 | return 0; |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 560 | } |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 561 | |
| 562 | if (pathlen < 0 || pathlen > MAXPATHLEN) { |
| 563 | xfs_alert(mp, "%s: inode (0x%llx) bad symlink length (%d)", |
| 564 | __func__, (unsigned long long)ip->i_ino, pathlen); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 565 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 566 | ASSERT(0); |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 567 | return -EFSCORRUPTED; |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 568 | } |
| 569 | |
| 570 | if (ip->i_df.if_flags & XFS_IFINLINE) { |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 571 | if (ip->i_df.if_bytes > 0) |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 572 | xfs_idata_realloc(ip, -(ip->i_df.if_bytes), |
| 573 | XFS_DATA_FORK); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 574 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 575 | ASSERT(ip->i_df.if_bytes == 0); |
| 576 | return 0; |
| 577 | } |
| 578 | |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 579 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
| 580 | |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 581 | /* remove the remote symlink */ |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 582 | return xfs_inactive_symlink_rmt(ip); |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 583 | } |