Video playback colorspace conversion corruption with certain H264 VUI params
Reported by
iricha...@nvidia.com,
Mar 14 2018
|
||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.162 Safari/537.36 Example URL: https://drive.google.com/file/d/1Lrx3xO54zWDAokYKsboZa3tLaAgWgDPe Steps to reproduce the problem: 1. Download video file from https://drive.google.com/file/d/1Lrx3xO54zWDAokYKsboZa3tLaAgWgDPe (must download, not play from drive) 2. Play it back in the browser, observe the moving black spots in the middle of the video. What is the expected behavior? No corruption during video playback. What went wrong? This video has certain H264 VUI params (transform function 'GAMMA22' / range 'limited') that appears to cause a problem in the CSC shader. Did this work before? N/A Is it a problem with Flash or HTML5? HTML5 Does this work in other browsers? Yes Chrome version: 65.0.3325.162 Channel: stable OS Version: 10.0 Flash Version: Contents of chrome://gpu: Graphics Feature Status Canvas: Hardware accelerated CheckerImaging: Disabled Flash: Hardware accelerated Flash Stage3D: Hardware accelerated Flash Stage3D Baseline profile: Hardware accelerated Compositing: Hardware accelerated Multiple Raster Threads: Enabled Native GpuMemoryBuffers: Software only. Hardware acceleration disabled Rasterization: Hardware accelerated Video Decode: Hardware accelerated WebGL: Hardware accelerated WebGL2: Hardware accelerated Driver Bug Workarounds clear_uniforms_before_first_program_use decode_encode_srgb_for_generatemipmap disable_delayed_copy_nv12 disable_discard_framebuffer disable_framebuffer_cmaa exit_on_context_lost force_cube_complete scalarize_vec_and_mat_constructor_args texsubimage_faster_than_teximage Problems Detected Some drivers are unable to reset the D3D device in the GPU process sandbox Applied Workarounds: exit_on_context_lost TexSubImage is faster for full uploads on ANGLE Applied Workarounds: texsubimage_faster_than_teximage Clear uniforms before first program use on all platforms: 124764, 349137 Applied Workarounds: clear_uniforms_before_first_program_use Always rewrite vec/mat constructors to be consistent: 398694 Applied Workarounds: scalarize_vec_and_mat_constructor_args ANGLE crash on glReadPixels from incomplete cube map texture: 518889 Applied Workarounds: force_cube_complete Framebuffer discarding can hurt performance on non-tilers: 570897 Applied Workarounds: disable_discard_framebuffer Use GL_INTEL_framebuffer_CMAA on ChromeOS: 535198 Applied Workarounds: disable_framebuffer_cmaa Disable KHR_blend_equation_advanced until cc shaders are updated: 661715 Applied Workarounds: disable(GL_KHR_blend_equation_advanced), disable(GL_KHR_blend_equation_advanced_coherent) Decode and Encode before generateMipmap for srgb format textures on Windows: 634519 Applied Workarounds: decode_encode_srgb_for_generatemipmap Delayed copy NV12 displays incorrect colors on NVIDIA drivers.: 728670 Applied Workarounds: disable_delayed_copy_nv12 Don't expose disjoint_timer_query extensions to WebGL: 808744 Native GpuMemoryBuffers have been disabled, either via about:flags or command line. Disabled Features: native_gpu_memory_buffers Checker-imaging has been disabled via finch trial or the command line. Disabled Features: checker_imaging Version Information Data exported 2018-03-14T20:21:39.172Z Chrome version Chrome/65.0.3325.162 Operating system Windows NT 10.0.16299 Software rendering list URL https://chromium.googlesource.com/chromium/src/+/5d04e9e9c8ce31bee0923a8c326a7e9e19c492a3/gpu/config/software_rendering_list.json Driver bug list URL https://chromium.googlesource.com/chromium/src/+/5d04e9e9c8ce31bee0923a8c326a7e9e19c492a3/gpu/config/gpu_driver_bug_list.json ANGLE commit id 2c9cc8b6e810 2D graphics backend Skia/65 bd0dafbc8112f6cfa92a8096d8cb5696d8535ef9- Command Line "C:\Program Files (x86)\Google\Chrome\Application\chrome.exe" --flag-switches-begin --flag-switches-end --restore-last-session --flag-switches-begin --flag-switches-end Driver Information Initialization time 182 In-process GPU false Passthrough Command Decoder false Direct Composition true Supports overlays false Sandboxed false GPU0 VENDOR = 0x10de, DEVICE= 0x1617 *ACTIVE* Optimus false Optimus false AMD switchable false Desktop compositing Aero Glass Diagonal Monitor Size of \\.\DISPLAY1 27.1" Diagonal Monitor Size of \\.\DISPLAY2 27.1" Diagonal Monitor Size of \\.\DISPLAY3 17.2" Diagonal Monitor Size of \\.\DISPLAY4 17.2" Driver vendor NVIDIA Driver version 24.21.13.9681 Driver date 3-10-2018 Pixel shader version 5.0 Vertex shader version 5.0 Max. MSAA samples 8 Machine model name Machine model version GL_VENDOR Google Inc. GL_RENDERER ANGLE (NVIDIA GeForce GTX 980M Direct3D11 vs_5_0 ps_5_0) GL_VERSION OpenGL ES 2.0 (ANGLE 2.1.0.2c9cc8b6e810) GL_EXTENSIONS GL_ANGLE_client_arrays GL_ANGLE_depth_texture GL_ANGLE_framebuffer_blit GL_ANGLE_framebuffer_multisample GL_ANGLE_instanced_arrays GL_ANGLE_lossy_etc_decode GL_ANGLE_pack_reverse_row_order GL_ANGLE_program_cache_control GL_ANGLE_request_extension GL_ANGLE_robust_client_memory GL_ANGLE_texture_compression_dxt3 GL_ANGLE_texture_compression_dxt5 GL_ANGLE_texture_usage GL_ANGLE_translated_shader_source GL_CHROMIUM_bind_generates_resource GL_CHROMIUM_bind_uniform_location GL_CHROMIUM_color_buffer_float_rgb GL_CHROMIUM_color_buffer_float_rgba GL_CHROMIUM_copy_compressed_texture GL_CHROMIUM_copy_texture GL_CHROMIUM_sync_query GL_EXT_blend_minmax GL_EXT_color_buffer_half_float GL_EXT_debug_marker GL_EXT_discard_framebuffer GL_EXT_disjoint_timer_query GL_EXT_draw_buffers GL_EXT_frag_depth GL_EXT_map_buffer_range GL_EXT_occlusion_query_boolean GL_EXT_read_format_bgra GL_EXT_robustness GL_EXT_sRGB GL_EXT_shader_texture_lod GL_EXT_texture_compression_dxt1 GL_EXT_texture_compression_s3tc_srgb GL_EXT_texture_filter_anisotropic GL_EXT_texture_format_BGRA8888 GL_EXT_texture_rg GL_EXT_texture_storage GL_EXT_unpack_subimage GL_KHR_debug GL_NV_EGL_stream_consumer_external GL_NV_fence GL_NV_pack_subimage GL_NV_pixel_buffer_object GL_OES_EGL_image GL_OES_EGL_image_external GL_OES_compressed_ETC1_RGB8_texture GL_OES_depth32 GL_OES_element_index_uint GL_OES_get_program_binary GL_OES_mapbuffer GL_OES_packed_depth_stencil GL_OES_rgb8_rgba8 GL_OES_standard_derivatives GL_OES_surfaceless_context GL_OES_texture_float GL_OES_texture_float_linear GL_OES_texture_half_float GL_OES_texture_half_float_linear GL_OES_texture_npot GL_OES_vertex_array_object Disabled Extensions GL_KHR_blend_equation_advanced GL_KHR_blend_equation_advanced_coherent Disabled WebGL Extensions EXT_disjoint_timer_query EXT_disjoint_timer_query_webgl2 Window system binding vendor Google Inc. (adapter LUID: 0000000000009122) Window system binding version 1.4 (ANGLE 2.1.0.2c9cc8b6e810) Window system binding extensions EGL_EXT_create_context_robustness EGL_ANGLE_d3d_share_handle_client_buffer EGL_ANGLE_d3d_texture_client_buffer EGL_ANGLE_surface_d3d_texture_2d_share_handle EGL_ANGLE_query_surface_pointer EGL_ANGLE_window_fixed_size EGL_ANGLE_keyed_mutex EGL_ANGLE_surface_orientation EGL_ANGLE_direct_composition EGL_NV_post_sub_buffer EGL_KHR_create_context EGL_EXT_device_query EGL_KHR_image EGL_KHR_image_base EGL_KHR_gl_texture_2D_image EGL_KHR_gl_texture_cubemap_image EGL_KHR_gl_renderbuffer_image EGL_KHR_get_all_proc_addresses EGL_KHR_stream EGL_KHR_stream_consumer_gltexture EGL_NV_stream_consumer_gltexture_yuv EGL_ANGLE_flexible_surface_compatibility EGL_ANGLE_stream_producer_d3d_texture EGL_ANGLE_create_context_webgl_compatibility EGL_CHROMIUM_create_context_bind_generates_resource EGL_CHROMIUM_sync_control EGL_EXT_pixel_format_float EGL_KHR_surfaceless_context EGL_ANGLE_display_texture_share_group EGL_ANGLE_create_context_client_arrays EGL_ANGLE_program_cache_control EGL_ANGLE_robust_resource_initialization Direct rendering Yes Reset notification strategy 0x8252 GPU process crash count 0 Compositor Information Tile Update Mode One-copy Partial Raster Enabled GpuMemoryBuffers Status ATC Software only ATCIA Software only DXT1 Software only DXT5 Software only ETC1 Software only R_8 Software only R_16 Software only RG_88 Software only BGR_565 Software only RGBA_4444 Software only RGBX_8888 GPU_READ, SCANOUT RGBA_8888 GPU_READ, SCANOUT BGRX_8888 Software only BGRX_1010102 Software only BGRA_8888 Software only RGBA_F16 Software only YVU_420 Software only YUV_420_BIPLANAR Software only UYVY_422 Software only Display(s) Information Info Display[2528732444] bounds=2560,0 2560x1440, workarea=2560,0 2560x1400, scale=1.5, external Color space information {primaries:[[0.4407,0.3712,0.1523,],[0.2234,0.7121,0.0645,],[0.0061,0.0545,0.7644,],], transfer:0.0000*x + 0.0000 if x < 0.0000 else (1.0000*x + 0.0000)**2.2000 + 0.0000, matrix:RGB, range:FULL} Bits per color component 8 Bits per pixel 24 Info Display[2779098405] bounds=0,0 2560x1440, workarea=0,0 2560x1400, scale=1.5, external Color space information {primaries:[[0.4407,0.3712,0.1523,],[0.2234,0.7121,0.0645,],[0.0061,0.0545,0.7644,],], transfer:0.0000*x + 0.0000 if x < 0.0000 else (1.0000*x + 0.0000)**2.2000 + 0.0000, matrix:RGB, range:FULL} Bits per color component 8 Bits per pixel 24 Video Acceleration Information Decode h264 baseline up to 4096x2304 pixels Decode h264 main up to 4096x2304 pixels Decode h264 high up to 4096x2304 pixels Decode vp8 up to 4096x2160 pixels Decode vp9 profile0 up to 4096x2160 pixels Decode vp9 profile1 up to 4096x2160 pixels Decode vp9 profile2 up to 4096x2160 pixels Decode vp9 profile3 up to 4096x2160 pixels Encode h264 baseline up to 3840x2176 pixels and/or 30.000 fps Encode h264 main up to 3840x2176 pixels and/or 30.000 fps Encode h264 high up to 3840x2176 pixels and/or 30.000 fps Diagnostics 0 b3DAccelerationEnabled true b3DAccelerationExists true bAGPEnabled true bAGPExistenceValid true bAGPExists true bCanRenderWindow true bDDAccelerationEnabled true bDriverBeta false bDriverDebug false bDriverSigned false bDriverSignedValid false bNoHardware false dwBpp 32 dwDDIVersion 12 dwHeight 2160 dwRefreshRate 60 dwWHQLLevel 0 dwWidth 3840 iAdapter 1 lDriverSize 933632 lMiniVddSize 0 szAGPStatusEnglish Enabled szAGPStatusLocalized Enabled szChipType GeForce GTX 980M szD3DStatusEnglish Enabled szD3DStatusLocalized Enabled szDACType Integrated RAMDAC szDDIVersionEnglish 12 szDDIVersionLocalized 12 szDDStatusEnglish Enabled szDDStatusLocalized Enabled szDXVAHDEnglish Supported szDXVAModes szDescription NVIDIA GeForce GTX 980M szDeviceId 0x1617 szDeviceIdentifier {D7B71E3E-5557-11CF-2256-E03C1BC2D535} szDeviceName \\.\DISPLAY1 szDisplayMemoryEnglish 20406 MB szDisplayMemoryLocalized 20406 MB szDisplayModeEnglish 3840 x 2160 (32 bit) (60Hz) szDisplayModeLocalized 3840 x 2160 (32 bit) (60Hz) szDriverAssemblyVersion 24.21.13.9681 szDriverAttributes Final Retail szDriverDateEnglish 3/9/2018 8:00:00 PM szDriverDateLocalized 3/9/2018 20:00:00 szDriverLanguageEnglish English szDriverLanguageLocalized English szDriverModelEnglish WDDM 2.3 szDriverModelLocalized WDDM 2.3 szDriverName C:\WINDOWS\System32\DriverStore\FileRepository\nvami.inf_amd64_50cc5e17ee8404dc\nvldumdx.dll,C:\WINDOWS\System32\DriverStore\FileRepository\nvami.inf_amd64_50cc5e17ee8404dc\nvldumdx.dll,C:\WINDOWS\System32\DriverStore\FileRepository\nvami.inf_amd64_50cc5e17ee8404dc\nvldumdx.dll,C:\WINDOWS\System32\DriverStore\FileRepository\nvami.inf_amd64_50cc5e17ee8404dc\nvldumdx.dll szDriverNodeStrongName oem72.inf:0f066de3aa1d5595:Section100:24.21.13.9681:pci\ven_10de&dev_1617&subsys_1ced1043 szDriverSignDate Unknown szDriverVersion 24.21.0013.9681 szKeyDeviceID Enum\PCI\VEN_10DE&DEV_1617&SUBSYS_1CED1043&REV_A1 szKeyDeviceKey \Registry\Machine\System\CurrentControlSet\Control\Video\{ACC3051F-25D6-11E8-8748-9C5C8E366082}\0000 szManufacturer NVIDIA szMiniVdd unknown szMiniVddDateEnglish Unknown szMiniVddDateLocalized unknown szMonitorMaxRes Unknown szMonitorName LG ULTRA HD(DisplayPort) szNotesEnglish No problems found. szNotesLocalized No problems found. szOverlayEnglish Supported szRankOfInstalledDriver 00D10001 szRegHelpText Unknown szRevision Unknown szRevisionId 0x00A1 szSubSysId 0x1CED1043 szTestResultD3D7English Not run szTestResultD3D7Localized Not run szTestResultD3D8English Not run szTestResultD3D8Localized Not run szTestResultD3D9English Not run szTestResultD3D9Localized Not run szTestResultDDEnglish Not run szTestResultDDLocalized Not run szVdd unknown szVendorId 0x10DE 1 b3DAccelerationEnabled true b3DAccelerationExists true bAGPEnabled true bAGPExistenceValid true bAGPExists true bCanRenderWindow true bDDAccelerationEnabled true bDriverBeta false bDriverDebug false bDriverSigned false bDriverSignedValid false bNoHardware false dwBpp 32 dwDDIVersion 12 dwHeight 2160 dwRefreshRate 60 dwWHQLLevel 0 dwWidth 3840 iAdapter 0 lDriverSize 933632 lMiniVddSize 0 szAGPStatusEnglish Enabled szAGPStatusLocalized Enabled szChipType GeForce GTX 980M szD3DStatusEnglish Enabled szD3DStatusLocalized Enabled szDACType Integrated RAMDAC szDDIVersionEnglish 12 szDDIVersionLocalized 12 szDDStatusEnglish Enabled szDDStatusLocalized Enabled szDXVAHDEnglish Supported szDXVAModes szDescription NVIDIA GeForce GTX 980M szDeviceId 0x1617 szDeviceIdentifier {D7B71E3E-5557-11CF-2256-E03C1BC2D535} szDeviceName \\.\DISPLAY2 szDisplayMemoryEnglish 20406 MB szDisplayMemoryLocalized 20406 MB szDisplayModeEnglish 3840 x 2160 (32 bit) (60Hz) szDisplayModeLocalized 3840 x 2160 (32 bit) (60Hz) szDriverAssemblyVersion 24.21.13.9681 szDriverAttributes Final Retail szDriverDateEnglish 3/9/2018 8:00:00 PM szDriverDateLocalized 3/9/2018 20:00:00 szDriverLanguageEnglish English szDriverLanguageLocalized English szDriverModelEnglish WDDM 2.3 szDriverModelLocalized WDDM 2.3 szDriverName C:\WINDOWS\System32\DriverStore\FileRepository\nvami.inf_amd64_50cc5e17ee8404dc\nvldumdx.dll,C:\WINDOWS\System32\DriverStore\FileRepository\nvami.inf_amd64_50cc5e17ee8404dc\nvldumdx.dll,C:\WINDOWS\System32\DriverStore\FileRepository\nvami.inf_amd64_50cc5e17ee8404dc\nvldumdx.dll,C:\WINDOWS\System32\DriverStore\FileRepository\nvami.inf_amd64_50cc5e17ee8404dc\nvldumdx.dll szDriverNodeStrongName oem72.inf:0f066de3aa1d5595:Section100:24.21.13.9681:pci\ven_10de&dev_1617&subsys_1ced1043 szDriverSignDate Unknown szDriverVersion 24.21.0013.9681 szKeyDeviceID Enum\PCI\VEN_10DE&DEV_1617&SUBSYS_1CED1043&REV_A1 szKeyDeviceKey \Registry\Machine\System\CurrentControlSet\Control\Video\{ACC3051F-25D6-11E8-8748-9C5C8E366082}\0001 szManufacturer NVIDIA szMiniVdd unknown szMiniVddDateEnglish Unknown szMiniVddDateLocalized unknown szMonitorMaxRes Unknown szMonitorName LG ULTRA HD(DisplayPort) szNotesEnglish No problems found. szNotesLocalized No problems found. szOverlayEnglish Supported szRankOfInstalledDriver 00D10001 szRegHelpText Unknown szRevision Unknown szRevisionId 0x00A1 szSubSysId 0x1CED1043 szTestResultD3D7English Not run szTestResultD3D7Localized Not run szTestResultD3D8English Not run szTestResultD3D8Localized Not run szTestResultD3D9English Not run szTestResultD3D9Localized Not run szTestResultDDEnglish Not run szTestResultDDLocalized Not run szVdd unknown szVendorId 0x10DE Log Messages GpuProcessHostUIShim: The GPU process exited normally. Everything is okay. Disabling GPU support in chrome allows the video to play without corruption on screen.
,
Mar 14 2018
,
Mar 14 2018
,
Mar 15 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5391302f122db79a527d86beb976adbbfb0d014f commit 5391302f122db79a527d86beb976adbbfb0d014f Author: Fredrik Hubinette <hubbe@google.com> Date: Thu Mar 15 00:48:23 2018 color: avoid negative pow Avoid calling pow() on negative numbers as that causes undefined behavior in the shader. Bug: 821967 Change-Id: Iab3dc9840c17d90aa5952adf17b8b24e2890d055 Reviewed-on: https://chromium-review.googlesource.com/962969 Reviewed-by: ccameron <ccameron@chromium.org> Commit-Queue: Fredrik Hubinette <hubbe@chromium.org> Cr-Commit-Position: refs/heads/master@{#543263} [modify] https://crrev.com/5391302f122db79a527d86beb976adbbfb0d014f/ui/gfx/color_transform.cc [modify] https://crrev.com/5391302f122db79a527d86beb976adbbfb0d014f/ui/gfx/color_transform_unittest.cc
,
Mar 15 2018
How common is this; should we merge back to 66 or even 65?
,
Mar 15 2018
,
Mar 15 2018
,
Mar 15 2018
,
Mar 15 2018
It can potentially happen a lot, but most encoders are pretty good about only using "allowed" values instead of assuming that the values get clipped. Not sure how often it really happens.
,
Mar 15 2018
Do we need to take the CL into M65 and M66 if so please do merge request for the respective channels.
,
Mar 15 2018
@irichards - how critical is this issue to nvidia?
,
Mar 15 2018
Marking as "RBS" for tracking purpose. Thank you.
,
Mar 15 2018
Verified fixed on canary.
,
Mar 15 2018
[Auto-generated comment by a script] We noticed that this issue is targeted for M-65; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-65 label, otherwise remove Merge-TBD label. Thanks.
,
Mar 15 2018
It's not terribly critical - this does happen on basically all game recordings made by shadowplay (we're re-evaluating the encoder params because of this), but playing them back directly in a browser really isn't a primary usecase. The problem goes away whenever things get transcoded on upload to youtube or whereever. Thanks for the quick fix!
,
Mar 15 2018
So it looks like this bug has been around for about a year. However, on systems with hardware-decoded video, it may or may not show up depending on what workarounds are in effect. Probably not worth merging.
,
Mar 15 2018
Seems reasonable to merge to M66 at least though, just not M65 probably.
,
Mar 15 2018
Thank you irichards@, hubbe@ and dalecurtis@. Removing "RBS" label. hubbe@, pls request a merge to M66 per comment #17.
,
Mar 15 2018
,
Mar 15 2018
Merge to M65 is not needed per comment #17 and Merge to M66 is already requested at #19.
,
Mar 16 2018
This bug requires manual review: M66 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: cmasso@(Android), cmasso@(iOS), josafat@(ChromeOS), abdulsyed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 19 2018
Let's just target this for M67. This has been around for a year, currently in M65. Since we're already past branch, let's target this for M67. |
||||||||||||||
►
Sign in to add a comment |
||||||||||||||
Comment 1 by pbomm...@chromium.org
, Mar 14 2018