import static com.google.common.base.Verify.verify;
import static java.util.Objects.requireNonNull;
-import com.google.common.base.MoreObjects;
-import com.google.common.base.MoreObjects.ToStringHelper;
-import java.util.Optional;
-import java.util.concurrent.atomic.AtomicReferenceFieldUpdater;
+import java.lang.invoke.MethodHandles;
+import java.lang.invoke.VarHandle;
import org.eclipse.jdt.annotation.NonNull;
import org.eclipse.jdt.annotation.Nullable;
import org.opendaylight.yangtools.yang.binding.DataObject;
-import org.opendaylight.yangtools.yang.binding.Identifier;
+import org.opendaylight.yangtools.yang.data.api.schema.DistinctNodeContainer;
import org.opendaylight.yangtools.yang.data.api.schema.MapEntryNode;
import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
-import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNodeContainer;
/**
* A base class for {@link DataObject}s backed by {@link DataObjectCodecContext}. While this class is public, it not
// An object representing a null value in a member field.
private static final @NonNull Object NULL_VALUE = new Object();
+ private static final VarHandle CACHED_HASH_CODE;
+
+ static {
+ try {
+ CACHED_HASH_CODE = MethodHandles.lookup().findVarHandle(CodecDataObject.class, "cachedHashcode",
+ Integer.class);
+ } catch (NoSuchFieldException | IllegalAccessException e) {
+ throw new ExceptionInInitializerError(e);
+ }
+ }
+
private final @NonNull DataObjectCodecContext<T, ?> context;
@SuppressWarnings("rawtypes")
- private final @NonNull NormalizedNodeContainer data;
+ private final @NonNull DistinctNodeContainer data;
- private volatile Integer cachedHashcode = null;
+ // Accessed via a VarHandle
+ @SuppressWarnings("unused")
+ // FIXME: consider using a primitive int-based cache (with 0 being uninit)
+ private volatile Integer cachedHashcode;
- protected CodecDataObject(final DataObjectCodecContext<T, ?> context, final NormalizedNodeContainer<?, ?, ?> data) {
+ protected CodecDataObject(final DataObjectCodecContext<T, ?> context, final DistinctNodeContainer<?, ?> data) {
this.data = requireNonNull(data, "Data must not be null");
this.context = requireNonNull(context, "Context must not be null");
}
@Override
public final int hashCode() {
- final Integer cached = cachedHashcode;
- if (cached != null) {
- return cached;
- }
-
- final int result = codecAugmentedHashCode();
- cachedHashcode = result;
- return result;
+ final Integer cached = (Integer) CACHED_HASH_CODE.getAcquire(this);
+ return cached != null ? cached : loadHashCode();
}
@Override
public final boolean equals(final Object obj) {
- if (obj == this) {
- return true;
- }
- final Class<? extends DataObject> iface = implementedInterface();
- if (!iface.isInstance(obj)) {
- return false;
- }
- @SuppressWarnings("unchecked")
- final T other = (T) iface.cast(obj);
- if (other instanceof CodecDataObject) {
- return data.equals(((CodecDataObject<?>) obj).data);
- }
- return codecAugmentedEquals(other);
+ // Indirection to keep checkstyle happy
+ return codecEquals(obj);
}
@Override
- public final String toString() {
- return codecAugmentedFillToString(MoreObjects.toStringHelper(implementedInterface()).omitNullValues())
- .toString();
- }
+ public abstract String toString();
- // TODO: consider switching to VarHandles for Java 9+, as that would disconnect us from the need to use a volatile
- // field and use acquire/release mechanics -- see http://gee.cs.oswego.edu/dl/html/j9mm.html for details.
- protected final Object codecMember(final AtomicReferenceFieldUpdater<CodecDataObject<?>, Object> updater,
- final String localName) {
- final Object cached = updater.get(this);
- return cached != null ? unmaskNull(cached) : loadMember(updater, context.getLeafChild(localName));
+ protected final Object codecMember(final VarHandle handle, final String localName) {
+ final Object cached = handle.getAcquire(this);
+ return cached != null ? unmaskNull(cached) : loadMember(handle, context.getLeafChild(localName));
}
- protected final Object codecMember(final AtomicReferenceFieldUpdater<CodecDataObject<?>, Object> updater,
- final Class<? extends DataObject> bindingClass) {
- final Object cached = updater.get(this);
- return cached != null ? unmaskNull(cached) : loadMember(updater, context.streamChild(bindingClass));
+ protected final Object codecMember(final VarHandle handle, final Class<? extends DataObject> bindingClass) {
+ final Object cached = handle.getAcquire(this);
+ return cached != null ? unmaskNull(cached) : loadMember(handle, context.streamChild(bindingClass));
}
- protected final Object codecMember(final AtomicReferenceFieldUpdater<CodecDataObject<?>, Object> updater,
- final NodeContextSupplier supplier) {
- final Object cached = updater.get(this);
- return cached != null ? unmaskNull(cached) : loadMember(updater, supplier.get());
+ protected final Object codecMember(final VarHandle handle, final NodeContextSupplier supplier) {
+ final Object cached = handle.getAcquire(this);
+ return cached != null ? unmaskNull(cached) : loadMember(handle, supplier.get());
}
- protected final Object codecKey(final AtomicReferenceFieldUpdater<CodecDataObject<?>, Object> updater) {
- final Object cached = updater.get(this);
- return cached != null ? cached : loadKey(updater);
+ protected final @NonNull Object codecKey(final VarHandle handle) {
+ final Object cached = handle.getAcquire(this);
+ return cached != null ? cached : loadKey(handle);
}
protected abstract int codecHashCode();
- protected abstract boolean codecEquals(T other);
-
- protected abstract ToStringHelper codecFillToString(ToStringHelper helper);
+ protected abstract boolean codecEquals(Object obj);
final @NonNull DataObjectCodecContext<T, ?> codecContext() {
return context;
}
@SuppressWarnings("rawtypes")
- final @NonNull NormalizedNodeContainer codecData() {
+ final @NonNull DistinctNodeContainer codecData() {
return data;
}
- // Non-final to allow specialization in AugmentableCodecDataObject
- int codecAugmentedHashCode() {
- return codecHashCode();
- }
-
- // Non-final to allow specialization in AugmentableCodecDataObject
- boolean codecAugmentedEquals(final T other) {
- return codecEquals(other);
- }
-
- // Non-final to allow specialization in AugmentableCodecDataObject
- ToStringHelper codecAugmentedFillToString(final ToStringHelper helper) {
- return codecFillToString(helper);
- }
-
- // Helpers split out of codecMember to aid its inlining
- private Object loadMember(final AtomicReferenceFieldUpdater<CodecDataObject<?>, Object> updater,
- final NodeCodecContext childCtx) {
+ // Helper split out of codecMember to aid its inlining
+ private Object loadMember(final VarHandle handle, final NodeCodecContext childCtx) {
@SuppressWarnings("unchecked")
- final Optional<NormalizedNode<?, ?>> child = data.getChild(childCtx.getDomPathArgument());
+ final NormalizedNode child = data.childByArg(childCtx.getDomPathArgument());
// We do not want to use Optional.map() here because we do not want to invoke defaultObject() when we have
// normal value because defaultObject() may end up throwing an exception intentionally.
- final Object obj = child.isPresent() ? childCtx.deserializeObject(child.get()) : childCtx.defaultObject();
- return updater.compareAndSet(this, null, maskNull(obj)) ? obj : unmaskNull(updater.get(this));
+ final Object obj = child != null ? childCtx.deserializeObject(child) : childCtx.defaultObject();
+ final Object witness = handle.compareAndExchangeRelease(this, null, maskNull(obj));
+ return witness == null ? obj : unmaskNull(witness);
}
- // Helpers split out of codecMember to aid its inlining
- private Object loadKey(final AtomicReferenceFieldUpdater<CodecDataObject<?>, Object> updater) {
+ // Helper split out of codecKey to aid its inlining
+ private @NonNull Object loadKey(final VarHandle handle) {
verify(data instanceof MapEntryNode, "Unsupported value %s", data);
verify(context instanceof KeyedListNodeCodecContext, "Unexpected context %s", context);
- final Identifier<?> key = ((KeyedListNodeCodecContext<?>) context).deserialize(
- ((MapEntryNode) data).getIdentifier());
+ final Object obj = ((KeyedListNodeCodecContext<?, ?>) context)
+ .deserialize(((MapEntryNode) data).getIdentifier());
// key is known to be non-null, no need to mask it
- return updater.compareAndSet(this, null, key) ? key : updater.get(this);
+ final Object witness = handle.compareAndExchangeRelease(this, null, obj);
+ return witness == null ? obj : witness;
+ }
+
+ // Helper split out of hashCode() to aid its inlining
+ private int loadHashCode() {
+ final int result = codecHashCode();
+ final Object witness = CACHED_HASH_CODE.compareAndExchangeRelease(this, null, result);
+ return witness == null ? result : (Integer) witness;
}
private static @NonNull Object maskNull(final @Nullable Object unmasked) {
Code Review / mdsal.git / blobdiff