0
Commit Graph

16 Commits

Author SHA1 Message Date
92d4fc59fa Update GPU process debugging documentation
Updates documentation on attaching a debugger to the GPU process based
on recent attempts to do so.

Change-Id: I2307d8823bd50759e2bee442b7dfd56ff355f927
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6336900
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Commit-Queue: Yuly Novikov <ynovikov@chromium.org>
Auto-Submit: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1429884}
2025-03-08 02:02:23 -08:00
d2a12773e7 Add documentation of --enable-gpu-service-tracing flag.
Bug: b/189215577
Change-Id: I0f299703b109bf178f6c536f5cd9d00e5a1d680a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2953111
Auto-Submit: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Commit-Queue: Zhenyao Mo <zmo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#891408}
2021-06-10 22:11:17 +00:00
046f987e02 [COIL] Change chromium code search type links to use main rather than master for docs
See this doc: go/chromium-coil-change for more info
    BUG=1210385

Change-Id: I6cd8548301e360bae8e71384dbd8ef11aa6c985f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2902904
Reviewed-by: My Nguyen <myy@chromium.org>
Commit-Queue: My Nguyen <myy@chromium.org>
Auto-Submit: John Palmer <jopalmer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#885851}
2021-05-24 01:24:56 +00:00
e660b15416 //gpu: rename switch to --ignore-gpu-blocklist
This switches is a relatively well known switch, so the old version is
kept around and produces a deprecation LOG(ERROR) when used.

Bug: chromium:1101491
Change-Id: I89aea90514bfe3a6c5a52e960b8e9434520ffbfd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2289414
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Sean Topping <seantopping@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#788661}
2020-07-15 16:07:54 +00:00
f533325e3f Revert "Re-Reland: Rename gl_test to gpu_tests"
This reverts commit d921069a00.

Reason for revert: broke https://ci.chromium.org/p/chromium/builders/ci/Deterministic%20Linux/25579

Original change's description:
> Re-Reland: Rename gl_test to gpu_tests
> 
> Reland keeping a group forwarding //gpu:gl_tests to //gpu:gpu_tests
> ReReland keeping gl_tests in the isolate map but making it point to
> gpu_tests.
> 
> This is to make the name reflect that there aren't just OpenGL tests in
> this executable now, there are WebGPU / Dawn tests too.
> 
> TBR=zmo@chromium.org
> 
> Bug: 1017035
> Change-Id: Ibcf866f3b91dd10a8825d534d160b6252de8c7e0
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1897927
> Reviewed-by: Corentin Wallez <cwallez@chromium.org>
> Reviewed-by: John Budorick <jbudorick@chromium.org>
> Commit-Queue: Corentin Wallez <cwallez@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#713002}

TBR=zmo@chromium.org,cwallez@chromium.org,jbudorick@chromium.org

Change-Id: Ia04386ef6de4415ab6fd17b54a0d1f3a5d5ad9a6
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1017035
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899191
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#713022}
2019-11-06 16:07:08 +00:00
d921069a00 Re-Reland: Rename gl_test to gpu_tests
Reland keeping a group forwarding //gpu:gl_tests to //gpu:gpu_tests
ReReland keeping gl_tests in the isolate map but making it point to
gpu_tests.

This is to make the name reflect that there aren't just OpenGL tests in
this executable now, there are WebGPU / Dawn tests too.

TBR=zmo@chromium.org

Bug: 1017035
Change-Id: Ibcf866f3b91dd10a8825d534d160b6252de8c7e0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1897927
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#713002}
2019-11-06 14:47:31 +00:00
1309d9a0bc Revert "Reland: Rename gl_test to gpu_tests"
This reverts commit e1aacdeb87.

Reason for revert: chromium-internal-arm-builder's generate_build_files is broken by this, see https://ci.chromium.org/p/chrome/builders/ci/chromium-internal-arm-builder/7799

Original change's description:
> Reland: Rename gl_test to gpu_tests
> 
> This is to make the name reflect that there aren't just OpenGL tests in
> this executable now, there are WebGPU / Dawn tests too.
> 
> TBR=zmo@chromium.org
> 
> Bug: 1017035
> Change-Id: I5f0f04a2082533eda1cd6c474c85214e2195304d
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1888936
> Reviewed-by: Corentin Wallez <cwallez@chromium.org>
> Reviewed-by: John Budorick <jbudorick@chromium.org>
> Commit-Queue: Corentin Wallez <cwallez@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#710854}

TBR=zmo@chromium.org,cwallez@chromium.org,jbudorick@chromium.org

Change-Id: Id0dcd22a2246be1f4a1507f336e8f63eddb03557
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1017035
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1891469
Reviewed-by: Sky Malice <skym@chromium.org>
Commit-Queue: Sky Malice <skym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#711018}
2019-10-30 22:22:25 +00:00
e1aacdeb87 Reland: Rename gl_test to gpu_tests
This is to make the name reflect that there aren't just OpenGL tests in
this executable now, there are WebGPU / Dawn tests too.

TBR=zmo@chromium.org

Bug: 1017035
Change-Id: I5f0f04a2082533eda1cd6c474c85214e2195304d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1888936
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710854}
2019-10-30 17:36:19 +00:00
46a8ee50fd Revert "Rename gl_test to gpu_tests"
This reverts commit 667d336eed.

Reason for revert: Breaks downstream Android builds.
https://ci.chromium.org/p/chrome/g/internal.client.clank/console

Original change's description:
> Rename gl_test to gpu_tests
> 
> This is to make the name reflect that there aren't just OpenGL tests in
> this executable now, there are WebGPU / Dawn tests too.
> 
> Bug: 1017035
> Change-Id: Id685700c01dfe754fc23220c34d49c96f835cc5a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1885470
> Reviewed-by: Zhenyao Mo <zmo@chromium.org>
> Reviewed-by: John Budorick <jbudorick@chromium.org>
> Commit-Queue: Corentin Wallez <cwallez@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#710382}

TBR=zmo@chromium.org,cwallez@chromium.org,jbudorick@chromium.org

Change-Id: I7e1ae7d108568a704f4900261b1f4b318d58b26c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1017035
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1887665
Reviewed-by: Christopher Grant <cjgrant@chromium.org>
Commit-Queue: Christopher Grant <cjgrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710401}
2019-10-29 18:27:54 +00:00
667d336eed Rename gl_test to gpu_tests
This is to make the name reflect that there aren't just OpenGL tests in
this executable now, there are WebGPU / Dawn tests too.

Bug: 1017035
Change-Id: Id685700c01dfe754fc23220c34d49c96f835cc5a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1885470
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710382}
2019-10-29 17:18:04 +00:00
78daa635cb Add enable_gpu_service_logging GN argument and about flag.
1. Add enable_gpu_service_logging GN argument. This can be used in
Release builds without dcheck_always_on=true, and allows the
--enable-gpu-service-logging command line argument to work.

2. Add about flag 'Enable gpu service logging' to let non-rooted
devices enable the --enable-gpu-service-logging flag.

Bug: 839769
Change-Id: Ic6fe4d97aaa6deb51b1f4d30ac4e7aeeaf8b4700
Reviewed-on: https://chromium-review.googlesource.com/c/1401616
Commit-Queue: vikas soni <vikassoni@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622252}
2019-01-12 01:32:52 +00:00
dca586f615 Add and document enable_gpu_client_logging GN argument.
This can be used in Release builds without dcheck_always_on=true, and
allows the --enable-gpu-client-logging command line argument to work.

Bug: 910783
Change-Id: I0720b7eb0c735de4d08ba50d69b41060eb387d63
Reviewed-on: https://chromium-review.googlesource.com/c/1357623
Reviewed-by: Victor Miura <vmiura@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#612900}
2018-12-01 01:53:19 +00:00
f73f0df88a Remove references to gyp from DEPS and docs
Bug: 826218
Change-Id: I176e1aeb0b24b21c6b4e5ee40910dce2bce52c95
Reviewed-on: https://chromium-review.googlesource.com/1239461
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#593522}
2018-09-24 13:52:49 +00:00
a33b7c45d8 Revert "Modify markdown to satisfy Windows Defender"
This reverts commit f17419f416.

Reason for revert: Microsoft responded to twitter reports and fixed
their detection so this isn't needed anymore. I'm retaining the typo
fix though!

Original change's description:
> Modify markdown to satisfy Windows Defender
> 
> debugging_gpu_related_code.md explains how to use a Chrome command-line
> switch which has been used for arbitrary code execution. Windows
> Defender therefore treats it as malware. This means that some Windows
> Chromium developers will get anti-virus warnings (with mandatory
> quarantining of the file). If Chrome is in a directory that is
> excluded from scanning then they will avoid warnings but will hit
> problems if they try to use Windows backup.
> 
> This is all very silly since there is no way - short of manually
> following the steps, modify to be malicious - to activate this payload.
> Windows Defender even complains about the file created by this:
> 
>   > echo --no-sandbox --gpu-launcher="x" >foo.txt
> 
> But, sometimes it's not worth arguing.
> 
> This also fixes a typo.
> 
> R=​kainino@chromium.org
> BUG=851562
> 
> Change-Id: I85403a1cb1667f45784684179927119058608d40
> Reviewed-on: https://chromium-review.googlesource.com/1096311
> Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
> Reviewed-by: Kai Ninomiya <kainino@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#566245}

TBR=brucedawson@chromium.org,kainino@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 851562
Change-Id: I60b09d8e6a698c1646f5c4bb0ecab16f999bfb9c
Reviewed-on: https://chromium-review.googlesource.com/1100255
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#567294}
2018-06-14 16:22:23 +00:00
f17419f416 Modify markdown to satisfy Windows Defender
debugging_gpu_related_code.md explains how to use a Chrome command-line
switch which has been used for arbitrary code execution. Windows
Defender therefore treats it as malware. This means that some Windows
Chromium developers will get anti-virus warnings (with mandatory
quarantining of the file). If Chrome is in a directory that is
excluded from scanning then they will avoid warnings but will hit
problems if they try to use Windows backup.

This is all very silly since there is no way - short of manually
following the steps, modify to be malicious - to activate this payload.
Windows Defender even complains about the file created by this:

  > echo --no-sandbox --gpu-launcher="x" >foo.txt

But, sometimes it's not worth arguing.

This also fixes a typo.

R=kainino@chromium.org
BUG=851562

Change-Id: I85403a1cb1667f45784684179927119058608d40
Reviewed-on: https://chromium-review.googlesource.com/1096311
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Reviewed-by: Kai Ninomiya <kainino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#566245}
2018-06-12 01:47:03 +00:00
a6429fb3a6 Port GPU documentation to Markdown
This ports the following wiki pages into markdown:
https://www.chromium.org/developers/testing/gpu-testing
https://www.chromium.org/developers/testing/gpu-testing/gpu-bot-details
https://www.chromium.org/developers/how-tos/gpu-wrangling
https://www.chromium.org/developers/how-tos/debugging-gpu-related-code

and updates *some* of the old outdated content.

Bug: 813153
Change-Id: Ic5f1b58659bbdb691343785cb18c50f4d55c177f
Reviewed-on: https://chromium-review.googlesource.com/987233
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547060}
2018-03-30 01:30:56 +00:00