48926e8344
This reverts commit ea818f0733
.
Reason for revert: Test failure in Linux64 UBSan https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20UBSan/15251/overview
Original change's description:
> cppgc: Fix testing APIs that enable garbage collection
>
> The APIs require that the CppHeap is moved into a permanently detached
> state that moves the heap out of a no-gc scope.
>
> Bug: chromium:1056170
> Change-Id: I1fc08451b3fdfaa4cfe58e6a1ddbe5dbed7efe5c
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2718146
> Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
> Reviewed-by: Omer Katz <omerkatz@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#73025}
Bug: chromium:1056170
Change-Id: Id00cb18274cbe7d255e7e95bd9e8e4dbc4b0c6e7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2718658
Auto-Submit: Zhi An Ng <zhin@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Francis McCabe <fgm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#73029}
154 lines
5.9 KiB
C++
154 lines
5.9 KiB
C++
// Copyright 2020 the V8 project authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
#include "include/cppgc/allocation.h"
|
|
#include "include/cppgc/garbage-collected.h"
|
|
#include "include/cppgc/persistent.h"
|
|
#include "include/cppgc/platform.h"
|
|
#include "include/v8-cppgc.h"
|
|
#include "include/v8.h"
|
|
#include "src/api/api-inl.h"
|
|
#include "src/heap/cppgc-js/cpp-heap.h"
|
|
#include "src/heap/cppgc/sweeper.h"
|
|
#include "src/objects/objects-inl.h"
|
|
#include "test/unittests/heap/heap-utils.h"
|
|
#include "test/unittests/heap/unified-heap-utils.h"
|
|
|
|
namespace v8 {
|
|
namespace internal {
|
|
|
|
namespace {
|
|
|
|
class Wrappable final : public cppgc::GarbageCollected<Wrappable> {
|
|
public:
|
|
static size_t destructor_callcount;
|
|
|
|
~Wrappable() { destructor_callcount++; }
|
|
|
|
void Trace(cppgc::Visitor* visitor) const { visitor->Trace(wrapper_); }
|
|
|
|
void SetWrapper(v8::Isolate* isolate, v8::Local<v8::Object> wrapper) {
|
|
wrapper_.Reset(isolate, wrapper);
|
|
}
|
|
|
|
TracedReference<v8::Object>& wrapper() { return wrapper_; }
|
|
|
|
private:
|
|
TracedReference<v8::Object> wrapper_;
|
|
};
|
|
|
|
size_t Wrappable::destructor_callcount = 0;
|
|
|
|
using UnifiedHeapDetachedTest = TestWithHeapInternals;
|
|
|
|
} // namespace
|
|
|
|
TEST_F(UnifiedHeapTest, OnlyGC) { CollectGarbageWithEmbedderStack(); }
|
|
|
|
TEST_F(UnifiedHeapTest, FindingV8ToBlinkReference) {
|
|
v8::HandleScope scope(v8_isolate());
|
|
v8::Local<v8::Context> context = v8::Context::New(v8_isolate());
|
|
v8::Context::Scope context_scope(context);
|
|
uint16_t wrappable_type = WrapperHelper::kTracedEmbedderId;
|
|
v8::Local<v8::Object> api_object = WrapperHelper::CreateWrapper(
|
|
context, &wrappable_type,
|
|
cppgc::MakeGarbageCollected<Wrappable>(allocation_handle()));
|
|
Wrappable::destructor_callcount = 0;
|
|
EXPECT_FALSE(api_object.IsEmpty());
|
|
EXPECT_EQ(0u, Wrappable::destructor_callcount);
|
|
CollectGarbageWithoutEmbedderStack(cppgc::Heap::SweepingType::kAtomic);
|
|
EXPECT_EQ(0u, Wrappable::destructor_callcount);
|
|
WrapperHelper::ResetWrappableConnection(api_object);
|
|
CollectGarbageWithoutEmbedderStack(cppgc::Heap::SweepingType::kAtomic);
|
|
EXPECT_EQ(1u, Wrappable::destructor_callcount);
|
|
}
|
|
|
|
TEST_F(UnifiedHeapTest, WriteBarrierV8ToCppReference) {
|
|
v8::HandleScope scope(v8_isolate());
|
|
v8::Local<v8::Context> context = v8::Context::New(v8_isolate());
|
|
v8::Context::Scope context_scope(context);
|
|
void* wrappable = cppgc::MakeGarbageCollected<Wrappable>(allocation_handle());
|
|
v8::Local<v8::Object> api_object =
|
|
WrapperHelper::CreateWrapper(context, nullptr, nullptr);
|
|
Wrappable::destructor_callcount = 0;
|
|
WrapperHelper::ResetWrappableConnection(api_object);
|
|
SimulateIncrementalMarking();
|
|
{
|
|
// The following snippet shows the embedder code for implementing a GC-safe
|
|
// setter for JS to C++ references.
|
|
WrapperHelper::SetWrappableConnection(api_object, wrappable, wrappable);
|
|
JSHeapConsistency::WriteBarrierParams params;
|
|
auto barrier_type = JSHeapConsistency::GetWriteBarrierType(
|
|
api_object, 1, wrappable, params,
|
|
[this]() -> cppgc::HeapHandle& { return cpp_heap().GetHeapHandle(); });
|
|
EXPECT_EQ(JSHeapConsistency::WriteBarrierType::kMarking, barrier_type);
|
|
JSHeapConsistency::DijkstraMarkingBarrier(
|
|
params, cpp_heap().GetHeapHandle(), wrappable);
|
|
}
|
|
CollectGarbageWithoutEmbedderStack(cppgc::Heap::SweepingType::kAtomic);
|
|
EXPECT_EQ(0u, Wrappable::destructor_callcount);
|
|
}
|
|
|
|
TEST_F(UnifiedHeapTest, WriteBarrierCppToV8Reference) {
|
|
v8::HandleScope scope(v8_isolate());
|
|
v8::Local<v8::Context> context = v8::Context::New(v8_isolate());
|
|
v8::Context::Scope context_scope(context);
|
|
cppgc::Persistent<Wrappable> wrappable =
|
|
cppgc::MakeGarbageCollected<Wrappable>(allocation_handle());
|
|
Wrappable::destructor_callcount = 0;
|
|
SimulateIncrementalMarking();
|
|
// Pick a sentinel to compare against.
|
|
void* kMagicAddress = &Wrappable::destructor_callcount;
|
|
{
|
|
// The following snippet shows the embedder code for implementing a GC-safe
|
|
// setter for C++ to JS references.
|
|
v8::HandleScope nested_scope(v8_isolate());
|
|
v8::Local<v8::Object> api_object =
|
|
WrapperHelper::CreateWrapper(context, nullptr, nullptr);
|
|
// Setting only one field to avoid treating this as wrappable backref, see
|
|
// `LocalEmbedderHeapTracer::ExtractWrapperInfo`.
|
|
api_object->SetAlignedPointerInInternalField(1, kMagicAddress);
|
|
wrappable->SetWrapper(v8_isolate(), api_object);
|
|
JSHeapConsistency::WriteBarrierParams params;
|
|
auto barrier_type = JSHeapConsistency::GetWriteBarrierType(
|
|
wrappable->wrapper(), params,
|
|
[this]() -> cppgc::HeapHandle& { return cpp_heap().GetHeapHandle(); });
|
|
EXPECT_EQ(JSHeapConsistency::WriteBarrierType::kMarking, barrier_type);
|
|
JSHeapConsistency::DijkstraMarkingBarrier(
|
|
params, cpp_heap().GetHeapHandle(), wrappable->wrapper());
|
|
}
|
|
CollectGarbageWithoutEmbedderStack(cppgc::Heap::SweepingType::kAtomic);
|
|
EXPECT_EQ(0u, Wrappable::destructor_callcount);
|
|
EXPECT_EQ(kMagicAddress,
|
|
wrappable->wrapper()->GetAlignedPointerFromInternalField(1));
|
|
}
|
|
|
|
TEST_F(UnifiedHeapDetachedTest, AllocationBeforeConfigureHeap) {
|
|
auto heap = v8::CppHeap::Create(
|
|
V8::GetCurrentPlatform(),
|
|
CppHeapCreateParams{{}, WrapperHelper::DefaultWrapperDescriptor()});
|
|
auto* object =
|
|
cppgc::MakeGarbageCollected<Wrappable>(heap->GetAllocationHandle());
|
|
cppgc::WeakPersistent<Wrappable> weak_holder{object};
|
|
|
|
auto& js_heap = *isolate()->heap();
|
|
js_heap.AttachCppHeap(heap.get());
|
|
auto& cpp_heap = *CppHeap::From(isolate()->heap()->cpp_heap());
|
|
{
|
|
CollectGarbage(OLD_SPACE);
|
|
cpp_heap.AsBase().sweeper().FinishIfRunning();
|
|
EXPECT_TRUE(weak_holder);
|
|
}
|
|
{
|
|
js_heap.SetEmbedderStackStateForNextFinalization(
|
|
EmbedderHeapTracer::EmbedderStackState::kNoHeapPointers);
|
|
CollectGarbage(OLD_SPACE);
|
|
cpp_heap.AsBase().sweeper().FinishIfRunning();
|
|
EXPECT_FALSE(weak_holder);
|
|
}
|
|
}
|
|
|
|
} // namespace internal
|
|
} // namespace v8
|