Aren
/
NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed issues with uvm_physseg_plug test case. 

There is a condition check which failed for VM_PHYSSEG_MAX == 2 (not for
1 or 3 and above), in case of 2, pgs == slab + npages1 + npages3, so we
need to change ">" check to ">=" check.

Reviewed by <cherry>
This commit is contained in:
fox 2019-01-16 13:35:51 +00:00
parent 497a8f2bfd
commit ff43743a55
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tests/sys/uvm/t_uvm_physseg.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: t_uvm_physseg.c,v 1.5 2019/01/16 13:21:02 fox Exp $ */ /* $NetBSD: t_uvm_physseg.c,v 1.6 2019/01/16 13:35:51 fox Exp $ */
/*- /*-
* Copyright (c) 2015, 2016 The NetBSD Foundation, Inc. * Copyright (c) 2015, 2016 The NetBSD Foundation, Inc.
@ -31,7 +31,7 @@
*/ */
#include <sys/cdefs.h> #include <sys/cdefs.h>
__RCSID("$NetBSD: t_uvm_physseg.c,v 1.5 2019/01/16 13:21:02 fox Exp $"); __RCSID("$NetBSD: t_uvm_physseg.c,v 1.6 2019/01/16 13:35:51 fox Exp $");
/* /*
* If this line is commented out tests related to uvm_physseg_get_pmseg() * If this line is commented out tests related to uvm_physseg_get_pmseg()
@ -522,7 +522,7 @@ ATF_TC_BODY(uvm_physseg_plug, tc)
ATF_REQUIRE_EQ(uvm_physseg_plug(VALID_START_PFN_4, npages4, &upm4), true); ATF_REQUIRE_EQ(uvm_physseg_plug(VALID_START_PFN_4, npages4, &upm4), true);
/* The hot plug slab should have nothing to do with the original slab */ /* The hot plug slab should have nothing to do with the original slab */
pgs = uvm_physseg_get_pg(upm4, 0); pgs = uvm_physseg_get_pg(upm4, 0);
ATF_REQUIRE(pgs < slab || pgs > (slab + npages1 ATF_REQUIRE(pgs < slab || pgs >= (slab + npages1
#if VM_PHYSSEG_MAX > 2 #if VM_PHYSSEG_MAX > 2
+ npages2 + npages2
#endif #endif