Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/sys/ufs/lfs Fill in some XXXs with the exact action describe...
details: https://anonhg.NetBSD.org/src/rev/0e974a75c393
branches: trunk
changeset: 822310:0e974a75c393
user: maya <maya%NetBSD.org@localhost>
date: Mon Mar 13 20:15:50 2017 +0000
description:
Fill in some XXXs with the exact action described in them. match
lfs_valloc behaviour.
diffstat:
sys/ufs/lfs/lfs_alloc.c | 20 ++++++++++----------
1 files changed, 10 insertions(+), 10 deletions(-)
diffs (62 lines):
diff -r 425f0f0df5fb -r 0e974a75c393 sys/ufs/lfs/lfs_alloc.c
--- a/sys/ufs/lfs/lfs_alloc.c Mon Mar 13 14:24:20 2017 +0000
+++ b/sys/ufs/lfs/lfs_alloc.c Mon Mar 13 20:15:50 2017 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: lfs_alloc.c,v 1.134 2017/03/13 14:24:20 riastradh Exp $ */
+/* $NetBSD: lfs_alloc.c,v 1.135 2017/03/13 20:15:50 maya Exp $ */
/*-
* Copyright (c) 1999, 2000, 2001, 2002, 2003, 2007 The NetBSD Foundation, Inc.
@@ -60,7 +60,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: lfs_alloc.c,v 1.134 2017/03/13 14:24:20 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: lfs_alloc.c,v 1.135 2017/03/13 20:15:50 maya Exp $");
#if defined(_KERNEL_OPT)
#include "opt_quota.h"
@@ -318,9 +318,6 @@
*
* Called in the same context as lfs_valloc and therefore shares the
* same locking assumptions.
- *
- * XXX: WHICH MEANS IT OUGHT TO TAKE THE SEGLOCK WHILE FROBBING THIS
- * XXX: STUFF. REALLY.
*/
int
lfs_valloc_fixed(struct lfs *fs, ino_t ino, int vers)
@@ -330,9 +327,12 @@
ino_t headino, thisino, oldnext;
CLEANERINFO *cip;
- /* XXX: check for readonly */
- /* XXX: assert no seglock */
- /* XXX: should take seglock (as noted above) */
+ if (fs->lfs_ronly)
+ return EROFS;
+
+ ASSERT_NO_SEGLOCK(fs);
+
+ lfs_seglock(fs, SEGM_PROT);
/*
* If the ifile is too short to contain this inum, extend it.
@@ -381,7 +381,7 @@
if (nextfree == LFS_UNUSED_INUM) {
/* hit the end -- this inode is not available */
brelse(bp, 0);
- /* XXX release seglock (see above) */
+ lfs_segunlock(fs);
return ENOENT;
}
/* found it; update the next pointer */
@@ -391,7 +391,7 @@
}
/* done */
- /* XXX release seglock (see above) */
+ lfs_segunlock(fs);
return 0;
}
Home |
Main Index |
Thread Index |
Old Index