xfs: Factor out xfs_attr_rmtval_invalidate
[linux-2.6-microblaze.git] / fs / xfs / libxfs / xfs_attr_remote.c
1 // SPDX-License-Identifier: GPL-2.0
2 /*
3  * Copyright (c) 2000-2005 Silicon Graphics, Inc.
4  * Copyright (c) 2013 Red Hat, Inc.
5  * All Rights Reserved.
6  */
7 #include "xfs.h"
8 #include "xfs_fs.h"
9 #include "xfs_shared.h"
10 #include "xfs_format.h"
11 #include "xfs_log_format.h"
12 #include "xfs_trans_resv.h"
13 #include "xfs_bit.h"
14 #include "xfs_mount.h"
15 #include "xfs_defer.h"
16 #include "xfs_da_format.h"
17 #include "xfs_da_btree.h"
18 #include "xfs_inode.h"
19 #include "xfs_trans.h"
20 #include "xfs_bmap.h"
21 #include "xfs_attr.h"
22 #include "xfs_attr_remote.h"
23 #include "xfs_trace.h"
24 #include "xfs_error.h"
25
26 #define ATTR_RMTVALUE_MAPSIZE   1       /* # of map entries at once */
27
28 /*
29  * Remote Attribute Values
30  * =======================
31  *
32  * Remote extended attribute values are conceptually simple -- they're written
33  * to data blocks mapped by an inode's attribute fork, and they have an upper
34  * size limit of 64k.  Setting a value does not involve the XFS log.
35  *
36  * However, on a v5 filesystem, maximally sized remote attr values require one
37  * block more than 64k worth of space to hold both the remote attribute value
38  * header (64 bytes).  On a 4k block filesystem this results in a 68k buffer;
39  * on a 64k block filesystem, this would be a 128k buffer.  Note that the log
40  * format can only handle a dirty buffer of XFS_MAX_BLOCKSIZE length (64k).
41  * Therefore, we /must/ ensure that remote attribute value buffers never touch
42  * the logging system and therefore never have a log item.
43  */
44
45 /*
46  * Each contiguous block has a header, so it is not just a simple attribute
47  * length to FSB conversion.
48  */
49 int
50 xfs_attr3_rmt_blocks(
51         struct xfs_mount *mp,
52         int             attrlen)
53 {
54         if (xfs_sb_version_hascrc(&mp->m_sb)) {
55                 int buflen = XFS_ATTR3_RMT_BUF_SPACE(mp, mp->m_sb.sb_blocksize);
56                 return (attrlen + buflen - 1) / buflen;
57         }
58         return XFS_B_TO_FSB(mp, attrlen);
59 }
60
61 /*
62  * Checking of the remote attribute header is split into two parts. The verifier
63  * does CRC, location and bounds checking, the unpacking function checks the
64  * attribute parameters and owner.
65  */
66 static xfs_failaddr_t
67 xfs_attr3_rmt_hdr_ok(
68         void                    *ptr,
69         xfs_ino_t               ino,
70         uint32_t                offset,
71         uint32_t                size,
72         xfs_daddr_t             bno)
73 {
74         struct xfs_attr3_rmt_hdr *rmt = ptr;
75
76         if (bno != be64_to_cpu(rmt->rm_blkno))
77                 return __this_address;
78         if (offset != be32_to_cpu(rmt->rm_offset))
79                 return __this_address;
80         if (size != be32_to_cpu(rmt->rm_bytes))
81                 return __this_address;
82         if (ino != be64_to_cpu(rmt->rm_owner))
83                 return __this_address;
84
85         /* ok */
86         return NULL;
87 }
88
89 static xfs_failaddr_t
90 xfs_attr3_rmt_verify(
91         struct xfs_mount        *mp,
92         struct xfs_buf          *bp,
93         void                    *ptr,
94         int                     fsbsize,
95         xfs_daddr_t             bno)
96 {
97         struct xfs_attr3_rmt_hdr *rmt = ptr;
98
99         if (!xfs_sb_version_hascrc(&mp->m_sb))
100                 return __this_address;
101         if (!xfs_verify_magic(bp, rmt->rm_magic))
102                 return __this_address;
103         if (!uuid_equal(&rmt->rm_uuid, &mp->m_sb.sb_meta_uuid))
104                 return __this_address;
105         if (be64_to_cpu(rmt->rm_blkno) != bno)
106                 return __this_address;
107         if (be32_to_cpu(rmt->rm_bytes) > fsbsize - sizeof(*rmt))
108                 return __this_address;
109         if (be32_to_cpu(rmt->rm_offset) +
110                                 be32_to_cpu(rmt->rm_bytes) > XFS_XATTR_SIZE_MAX)
111                 return __this_address;
112         if (rmt->rm_owner == 0)
113                 return __this_address;
114
115         return NULL;
116 }
117
118 static int
119 __xfs_attr3_rmt_read_verify(
120         struct xfs_buf  *bp,
121         bool            check_crc,
122         xfs_failaddr_t  *failaddr)
123 {
124         struct xfs_mount *mp = bp->b_mount;
125         char            *ptr;
126         int             len;
127         xfs_daddr_t     bno;
128         int             blksize = mp->m_attr_geo->blksize;
129
130         /* no verification of non-crc buffers */
131         if (!xfs_sb_version_hascrc(&mp->m_sb))
132                 return 0;
133
134         ptr = bp->b_addr;
135         bno = bp->b_bn;
136         len = BBTOB(bp->b_length);
137         ASSERT(len >= blksize);
138
139         while (len > 0) {
140                 if (check_crc &&
141                     !xfs_verify_cksum(ptr, blksize, XFS_ATTR3_RMT_CRC_OFF)) {
142                         *failaddr = __this_address;
143                         return -EFSBADCRC;
144                 }
145                 *failaddr = xfs_attr3_rmt_verify(mp, bp, ptr, blksize, bno);
146                 if (*failaddr)
147                         return -EFSCORRUPTED;
148                 len -= blksize;
149                 ptr += blksize;
150                 bno += BTOBB(blksize);
151         }
152
153         if (len != 0) {
154                 *failaddr = __this_address;
155                 return -EFSCORRUPTED;
156         }
157
158         return 0;
159 }
160
161 static void
162 xfs_attr3_rmt_read_verify(
163         struct xfs_buf  *bp)
164 {
165         xfs_failaddr_t  fa;
166         int             error;
167
168         error = __xfs_attr3_rmt_read_verify(bp, true, &fa);
169         if (error)
170                 xfs_verifier_error(bp, error, fa);
171 }
172
173 static xfs_failaddr_t
174 xfs_attr3_rmt_verify_struct(
175         struct xfs_buf  *bp)
176 {
177         xfs_failaddr_t  fa;
178         int             error;
179
180         error = __xfs_attr3_rmt_read_verify(bp, false, &fa);
181         return error ? fa : NULL;
182 }
183
184 static void
185 xfs_attr3_rmt_write_verify(
186         struct xfs_buf  *bp)
187 {
188         struct xfs_mount *mp = bp->b_mount;
189         xfs_failaddr_t  fa;
190         int             blksize = mp->m_attr_geo->blksize;
191         char            *ptr;
192         int             len;
193         xfs_daddr_t     bno;
194
195         /* no verification of non-crc buffers */
196         if (!xfs_sb_version_hascrc(&mp->m_sb))
197                 return;
198
199         ptr = bp->b_addr;
200         bno = bp->b_bn;
201         len = BBTOB(bp->b_length);
202         ASSERT(len >= blksize);
203
204         while (len > 0) {
205                 struct xfs_attr3_rmt_hdr *rmt = (struct xfs_attr3_rmt_hdr *)ptr;
206
207                 fa = xfs_attr3_rmt_verify(mp, bp, ptr, blksize, bno);
208                 if (fa) {
209                         xfs_verifier_error(bp, -EFSCORRUPTED, fa);
210                         return;
211                 }
212
213                 /*
214                  * Ensure we aren't writing bogus LSNs to disk. See
215                  * xfs_attr3_rmt_hdr_set() for the explanation.
216                  */
217                 if (rmt->rm_lsn != cpu_to_be64(NULLCOMMITLSN)) {
218                         xfs_verifier_error(bp, -EFSCORRUPTED, __this_address);
219                         return;
220                 }
221                 xfs_update_cksum(ptr, blksize, XFS_ATTR3_RMT_CRC_OFF);
222
223                 len -= blksize;
224                 ptr += blksize;
225                 bno += BTOBB(blksize);
226         }
227
228         if (len != 0)
229                 xfs_verifier_error(bp, -EFSCORRUPTED, __this_address);
230 }
231
232 const struct xfs_buf_ops xfs_attr3_rmt_buf_ops = {
233         .name = "xfs_attr3_rmt",
234         .magic = { 0, cpu_to_be32(XFS_ATTR3_RMT_MAGIC) },
235         .verify_read = xfs_attr3_rmt_read_verify,
236         .verify_write = xfs_attr3_rmt_write_verify,
237         .verify_struct = xfs_attr3_rmt_verify_struct,
238 };
239
240 STATIC int
241 xfs_attr3_rmt_hdr_set(
242         struct xfs_mount        *mp,
243         void                    *ptr,
244         xfs_ino_t               ino,
245         uint32_t                offset,
246         uint32_t                size,
247         xfs_daddr_t             bno)
248 {
249         struct xfs_attr3_rmt_hdr *rmt = ptr;
250
251         if (!xfs_sb_version_hascrc(&mp->m_sb))
252                 return 0;
253
254         rmt->rm_magic = cpu_to_be32(XFS_ATTR3_RMT_MAGIC);
255         rmt->rm_offset = cpu_to_be32(offset);
256         rmt->rm_bytes = cpu_to_be32(size);
257         uuid_copy(&rmt->rm_uuid, &mp->m_sb.sb_meta_uuid);
258         rmt->rm_owner = cpu_to_be64(ino);
259         rmt->rm_blkno = cpu_to_be64(bno);
260
261         /*
262          * Remote attribute blocks are written synchronously, so we don't
263          * have an LSN that we can stamp in them that makes any sense to log
264          * recovery. To ensure that log recovery handles overwrites of these
265          * blocks sanely (i.e. once they've been freed and reallocated as some
266          * other type of metadata) we need to ensure that the LSN has a value
267          * that tells log recovery to ignore the LSN and overwrite the buffer
268          * with whatever is in it's log. To do this, we use the magic
269          * NULLCOMMITLSN to indicate that the LSN is invalid.
270          */
271         rmt->rm_lsn = cpu_to_be64(NULLCOMMITLSN);
272
273         return sizeof(struct xfs_attr3_rmt_hdr);
274 }
275
276 /*
277  * Helper functions to copy attribute data in and out of the one disk extents
278  */
279 STATIC int
280 xfs_attr_rmtval_copyout(
281         struct xfs_mount *mp,
282         struct xfs_buf  *bp,
283         xfs_ino_t       ino,
284         int             *offset,
285         int             *valuelen,
286         uint8_t         **dst)
287 {
288         char            *src = bp->b_addr;
289         xfs_daddr_t     bno = bp->b_bn;
290         int             len = BBTOB(bp->b_length);
291         int             blksize = mp->m_attr_geo->blksize;
292
293         ASSERT(len >= blksize);
294
295         while (len > 0 && *valuelen > 0) {
296                 int hdr_size = 0;
297                 int byte_cnt = XFS_ATTR3_RMT_BUF_SPACE(mp, blksize);
298
299                 byte_cnt = min(*valuelen, byte_cnt);
300
301                 if (xfs_sb_version_hascrc(&mp->m_sb)) {
302                         if (xfs_attr3_rmt_hdr_ok(src, ino, *offset,
303                                                   byte_cnt, bno)) {
304                                 xfs_alert(mp,
305 "remote attribute header mismatch bno/off/len/owner (0x%llx/0x%x/Ox%x/0x%llx)",
306                                         bno, *offset, byte_cnt, ino);
307                                 return -EFSCORRUPTED;
308                         }
309                         hdr_size = sizeof(struct xfs_attr3_rmt_hdr);
310                 }
311
312                 memcpy(*dst, src + hdr_size, byte_cnt);
313
314                 /* roll buffer forwards */
315                 len -= blksize;
316                 src += blksize;
317                 bno += BTOBB(blksize);
318
319                 /* roll attribute data forwards */
320                 *valuelen -= byte_cnt;
321                 *dst += byte_cnt;
322                 *offset += byte_cnt;
323         }
324         return 0;
325 }
326
327 STATIC void
328 xfs_attr_rmtval_copyin(
329         struct xfs_mount *mp,
330         struct xfs_buf  *bp,
331         xfs_ino_t       ino,
332         int             *offset,
333         int             *valuelen,
334         uint8_t         **src)
335 {
336         char            *dst = bp->b_addr;
337         xfs_daddr_t     bno = bp->b_bn;
338         int             len = BBTOB(bp->b_length);
339         int             blksize = mp->m_attr_geo->blksize;
340
341         ASSERT(len >= blksize);
342
343         while (len > 0 && *valuelen > 0) {
344                 int hdr_size;
345                 int byte_cnt = XFS_ATTR3_RMT_BUF_SPACE(mp, blksize);
346
347                 byte_cnt = min(*valuelen, byte_cnt);
348                 hdr_size = xfs_attr3_rmt_hdr_set(mp, dst, ino, *offset,
349                                                  byte_cnt, bno);
350
351                 memcpy(dst + hdr_size, *src, byte_cnt);
352
353                 /*
354                  * If this is the last block, zero the remainder of it.
355                  * Check that we are actually the last block, too.
356                  */
357                 if (byte_cnt + hdr_size < blksize) {
358                         ASSERT(*valuelen - byte_cnt == 0);
359                         ASSERT(len == blksize);
360                         memset(dst + hdr_size + byte_cnt, 0,
361                                         blksize - hdr_size - byte_cnt);
362                 }
363
364                 /* roll buffer forwards */
365                 len -= blksize;
366                 dst += blksize;
367                 bno += BTOBB(blksize);
368
369                 /* roll attribute data forwards */
370                 *valuelen -= byte_cnt;
371                 *src += byte_cnt;
372                 *offset += byte_cnt;
373         }
374 }
375
376 /*
377  * Read the value associated with an attribute from the out-of-line buffer
378  * that we stored it in.
379  *
380  * Returns 0 on successful retrieval, otherwise an error.
381  */
382 int
383 xfs_attr_rmtval_get(
384         struct xfs_da_args      *args)
385 {
386         struct xfs_bmbt_irec    map[ATTR_RMTVALUE_MAPSIZE];
387         struct xfs_mount        *mp = args->dp->i_mount;
388         struct xfs_buf          *bp;
389         xfs_dablk_t             lblkno = args->rmtblkno;
390         uint8_t                 *dst = args->value;
391         int                     valuelen;
392         int                     nmap;
393         int                     error;
394         int                     blkcnt = args->rmtblkcnt;
395         int                     i;
396         int                     offset = 0;
397
398         trace_xfs_attr_rmtval_get(args);
399
400         ASSERT(args->valuelen != 0);
401         ASSERT(args->rmtvaluelen == args->valuelen);
402
403         valuelen = args->rmtvaluelen;
404         while (valuelen > 0) {
405                 nmap = ATTR_RMTVALUE_MAPSIZE;
406                 error = xfs_bmapi_read(args->dp, (xfs_fileoff_t)lblkno,
407                                        blkcnt, map, &nmap,
408                                        XFS_BMAPI_ATTRFORK);
409                 if (error)
410                         return error;
411                 ASSERT(nmap >= 1);
412
413                 for (i = 0; (i < nmap) && (valuelen > 0); i++) {
414                         xfs_daddr_t     dblkno;
415                         int             dblkcnt;
416
417                         ASSERT((map[i].br_startblock != DELAYSTARTBLOCK) &&
418                                (map[i].br_startblock != HOLESTARTBLOCK));
419                         dblkno = XFS_FSB_TO_DADDR(mp, map[i].br_startblock);
420                         dblkcnt = XFS_FSB_TO_BB(mp, map[i].br_blockcount);
421                         error = xfs_buf_read(mp->m_ddev_targp, dblkno, dblkcnt,
422                                         0, &bp, &xfs_attr3_rmt_buf_ops);
423                         if (error)
424                                 return error;
425
426                         error = xfs_attr_rmtval_copyout(mp, bp, args->dp->i_ino,
427                                                         &offset, &valuelen,
428                                                         &dst);
429                         xfs_buf_relse(bp);
430                         if (error)
431                                 return error;
432
433                         /* roll attribute extent map forwards */
434                         lblkno += map[i].br_blockcount;
435                         blkcnt -= map[i].br_blockcount;
436                 }
437         }
438         ASSERT(valuelen == 0);
439         return 0;
440 }
441
442 /*
443  * Find a "hole" in the attribute address space large enough for us to drop the
444  * new attribute's value into
445  */
446 STATIC int
447 xfs_attr_rmt_find_hole(
448         struct xfs_da_args      *args)
449 {
450         struct xfs_inode        *dp = args->dp;
451         struct xfs_mount        *mp = dp->i_mount;
452         int                     error;
453         int                     blkcnt;
454         xfs_fileoff_t           lfileoff = 0;
455
456         /*
457          * Because CRC enable attributes have headers, we can't just do a
458          * straight byte to FSB conversion and have to take the header space
459          * into account.
460          */
461         blkcnt = xfs_attr3_rmt_blocks(mp, args->rmtvaluelen);
462         error = xfs_bmap_first_unused(args->trans, args->dp, blkcnt, &lfileoff,
463                                                    XFS_ATTR_FORK);
464         if (error)
465                 return error;
466
467         args->rmtblkno = (xfs_dablk_t)lfileoff;
468         args->rmtblkcnt = blkcnt;
469
470         return 0;
471 }
472
473 STATIC int
474 xfs_attr_rmtval_set_value(
475         struct xfs_da_args      *args)
476 {
477         struct xfs_inode        *dp = args->dp;
478         struct xfs_mount        *mp = dp->i_mount;
479         struct xfs_bmbt_irec    map;
480         xfs_dablk_t             lblkno;
481         uint8_t                 *src = args->value;
482         int                     blkcnt;
483         int                     valuelen;
484         int                     nmap;
485         int                     error;
486         int                     offset = 0;
487
488         /*
489          * Roll through the "value", copying the attribute value to the
490          * already-allocated blocks.  Blocks are written synchronously
491          * so that we can know they are all on disk before we turn off
492          * the INCOMPLETE flag.
493          */
494         lblkno = args->rmtblkno;
495         blkcnt = args->rmtblkcnt;
496         valuelen = args->rmtvaluelen;
497         while (valuelen > 0) {
498                 struct xfs_buf  *bp;
499                 xfs_daddr_t     dblkno;
500                 int             dblkcnt;
501
502                 ASSERT(blkcnt > 0);
503
504                 nmap = 1;
505                 error = xfs_bmapi_read(dp, (xfs_fileoff_t)lblkno,
506                                        blkcnt, &map, &nmap,
507                                        XFS_BMAPI_ATTRFORK);
508                 if (error)
509                         return error;
510                 ASSERT(nmap == 1);
511                 ASSERT((map.br_startblock != DELAYSTARTBLOCK) &&
512                        (map.br_startblock != HOLESTARTBLOCK));
513
514                 dblkno = XFS_FSB_TO_DADDR(mp, map.br_startblock),
515                 dblkcnt = XFS_FSB_TO_BB(mp, map.br_blockcount);
516
517                 error = xfs_buf_get(mp->m_ddev_targp, dblkno, dblkcnt, &bp);
518                 if (error)
519                         return error;
520                 bp->b_ops = &xfs_attr3_rmt_buf_ops;
521
522                 xfs_attr_rmtval_copyin(mp, bp, args->dp->i_ino, &offset,
523                                        &valuelen, &src);
524
525                 error = xfs_bwrite(bp); /* GROT: NOTE: synchronous write */
526                 xfs_buf_relse(bp);
527                 if (error)
528                         return error;
529
530
531                 /* roll attribute extent map forwards */
532                 lblkno += map.br_blockcount;
533                 blkcnt -= map.br_blockcount;
534         }
535         ASSERT(valuelen == 0);
536         return 0;
537 }
538
539 /* Mark stale any incore buffers for the remote value. */
540 int
541 xfs_attr_rmtval_stale(
542         struct xfs_inode        *ip,
543         struct xfs_bmbt_irec    *map,
544         xfs_buf_flags_t         incore_flags)
545 {
546         struct xfs_mount        *mp = ip->i_mount;
547         struct xfs_buf          *bp;
548
549         ASSERT(xfs_isilocked(ip, XFS_ILOCK_EXCL));
550
551         if (XFS_IS_CORRUPT(mp, map->br_startblock == DELAYSTARTBLOCK) ||
552             XFS_IS_CORRUPT(mp, map->br_startblock == HOLESTARTBLOCK))
553                 return -EFSCORRUPTED;
554
555         bp = xfs_buf_incore(mp->m_ddev_targp,
556                         XFS_FSB_TO_DADDR(mp, map->br_startblock),
557                         XFS_FSB_TO_BB(mp, map->br_blockcount), incore_flags);
558         if (bp) {
559                 xfs_buf_stale(bp);
560                 xfs_buf_relse(bp);
561         }
562
563         return 0;
564 }
565
566 /*
567  * Write the value associated with an attribute into the out-of-line buffer
568  * that we have defined for it.
569  */
570 int
571 xfs_attr_rmtval_set(
572         struct xfs_da_args      *args)
573 {
574         struct xfs_inode        *dp = args->dp;
575         struct xfs_bmbt_irec    map;
576         xfs_dablk_t             lblkno;
577         int                     blkcnt;
578         int                     nmap;
579         int                     error;
580
581         trace_xfs_attr_rmtval_set(args);
582
583         error = xfs_attr_rmt_find_hole(args);
584         if (error)
585                 return error;
586
587         blkcnt = args->rmtblkcnt;
588         lblkno = (xfs_dablk_t)args->rmtblkno;
589         /*
590          * Roll through the "value", allocating blocks on disk as required.
591          */
592         while (blkcnt > 0) {
593                 /*
594                  * Allocate a single extent, up to the size of the value.
595                  *
596                  * Note that we have to consider this a data allocation as we
597                  * write the remote attribute without logging the contents.
598                  * Hence we must ensure that we aren't using blocks that are on
599                  * the busy list so that we don't overwrite blocks which have
600                  * recently been freed but their transactions are not yet
601                  * committed to disk. If we overwrite the contents of a busy
602                  * extent and then crash then the block may not contain the
603                  * correct metadata after log recovery occurs.
604                  */
605                 nmap = 1;
606                 error = xfs_bmapi_write(args->trans, dp, (xfs_fileoff_t)lblkno,
607                                   blkcnt, XFS_BMAPI_ATTRFORK, args->total, &map,
608                                   &nmap);
609                 if (error)
610                         return error;
611                 error = xfs_defer_finish(&args->trans);
612                 if (error)
613                         return error;
614
615                 ASSERT(nmap == 1);
616                 ASSERT((map.br_startblock != DELAYSTARTBLOCK) &&
617                        (map.br_startblock != HOLESTARTBLOCK));
618                 lblkno += map.br_blockcount;
619                 blkcnt -= map.br_blockcount;
620
621                 /*
622                  * Start the next trans in the chain.
623                  */
624                 error = xfs_trans_roll_inode(&args->trans, dp);
625                 if (error)
626                         return error;
627         }
628
629         return xfs_attr_rmtval_set_value(args);
630 }
631
632 /*
633  * Remove the value associated with an attribute by deleting the
634  * out-of-line buffer that it is stored on.
635  */
636 int
637 xfs_attr_rmtval_invalidate(
638         struct xfs_da_args      *args)
639 {
640         xfs_dablk_t             lblkno;
641         int                     blkcnt;
642         int                     error;
643
644         /*
645          * Roll through the "value", invalidating the attribute value's blocks.
646          */
647         lblkno = args->rmtblkno;
648         blkcnt = args->rmtblkcnt;
649         while (blkcnt > 0) {
650                 struct xfs_bmbt_irec    map;
651                 int                     nmap;
652
653                 /*
654                  * Try to remember where we decided to put the value.
655                  */
656                 nmap = 1;
657                 error = xfs_bmapi_read(args->dp, (xfs_fileoff_t)lblkno,
658                                        blkcnt, &map, &nmap, XFS_BMAPI_ATTRFORK);
659                 if (error)
660                         return error;
661                 if (XFS_IS_CORRUPT(args->dp->i_mount, nmap != 1))
662                         return -EFSCORRUPTED;
663                 error = xfs_attr_rmtval_stale(args->dp, &map, XBF_TRYLOCK);
664                 if (error)
665                         return error;
666
667                 lblkno += map.br_blockcount;
668                 blkcnt -= map.br_blockcount;
669         }
670         return 0;
671 }
672
673 /*
674  * Remove the value associated with an attribute by deleting the
675  * out-of-line buffer that it is stored on.
676  */
677 int
678 xfs_attr_rmtval_remove(
679         struct xfs_da_args      *args)
680 {
681         xfs_dablk_t             lblkno;
682         int                     blkcnt;
683         int                     error = 0;
684         int                     done = 0;
685
686         trace_xfs_attr_rmtval_remove(args);
687
688         error = xfs_attr_rmtval_invalidate(args);
689         if (error)
690                 return error;
691         /*
692          * Keep de-allocating extents until the remote-value region is gone.
693          */
694         lblkno = args->rmtblkno;
695         blkcnt = args->rmtblkcnt;
696         while (!done) {
697                 error = xfs_bunmapi(args->trans, args->dp, lblkno, blkcnt,
698                                     XFS_BMAPI_ATTRFORK, 1, &done);
699                 if (error)
700                         return error;
701                 error = xfs_defer_finish(&args->trans);
702                 if (error)
703                         return error;
704
705                 /*
706                  * Close out trans and start the next one in the chain.
707                  */
708                 error = xfs_trans_roll_inode(&args->trans, args->dp);
709                 if (error)
710                         return error;
711         }
712         return 0;
713 }