Re: [Intel-gfx] [RFC PATCH 00/18] Generic DRRS implementation across the encoders
On Tue, Jun 30, 2015 at 11:59:30AM +0530, Ramalingam C wrote: > > On Monday 29 June 2015 09:57 PM, Daniel Vetter wrote: > >On Mon, Jun 29, 2015 at 04:52:14PM +0530, Ramalingam C wrote: > >>On Friday 26 June 2015 10:46 PM, Daniel Vetter wrote: > >>>On Fri, Jun 26, 2015 at 07:21:44PM +0530, Ramalingam C wrote: > >>>- Static DRRS and generalized seamless DRRS are imo separate features and > >>> we should split the patch series. seamless DRRS is already implemented > >>> using the fb tracking, maybe extended with some hints to userspace. > >>> > >>> Static DRRS otoh is just a modeset with a different clock (plus a better > >>> internal implementation to avoid flicker). So from that pov two > >>> completely different features, both in the implementation and the > >>> userspace ABI. > >>Yup. Static is not even in our development radar :). All the code that I > >>have shared is > >>concerned only with seamless DRRS and it's two usecase scenarios( Idleness > >>and Content based). > >>Mentioned the Static DRRS in cover letter, just to give the two different > >>DRRS supports available in general. > >Hm then I'm confused how the content based DRRS is supposed to work. I > >thought userspace requires a precise vrefresh rate (adjusted to content, > >within the limits of what the panel can do ofc) and the kernel tries to > >obey. That's what I consider static DRRS, i.e. userspace makes a fixed > >request, kernel executes it exactly. > If panel does support the static DRRS only and you are ok to do a complete > modeset on a usecase then > Whatever you have explained stands true for content based DRRS. I believe > its another complete modeset w.r.t kernel. > Which I am not addressing in this RFC. > > But If the panel supports seamless DRRS and userspace want a precise > vrefresh rate (adjusted to the > content, within the limits of what panel can do) then kernel can achieve the > vrefresh change seamlessly > (the way its done with Idleness scenario). This is the second usecase of > seamless DRRS support of the panel. > Infact We have implemented this and verified on android already for video > playback. > > > >Seamless DRRS for me is when the kernel tries to change vrefresh when > >everything is idle behind everyone's back. > Seamless DRRS is a capability of the host and the panel. Usercase definition > is upto us to do. > > > >I'm doing this split since seamless doesn't require new userspace > >interfaces (we just use the frontbuffer tracking system), whereas static > >needs explicit action from userspace and hence the dreaded userspace ABI > >problem starts to kick in. > Based on above usecase of the seamless drrs we need a fast modeset path > which I have addressed at the RFC PATCH 16/18. > And drm connector property to expose the SEAMLESS DRRS capability to the > userspace. implemented at RFC PATCH 18/18 Summarizing our discussion from the mtg, there's 3 different cases here: - transparent seamless DRRS where the kernel lowers vrefresh on idle behind userspace's back. This is what we currently have integrated. No new ABI required. - seamless DRRS but upon explicit request from userspace. This is essentially just a mode change, but excuted by the kernel without any blanking or otherwise stalling. We can reuse the modeset ioctls for this, just need a fast path (similar to the fast pfit update Maarten already has in his atomic branches). For figuring out whether seamless is possible we could reuse the atomic TEST_ONLY and ALLOW_MODESET flags. Good chances that we don't need a new ABI. - just changing the vrefresh of the panel, with full modeset/blanking. We have the interface for that (we can do modesets), but currently our panel compute_config code hardcodes the vrefresh to the preferred mode of the panel. We need to change that and pick the vrefresh userspace asked for. I think from an implementation pov it makes sense to first fix up this case, and then in a 2nd step implement the seamless DRRS for explicit mode changes. Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [RFC PATCH 00/18] Generic DRRS implementation across the encoders
On Monday 29 June 2015 09:57 PM, Daniel Vetter wrote: On Mon, Jun 29, 2015 at 04:52:14PM +0530, Ramalingam C wrote: On Friday 26 June 2015 10:46 PM, Daniel Vetter wrote: On Fri, Jun 26, 2015 at 07:21:44PM +0530, Ramalingam C wrote: - Static DRRS and generalized seamless DRRS are imo separate features and we should split the patch series. seamless DRRS is already implemented using the fb tracking, maybe extended with some hints to userspace. Static DRRS otoh is just a modeset with a different clock (plus a better internal implementation to avoid flicker). So from that pov two completely different features, both in the implementation and the userspace ABI. Yup. Static is not even in our development radar :). All the code that I have shared is concerned only with seamless DRRS and it's two usecase scenarios( Idleness and Content based). Mentioned the Static DRRS in cover letter, just to give the two different DRRS supports available in general. Hm then I'm confused how the content based DRRS is supposed to work. I thought userspace requires a precise vrefresh rate (adjusted to content, within the limits of what the panel can do ofc) and the kernel tries to obey. That's what I consider static DRRS, i.e. userspace makes a fixed request, kernel executes it exactly. If panel does support the static DRRS only and you are ok to do a complete modeset on a usecase then Whatever you have explained stands true for content based DRRS. I believe its another complete modeset w.r.t kernel. Which I am not addressing in this RFC. But If the panel supports seamless DRRS and userspace want a precise vrefresh rate (adjusted to the content, within the limits of what panel can do) then kernel can achieve the vrefresh change seamlessly (the way its done with Idleness scenario). This is the second usecase of seamless DRRS support of the panel. Infact We have implemented this and verified on android already for video playback. Seamless DRRS for me is when the kernel tries to change vrefresh when everything is idle behind everyone's back. Seamless DRRS is a capability of the host and the panel. Usercase definition is upto us to do. I'm doing this split since seamless doesn't require new userspace interfaces (we just use the frontbuffer tracking system), whereas static needs explicit action from userspace and hence the dreaded userspace ABI problem starts to kick in. Based on above usecase of the seamless drrs we need a fast modeset path which I have addressed at the RFC PATCH 16/18. And drm connector property to expose the SEAMLESS DRRS capability to the userspace. implemented at RFC PATCH 18/18 Do you have different definitions? What exactly are those? -Daniel -- Thanks, --Ram ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [RFC PATCH 00/18] Generic DRRS implementation across the encoders
On Mon, Jun 29, 2015 at 04:52:14PM +0530, Ramalingam C wrote: > On Friday 26 June 2015 10:46 PM, Daniel Vetter wrote: > >On Fri, Jun 26, 2015 at 07:21:44PM +0530, Ramalingam C wrote: > >- Static DRRS and generalized seamless DRRS are imo separate features and > > we should split the patch series. seamless DRRS is already implemented > > using the fb tracking, maybe extended with some hints to userspace. > > > > Static DRRS otoh is just a modeset with a different clock (plus a better > > internal implementation to avoid flicker). So from that pov two > > completely different features, both in the implementation and the > > userspace ABI. > Yup. Static is not even in our development radar :). All the code that I > have shared is > concerned only with seamless DRRS and it's two usecase scenarios( Idleness > and Content based). > Mentioned the Static DRRS in cover letter, just to give the two different > DRRS supports available in general. Hm then I'm confused how the content based DRRS is supposed to work. I thought userspace requires a precise vrefresh rate (adjusted to content, within the limits of what the panel can do ofc) and the kernel tries to obey. That's what I consider static DRRS, i.e. userspace makes a fixed request, kernel executes it exactly. Seamless DRRS for me is when the kernel tries to change vrefresh when everything is idle behind everyone's back. I'm doing this split since seamless doesn't require new userspace interfaces (we just use the frontbuffer tracking system), whereas static needs explicit action from userspace and hence the dreaded userspace ABI problem starts to kick in. Do you have different definitions? What exactly are those? -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [RFC PATCH 00/18] Generic DRRS implementation across the encoders
On Friday 26 June 2015 10:46 PM, Daniel Vetter wrote: On Fri, Jun 26, 2015 at 07:21:44PM +0530, Ramalingam C wrote: Display Refresh Rate Switching (DRRS) is a power conservation feature which enables swtching between low and high refresh rates, dynamically, based on the usage scenario to save power. This feature is applicable for internal panels. Indication that the panel supports DRRS is given by the panel EDID, which would list multiple refresh rates for one resolution along with VBT fields like drrs_min_vrefresh. DRRS is of 2 types - static and seamless. Static needs full modeset. Which can be used in docking and undocking scenarios. Seamless DRRS involves changing RR without any visual effect to the user and can be used during normal system usage. This is done by programming certain registers. Support for static/seamless DRRS may be indicated in the VBT based on inputs from the panel spec. We have implemented two DRRS use case scenrios : 1. Idleness DRRS, 2. Media playback DRRS (Content's FPS based) Idleness DRRS:- The implementation is based on frontbuffer tracking implementation. When there is a disturbance on the screen triggered by user activity or a periodic system activity, DRRS is disabled (RR is changed to high RR). When there is no movement on screen, after a timeout, a switch to low RR is made. For integration with frontbuffer tracking code, intel_drrs_invalidate() and intel_drrs_flush() are called. Media playback DRRS (Content's FPS based): Based on the content's Frames Per Second, minimum required vrefresh is called in userspace. If the panel supports the minimum required vrefresh and if the DRRS is supported on the platform, userspace will make a modeset request with same mode and FB but with different vrefresh. Panel's vrefresh range will be exposed through connector's modelist and connector property is created to expose the DRRS capability to the userspace. In DRRS state machine Media plyabck DRRS holds the highest priority. Idleness DRRS is disabled as soon as Media playback DRRS is requested. Idleness DRRS is reenabled when the userspace indicates the exit of Media playback DRRS by requesting for highest vrefresh supported. NOTE: 1. DRM documentation will be made once the RFC is closed along with IGT. 2. Currently in RFC, platform support is developed only for VLV. it will be extended to other platform once the design is approved. 3. Basic design of DSI DRRS and Content based DRRS is functionally tested on Android(VLV & CHT). Daniel. First of all Thank you so much for your comments. Sorry for such a huge patch series. Tolerate me on that one. I've made a few drive-by comments, but big picture comments repeated here: - Don't throw existing code away as in patch 1. When extending a feature/code, we'll refactor the existing stuff and keep it working all the time. Otherwise we invalidate all the previous testing, which throws out a lot of value. Ok agreed. I will work on this comment. And will bring the current existing eDP DRRS code to the generic design, retaining the bug fixes we have done. - Static DRRS and generalized seamless DRRS are imo separate features and we should split the patch series. seamless DRRS is already implemented using the fb tracking, maybe extended with some hints to userspace. Static DRRS otoh is just a modeset with a different clock (plus a better internal implementation to avoid flicker). So from that pov two completely different features, both in the implementation and the userspace ABI. Yup. Static is not even in our development radar :). All the code that I have shared is concerned only with seamless DRRS and it's two usecase scenarios( Idleness and Content based). Mentioned the Static DRRS in cover letter, just to give the two different DRRS supports available in general. - I guess we'll have a proper design discussion in internal mtgs about this too? Sure Daniel we will have one. Thanks, Daniel Ramalingam C (18): drm/i915: Removing the eDP specific DRRS implementation drm/i915: Generic DRRS state Machine drm/i915: Addition of the drrs_min_vrefresh in VBT drm/i915: Implementation of Generic DSI DRRS drm/i915: Adjusting the pclk for dual link and burst mode drm/i915: VLV dsi drrs support drm/i915: Generic eDP DRRS implementation drm/i915: VLV eDP DRRS methods drm/i915: Cloned mode check drm/i915: Initializing DRRS for all connectors drm/i915: Updating the crtc modes in DRRS transitions drm/i915: Redesigning dp_set_m_n to receive divider values drm/i915: MEDIA_RR support in general DRRS state machine drm/i915: MEDIA_RR support in eDP DRRS module drm/i915: MEDIA_RR support in DSI DRRS module drm/i915: Filtering media playback DRRS requests drm/i915: Addition of downclock mode to connector modelist drm/i915: Connector property for DRRS capability drivers/gpu/drm/i915/Makefile|3
Re: [Intel-gfx] [RFC PATCH 00/18] Generic DRRS implementation across the encoders
On Fri, Jun 26, 2015 at 07:21:44PM +0530, Ramalingam C wrote: > Display Refresh Rate Switching (DRRS) is a power conservation feature > which enables swtching between low and high refresh rates, > dynamically, based on the usage scenario to save power. > > This feature is applicable for internal panels. > > Indication that the panel supports DRRS is given by the panel EDID, which > would list multiple refresh rates for one resolution along with VBT fields > like drrs_min_vrefresh. > > DRRS is of 2 types - static and seamless. > Static needs full modeset. Which can be used in docking and undocking > scenarios. > Seamless DRRS involves changing RR without any visual effect to the user > and can be used during normal system usage. This is done by programming > certain registers. > > Support for static/seamless DRRS may be indicated in the VBT based on > inputs from the panel spec. > > We have implemented two DRRS use case scenrios : > 1. Idleness DRRS, 2. Media playback DRRS (Content's FPS based) > > Idleness DRRS:- > The implementation is based on frontbuffer tracking implementation. > When there is a disturbance on the screen triggered by user activity or a > periodic system activity, DRRS is disabled (RR is changed to high RR). > When there is no movement on screen, after a timeout, a switch > to low RR is made. > For integration with frontbuffer tracking code, > intel_drrs_invalidate() and intel_drrs_flush() are called. > > Media playback DRRS (Content's FPS based): > Based on the content's Frames Per Second, minimum required vrefresh > is called in userspace. If the panel supports the minimum required vrefresh > and if the DRRS is supported on the platform, userspace will make a modeset > request with same mode and FB but with different vrefresh. > Panel's vrefresh range will be exposed through connector's modelist and > connector property is created to expose the DRRS capability to the userspace. > > In DRRS state machine Media plyabck DRRS holds the highest priority. > Idleness DRRS is disabled as soon as Media playback DRRS is requested. > Idleness DRRS is reenabled when the userspace indicates the exit of Media > playback DRRS by requesting for highest vrefresh supported. > > NOTE: > 1. DRM documentation will be made once the RFC is closed along with IGT. > 2. Currently in RFC, platform support is developed only for VLV. >it will be extended to other platform once the design is approved. > 3. Basic design of DSI DRRS and Content based DRRS is functionally tested on >Android(VLV & CHT). I've made a few drive-by comments, but big picture comments repeated here: - Don't throw existing code away as in patch 1. When extending a feature/code, we'll refactor the existing stuff and keep it working all the time. Otherwise we invalidate all the previous testing, which throws out a lot of value. - Static DRRS and generalized seamless DRRS are imo separate features and we should split the patch series. seamless DRRS is already implemented using the fb tracking, maybe extended with some hints to userspace. Static DRRS otoh is just a modeset with a different clock (plus a better internal implementation to avoid flicker). So from that pov two completely different features, both in the implementation and the userspace ABI. - I guess we'll have a proper design discussion in internal mtgs about this too? Thanks, Daniel > > Ramalingam C (18): > drm/i915: Removing the eDP specific DRRS implementation > drm/i915: Generic DRRS state Machine > drm/i915: Addition of the drrs_min_vrefresh in VBT > drm/i915: Implementation of Generic DSI DRRS > drm/i915: Adjusting the pclk for dual link and burst mode > drm/i915: VLV dsi drrs support > drm/i915: Generic eDP DRRS implementation > drm/i915: VLV eDP DRRS methods > drm/i915: Cloned mode check > drm/i915: Initializing DRRS for all connectors > drm/i915: Updating the crtc modes in DRRS transitions > drm/i915: Redesigning dp_set_m_n to receive divider values > drm/i915: MEDIA_RR support in general DRRS state machine > drm/i915: MEDIA_RR support in eDP DRRS module > drm/i915: MEDIA_RR support in DSI DRRS module > drm/i915: Filtering media playback DRRS requests > drm/i915: Addition of downclock mode to connector modelist > drm/i915: Connector property for DRRS capability > > drivers/gpu/drm/i915/Makefile|3 + > drivers/gpu/drm/i915/i915_debugfs.c | 110 - > drivers/gpu/drm/i915/i915_drv.h | 91 +++- > drivers/gpu/drm/i915/i915_reg.h |4 + > drivers/gpu/drm/i915/intel_bios.c|3 + > drivers/gpu/drm/i915/intel_bios.h|9 +- > drivers/gpu/drm/i915/intel_ddi.c |2 - > drivers/gpu/drm/i915/intel_display.c | 57 ++- > drivers/gpu/drm/i915/intel_dp.c | 397 +--- > drivers/gpu/drm/i915/intel_drrs.c| 728 > ++ > drivers/gpu/drm/i915
[Intel-gfx] [RFC PATCH 00/18] Generic DRRS implementation across the encoders
Display Refresh Rate Switching (DRRS) is a power conservation feature which enables swtching between low and high refresh rates, dynamically, based on the usage scenario to save power. This feature is applicable for internal panels. Indication that the panel supports DRRS is given by the panel EDID, which would list multiple refresh rates for one resolution along with VBT fields like drrs_min_vrefresh. DRRS is of 2 types - static and seamless. Static needs full modeset. Which can be used in docking and undocking scenarios. Seamless DRRS involves changing RR without any visual effect to the user and can be used during normal system usage. This is done by programming certain registers. Support for static/seamless DRRS may be indicated in the VBT based on inputs from the panel spec. We have implemented two DRRS use case scenrios : 1. Idleness DRRS, 2. Media playback DRRS (Content's FPS based) Idleness DRRS:- The implementation is based on frontbuffer tracking implementation. When there is a disturbance on the screen triggered by user activity or a periodic system activity, DRRS is disabled (RR is changed to high RR). When there is no movement on screen, after a timeout, a switch to low RR is made. For integration with frontbuffer tracking code, intel_drrs_invalidate() and intel_drrs_flush() are called. Media playback DRRS (Content's FPS based): Based on the content's Frames Per Second, minimum required vrefresh is called in userspace. If the panel supports the minimum required vrefresh and if the DRRS is supported on the platform, userspace will make a modeset request with same mode and FB but with different vrefresh. Panel's vrefresh range will be exposed through connector's modelist and connector property is created to expose the DRRS capability to the userspace. In DRRS state machine Media plyabck DRRS holds the highest priority. Idleness DRRS is disabled as soon as Media playback DRRS is requested. Idleness DRRS is reenabled when the userspace indicates the exit of Media playback DRRS by requesting for highest vrefresh supported. NOTE: 1. DRM documentation will be made once the RFC is closed along with IGT. 2. Currently in RFC, platform support is developed only for VLV. it will be extended to other platform once the design is approved. 3. Basic design of DSI DRRS and Content based DRRS is functionally tested on Android(VLV & CHT). Ramalingam C (18): drm/i915: Removing the eDP specific DRRS implementation drm/i915: Generic DRRS state Machine drm/i915: Addition of the drrs_min_vrefresh in VBT drm/i915: Implementation of Generic DSI DRRS drm/i915: Adjusting the pclk for dual link and burst mode drm/i915: VLV dsi drrs support drm/i915: Generic eDP DRRS implementation drm/i915: VLV eDP DRRS methods drm/i915: Cloned mode check drm/i915: Initializing DRRS for all connectors drm/i915: Updating the crtc modes in DRRS transitions drm/i915: Redesigning dp_set_m_n to receive divider values drm/i915: MEDIA_RR support in general DRRS state machine drm/i915: MEDIA_RR support in eDP DRRS module drm/i915: MEDIA_RR support in DSI DRRS module drm/i915: Filtering media playback DRRS requests drm/i915: Addition of downclock mode to connector modelist drm/i915: Connector property for DRRS capability drivers/gpu/drm/i915/Makefile|3 + drivers/gpu/drm/i915/i915_debugfs.c | 110 - drivers/gpu/drm/i915/i915_drv.h | 91 +++- drivers/gpu/drm/i915/i915_reg.h |4 + drivers/gpu/drm/i915/intel_bios.c|3 + drivers/gpu/drm/i915/intel_bios.h|9 +- drivers/gpu/drm/i915/intel_ddi.c |2 - drivers/gpu/drm/i915/intel_display.c | 57 ++- drivers/gpu/drm/i915/intel_dp.c | 397 +--- drivers/gpu/drm/i915/intel_drrs.c| 728 ++ drivers/gpu/drm/i915/intel_drrs.h| 38 ++ drivers/gpu/drm/i915/intel_drv.h | 45 +- drivers/gpu/drm/i915/intel_dsi.c | 29 +- drivers/gpu/drm/i915/intel_dsi.h |3 + drivers/gpu/drm/i915/intel_dsi_drrs.c| 415 + drivers/gpu/drm/i915/intel_dsi_drrs.h| 66 +++ drivers/gpu/drm/i915/intel_dsi_pll.c | 303 +++-- drivers/gpu/drm/i915/intel_edp_drrs.c| 196 drivers/gpu/drm/i915/intel_edp_drrs.h| 29 ++ drivers/gpu/drm/i915/intel_frontbuffer.c |4 +- drivers/gpu/drm/i915/intel_modes.c | 28 ++ 21 files changed, 1959 insertions(+), 601 deletions(-) create mode 100644 drivers/gpu/drm/i915/intel_drrs.c create mode 100644 drivers/gpu/drm/i915/intel_drrs.h create mode 100644 drivers/gpu/drm/i915/intel_dsi_drrs.c create mode 100644 drivers/gpu/drm/i915/intel_dsi_drrs.h create mode 100644 drivers/gpu/drm/i915/intel_edp_drrs.c create mode 100644 drivers/gpu/drm/i915/intel_edp_drrs.h -- 1.7.9.5 ___ Intel-gfx mailing list Intel-gfx@li