This is an archive of the discontinued LLVM Phabricator instance.

Increase the treshold in unit test to accomodate for qurantine size increase.
ClosedPublic

Authored by alekseyshl on Dec 21 2016, 1:42 PM.

Reviewers

Commits

Build Status

Buildable 2350
Build 2350: arc lint + arc unit

alekseyshl retitled this revision from to Increase the treshold in unit test to accomodate for qurantine size increase..

alekseyshl updated this object.

alekseyshl set the repository for this revision to rL LLVM.

LGTM
Maybe add a comment saying that we don't really understand why it is using that much memory

This revision is now accepted and ready to land.Dec 21 2016, 1:48 PM

Done

This revision was automatically updated to reflect the committed changes.

Path

Size

lib/

asan/

tests/

5 lines

	Show First 20 Lines • Show All 170 Lines • ▼ Show 20 Lines
	// destroyed.			// destroyed.
	TEST(AddressSanitizer, ThreadedQuarantineTest) {			TEST(AddressSanitizer, ThreadedQuarantineTest) {
	const int n_threads = 3000;			const int n_threads = 3000;
	size_t mmaped1 = __sanitizer_get_heap_size();			size_t mmaped1 = __sanitizer_get_heap_size();
	for (int i = 0; i < n_threads; i++) {			for (int i = 0; i < n_threads; i++) {
	pthread_t t;			pthread_t t;
	PTHREAD_CREATE(&t, NULL, ThreadedQuarantineTestWorker, 0);			PTHREAD_CREATE(&t, NULL, ThreadedQuarantineTestWorker, 0);
	PTHREAD_JOIN(t, 0);			PTHREAD_JOIN(t, 0);
	size_t mmaped2 = __sanitizer_get_heap_size();			size_t mmaped2 = __sanitizer_get_heap_size();i
	EXPECT_LT(mmaped2 - mmaped1, 320U * (1 << 20));			// TODO: Figure out why this much memory is required.
				EXPECT_LT(mmaped2 - mmaped1, 352U * (1 << 20));
	}			}
	}			}

	void ThreadedOneSizeMallocStress(void unused) {			void ThreadedOneSizeMallocStress(void unused) {
	(void)unused;			(void)unused;
	BufferedStackTrace stack;			BufferedStackTrace stack;
	stack.trace_buffer[0] = 0x890;			stack.trace_buffer[0] = 0x890;
	stack.size = 1;			stack.size = 1;
	▲ Show 20 Lines • Show All 76 Lines • Show Last 20 Lines