| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Revert "Revert of Added SyncToken command buffer trait to help with IPC messages. (patchset #1 id:1 of https://codereview.chromium.org/1394543003/ )"
This reverts commit 77970f1c3cd71d535a248d6e15f14081312b300a.
This CL is relands the SyncToken command buffer trait changes but also
fixes the alignment issues found on android.
R=piman@chromium.org
BUG=514815
Review URL: https://codereview.chromium.org/1399173002
Cr-Commit-Position: refs/heads/master@{#353555}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #1 id:1 of https://codereview.chromium.org/1394543003/ )
Reason for revert:
Broke gpu_unittests on Android. e.g. from the tombstones for this CL's run:
I 42.987s Main signal 7 (SIGBUS) at 0x7569c97a (code=1), thread 15602 (st:test_process)
I 42.987s Main
I 42.987s Main tombstone_06 created on 2015-10-08 01:00:00, about this long ago: 0:11:29 Device: 03d130ac006adc26
I 42.987s Main Reading native crash info from stdin
I 42.987s Main Reading Android symbols from /b/build/slave/android/build/src/third_party/android_platform/development/scripts/../../../..
I 42.987s Main Reading Chrome symbols from /b/build/slave/android/build/src/third_party/android_platform/development/scripts/../../../..
I 42.987s Main Using toolchain from :/b/build/slave/android/build/src/third_party/android_platform/development/scripts/../../../../third_party/android_tools/ndk/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64/bin/arm-linux-androideabi-
I 42.987s Main pid: 15669, tid: 15669, name: st:test_process >>> org.chromium.native_test:test_process <<<
I 42.987s Main signal 7 (SIGBUS), code 1 (BUS_ADRALN), fault addr 7569c97a
I 42.987s Main r0 7569c972 r1 00000000 r2 00000000 r3 00000000
I 42.987s Main r4 7569c972 r5 00000000 r6 0000007b r7 00000000
I 42.987s Main r8 00000052 r9 4004e678 sl 00000001 fp 400b2384
I 42.987s Main ip 00000000 sp bef25480 lr 75441b59 pc 75441b5c
I 42.987s Main
I 42.987s Main Stack Trace:
I 42.987s Main RELADDR FUNCTION FILE:LINE
I 42.987s Main 00329b5c gpu::SyncToken::SyncToken(gpu::CommandBufferNamespace, unsigned long long, unsigned long long) /data/app-lib/org.chromium.native_test-1/libgpu_unittests.so
I 42.987s Main 00077699 __static_initialization_and_destruction_0(int, int) [clone .constprop.176] /data/app-lib/org.chromium.native_test-1/libgpu_unittests.so
Original issue's description:
> Added SyncToken command buffer trait to help with IPC messages.
>
> R=piman@chromium.org
> BUG=514815
>
> Committed: https://crrev.com/6a219f8ccdd5148f28458b8184f31859babb9848
> Cr-Commit-Position: refs/heads/master@{#353170}
TBR=piman@chromium.org,dyen@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=514815
Review URL: https://codereview.chromium.org/1386323004
Cr-Commit-Position: refs/heads/master@{#353440}
|
|
|
|
|
|
|
|
|
| |
R=piman@chromium.org
BUG=514815
Review URL: https://codereview.chromium.org/1394543003
Cr-Commit-Position: refs/heads/master@{#353170}
|
|
|
|
|
|
|
|
|
|
| |
Required for the SubscribeUniform WebGL extension.
BUG=422978
Review URL: https://codereview.chromium.org/634313002
Cr-Commit-Position: refs/heads/master@{#306555}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make its current clients use base/macros.h and stdint.h instead.
No behavior change.
BUG=138542
NOTRY=true
Review URL: https://codereview.chromium.org/268063002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@268315 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Orignal goal of this CL is to make
InProcessCommandBuffer::CreateGpuMemoryBuffer and
DestroyGpuMemoryBuffer thread safe. Before this, Create runs
on the client thread and Destroy runs on the service thread
without any kind of synchronization. This change makes the
division closer to the cross-process implementation, moving
parts of the implementation from GpuControlService to
GpuControl/InProcessViewRenderer.
As a result, GpuControlService no longer needs to inherit
GpuControl. And GLES2Decoder has enough information to
decide on all gpu::Capabilities.
Need to implement the bare minimum client GpuControl for
gl_tests and gles2_conform_test. This currently involves
some boilerplate and duplication.
BUG=362346
Review URL: https://codereview.chromium.org/235563002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267753 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We tend to duplicate shared memory within the same process
when we don't need to, causing extra page faults etc when
accessing the memory.
With reference counting we can more easily re-use blocks of
shared memory in the same process, and make security easier
(objects in shared memory can hold a reference to their
memory and prevent use-after-free).
BUG=177063, 353822
Review URL: https://codereview.chromium.org/209353002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259491 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
gpu::Mailbox by itself can hold only a texture id, but in common usage it comes
with texture target and syncpoint information for cross-context sharing. To
reduce repetition of this pattern, gpu::MailboxHolder holds:
* a gpu::Mailbox
* a GL texture target
* a syncpoint index
Refactor other classes to use a gpu::MailboxHolder instead of separate
gpu::Mailbox and associated state.
Syncpoints are created with uint32 indices; make sure all uses of syncpoints
use the appropriate type.
BUG=None
TEST=local build, unittests on CrOS snow, desktop Linux
TBR=piman@chromium.org, enn@chromium.orge, cevans@chromium.org, scherkus@chromium.org
Review URL: https://codereview.chromium.org/132233041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248612 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 9ee2343406a6fae9c8aacc7303f7b1e5a04aab9e.
This patch is the source of the top crash in canary. See the bug
for more details.
TBR=sheu
NOTREECHECKS=true
BUG=336040
Review URL: https://codereview.chromium.org/129873005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245959 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
gpu::Mailbox by itself can hold only a texture id, but in common usage it comes
with texture target and syncpoint information for cross-context sharing. To
reduce repetition of this pattern, gpu::MailboxHolder holds:
* a gpu::Mailbox
* a GL texture target
* a syncpoint index
Refactor other classes to use a gpu::MailboxHolder instead of separate
gpu::Mailbox and associated state.
Syncpoints are created with uint32 indices; make sure all uses of syncpoints use uint32.
BUG=None
TEST=local build, unittests on CrOS snow, desktop Linux
Review URL: https://codereview.chromium.org/105743004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@245730 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
gpu::Capabilities is a subset of FeatureInfo that is exposed
to the command buffer client.
It is populated in GlES2Implementation::Initialize,
piggy-backing on the synchronous init ipc.
Make ContextProviderInProcess use gpu::Capabilities as well
instead of hardcoding them.
Merged GpuControl::SupportsGpuMemoryBuffer into capabilities.
BUG=285390
Review URL: https://codereview.chromium.org/73263003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238779 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
command_buffer was historically included in nacl-compiled things, and base
wasn't. These days, base is included in nacl-compiled things too, so the
duplication can be removed.
BUG=none
NOTRY=true
Review URL: https://codereview.chromium.org/57583008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232944 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This removes the ImageFactory interface and adjusts the buffer
allocation system for future multi-process support.
Also includes proper plumbing of internalformat to GLImage implementation
and makes sure the compositor is using the correct format.
TEST=gpu_unittests --gtest_filter=MockGpuMemoryBufferTest.Lifecycle
BUG=261649
Review URL: https://chromiumcodereview.appspot.com/20017005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218034 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mailbox is safer to IPC than coaxing it into a string, because otherwise we need
to validate the size when coming from untrusted sources.
This is to enable passing Mailbox through IPC without having to depend on cc.
Among others I will need it to move Pepper to mailboxes, and pepper can't depend
on cc.
BUG=164095
Review URL: https://chromiumcodereview.appspot.com/12378053
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@186006 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
I suspect that the target name collision is causing compile failures on Win (e.g. http://build.chromium.org/p/chromium.win/builders/Win%20Builder/builds/4176/steps/compile/logs/stdio)
BUG=none
TBR=jochen@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/11412217
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@169895 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
instead of hardcoding it all over the place (literally).
This is one of the steps towards use_system_mesa build option.
BUG=161389
Review URL: https://chromiumcodereview.appspot.com/11416085
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168931 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Increases query result size to be 64 bits.
Allows result to be used as input on the query begin.
BUG=
Review URL: https://chromiumcodereview.appspot.com/11235012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163683 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=141700
Review URL: https://chromiumcodereview.appspot.com/10836185
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151194 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=116317
TEST=builds,manual
With this change, the IPC-based PPAPI proxy builds all code needed for 3d and OpenGL ES.
Review URL: https://chromiumcodereview.appspot.com/10837145
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150432 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
to untrusted build.
BUG=116317
TEST=builds,manual
With this change, the IPC-based PPAPI proxy builds and runs the tumbler example.
Review URL: https://chromiumcodereview.appspot.com/10796038
TBR=bbudge@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10830197
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150345 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=116317
TEST=builds,manual
With this change, the IPC-based PPAPI proxy builds and runs the tumbler example.
Review URL: https://chromiumcodereview.appspot.com/10796038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150343 0039d316-1c4b-4281-b951-d872f2087c98
|
|
There should be no change in behavior.
BUG=114261
TEST=none
Review URL: https://chromiumcodereview.appspot.com/9359055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122413 0039d316-1c4b-4281-b951-d872f2087c98
|