New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 864695 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Jul 30
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

virglrenderer no provider failures for glSampleMaski

Project Member Reported by davidri...@chromium.org, Jul 17

Issue description

No provider of glSampleMaski found.  Requires one of:
    Desktop OpenGL 3.2
    GL extension "GL_ARB_texture_multisample"
    OpenGL ES 3.1
==158697== ERROR: libFuzzer: deadly signal
    #0 0x5643c1902ef7 in __sanitizer_print_stack_trace (/usr/libexec/fuzzers/virgl_fuzzer+0xfcef7)
    #1 0x5643c1831785 in _init (/usr/libexec/fuzzers/virgl_fuzzer+0x2b785)
    #2 0x5643c183173f in _init (/usr/libexec/fuzzers/virgl_fuzzer+0x2b73f)
    #3 0x7f55425fa2df  (/lib64/libpthread.so.0+0x112df)
    #4 0x7f5541e65dd1 in gsignal /var/tmp/portage/cross-x86_64-cros-linux-gnu/glibc-2.23-r18/work/glibc-2.23/signal/../sysdeps/unix/sysv/linux/raise.c:54
    #5 0x7f5541e67bf5 in abort /var/tmp/portage/cross-x86_64-cros-linux-gnu/glibc-2.23-r18/work/glibc-2.23/stdlib/abort.c:89
    #6 0x7f5542f2a144 in gl_provider_resolver /build/amd64-generic/tmp/portage/media-libs/libepoxy-1.4.0/work/libepoxy-1.4.0-abi_x86_64.amd64/src/gl_generated_dispatch.c:72518:5

NOTE: libFuzzer has rudimentary signal handlers.
      Combine libFuzzer with AddressSanitizer or similar for better crash reports.
SUMMARY: libFuzzer: deadly signal
MS: 0 ; base unit: 0000000000000000000000000000000000000000
0x18,0xa,0x1,0x0,0x81,0xf,0x0,0x14,
\x18\x0a\x01\x00\x81\x0f\x00\x14
artifact_prefix='./'; Test unit written to ./crash-53ecef5b51084c61a2e109b7689be23228b88ade
Base64: GAoBAIEPABQ=

 
Do you have a crash file for this?
Sorry, crbug was having issues and my attachment didn't upload properly.  Here it is.
crash-53ecef5b51084c61a2e109b7689be23228b88ade
8 bytes View Download
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 28

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/02c8bd18a6cecea1cb0b8acff9dba529454f89d0

commit 02c8bd18a6cecea1cb0b8acff9dba529454f89d0
Author: David Riley <davidriley@chromium.org>
Date: Sat Jul 28 05:33:19 2018

virglrenderer: Update to upstream, remove local fuzzer patch.

From: 0fb73b11e4cdadced885e52848002b2e9c79e3f5
To: 9c420d224d86215d408dff8dea599ed9414a24d6

9c420d2 vrend, caps: Move GL only caps into newly created function
f4ac4c6 vrend, caps: Move the sanity checks up in the call hierarchy
60521af vrend, caps: Split GL/GLES version checking and move caps set check up
97ddb62 vrend: remove superfluous initializations
c2e457e vrend: correct the stride if the client sends it
cd14ff1 renderer: Protect glSampleMaski and GL_SAMPLE_MASK.
519a091 shader_buffers: fix macros and use in decode.
42e2a4c vrend: use the row-stride when directly reading back to an IOV
34809ef vrend: Set scissor_state_dirty correctly.
58e521c get rid of yet another bind-flag set
2e84388 discourage using legacy-definitions
dc1bc1e get rid of diplicate definition of VREND_RES_BIND-flags
5ff40d5 add VIRGL_BIND_*-flags from mesa
9eaf2c8 features: disallow ssbos if we don't have the feature (v2)
6a6f3c4 features: add ubo feature (v2)
7958225 features: add transform_feedback feature (v2)
4145714 features: add multisample texture feature.
dd2f62b features: add cube map array feature.
c8d3c59 features: add texture array feature
4593bef features: add conditional render inverted.
0dc96e9 features: add transform feedback overflow query
ea7f3c1 features: add geometry shader feature
e8eeea7 features: add dual src blend support
1497dd9 features: add viewport array feature
4ed679c features: use correct extensions for tbo size
2704d81 features: add independent blend function feature.
c402e82 features: add indirect draw feature.
35356ec features: add independent blend enable feature
36ac335 features: add base instance feature.
7c23f33 features: add draw_instance feature.
c8269ae renderer: get return value from draw vbo.
31049f6 features: move existing features to a table init (v2)
87d8671 features: add transform_feedback3 feature
6ff41a3 features: move some caps to use has_feature flags
edd2478 Fix create_shader buf boundary check
fe7a1ef gles: report maximum vertex-attrib stride to guest
e898b8f Avoid needless repetition
2c0d096 use short-hand state accessors
7a37a36 Fix NULL dereference in vrend_draw_bind_samplers_shader
87b346a fuzzer: Add a libFuzzer based fuzzer.
5057fb9 tests: Fix virgl_init_cbs_wrong_ver test
79479ac blitter, GL blit fallback: clean up framebuffer after use
eb9555c features: convert current feature list to an array (v2)
cdf8860 renderer: fix ambiguous else warning
97b9df0 add a cap for TGSI precise modifiers
47387e4 emit precise keyword
ef70cef tgsi/text: parse _PRECISE modifier
46d2cf8 tgsi: populate precise
654647c protect calls to glPrimitiveRestart on GLES 3.1
39add38 protect gl{Begin, End}ConditionalRendering calls
4349893 protect call to glDeleteSamplers
89f7995 protect call to glPrimitiveRestartIndex
ec454b9 renderer: fix ssbo != -1 comparison.
df7322e ssbo: reorder var assignment
083d97f renderer: add shader_storage_buffer_object support. (v4)
1800bd4 shader: add basic shader_storage_buffer_object parsing. (v4)
4013fbc gallium: import MAX_SHADER_BUFFERS from mesa
dfa1e8c u_math: bring over u_bit_scan_consecutive_range.
7f96206 shader: drop unused sviews_used
249fb00 shader: pass sinfo/dinfo into translate_tex
a04a63e virgl-caps: Report support for GL_ARB_copy_image to the guest
8ad0201 vrend_formats: Replace RGB(8|16) formats with RGBX(8|16)
2846dcf vrend: If available use glCopyImageSubData to execute memcopy like blits
be3b107 vrend: Remove bad sRGB warning on GLES
cae96e1 shader: drop unused function.
e387116 report maximum vertex-attrib stride to host
6a4ef6d renderer: swizzle sampler border color channel if we emulate alpha format

BUG= chromium:852111 , chromium:864689 , chromium:862699 , chromium:864695 , chromium:864792 
TEST=ASAN_OPTIONS="log_path=stderr" /usr/libexec/fuzzers/virgl_fuzzer

Change-Id: I6e9b40675053dc1f18af6dfd888a145caecf13b7
Reviewed-on: https://chromium-review.googlesource.com/1153607
Commit-Ready: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Pohsien Wang <pwang@chromium.org>

[delete] https://crrev.com/6c1c4a5360e6a3a124cc64ebdc22943fbeb8211a/media-libs/virglrenderer/files/virglrenderer-0.6.0-fuzzer.patch
[delete] https://crrev.com/6c1c4a5360e6a3a124cc64ebdc22943fbeb8211a/media-libs/virglrenderer/virglrenderer-0.6.0_p20180716-r2.ebuild
[modify] https://crrev.com/02c8bd18a6cecea1cb0b8acff9dba529454f89d0/media-libs/virglrenderer/Manifest
[rename] https://crrev.com/02c8bd18a6cecea1cb0b8acff9dba529454f89d0/media-libs/virglrenderer/virglrenderer-0.6.0_p20180727.ebuild

Status: Fixed (was: Untriaged)

Sign in to add a comment