[src/trunk]: src/sbin/fsck_lfs PR/51422: Jose Luis Rodriguez Garcia: complete...

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

details:   https://anonhg.NetBSD.org/src/rev/cf74b68af2aa
branches:  trunk
changeset: 347210:cf74b68af2aa
user:      christos <christos%NetBSD.org@localhost>
date:      Thu Aug 18 08:08:02 2016 +0000

description:
PR/51422: Jose Luis Rodriguez Garcia: completely remove B_NEEDCOMMIT

diffstat:

 sbin/fsck_lfs/bufcache.c |  4 +---
 sbin/fsck_lfs/bufcache.h |  3 +--
 2 files changed, 2 insertions(+), 5 deletions(-)

diffs (42 lines):

diff -r 0f8d43d8219a -r cf74b68af2aa sbin/fsck_lfs/bufcache.c
--- a/sbin/fsck_lfs/bufcache.c  Thu Aug 18 08:06:53 2016 +0000
+++ b/sbin/fsck_lfs/bufcache.c  Thu Aug 18 08:08:02 2016 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: bufcache.c,v 1.17 2016/08/18 08:04:28 christos Exp $ */
+/* $NetBSD: bufcache.c,v 1.18 2016/08/18 08:08:02 christos Exp $ */
 /*-
  * Copyright (c) 2003 The NetBSD Foundation, Inc.
  * All rights reserved.
@@ -230,7 +230,6 @@
         * the buffer, its contents are invalid; but shrinking is okay.
         */
        if ((bp = incore(vp, lbn)) != NULL) {
-               assert(!(bp->b_flags & B_NEEDCOMMIT));
                assert(!(bp->b_flags & B_BUSY));
                bp->b_flags |= B_BUSY;
                bremfree(bp);
@@ -308,7 +307,6 @@
 {
        int age;
 
-       assert(!(bp->b_flags & B_NEEDCOMMIT));
        assert(bp->b_flags & B_BUSY);
 
        bp->b_flags |= set;
diff -r 0f8d43d8219a -r cf74b68af2aa sbin/fsck_lfs/bufcache.h
--- a/sbin/fsck_lfs/bufcache.h  Thu Aug 18 08:06:53 2016 +0000
+++ b/sbin/fsck_lfs/bufcache.h  Thu Aug 18 08:08:02 2016 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: bufcache.h,v 1.12 2015/03/29 19:35:58 chopps Exp $     */
+/*     $NetBSD: bufcache.h,v 1.13 2016/08/18 08:08:02 christos Exp $   */
 
 /*-
  * Copyright (c) 1999, 2000 The NetBSD Foundation, Inc.
@@ -92,7 +92,6 @@
  * These flags are kept in b_flags.
  */
 #define        B_AGE           0x00000001      /* Move to age queue when I/O done. */
-#define        B_NEEDCOMMIT    0x00000002      /* Needs committing to stable storage */
 #define        B_BUSY          0x00000010      /* I/O in progress. */
 #define        B_DELWRI        0x00000080      /* Delay I/O until buffer reused. */
 #define        B_DONE          0x00000200      /* I/O completed. */