[src/trunk]: src/sys/ufs/ffs Need wapbl transaction around ffs_blkfree_cg. F...

[riastradh <riastradh%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/f91959f6fa49
branches:  trunk
changeset: 339816:f91959f6fa49
user:      riastradh <riastradh%NetBSD.org@localhost>
date:      Wed Aug 12 14:52:35 2015 +0000

description:
Need wapbl transaction around ffs_blkfree_cg.  Fixes wapbl+discard.

diffstat:

 sys/ufs/ffs/ffs_alloc.c |  13 +++++++++++--
 1 files changed, 11 insertions(+), 2 deletions(-)

diffs (40 lines):

diff -r 8f9640ff7a33 -r f91959f6fa49 sys/ufs/ffs/ffs_alloc.c
--- a/sys/ufs/ffs/ffs_alloc.c   Wed Aug 12 07:53:56 2015 +0000
+++ b/sys/ufs/ffs/ffs_alloc.c   Wed Aug 12 14:52:35 2015 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: ffs_alloc.c,v 1.150 2015/08/08 08:18:52 mlelstv Exp $  */
+/*     $NetBSD: ffs_alloc.c,v 1.151 2015/08/12 14:52:35 riastradh Exp $        */
 
 /*-
  * Copyright (c) 2008, 2009 The NetBSD Foundation, Inc.
@@ -70,7 +70,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ffs_alloc.c,v 1.150 2015/08/08 08:18:52 mlelstv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ffs_alloc.c,v 1.151 2015/08/12 14:52:35 riastradh Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_ffs.h"
@@ -1553,12 +1553,21 @@
 static void
 ffs_blkfree_td(struct fs *fs, struct discardopdata *td)
 {
+       struct mount *mp = spec_node_getmountedfs(td->devvp);
        long todo;
+       int error;
 
        while (td->size) {
                todo = min(td->size,
                  ffs_lfragtosize(fs, (fs->fs_frag - ffs_fragnum(fs, td->bno))));
+               error = UFS_WAPBL_BEGIN(mp);
+               if (error) {
+                       printf("ffs: failed to begin wapbl transaction"
+                           " for discard: %d\n", error);
+                       break;
+               }
                ffs_blkfree_cg(fs, td->devvp, td->bno, todo);
+               UFS_WAPBL_END(mp);
                td->bno += ffs_numfrags(fs, todo);
                td->size -= todo;
        }