From a74f6896ea492a746f23fe0147cf152c9cf45109 Mon Sep 17 00:00:00 2001 From: Amber Lin Date: Tue, 29 Aug 2017 15:10:05 -0400 Subject: [PATCH] Revert "Set guard page as disabled as default" This reverts commit 65d680c035b380ebe6374fdab3ce6e6259a5c43d. Change-Id: I09b7e7915ec4759cab57d5863089a2c4a44dfacd Signed-off-by: Amber Lin --- src/fmm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/fmm.c b/src/fmm.c index 063944b39c..5756df4e62 100644 --- a/src/fmm.c +++ b/src/fmm.c @@ -1432,7 +1432,7 @@ HSAKMT_STATUS fmm_init_process_apertures(unsigned int NumNodes) uint32_t num_of_nodes; HSAKMT_STATUS ret = HSAKMT_STATUS_SUCCESS; char *disableCache, *pagedUserptr, *checkUserptr, *guardPagesStr; - unsigned int guardPages = 0; + unsigned int guardPages = 1; struct pci_access *pacc; /* If HSA_DISABLE_CACHE is set to a non-0 value, disable caching */ @@ -1452,10 +1452,10 @@ HSAKMT_STATUS fmm_init_process_apertures(unsigned int NumNodes) checkUserptr = getenv("HSA_CHECK_USERPTR"); svm.check_userptr = (checkUserptr && strcmp(checkUserptr, "0")); - /* Specify number of guard pages for SVM apertures, default is 0 */ + /* Specify number of guard pages for SVM apertures, default is 1 */ guardPagesStr = getenv("HSA_SVM_GUARD_PAGES"); if (!guardPagesStr || sscanf(guardPagesStr, "%u", &guardPages) != 1) - guardPages = 0; + guardPages = 1; /* Trade off - NumNodes includes GPU nodes + CPU Node. So in * systems with CPU node, slightly more memory is allocated than