On Mon, 04 Feb 2019, Daniel Vetter <daniel@xxxxxxxx> wrote: > On Mon, Feb 04, 2019 at 10:47:36AM +0200, Joonas Lahtinen wrote: >> Quoting Dave Airlie (2019-02-04 07:02:07) >> > On Sat, 2 Feb 2019 at 18:29, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote: >> > > >> > > Hi Dave and Daniel, >> > > >> > > Here goes another pull request for 5.1. >> > >> > dim complained: >> > >> > Chris committed this without an S-O-B, now because it's all Intel this >> > probably doesn't matter, so I'll pull it, put please try and let it >> > not happen again. >> >> It's a tooling issue. It even has the Link: tag, so it is applied with >> dim, which automatically should apply the S-o-b of committer. The issue >> should already have a fix. >> >> And we also concluded that as it's all Intel, it should be legally OK, >> and not worthy force pushing the history (as it was noticed rather >> late). >> >> But looks like the communication back to you fell short. Apologies for >> that. > > Hm yeah I thought Dave was on cc: but he wasn't. Some I was on cc: for > that thread though (no idea why that tbh). Also just noticed that we only > had the private subthread that Jani started, but never replied in public > (or to sfr or anyone). Sorry about that, the intention was to sort it out in private first, the follow-up failed. BR, Jani. > -Daniel > >> >> Regards, Joonas >> >> > Dave. >> > >> > commit 8e525cb4a622148fbe30134ee3a1a34ad839a43a >> > Author: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> >> > Commit: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> >> > >> > drm/i915/execlists: Move RPCS setup to context pin >> > >> > Configuring RPCS in context image just before pin is sufficient and will >> > come extra handy in one of the following patches. >> > >> > v2: >> > * Split image setup a bit differently. (Chris Wilson) >> > >> > v3: >> > * Update context image after reset as well - otherwise the application >> > of pinned default state clears the RPCS. >> > >> > v4: >> > * Use local variable throughout the function. (Chris Wilson) >> > >> > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> >> > Suggested-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> >> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> >> > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> >> > Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> >> > Link: https://patchwork.freedesktop.org/patch/msgid/20190125023005.1007-1-chris@xxxxxxxxxxxxxxxxxx >> > >> > > >> > > Maybe I will still send another next week. >> > > >> > > This pull also include a GVT one with: >> > > " >> > > Here is gvt-next stuff. This includes Coffeelake support for GVT, >> > > making kvmgt as self load module to have better dependence with >> > > vfio/mdev, with some const treatment and kernel type change. >> > > " >> > > >> > > And also it includes a drm change for constify drm_color_lut_check. >> > > >> > > Rest of details are on the tags below. >> > > >> > > drm-intel-next-2019-02-02: >> > > - Make background color and LUT more robust (Matt) >> > > - Icelake display fixes (Ville, Imre) >> > > - Workarounds fixes and reorg (Tvrtko, Talha) >> > > - Enable fastboot by default on VLV and CHV (Hans) >> > > - Add another PCI ID for Coffee Lake (Rodrigo) >> > > >> > > drm-intel-next-2019-01-29: >> > > - MOCS table rework for simplification and to add ICL (Lucas, Tomasz) >> > > - Move RPCS setup to context pin (Tvrtko) >> > > - Breadcrumb simplification and GPU Reset improvements (Chris) >> > > - Many fixes for TV modeset (Ville) >> > > - Clean up on atomic plane checks (Ville) >> > > - NV12 pich check fix (Raviraj) >> > > - Disable -Wuninitialized (Nathan) >> > > - Sanitize DPLL state for broken BIOSes on SNB (Ville) >> > > - Rework on vma locking and counting and introduce a concept of per-timeline >> > > HWSP (Chris) >> > > - Enable fastboot by default on Skylake and newer platforms (Hans) >> > > - Fix slk srckey mask bits (Ville) >> > > - Selftests fixes (Chris) >> > > - Execlists and preemption improvements and fixes (Chris) >> > > - drm consitify drm_color_lut_check (Ville) >> > > - Ice Lake clock fixes (Lucas) >> > > >> > > Thanks, >> > > Rodrigo. >> > > >> > > The following changes since commit 85baa5dbf79163026dcb78f742294c522e176432: >> > > >> > > drm/i915: Update DRIVER_DATE to 20190124 (2019-01-24 15:00:59 -0800) >> > > >> > > are available in the Git repository at: >> > > >> > > git://anongit.freedesktop.org/drm/drm-intel tags/drm-intel-next-2019-02-02 >> > > >> > > for you to fetch changes up to 46c0cd8c562bc3e4a99cbaa4ba0904b6871b7b4b: >> > > >> > > drm/i915: Update DRIVER_DATE to 20190202 (2019-02-02 00:14:28 -0800) >> > > >> > > ---------------------------------------------------------------- >> > > - Make background color and LUT more robust (Matt) >> > > - Icelake display fixes (Ville, Imre) >> > > - Workarounds fixes and reorg (Tvrtko, Talha) >> > > - Enable fastboot by default on VLV and CHV (Hans) >> > > - Add another PCI ID for Coffee Lake (Rodrigo) >> > > >> > > ---------------------------------------------------------------- >> > > Chris Wilson (27): >> > > drm/i915: Measure the required reserved size for request emission >> > > drm/i915: Remove manual breadcumb counting >> > > drm/i915: Compute the HWS offsets explicitly >> > > drm/i915: Make all GPU resets atomic >> > > drm/i915/guc: Disable global reset >> > > drm/i915: Remove GPU reset dependence on struct_mutex >> > > drm/i915/selftests: Trim struct_mutex duration for set-wedged selftest >> > > drm/i915: Issue engine resets onto idle engines >> > > drm/i915: Wait for a moment before forcibly resetting the device >> > > drm/i915: Stop tracking MRU activity on VMA >> > > drm/i915: Pull VM lists under the VM mutex. >> > > drm/i915: Move vma lookup to its own lock >> > > drm/i915: Always allocate an object/vma for the HWSP >> > > drm/i915: Move list of timelines under its own lock >> > > drm/i915: Introduce concept of per-timeline (context) HWSP >> > > drm/i915: Enlarge vma->pin_count >> > > drm/i915: Allocate a status page for each timeline >> > > drm/i915: Share per-timeline HWSP using a slab suballocator >> > > drm/i915: Track the context's seqno in its own timeline HWSP >> > > drm/i915: Track active timelines >> > > drm/i915/selftests: Apply a subtest filter >> > > drm/i915: Identify active requests >> > > drm/i915: Rename execlists->queue_priority to queue_priority_hint >> > > drm/i915/execlists: Suppress preempting self >> > > drm/i915: Remove the intel_engine_notify tracepoint >> > > drm/i915: Replace global breadcrumbs with per-context interrupt tracking >> > > drm/i915: Drop fake breadcrumb irq >> > > >> > > Hang Yuan (3): >> > > drm/i915/gvt: add functions to get default resolution >> > > drm/i915/gvt: add hotplug emulation >> > > drm/i915/gvt: add VFIO EDID region >> > > >> > > Hans de Goede (2): >> > > drm/i915: Enable fastboot by default on Skylake and newer >> > > drm/i915: Enable fastboot by default on VLV and CHV >> > > >> > > Imre Deak (2): >> > > drm/i915/icl: Add TypeC ports only if VBT is present >> > > drm/i915/icl: Work around broken VBTs for port F detection >> > > >> > > Lucas De Marchi (9): >> > > drm/i915: initialize unused MOCS entries to PTE >> > > drm/i915: Simplify MOCS table definition >> > > drm/i915: use a macro to define MOCS entries >> > > drm/i915: keep track of used entries in MOCS table >> > > drm/i915: cache number of MOCS entries >> > > drm/i915/icl: use tc_port in MG_PLL macros >> > > drm/i915: always return something on DDI clock selection >> > > drm/i915/icl: remove dpll from clk_sel >> > > drm/i915/icl: keep track of unused pll while looping >> > > >> > > Matt Roper (2): >> > > drm/i915: Force background color to black for gen9+ (v2) >> > > drm/i915: Apply LUT validation checks to platforms more accurately (v3) >> > > >> > > Nathan Chancellor (1): >> > > drm/i915: Disable -Wuninitialized >> > > >> > > P Raviraj Sitaram (1): >> > > drm/i915: correct the pitch check for NV12 framebuffer >> > > >> > > Rodrigo Vivi (5): >> > > Merge drm/drm-next into drm-intel-next-queued >> > > drm/i915: Update DRIVER_DATE to 20190129 >> > > Merge tag 'gvt-next-2019-02-01' of https://github.com/intel/gvt-linux into drm-intel-next-queued >> > > drm/i915/cfl: Adding another PCI Device ID. >> > > drm/i915: Update DRIVER_DATE to 20190202 >> > > >> > > Talha Nassar (1): >> > > drm/i915/icl: restore WaEnableFloatBlendOptimization >> > > >> > > Tomasz Lis (2): >> > > drm/i915/skl: Rework MOCS tables to keep common part in a define >> > > drm/i915/icl: Define MOCS table for Icelake >> > > >> > > Tvrtko Ursulin (3): >> > > drm/i915/execlists: Move RPCS setup to context pin >> > > drm/i915: Move workaround infrastructure code up >> > > drm/i915: Save some lines of source code in workarounds >> > > >> > > Ville Syrjälä (33): >> > > drm/i915: Don't try to use the hardware frame counter with i965gm TV output >> > > drm/i915/tv: Fix interlaced ysize calculation >> > > drm/i915/tv: Fix tv mode clocks >> > > drm/i915/tv: Store the TV oversampling factor in the TV mode >> > > drm/i915/tv: Use bools where appropriate >> > > drm/i915/tv: Nuke silly 0 initialzation of xpos/ypos >> > > drm/i915/tv: Deobfuscate preferred mode selection >> > > drm/i915/tv: Use drm_mode_set_name() to name TV modes >> > > drm/i915/tv: Make TV mode autoselection actually useable >> > > drm/i915/tv: Nuke reported_modes[] >> > > drm/i915/tv: Add 1080p30/50/60 TV modes >> > > drm/i915/tv: Generate better pipe timings for TV encoder >> > > drm/i915/tv: Fix >1024 modes on gen3 >> > > drm/i915/tv: Filter out >1024 wide modes that would need vertical scaling on gen3 >> > > drm/i915: Clean up intel_plane_atomic_check_with_state() >> > > drm/i915/tv: Fix return value for intel_tv_compute_config() >> > > drm/i915/tv: Use the scanline counter for timestamps on i965gm TV output >> > > drm/i915: Try to sanitize bogus DPLL state left over by broken SNB BIOSen >> > > drm/i915: Fix skl srckey mask bits >> > > drm: Constify drm_color_lut_check() >> > > drm/i915/tv: Fix adjusted_mode dotclock for interlaced modes >> > > drm/i915/tv: Bypass the vertical filter if possible >> > > drm/i915: Don't ignore level 0 lines watermark for glk+ >> > > drm/i915: Reinstate an early latency==0 check for skl+ >> > > drm/i915: Fix bits vs. bytes mixup in dbuf block size computation >> > > drm/i915: Fix > vs >= mismatch in watermark/ddb calculations >> > > drm/i915: Account for minimum ddb allocation restrictions >> > > drm/i915: Pass dev_priv to skl_needs_memory_bw_wa() >> > > drm/i915: Drop the definite article in front of SAGV >> > > drm/i915: Drop the pointless linetime==0 check >> > > drm/i915: Use IS_GEN9_LP() for the linetime w/a check >> > > drm/i915: Don't use the second dbuf slice on icl >> > > drm/i915: Pick the first unused PLL once again >> > > >> > > .../bindings/display/bridge/lvds-transmitter.txt | 12 +- >> > > .../bindings/display/bridge/thine,thc63lvdm83d.txt | 2 +- >> > > .../bindings/display/bridge/ti,ds90c185.txt | 55 ++ >> > > Documentation/gpu/todo.rst | 35 - >> > > drivers/gpu/drm/Kconfig | 4 - >> > > drivers/gpu/drm/bridge/lvds-encoder.c | 53 +- >> > > drivers/gpu/drm/drm_color_mgmt.c | 5 +- >> > > drivers/gpu/drm/drm_damage_helper.c | 41 + >> > > drivers/gpu/drm/drm_dp_helper.c | 11 +- >> > > drivers/gpu/drm/drm_fb_cma_helper.c | 137 +--- >> > > drivers/gpu/drm/drm_fb_helper.c | 12 +- >> > > drivers/gpu/drm/drm_gem_framebuffer_helper.c | 50 +- >> > > drivers/gpu/drm/i915/Makefile | 1 + >> > > drivers/gpu/drm/i915/gvt/display.c | 31 + >> > > drivers/gpu/drm/i915/gvt/display.h | 37 +- >> > > drivers/gpu/drm/i915/gvt/gvt.c | 1 + >> > > drivers/gpu/drm/i915/gvt/gvt.h | 3 + >> > > drivers/gpu/drm/i915/gvt/hypercall.h | 1 + >> > > drivers/gpu/drm/i915/gvt/kvmgt.c | 143 ++++ >> > > drivers/gpu/drm/i915/gvt/mpt.h | 17 + >> > > drivers/gpu/drm/i915/gvt/vgpu.c | 6 + >> > > drivers/gpu/drm/i915/i915_debugfs.c | 162 +--- >> > > drivers/gpu/drm/i915/i915_drv.h | 21 +- >> > > drivers/gpu/drm/i915/i915_gem.c | 195 ++--- >> > > drivers/gpu/drm/i915/i915_gem_context.c | 3 + >> > > drivers/gpu/drm/i915/i915_gem_context.h | 2 + >> > > drivers/gpu/drm/i915/i915_gem_evict.c | 89 ++- >> > > drivers/gpu/drm/i915/i915_gem_execbuffer.c | 12 + >> > > drivers/gpu/drm/i915/i915_gem_fence_reg.h | 1 - >> > > drivers/gpu/drm/i915/i915_gem_gtt.c | 30 +- >> > > drivers/gpu/drm/i915/i915_gem_gtt.h | 53 +- >> > > drivers/gpu/drm/i915/i915_gem_object.h | 45 +- >> > > drivers/gpu/drm/i915/i915_gem_shrinker.c | 12 +- >> > > drivers/gpu/drm/i915/i915_gem_stolen.c | 5 +- >> > > drivers/gpu/drm/i915/i915_gpu_error.c | 231 ++---- >> > > drivers/gpu/drm/i915/i915_gpu_error.h | 40 +- >> > > drivers/gpu/drm/i915/i915_irq.c | 118 +-- >> > > drivers/gpu/drm/i915/i915_params.c | 6 +- >> > > drivers/gpu/drm/i915/i915_params.h | 2 +- >> > > drivers/gpu/drm/i915/i915_pci.c | 11 +- >> > > drivers/gpu/drm/i915/i915_reg.h | 62 +- >> > > drivers/gpu/drm/i915/i915_request.c | 204 ++--- >> > > drivers/gpu/drm/i915/i915_request.h | 134 +++- >> > > drivers/gpu/drm/i915/i915_reset.c | 500 ++++++------ >> > > drivers/gpu/drm/i915/i915_reset.h | 3 + >> > > drivers/gpu/drm/i915/i915_scheduler.c | 29 +- >> > > drivers/gpu/drm/i915/i915_selftest.h | 1 + >> > > drivers/gpu/drm/i915/i915_timeline.c | 253 +++++- >> > > drivers/gpu/drm/i915/i915_timeline.h | 27 +- >> > > drivers/gpu/drm/i915/i915_trace.h | 25 - >> > > drivers/gpu/drm/i915/i915_vma.c | 84 +- >> > > drivers/gpu/drm/i915/i915_vma.h | 47 +- >> > > drivers/gpu/drm/i915/intel_atomic_plane.c | 36 +- >> > > drivers/gpu/drm/i915/intel_bios.c | 7 + >> > > drivers/gpu/drm/i915/intel_breadcrumbs.c | 863 +++++---------------- >> > > drivers/gpu/drm/i915/intel_color.c | 60 +- >> > > drivers/gpu/drm/i915/intel_ddi.c | 20 +- >> > > drivers/gpu/drm/i915/intel_device_info.h | 2 + >> > > drivers/gpu/drm/i915/intel_display.c | 150 +++- >> > > drivers/gpu/drm/i915/intel_dpll_mgr.c | 99 ++- >> > > drivers/gpu/drm/i915/intel_dpll_mgr.h | 2 +- >> > > drivers/gpu/drm/i915/intel_drv.h | 4 +- >> > > drivers/gpu/drm/i915/intel_engine_cs.c | 259 ++++--- >> > > drivers/gpu/drm/i915/intel_gpu_commands.h | 1 - >> > > drivers/gpu/drm/i915/intel_guc_submission.c | 22 +- >> > > drivers/gpu/drm/i915/intel_hangcheck.c | 30 +- >> > > drivers/gpu/drm/i915/intel_lrc.c | 365 ++++++--- >> > > drivers/gpu/drm/i915/intel_mocs.c | 406 ++++++---- >> > > drivers/gpu/drm/i915/intel_overlay.c | 2 - >> > > drivers/gpu/drm/i915/intel_pm.c | 130 ++-- >> > > drivers/gpu/drm/i915/intel_ringbuffer.c | 286 ++++--- >> > > drivers/gpu/drm/i915/intel_ringbuffer.h | 166 ++-- >> > > drivers/gpu/drm/i915/intel_sprite.c | 2 +- >> > > drivers/gpu/drm/i915/intel_tv.c | 718 +++++++++++++---- >> > > drivers/gpu/drm/i915/intel_workarounds.c | 70 +- >> > > drivers/gpu/drm/i915/selftests/i915_gem_evict.c | 7 +- >> > > drivers/gpu/drm/i915/selftests/i915_gem_gtt.c | 5 +- >> > > .../gpu/drm/i915/selftests/i915_live_selftests.h | 1 + >> > > .../gpu/drm/i915/selftests/i915_mock_selftests.h | 3 +- >> > > drivers/gpu/drm/i915/selftests/i915_random.c | 33 +- >> > > drivers/gpu/drm/i915/selftests/i915_random.h | 3 + >> > > drivers/gpu/drm/i915/selftests/i915_request.c | 425 ++++++++++ >> > > drivers/gpu/drm/i915/selftests/i915_selftest.c | 47 ++ >> > > drivers/gpu/drm/i915/selftests/i915_timeline.c | 464 ++++++++++- >> > > drivers/gpu/drm/i915/selftests/i915_vma.c | 4 +- >> > > drivers/gpu/drm/i915/selftests/igt_live_test.c | 7 - >> > > drivers/gpu/drm/i915/selftests/igt_spinner.c | 5 - >> > > drivers/gpu/drm/i915/selftests/intel_breadcrumbs.c | 470 ----------- >> > > drivers/gpu/drm/i915/selftests/intel_hangcheck.c | 88 +-- >> > > drivers/gpu/drm/i915/selftests/intel_lrc.c | 138 ++++ >> > > drivers/gpu/drm/i915/selftests/intel_workarounds.c | 3 - >> > > drivers/gpu/drm/i915/selftests/lib_sw_fence.c | 54 ++ >> > > drivers/gpu/drm/i915/selftests/lib_sw_fence.h | 3 + >> > > drivers/gpu/drm/i915/selftests/mock_engine.c | 58 +- >> > > drivers/gpu/drm/i915/selftests/mock_engine.h | 6 - >> > > drivers/gpu/drm/i915/selftests/mock_gem_device.c | 11 +- >> > > drivers/gpu/drm/i915/selftests/mock_timeline.c | 3 +- >> > > drivers/gpu/drm/panel/panel-innolux-p079zca.c | 11 +- >> > > drivers/gpu/drm/stm/ltdc.c | 4 +- >> > > drivers/gpu/drm/sun4i/sun4i_backend.c | 25 +- >> > > drivers/gpu/drm/sun4i/sun4i_drv.c | 1 + >> > > drivers/gpu/drm/sun4i/sun4i_frontend.c | 350 ++++++++- >> > > drivers/gpu/drm/sun4i/sun4i_frontend.h | 52 ++ >> > > drivers/gpu/drm/sun4i/sun4i_layer.c | 28 +- >> > > drivers/gpu/drm/tinydrm/core/tinydrm-core.c | 21 +- >> > > drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c | 100 +-- >> > > drivers/gpu/drm/tinydrm/core/tinydrm-pipe.c | 31 - >> > > drivers/gpu/drm/tinydrm/hx8357d.c | 2 +- >> > > drivers/gpu/drm/tinydrm/ili9225.c | 149 ++-- >> > > drivers/gpu/drm/tinydrm/ili9341.c | 2 +- >> > > drivers/gpu/drm/tinydrm/mi0283qt.c | 2 +- >> > > drivers/gpu/drm/tinydrm/mipi-dbi.c | 94 ++- >> > > drivers/gpu/drm/tinydrm/repaper.c | 43 +- >> > > drivers/gpu/drm/tinydrm/st7586.c | 86 +- >> > > drivers/gpu/drm/tinydrm/st7735r.c | 2 +- >> > > include/drm/drm_color_mgmt.h | 3 +- >> > > include/drm/drm_damage_helper.h | 3 + >> > > include/drm/drm_fb_cma_helper.h | 22 - >> > > include/drm/drm_fourcc.h | 117 +++ >> > > include/drm/drm_gem_framebuffer_helper.h | 3 + >> > > include/drm/drm_util.h | 2 +- >> > > include/drm/i915_pciids.h | 4 + >> > > include/drm/tinydrm/mipi-dbi.h | 5 +- >> > > include/drm/tinydrm/tinydrm-helpers.h | 20 +- >> > > include/drm/tinydrm/tinydrm.h | 26 - >> > > include/linux/dma-fence-array.h | 1 + >> > > include/uapi/drm/drm_fourcc.h | 16 + >> > > 127 files changed, 5732 insertions(+), 4095 deletions(-) >> > > create mode 100644 Documentation/devicetree/bindings/display/bridge/ti,ds90c185.txt >> > > delete mode 100644 drivers/gpu/drm/i915/selftests/intel_breadcrumbs.c -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel