test: skip wasm-allocation tests for pointer compression builds · nodejs/node@cccb15d · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 631
• Pull requests 747
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit cccb15d

Browse filesBrowse the repository at this point in the historyBrowse files

joyeecheung

authored and

aduh95

committed

test: skip wasm-allocation tests for pointer compression builds

V8 isolate group initialization forces allocation of the virtual memory cage with pointer compression builds and simply would not work when there is a smaller hard limit on the virtual memory. PR-URL: #58171 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>

1 parent 3f954ac commit cccb15d
Copy full SHA for cccb15d

2 files changed

+5-1Lines changed: 5 additions & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/wasm-allocation
  • wasm-allocation.status
• tools
  • test.py

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/wasm-allocation/wasm-allocation.status‎

Copy file name to clipboardExpand all lines: test/wasm-allocation/wasm-allocation.status

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -6,5 +6,5 @@ prefix wasm-allocation
6	6
7	7	[true] # This section applies to all platforms
8	8
9		- [$system!=linux || $asan==on]
	9	+ [$system!=linux || $asan==on || $pointer_compression==on]
10	10	test-wasm-allocation: SKIP

Collapse file

‎tools/test.py‎

Copy file name to clipboardExpand all lines: tools/test.py

+4Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -1619,6 +1619,9 @@ def get_asan_state(vm, context):
1619	1619	asan = Execute([vm, '-p', 'process.config.variables.asan'], context).stdout.strip()
1620	1620	return "on" if asan == "1" else "off"
1621	1621
	1622	+ def get_pointer_compression_state(vm, context):
	1623	+ pointer_compression = Execute([vm, '-p', 'process.config.variables.v8_enable_pointer_compression'], context).stdout.strip()
	1624	+ return "on" if pointer_compression == "1" else "off"
1622	1625
1623	1626	def Main():
1624	1627	parser = BuildOptions()
@@ -1717,6 +1720,7 @@ def Main():
1717	1720	'arch': vmArch,
1718	1721	'type': get_env_type(vm, options.type, context),
1719	1722	'asan': get_asan_state(vm, context),
	1723	+ 'pointer_compression': get_pointer_compression_state(vm, context),
1720	1724	}
1721	1725	test_list = root.ListTests([], path, context, arch, mode)
1722	1726	unclassified_tests += test_list

0 commit comments

Comments

0 (0)