Hash :
fbff065c
Author :
Date :
2023-05-03T22:19:07
Replace GetResourceFromHashSet with map lookup Function GetResourceFromHashSet had linear time complexity because it was sweeping through the set until a resource with a matching ID was found. This change replaces hash sets with hash maps to get constant time lookup. This solves, among other things, O(N^2) time complexity for rendering scenes containing a large number of surfaces. Function GetResourceFromHashSet was consuming over 50% of all CPU time on the main thread of Chrome's GPU process while running the MotionMark 1.2 Images test. With this change, the benchmark score increases by 70% on an M1 MacBook running a PGO official build of Chrome. Bug: chromium:1435066 Change-Id: I895ac0141a91d324c63adec2c0efb8e030d9675b Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/4505950 Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org> Commit-Queue: Justin Novosad <junov@chromium.org> Reviewed-by: Igor Nazarov <i.nazarov@samsung.com>
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165
//
// Copyright 2014 The ANGLE 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.
//
// DisplayImpl.cpp: Implementation methods of egl::Display
#include "libANGLE/renderer/DisplayImpl.h"
#include "libANGLE/Display.h"
#include "libANGLE/Surface.h"
#include "libANGLE/renderer/DeviceImpl.h"
namespace rx
{
namespace
{
// For back-ends that do not implement EGLDevice.
class MockDevice : public DeviceImpl
{
public:
MockDevice() = default;
egl::Error initialize() override { return egl::NoError(); }
egl::Error getAttribute(const egl::Display *display, EGLint attribute, void **outValue) override
{
UNREACHABLE();
return egl::EglBadAttribute();
}
EGLint getType() override
{
UNREACHABLE();
return EGL_NONE;
}
void generateExtensions(egl::DeviceExtensions *outExtensions) const override
{
*outExtensions = egl::DeviceExtensions();
}
};
} // anonymous namespace
DisplayImpl::DisplayImpl(const egl::DisplayState &state)
: mState(state), mExtensionsInitialized(false), mCapsInitialized(false), mBlobCache(nullptr)
{}
DisplayImpl::~DisplayImpl()
{
ASSERT(mState.surfaceMap.empty());
}
egl::Error DisplayImpl::prepareForCall()
{
return egl::NoError();
}
egl::Error DisplayImpl::releaseThread()
{
return egl::NoError();
}
const egl::DisplayExtensions &DisplayImpl::getExtensions() const
{
if (!mExtensionsInitialized)
{
generateExtensions(&mExtensions);
mExtensionsInitialized = true;
}
return mExtensions;
}
egl::Error DisplayImpl::handleGPUSwitch()
{
return egl::NoError();
}
egl::Error DisplayImpl::forceGPUSwitch(EGLint gpuIDHigh, EGLint gpuIDLow)
{
return egl::NoError();
}
egl::Error DisplayImpl::waitUntilWorkScheduled()
{
return egl::NoError();
}
egl::Error DisplayImpl::validateClientBuffer(const egl::Config *configuration,
EGLenum buftype,
EGLClientBuffer clientBuffer,
const egl::AttributeMap &attribs) const
{
UNREACHABLE();
return egl::EglBadDisplay() << "DisplayImpl::validateClientBuffer unimplemented.";
}
egl::Error DisplayImpl::validateImageClientBuffer(const gl::Context *context,
EGLenum target,
EGLClientBuffer clientBuffer,
const egl::AttributeMap &attribs) const
{
UNREACHABLE();
return egl::EglBadDisplay() << "DisplayImpl::validateImageClientBuffer unimplemented.";
}
egl::Error DisplayImpl::validatePixmap(const egl::Config *config,
EGLNativePixmapType pixmap,
const egl::AttributeMap &attributes) const
{
UNREACHABLE();
return egl::EglBadDisplay() << "DisplayImpl::valdiatePixmap unimplemented.";
}
const egl::Caps &DisplayImpl::getCaps() const
{
if (!mCapsInitialized)
{
generateCaps(&mCaps);
mCapsInitialized = true;
}
return mCaps;
}
DeviceImpl *DisplayImpl::createDevice()
{
return new MockDevice();
}
bool DisplayImpl::isX11() const
{
return false;
}
bool DisplayImpl::isWayland() const
{
return false;
}
bool DisplayImpl::isGBM() const
{
return false;
}
bool DisplayImpl::supportsDmaBufFormat(EGLint format) const
{
UNREACHABLE();
return false;
}
egl::Error DisplayImpl::queryDmaBufFormats(EGLint max_formats, EGLint *formats, EGLint *num_formats)
{
UNREACHABLE();
return egl::NoError();
}
egl::Error DisplayImpl::queryDmaBufModifiers(EGLint format,
EGLint max_modifiers,
EGLuint64KHR *modifiers,
EGLBoolean *external_only,
EGLint *num_modifiers)
{
UNREACHABLE();
return egl::NoError();
}
} // namespace rx