| Message ID | 1535395832-32264-1-git-send-email-abhay.kumar@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v1] drm/i915: Skip modeset for cdclk changes if possible | expand |
On Mon, Aug 27, 2018 at 11:50:32AM -0700, Abhay Kumar wrote: > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > If we have only a single active pipe and the cdclk change only requires > the cd2x divider to be updated bxt+ can do the update with forcing a full > modeset on the pipe. Try to hook that up. > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > Signed-off-by: Abhay Kumar <abhay.kumar@intel.com> > --- > drivers/gpu/drm/i915/i915_drv.h | 3 +- > drivers/gpu/drm/i915/i915_reg.h | 3 +- > drivers/gpu/drm/i915/intel_cdclk.c | 105 +++++++++++++++++++++++++---------- > drivers/gpu/drm/i915/intel_display.c | 20 ++++++- > drivers/gpu/drm/i915/intel_drv.h | 9 ++- > 5 files changed, 105 insertions(+), 35 deletions(-) > <snip> > @@ -12252,12 +12253,24 @@ static int intel_modeset_checks(struct drm_atomic_state *state) > return ret; > } > > + > /* All pipes must be switched off while we change the cdclk. */ > - if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, > - &intel_state->cdclk.actual)) { > + if (is_power_of_2(intel_state->active_crtcs) && > + intel_cdclk_needs_cd2x_update(dev_priv, > + &dev_priv->cdclk.actual, > + &intel_state->cdclk.actual)) { > + ret = intel_lock_all_pipes(state); > + if (ret < 0) > + return ret; > + > + intel_state->cdclk.pipe = ilog2(intel_state->active_crtcs); BTW on further reflection this probably isn't quite sufficient. Let's say we have a commit with allow_modeset=true, but we aren't actually required to do a modeset based on any of the state changes. If we still have to change cdclk we should actually be doing the cd2x update atomically with the plane updates, or we should do it before or after the plane updates depending on whether the cdclk freq is going up or down. Doing the update atomically with the plane updates might be nicer in the end, but for that we would likely need to split the .set_cdclk() hooks into three parts (pre+commit+post). Whereas just doing the update before or after the plane updates as needed would probably be a little simpler. > + } else if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, > + &intel_state->cdclk.actual)) { > ret = intel_modeset_all_pipes(state); > if (ret < 0) > return ret; > + > + intel_state->cdclk.pipe = INVALID_PIPE; > } > > DRM_DEBUG_KMS("New cdclk calculated to be logical %u kHz, actual %u kHz\n",
On 8/28/2018 5:39 AM, Ville Syrjälä wrote: > On Mon, Aug 27, 2018 at 11:50:32AM -0700, Abhay Kumar wrote: >> From: Ville Syrjälä <ville.syrjala@linux.intel.com> >> >> If we have only a single active pipe and the cdclk change only requires >> the cd2x divider to be updated bxt+ can do the update with forcing a full >> modeset on the pipe. Try to hook that up. >> >> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> >> Signed-off-by: Abhay Kumar <abhay.kumar@intel.com> >> --- >> drivers/gpu/drm/i915/i915_drv.h | 3 +- >> drivers/gpu/drm/i915/i915_reg.h | 3 +- >> drivers/gpu/drm/i915/intel_cdclk.c | 105 +++++++++++++++++++++++++---------- >> drivers/gpu/drm/i915/intel_display.c | 20 ++++++- >> drivers/gpu/drm/i915/intel_drv.h | 9 ++- >> 5 files changed, 105 insertions(+), 35 deletions(-) >> > <snip> >> @@ -12252,12 +12253,24 @@ static int intel_modeset_checks(struct drm_atomic_state *state) >> return ret; >> } >> >> + >> /* All pipes must be switched off while we change the cdclk. */ >> - if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, >> - &intel_state->cdclk.actual)) { >> + if (is_power_of_2(intel_state->active_crtcs) && >> + intel_cdclk_needs_cd2x_update(dev_priv, >> + &dev_priv->cdclk.actual, >> + &intel_state->cdclk.actual)) { >> + ret = intel_lock_all_pipes(state); >> + if (ret < 0) >> + return ret; >> + >> + intel_state->cdclk.pipe = ilog2(intel_state->active_crtcs); > BTW on further reflection this probably isn't quite sufficient. Let's > say we have a commit with allow_modeset=true, but we aren't actually > required to do a modeset based on any of the state changes. If we still > have to change cdclk we should actually be doing the cd2x update > atomically with the plane updates, or we should do it before or after > the plane updates depending on whether the cdclk freq is going up or > down. > > Doing the update atomically with the plane updates might be nicer in the > end, but for that we would likely need to split the .set_cdclk() hooks > into three parts (pre+commit+post). Whereas just doing the update before > or after the plane updates as needed would probably be a little simpler. Yeah. That might also get rid of cdclk mismatch warning during multiple suspend resume cycle. [280.600259] cdclk state doesn't match! [280.600270] calling1-8+ @ 3110, parent: usb1, cb: usb_dev_resume [280.600276] WARNING: CPU: 3 PID: 5224 at /mnt/host/source/src/third_party/ker nel/v4.14/drivers/gpu/drm/i915/intel_cdclk.c:1867 intel_set_cdclk+0xaa/0xdb [280.600277] Modules linked in: cmac rfcomm uinput snd_soc_sst_bxt_da7219_max9 8357a snd_soc_hdac_hdmi snd_soc_dmic lzo lzo_compress snd_soc_skl snd_soc_skl_ip c snd_soc_sst_ipc snd_soc_sst_dsp snd_soc_acpi snd_hda_ext_core snd_hda_core [280.600307] callingphy0+ @ 3102, parent: 0000:00:0c.0, cb: wiphy_resume [cf g80211] [280.600308]zram snd_soc_max98357a acpi_als snd_soc_da7219 bridge stp llc ip t_MASQUERADE nf_nat_masquerade_ipv4 xt_mark fuse snd_seq_dummy snd_seq snd_seq_d evice btusb btrtl btbcm iio_trig_sysfs btintel uvcvideo bluetooth videobuf2_vmal loc videobuf2_memops videobuf2_v4l2 ecdh_generic videobuf2_core cros_ec_sensors cros_ec_sensors_ring cros_ec_sensors_core industrialio_triggered_buffer kfifo_bu [280.600346] RDX: ffffffffb8258dd0 RSI: 0000000000000002 RDI: ffffffffb8258db0 [280.600347] RBP: ffffbb9546b73aa0 R08: 0000000000000000 R09: 0000000000000000 [280.600348] R10: 0000000000000000 R11: ffffffffb86d8518 R12: ffffa22075790000 [280.600349] R13: ffffa22075793d24 R14: 00000000ffffffff R15: ffffa2202eec9800 00000000000 [280.600352] CS:0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [280.600353] CR2: 00007f43c04d0e50 CR3: 0000000224112000 CR4: 00000000003406e0 [280.600354] Call Trace: [280.600361]intel_atomic_commit_tail+0x20a/0xacb [280.600363]? intel_atomic_commit_ready+0x44/0x4c [280.600365]intel_atomic_commit+0x227/0x238 [280.600368]glk_force_audio_cdclk+0x9f/0x119 [280.600370]i915_audio_component_get_power+0x3e/0x4d [280.600376]snd_hdac_display_power+0x53/0x97 [snd_hda_core] [280.600379] calling1-9+ @ 3086, parent: usb1, cb: usb_dev_resume [280.600384]skl_resume+0x3a/0x17a [snd_soc_skl] [280.600387]? pci_pm_suspend_noirq+0x1e9/0x1e9 [280.600391]dpm_run_callback+0x59/0xbf [280.600394]device_resume+0x192/0x1d4 [280.600396]dpm_resume+0x145/0x1da [280.600398]dpm_resume_end+0x11/0x1a [280.600403]suspend_devices_and_enter+0x354/0x5c2 [280.600407]? remove_wait_queue+0x51/0x51 [280.600409]pm_suspend+0x29c/0x2e2 [280.600411]state_store+0xa2/0xcb [280.600415]kernfs_fop_write+0x103/0x14a [280.600420]__vfs_write+0x37/0xd0 [280.600424]? inode_security+0x19/0x20 [280.600426]? selinux_file_permission+0x78/0xad [280.600428]vfs_write+0xb9/0xfd [280.600430]SyS_write+0x5f/0xa3 [280.600434]do_syscall_64+0x64/0x72 [280.600438]entry_SYSCALL_64_after_hwframe+0x3d/0xa2 [280.600441] RIP: 0033:0x7f969d8af3b0 00001 [280.600443] RAX: ffffffffffffffda RBX: 00007f969ddf0000 RCX: 00007f969d8af3b0 [280.600444] RDX: 0000000000000006 RSI: 00007f969ddf0000 RDI: 0000000000000001 [280.600445] RBP: 00007ffd9e3f4880 R08: ffffffffffffffff R09: 0000000000000000 [280.600446] R10: 0000000000001000 R11: 0000000000000246 R12: 0000000000000006 [280.600447] R13: 0000000000020000 R14: 00007f969ddf0000 R15: 0000000000000001 > > >> + } else if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, >> + &intel_state->cdclk.actual)) { >> ret = intel_modeset_all_pipes(state); >> if (ret < 0) >> return ret; >> + >> + intel_state->cdclk.pipe = INVALID_PIPE; >> } >> >> DRM_DEBUG_KMS("New cdclk calculated to be logical %u kHz, actual %u kHz\n", <html> <head> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> <p><br> </p> <br> <div class="moz-cite-prefix">On 8/28/2018 5:39 AM, Ville Syrjälä wrote:<br> </div> <blockquote cite="mid:20180828123931.GV5565@intel.com" type="cite"> <pre wrap="">On Mon, Aug 27, 2018 at 11:50:32AM -0700, Abhay Kumar wrote: </pre> <blockquote type="cite"> <pre wrap="">From: Ville Syrjälä <a class="moz-txt-link-rfc2396E" href="mailto:ville.syrjala@linux.intel.com"><ville.syrjala@linux.intel.com></a> If we have only a single active pipe and the cdclk change only requires the cd2x divider to be updated bxt+ can do the update with forcing a full modeset on the pipe. Try to hook that up. Signed-off-by: Ville Syrjälä <a class="moz-txt-link-rfc2396E" href="mailto:ville.syrjala@linux.intel.com"><ville.syrjala@linux.intel.com></a> Signed-off-by: Abhay Kumar <a class="moz-txt-link-rfc2396E" href="mailto:abhay.kumar@intel.com"><abhay.kumar@intel.com></a> --- drivers/gpu/drm/i915/i915_drv.h | 3 +- drivers/gpu/drm/i915/i915_reg.h | 3 +- drivers/gpu/drm/i915/intel_cdclk.c | 105 +++++++++++++++++++++++++---------- drivers/gpu/drm/i915/intel_display.c | 20 ++++++- drivers/gpu/drm/i915/intel_drv.h | 9 ++- 5 files changed, 105 insertions(+), 35 deletions(-) </pre> </blockquote> <pre wrap=""><snip> </pre> <blockquote type="cite"> <pre wrap="">@@ -12252,12 +12253,24 @@ static int intel_modeset_checks(struct drm_atomic_state *state) return ret; } + /* All pipes must be switched off while we change the cdclk. */ - if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, - &intel_state->cdclk.actual)) { + if (is_power_of_2(intel_state->active_crtcs) && + intel_cdclk_needs_cd2x_update(dev_priv, + &dev_priv->cdclk.actual, + &intel_state->cdclk.actual)) { + ret = intel_lock_all_pipes(state); + if (ret < 0) + return ret; + + intel_state->cdclk.pipe = ilog2(intel_state->active_crtcs); </pre> </blockquote> <pre wrap=""> BTW on further reflection this probably isn't quite sufficient. Let's say we have a commit with allow_modeset=true, but we aren't actually required to do a modeset based on any of the state changes. If we still have to change cdclk we should actually be doing the cd2x update atomically with the plane updates, or we should do it before or after the plane updates depending on whether the cdclk freq is going up or down. Doing the update atomically with the plane updates might be nicer in the end, but for that we would likely need to split the .set_cdclk() hooks into three parts (pre+commit+post). Whereas just doing the update before or after the plane updates as needed would probably be a little simpler.</pre> </blockquote> Yeah. That might also get rid of cdclk mismatch warning during multiple suspend resume cycle.<br> <br> <meta http-equiv="Content-Type" content="text/html; charset=windows-1252"> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600259] cdclk state doesn't match!<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600270] calling<span style="mso-spacerun:yes"> </span>1-8+ @ 3110, parent: usb1, cb: usb_dev_resume<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600276] WARNING: CPU: 3 PID: 5224 at /mnt/host/source/src/third_party/ker<o:p></o:p></p> <p class="MsoPlainText">nel/v4.14/drivers/gpu/drm/i915/intel_cdclk.c:1867 intel_set_cdclk+0xaa/0xdb<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600277] Modules linked in: cmac rfcomm uinput snd_soc_sst_bxt_da7219_max9<o:p></o:p></p> <p class="MsoPlainText">8357a snd_soc_hdac_hdmi snd_soc_dmic lzo lzo_compress snd_soc_skl snd_soc_skl_ip<o:p></o:p></p> <p class="MsoPlainText">c snd_soc_sst_ipc snd_soc_sst_dsp snd_soc_acpi snd_hda_ext_core snd_hda_core<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600307] calling<span style="mso-spacerun:yes"> </span>phy0+ @ 3102, parent: 0000:00:0c.0, cb: wiphy_resume [cf<o:p></o:p></p> <p class="MsoPlainText">g80211]<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600308]<span style="mso-spacerun:yes"> </span>zram snd_soc_max98357a acpi_als snd_soc_da7219 bridge stp llc ip<o:p></o:p></p> <p class="MsoPlainText">t_MASQUERADE nf_nat_masquerade_ipv4 xt_mark fuse snd_seq_dummy snd_seq snd_seq_d<o:p></o:p></p> <p class="MsoPlainText">evice btusb btrtl btbcm iio_trig_sysfs btintel uvcvideo bluetooth videobuf2_vmal<o:p></o:p></p> <p class="MsoPlainText">loc videobuf2_memops videobuf2_v4l2 ecdh_generic videobuf2_core cros_ec_sensors<o:p></o:p></p> <p class="MsoPlainText">cros_ec_sensors_ring cros_ec_sensors_core industrialio_triggered_buffer kfifo_bu<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600346] RDX: ffffffffb8258dd0 RSI: 0000000000000002 RDI: ffffffffb8258db0<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600347] RBP: ffffbb9546b73aa0 R08: 0000000000000000 R09: 0000000000000000<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600348] R10: 0000000000000000 R11: ffffffffb86d8518 R12: ffffa22075790000<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600349] R13: ffffa22075793d24 R14: 00000000ffffffff R15: ffffa2202eec9800<o:p></o:p></p> <p class="MsoPlainText">00000000000<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600352] CS:<span style="mso-spacerun:yes"> </span>0010 DS: 0000 ES: 0000 CR0: 0000000080050033<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600353] CR2: 00007f43c04d0e50 CR3: 0000000224112000 CR4: 00000000003406e0<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600354] Call Trace:<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600361]<span style="mso-spacerun:yes"> </span>intel_atomic_commit_tail+0x20a/0xacb<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600363]<span style="mso-spacerun:yes"> </span>? intel_atomic_commit_ready+0x44/0x4c<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600365]<span style="mso-spacerun:yes"> </span>intel_atomic_commit+0x227/0x238<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600368]<span style="mso-spacerun:yes"> </span>glk_force_audio_cdclk+0x9f/0x119<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600370]<span style="mso-spacerun:yes"> </span>i915_audio_component_get_power+0x3e/0x4d<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600376]<span style="mso-spacerun:yes"> </span>snd_hdac_display_power+0x53/0x97 [snd_hda_core]<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600379] calling<span style="mso-spacerun:yes"> </span>1-9+ @ 3086, parent: usb1, cb: usb_dev_resume<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600384]<span style="mso-spacerun:yes"> </span>skl_resume+0x3a/0x17a [snd_soc_skl]<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600387]<span style="mso-spacerun:yes"> </span>? pci_pm_suspend_noirq+0x1e9/0x1e9<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600391]<span style="mso-spacerun:yes"> </span>dpm_run_callback+0x59/0xbf<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600394]<span style="mso-spacerun:yes"> </span>device_resume+0x192/0x1d4<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600396]<span style="mso-spacerun:yes"> </span>dpm_resume+0x145/0x1da<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600398]<span style="mso-spacerun:yes"> </span>dpm_resume_end+0x11/0x1a<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600403]<span style="mso-spacerun:yes"> </span>suspend_devices_and_enter+0x354/0x5c2<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600407]<span style="mso-spacerun:yes"> </span>? remove_wait_queue+0x51/0x51<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600409]<span style="mso-spacerun:yes"> </span>pm_suspend+0x29c/0x2e2<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600411]<span style="mso-spacerun:yes"> </span>state_store+0xa2/0xcb<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600415]<span style="mso-spacerun:yes"> </span>kernfs_fop_write+0x103/0x14a<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600420]<span style="mso-spacerun:yes"> </span>__vfs_write+0x37/0xd0<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600424]<span style="mso-spacerun:yes"> </span>? inode_security+0x19/0x20<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600426]<span style="mso-spacerun:yes"> </span>? selinux_file_permission+0x78/0xad<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600428]<span style="mso-spacerun:yes"> </span>vfs_write+0xb9/0xfd<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600430]<span style="mso-spacerun:yes"> </span>SyS_write+0x5f/0xa3<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600434]<span style="mso-spacerun:yes"> </span>do_syscall_64+0x64/0x72<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600438]<span style="mso-spacerun:yes"> </span>entry_SYSCALL_64_after_hwframe+0x3d/0xa2<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600441] RIP: 0033:0x7f969d8af3b0<o:p></o:p></p> <p class="MsoPlainText">00001<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600443] RAX: ffffffffffffffda RBX: 00007f969ddf0000 RCX: 00007f969d8af3b0<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600444] RDX: 0000000000000006 RSI: 00007f969ddf0000 RDI: 0000000000000001<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600445] RBP: 00007ffd9e3f4880 R08: ffffffffffffffff R09: 0000000000000000<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600446] R10: 0000000000001000 R11: 0000000000000246 R12: 0000000000000006<o:p></o:p></p> <p class="MsoPlainText">[<span style="mso-spacerun:yes"> </span>280.600447] R13: 0000000000020000 R14: 00007f969ddf0000 R15: 0000000000000001<o:p></o:p></p> <meta name="ProgId" content="Word.Document"> <meta name="Generator" content="Microsoft Word 15"> <meta name="Originator" content="Microsoft Word 15"> <link rel="File-List" href="file:///C:%5CUsers%5Cabhaykum%5CAppData%5CLocal%5CTemp%5Cmsohtmlclip1%5C01%5Cclip_filelist.xml"> <!--[if gte mso 9]><xml> <o:OfficeDocumentSettings> <o:AllowPNG/> </o:OfficeDocumentSettings> </xml><![endif]--> <link rel="themeData" href="file:///C:%5CUsers%5Cabhaykum%5CAppData%5CLocal%5CTemp%5Cmsohtmlclip1%5C01%5Cclip_themedata.thmx"> <link rel="colorSchemeMapping" href="file:///C:%5CUsers%5Cabhaykum%5CAppData%5CLocal%5CTemp%5Cmsohtmlclip1%5C01%5Cclip_colorschememapping.xml"> <!--[if gte mso 9]><xml> <w:WordDocument> <w:View>Normal</w:View> <w:Zoom>0</w:Zoom> <w:TrackMoves/> <w:TrackFormatting/> <w:PunctuationKerning/> <w:ValidateAgainstSchemas/> <w:SaveIfXMLInvalid>false</w:SaveIfXMLInvalid> <w:IgnoreMixedContent>false</w:IgnoreMixedContent> <w:AlwaysShowPlaceholderText>false</w:AlwaysShowPlaceholderText> <w:DoNotPromoteQF/> <w:LidThemeOther>EN-US</w:LidThemeOther> <w:LidThemeAsian>X-NONE</w:LidThemeAsian> <w:LidThemeComplexScript>X-NONE</w:LidThemeComplexScript> <w:Compatibility> <w:BreakWrappedTables/> <w:SnapToGridInCell/> <w:WrapTextWithPunct/> <w:UseAsianBreakRules/> <w:DontGrowAutofit/> <w:SplitPgBreakAndParaMark/> <w:EnableOpenTypeKerning/> <w:DontFlipMirrorIndents/> <w:OverrideTableStyleHps/> </w:Compatibility> <w:DoNotOptimizeForBrowser/> <m:mathPr> <m:mathFont m:val="Cambria Math"/> <m:brkBin m:val="before"/> <m:brkBinSub m:val="--"/> <m:smallFrac m:val="off"/> <m:dispDef/> <m:lMargin m:val="0"/> <m:rMargin m:val="0"/> <m:defJc m:val="centerGroup"/> <m:wrapIndent m:val="1440"/> <m:intLim m:val="subSup"/> <m:naryLim m:val="undOvr"/> </m:mathPr></w:WordDocument> </xml><![endif]--><!--[if gte mso 9]><xml> <w:LatentStyles DefLockedState="false" DefUnhideWhenUsed="false" DefSemiHidden="false" DefQFormat="false" DefPriority="99" LatentStyleCount="371"> <w:LsdException Locked="false" Priority="0" QFormat="true" Name="Normal"/> <w:LsdException Locked="false" Priority="9" QFormat="true" Name="heading 1"/> <w:LsdException Locked="false" Priority="9" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="heading 2"/> <w:LsdException Locked="false" Priority="9" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="heading 3"/> <w:LsdException Locked="false" Priority="9" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="heading 4"/> <w:LsdException Locked="false" Priority="9" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="heading 5"/> <w:LsdException Locked="false" Priority="9" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="heading 6"/> <w:LsdException Locked="false" Priority="9" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="heading 7"/> <w:LsdException Locked="false" Priority="9" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="heading 8"/> <w:LsdException Locked="false" Priority="9" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="heading 9"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index 4"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index 5"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index 6"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index 7"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index 8"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index 9"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" Name="toc 1"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" Name="toc 2"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" Name="toc 3"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" Name="toc 4"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" Name="toc 5"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" Name="toc 6"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" Name="toc 7"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" Name="toc 8"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" Name="toc 9"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Normal Indent"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="footnote text"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="annotation text"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="header"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="footer"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="index heading"/> <w:LsdException Locked="false" Priority="35" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="caption"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="table of figures"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="envelope address"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="envelope return"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="footnote reference"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="annotation reference"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="line number"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="page number"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="endnote reference"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="endnote text"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="table of authorities"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="macro"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="toa heading"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Bullet"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Number"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List 4"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List 5"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Bullet 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Bullet 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Bullet 4"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Bullet 5"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Number 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Number 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Number 4"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Number 5"/> <w:LsdException Locked="false" Priority="10" QFormat="true" Name="Title"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Closing"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Signature"/> <w:LsdException Locked="false" Priority="1" SemiHidden="true" UnhideWhenUsed="true" Name="Default Paragraph Font"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Body Text"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Body Text Indent"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Continue"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Continue 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Continue 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Continue 4"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="List Continue 5"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Message Header"/> <w:LsdException Locked="false" Priority="11" QFormat="true" Name="Subtitle"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Salutation"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Date"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Body Text First Indent"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Body Text First Indent 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Note Heading"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Body Text 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Body Text 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Body Text Indent 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Body Text Indent 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Block Text"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Hyperlink"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="FollowedHyperlink"/> <w:LsdException Locked="false" Priority="22" QFormat="true" Name="Strong"/> <w:LsdException Locked="false" Priority="20" QFormat="true" Name="Emphasis"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Document Map"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Plain Text"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="E-mail Signature"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Top of Form"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Bottom of Form"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Normal (Web)"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Acronym"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Address"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Cite"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Code"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Definition"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Keyboard"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Preformatted"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Sample"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Typewriter"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="HTML Variable"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Normal Table"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="annotation subject"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="No List"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Outline List 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Outline List 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Outline List 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Simple 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Simple 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Simple 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Classic 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Classic 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Classic 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Classic 4"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Colorful 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Colorful 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Colorful 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Columns 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Columns 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Columns 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Columns 4"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Columns 5"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Grid 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Grid 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Grid 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Grid 4"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Grid 5"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Grid 6"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Grid 7"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Grid 8"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table List 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table List 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table List 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table List 4"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table List 5"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table List 6"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table List 7"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table List 8"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table 3D effects 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table 3D effects 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table 3D effects 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Contemporary"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Elegant"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Professional"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Subtle 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Subtle 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Web 1"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Web 2"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Web 3"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Balloon Text"/> <w:LsdException Locked="false" Priority="39" Name="Table Grid"/> <w:LsdException Locked="false" SemiHidden="true" UnhideWhenUsed="true" Name="Table Theme"/> <w:LsdException Locked="false" SemiHidden="true" Name="Placeholder Text"/> <w:LsdException Locked="false" Priority="1" QFormat="true" Name="No Spacing"/> <w:LsdException Locked="false" Priority="60" Name="Light Shading"/> <w:LsdException Locked="false" Priority="61" Name="Light List"/> <w:LsdException Locked="false" Priority="62" Name="Light Grid"/> <w:LsdException Locked="false" Priority="63" Name="Medium Shading 1"/> <w:LsdException Locked="false" Priority="64" Name="Medium Shading 2"/> <w:LsdException Locked="false" Priority="65" Name="Medium List 1"/> <w:LsdException Locked="false" Priority="66" Name="Medium List 2"/> <w:LsdException Locked="false" Priority="67" Name="Medium Grid 1"/> <w:LsdException Locked="false" Priority="68" Name="Medium Grid 2"/> <w:LsdException Locked="false" Priority="69" Name="Medium Grid 3"/> <w:LsdException Locked="false" Priority="70" Name="Dark List"/> <w:LsdException Locked="false" Priority="71" Name="Colorful Shading"/> <w:LsdException Locked="false" Priority="72" Name="Colorful List"/> <w:LsdException Locked="false" Priority="73" Name="Colorful Grid"/> <w:LsdException Locked="false" Priority="60" Name="Light Shading Accent 1"/> <w:LsdException Locked="false" Priority="61" Name="Light List Accent 1"/> <w:LsdException Locked="false" Priority="62" Name="Light Grid Accent 1"/> <w:LsdException Locked="false" Priority="63" Name="Medium Shading 1 Accent 1"/> <w:LsdException Locked="false" Priority="64" Name="Medium Shading 2 Accent 1"/> <w:LsdException Locked="false" Priority="65" Name="Medium List 1 Accent 1"/> <w:LsdException Locked="false" SemiHidden="true" Name="Revision"/> <w:LsdException Locked="false" Priority="34" QFormat="true" Name="List Paragraph"/> <w:LsdException Locked="false" Priority="29" QFormat="true" Name="Quote"/> <w:LsdException Locked="false" Priority="30" QFormat="true" Name="Intense Quote"/> <w:LsdException Locked="false" Priority="66" Name="Medium List 2 Accent 1"/> <w:LsdException Locked="false" Priority="67" Name="Medium Grid 1 Accent 1"/> <w:LsdException Locked="false" Priority="68" Name="Medium Grid 2 Accent 1"/> <w:LsdException Locked="false" Priority="69" Name="Medium Grid 3 Accent 1"/> <w:LsdException Locked="false" Priority="70" Name="Dark List Accent 1"/> <w:LsdException Locked="false" Priority="71" Name="Colorful Shading Accent 1"/> <w:LsdException Locked="false" Priority="72" Name="Colorful List Accent 1"/> <w:LsdException Locked="false" Priority="73" Name="Colorful Grid Accent 1"/> <w:LsdException Locked="false" Priority="60" Name="Light Shading Accent 2"/> <w:LsdException Locked="false" Priority="61" Name="Light List Accent 2"/> <w:LsdException Locked="false" Priority="62" Name="Light Grid Accent 2"/> <w:LsdException Locked="false" Priority="63" Name="Medium Shading 1 Accent 2"/> <w:LsdException Locked="false" Priority="64" Name="Medium Shading 2 Accent 2"/> <w:LsdException Locked="false" Priority="65" Name="Medium List 1 Accent 2"/> <w:LsdException Locked="false" Priority="66" Name="Medium List 2 Accent 2"/> <w:LsdException Locked="false" Priority="67" Name="Medium Grid 1 Accent 2"/> <w:LsdException Locked="false" Priority="68" Name="Medium Grid 2 Accent 2"/> <w:LsdException Locked="false" Priority="69" Name="Medium Grid 3 Accent 2"/> <w:LsdException Locked="false" Priority="70" Name="Dark List Accent 2"/> <w:LsdException Locked="false" Priority="71" Name="Colorful Shading Accent 2"/> <w:LsdException Locked="false" Priority="72" Name="Colorful List Accent 2"/> <w:LsdException Locked="false" Priority="73" Name="Colorful Grid Accent 2"/> <w:LsdException Locked="false" Priority="60" Name="Light Shading Accent 3"/> <w:LsdException Locked="false" Priority="61" Name="Light List Accent 3"/> <w:LsdException Locked="false" Priority="62" Name="Light Grid Accent 3"/> <w:LsdException Locked="false" Priority="63" Name="Medium Shading 1 Accent 3"/> <w:LsdException Locked="false" Priority="64" Name="Medium Shading 2 Accent 3"/> <w:LsdException Locked="false" Priority="65" Name="Medium List 1 Accent 3"/> <w:LsdException Locked="false" Priority="66" Name="Medium List 2 Accent 3"/> <w:LsdException Locked="false" Priority="67" Name="Medium Grid 1 Accent 3"/> <w:LsdException Locked="false" Priority="68" Name="Medium Grid 2 Accent 3"/> <w:LsdException Locked="false" Priority="69" Name="Medium Grid 3 Accent 3"/> <w:LsdException Locked="false" Priority="70" Name="Dark List Accent 3"/> <w:LsdException Locked="false" Priority="71" Name="Colorful Shading Accent 3"/> <w:LsdException Locked="false" Priority="72" Name="Colorful List Accent 3"/> <w:LsdException Locked="false" Priority="73" Name="Colorful Grid Accent 3"/> <w:LsdException Locked="false" Priority="60" Name="Light Shading Accent 4"/> <w:LsdException Locked="false" Priority="61" Name="Light List Accent 4"/> <w:LsdException Locked="false" Priority="62" Name="Light Grid Accent 4"/> <w:LsdException Locked="false" Priority="63" Name="Medium Shading 1 Accent 4"/> <w:LsdException Locked="false" Priority="64" Name="Medium Shading 2 Accent 4"/> <w:LsdException Locked="false" Priority="65" Name="Medium List 1 Accent 4"/> <w:LsdException Locked="false" Priority="66" Name="Medium List 2 Accent 4"/> <w:LsdException Locked="false" Priority="67" Name="Medium Grid 1 Accent 4"/> <w:LsdException Locked="false" Priority="68" Name="Medium Grid 2 Accent 4"/> <w:LsdException Locked="false" Priority="69" Name="Medium Grid 3 Accent 4"/> <w:LsdException Locked="false" Priority="70" Name="Dark List Accent 4"/> <w:LsdException Locked="false" Priority="71" Name="Colorful Shading Accent 4"/> <w:LsdException Locked="false" Priority="72" Name="Colorful List Accent 4"/> <w:LsdException Locked="false" Priority="73" Name="Colorful Grid Accent 4"/> <w:LsdException Locked="false" Priority="60" Name="Light Shading Accent 5"/> <w:LsdException Locked="false" Priority="61" Name="Light List Accent 5"/> <w:LsdException Locked="false" Priority="62" Name="Light Grid Accent 5"/> <w:LsdException Locked="false" Priority="63" Name="Medium Shading 1 Accent 5"/> <w:LsdException Locked="false" Priority="64" Name="Medium Shading 2 Accent 5"/> <w:LsdException Locked="false" Priority="65" Name="Medium List 1 Accent 5"/> <w:LsdException Locked="false" Priority="66" Name="Medium List 2 Accent 5"/> <w:LsdException Locked="false" Priority="67" Name="Medium Grid 1 Accent 5"/> <w:LsdException Locked="false" Priority="68" Name="Medium Grid 2 Accent 5"/> <w:LsdException Locked="false" Priority="69" Name="Medium Grid 3 Accent 5"/> <w:LsdException Locked="false" Priority="70" Name="Dark List Accent 5"/> <w:LsdException Locked="false" Priority="71" Name="Colorful Shading Accent 5"/> <w:LsdException Locked="false" Priority="72" Name="Colorful List Accent 5"/> <w:LsdException Locked="false" Priority="73" Name="Colorful Grid Accent 5"/> <w:LsdException Locked="false" Priority="60" Name="Light Shading Accent 6"/> <w:LsdException Locked="false" Priority="61" Name="Light List Accent 6"/> <w:LsdException Locked="false" Priority="62" Name="Light Grid Accent 6"/> <w:LsdException Locked="false" Priority="63" Name="Medium Shading 1 Accent 6"/> <w:LsdException Locked="false" Priority="64" Name="Medium Shading 2 Accent 6"/> <w:LsdException Locked="false" Priority="65" Name="Medium List 1 Accent 6"/> <w:LsdException Locked="false" Priority="66" Name="Medium List 2 Accent 6"/> <w:LsdException Locked="false" Priority="67" Name="Medium Grid 1 Accent 6"/> <w:LsdException Locked="false" Priority="68" Name="Medium Grid 2 Accent 6"/> <w:LsdException Locked="false" Priority="69" Name="Medium Grid 3 Accent 6"/> <w:LsdException Locked="false" Priority="70" Name="Dark List Accent 6"/> <w:LsdException Locked="false" Priority="71" Name="Colorful Shading Accent 6"/> <w:LsdException Locked="false" Priority="72" Name="Colorful List Accent 6"/> <w:LsdException Locked="false" Priority="73" Name="Colorful Grid Accent 6"/> <w:LsdException Locked="false" Priority="19" QFormat="true" Name="Subtle Emphasis"/> <w:LsdException Locked="false" Priority="21" QFormat="true" Name="Intense Emphasis"/> <w:LsdException Locked="false" Priority="31" QFormat="true" Name="Subtle Reference"/> <w:LsdException Locked="false" Priority="32" QFormat="true" Name="Intense Reference"/> <w:LsdException Locked="false" Priority="33" QFormat="true" Name="Book Title"/> <w:LsdException Locked="false" Priority="37" SemiHidden="true" UnhideWhenUsed="true" Name="Bibliography"/> <w:LsdException Locked="false" Priority="39" SemiHidden="true" UnhideWhenUsed="true" QFormat="true" Name="TOC Heading"/> <w:LsdException Locked="false" Priority="41" Name="Plain Table 1"/> <w:LsdException Locked="false" Priority="42" Name="Plain Table 2"/> <w:LsdException Locked="false" Priority="43" Name="Plain Table 3"/> <w:LsdException Locked="false" Priority="44" Name="Plain Table 4"/> <w:LsdException Locked="false" Priority="45" Name="Plain Table 5"/> <w:LsdException Locked="false" Priority="40" Name="Grid Table Light"/> <w:LsdException Locked="false" Priority="46" Name="Grid Table 1 Light"/> <w:LsdException Locked="false" Priority="47" Name="Grid Table 2"/> <w:LsdException Locked="false" Priority="48" Name="Grid Table 3"/> <w:LsdException Locked="false" Priority="49" Name="Grid Table 4"/> <w:LsdException Locked="false" Priority="50" Name="Grid Table 5 Dark"/> <w:LsdException Locked="false" Priority="51" Name="Grid Table 6 Colorful"/> <w:LsdException Locked="false" Priority="52" Name="Grid Table 7 Colorful"/> <w:LsdException Locked="false" Priority="46" Name="Grid Table 1 Light Accent 1"/> <w:LsdException Locked="false" Priority="47" Name="Grid Table 2 Accent 1"/> <w:LsdException Locked="false" Priority="48" Name="Grid Table 3 Accent 1"/> <w:LsdException Locked="false" Priority="49" Name="Grid Table 4 Accent 1"/> <w:LsdException Locked="false" Priority="50" Name="Grid Table 5 Dark Accent 1"/> <w:LsdException Locked="false" Priority="51" Name="Grid Table 6 Colorful Accent 1"/> <w:LsdException Locked="false" Priority="52" Name="Grid Table 7 Colorful Accent 1"/> <w:LsdException Locked="false" Priority="46" Name="Grid Table 1 Light Accent 2"/> <w:LsdException Locked="false" Priority="47" Name="Grid Table 2 Accent 2"/> <w:LsdException Locked="false" Priority="48" Name="Grid Table 3 Accent 2"/> <w:LsdException Locked="false" Priority="49" Name="Grid Table 4 Accent 2"/> <w:LsdException Locked="false" Priority="50" Name="Grid Table 5 Dark Accent 2"/> <w:LsdException Locked="false" Priority="51" Name="Grid Table 6 Colorful Accent 2"/> <w:LsdException Locked="false" Priority="52" Name="Grid Table 7 Colorful Accent 2"/> <w:LsdException Locked="false" Priority="46" Name="Grid Table 1 Light Accent 3"/> <w:LsdException Locked="false" Priority="47" Name="Grid Table 2 Accent 3"/> <w:LsdException Locked="false" Priority="48" Name="Grid Table 3 Accent 3"/> <w:LsdException Locked="false" Priority="49" Name="Grid Table 4 Accent 3"/> <w:LsdException Locked="false" Priority="50" Name="Grid Table 5 Dark Accent 3"/> <w:LsdException Locked="false" Priority="51" Name="Grid Table 6 Colorful Accent 3"/> <w:LsdException Locked="false" Priority="52" Name="Grid Table 7 Colorful Accent 3"/> <w:LsdException Locked="false" Priority="46" Name="Grid Table 1 Light Accent 4"/> <w:LsdException Locked="false" Priority="47" Name="Grid Table 2 Accent 4"/> <w:LsdException Locked="false" Priority="48" Name="Grid Table 3 Accent 4"/> <w:LsdException Locked="false" Priority="49" Name="Grid Table 4 Accent 4"/> <w:LsdException Locked="false" Priority="50" Name="Grid Table 5 Dark Accent 4"/> <w:LsdException Locked="false" Priority="51" Name="Grid Table 6 Colorful Accent 4"/> <w:LsdException Locked="false" Priority="52" Name="Grid Table 7 Colorful Accent 4"/> <w:LsdException Locked="false" Priority="46" Name="Grid Table 1 Light Accent 5"/> <w:LsdException Locked="false" Priority="47" Name="Grid Table 2 Accent 5"/> <w:LsdException Locked="false" Priority="48" Name="Grid Table 3 Accent 5"/> <w:LsdException Locked="false" Priority="49" Name="Grid Table 4 Accent 5"/> <w:LsdException Locked="false" Priority="50" Name="Grid Table 5 Dark Accent 5"/> <w:LsdException Locked="false" Priority="51" Name="Grid Table 6 Colorful Accent 5"/> <w:LsdException Locked="false" Priority="52" Name="Grid Table 7 Colorful Accent 5"/> <w:LsdException Locked="false" Priority="46" Name="Grid Table 1 Light Accent 6"/> <w:LsdException Locked="false" Priority="47" Name="Grid Table 2 Accent 6"/> <w:LsdException Locked="false" Priority="48" Name="Grid Table 3 Accent 6"/> <w:LsdException Locked="false" Priority="49" Name="Grid Table 4 Accent 6"/> <w:LsdException Locked="false" Priority="50" Name="Grid Table 5 Dark Accent 6"/> <w:LsdException Locked="false" Priority="51" Name="Grid Table 6 Colorful Accent 6"/> <w:LsdException Locked="false" Priority="52" Name="Grid Table 7 Colorful Accent 6"/> <w:LsdException Locked="false" Priority="46" Name="List Table 1 Light"/> <w:LsdException Locked="false" Priority="47" Name="List Table 2"/> <w:LsdException Locked="false" Priority="48" Name="List Table 3"/> <w:LsdException Locked="false" Priority="49" Name="List Table 4"/> <w:LsdException Locked="false" Priority="50" Name="List Table 5 Dark"/> <w:LsdException Locked="false" Priority="51" Name="List Table 6 Colorful"/> <w:LsdException Locked="false" Priority="52" Name="List Table 7 Colorful"/> <w:LsdException Locked="false" Priority="46" Name="List Table 1 Light Accent 1"/> <w:LsdException Locked="false" Priority="47" Name="List Table 2 Accent 1"/> <w:LsdException Locked="false" Priority="48" Name="List Table 3 Accent 1"/> <w:LsdException Locked="false" Priority="49" Name="List Table 4 Accent 1"/> <w:LsdException Locked="false" Priority="50" Name="List Table 5 Dark Accent 1"/> <w:LsdException Locked="false" Priority="51" Name="List Table 6 Colorful Accent 1"/> <w:LsdException Locked="false" Priority="52" Name="List Table 7 Colorful Accent 1"/> <w:LsdException Locked="false" Priority="46" Name="List Table 1 Light Accent 2"/> <w:LsdException Locked="false" Priority="47" Name="List Table 2 Accent 2"/> <w:LsdException Locked="false" Priority="48" Name="List Table 3 Accent 2"/> <w:LsdException Locked="false" Priority="49" Name="List Table 4 Accent 2"/> <w:LsdException Locked="false" Priority="50" Name="List Table 5 Dark Accent 2"/> <w:LsdException Locked="false" Priority="51" Name="List Table 6 Colorful Accent 2"/> <w:LsdException Locked="false" Priority="52" Name="List Table 7 Colorful Accent 2"/> <w:LsdException Locked="false" Priority="46" Name="List Table 1 Light Accent 3"/> <w:LsdException Locked="false" Priority="47" Name="List Table 2 Accent 3"/> <w:LsdException Locked="false" Priority="48" Name="List Table 3 Accent 3"/> <w:LsdException Locked="false" Priority="49" Name="List Table 4 Accent 3"/> <w:LsdException Locked="false" Priority="50" Name="List Table 5 Dark Accent 3"/> <w:LsdException Locked="false" Priority="51" Name="List Table 6 Colorful Accent 3"/> <w:LsdException Locked="false" Priority="52" Name="List Table 7 Colorful Accent 3"/> <w:LsdException Locked="false" Priority="46" Name="List Table 1 Light Accent 4"/> <w:LsdException Locked="false" Priority="47" Name="List Table 2 Accent 4"/> <w:LsdException Locked="false" Priority="48" Name="List Table 3 Accent 4"/> <w:LsdException Locked="false" Priority="49" Name="List Table 4 Accent 4"/> <w:LsdException Locked="false" Priority="50" Name="List Table 5 Dark Accent 4"/> <w:LsdException Locked="false" Priority="51" Name="List Table 6 Colorful Accent 4"/> <w:LsdException Locked="false" Priority="52" Name="List Table 7 Colorful Accent 4"/> <w:LsdException Locked="false" Priority="46" Name="List Table 1 Light Accent 5"/> <w:LsdException Locked="false" Priority="47" Name="List Table 2 Accent 5"/> <w:LsdException Locked="false" Priority="48" Name="List Table 3 Accent 5"/> <w:LsdException Locked="false" Priority="49" Name="List Table 4 Accent 5"/> <w:LsdException Locked="false" Priority="50" Name="List Table 5 Dark Accent 5"/> <w:LsdException Locked="false" Priority="51" Name="List Table 6 Colorful Accent 5"/> <w:LsdException Locked="false" Priority="52" Name="List Table 7 Colorful Accent 5"/> <w:LsdException Locked="false" Priority="46" Name="List Table 1 Light Accent 6"/> <w:LsdException Locked="false" Priority="47" Name="List Table 2 Accent 6"/> <w:LsdException Locked="false" Priority="48" Name="List Table 3 Accent 6"/> <w:LsdException Locked="false" Priority="49" Name="List Table 4 Accent 6"/> <w:LsdException Locked="false" Priority="50" Name="List Table 5 Dark Accent 6"/> <w:LsdException Locked="false" Priority="51" Name="List Table 6 Colorful Accent 6"/> <w:LsdException Locked="false" Priority="52" Name="List Table 7 Colorful Accent 6"/> </w:LatentStyles> </xml><![endif]--> <style> <!-- /* Font Definitions */ @font-face {font-family:"Cambria Math"; panose-1:2 4 5 3 5 4 6 3 2 4; mso-font-alt:"Calisto MT"; mso-font-charset:0; mso-generic-font-family:roman; mso-font-pitch:variable; mso-font-signature:-536870145 1107305727 0 0 415 0;} @font-face {font-family:Calibri; panose-1:2 15 5 2 2 2 4 3 2 4; mso-font-alt:"Times New Roman"; mso-font-charset:0; mso-generic-font-family:swiss; mso-font-pitch:variable; mso-font-signature:-536870145 1073786111 1 0 415 0;} /* Style Definitions */ p.MsoNormal, li.MsoNormal, div.MsoNormal {mso-style-unhide:no; mso-style-qformat:yes; mso-style-parent:""; margin:0in; margin-bottom:.0001pt; mso-pagination:widow-orphan; font-size:11.0pt; font-family:"Calibri",sans-serif; mso-ascii-font-family:Calibri; mso-ascii-theme-font:minor-latin; mso-fareast-font-family:Calibri; mso-fareast-theme-font:minor-latin; mso-hansi-font-family:Calibri; mso-hansi-theme-font:minor-latin; mso-bidi-font-family:"Times New Roman"; mso-bidi-theme-font:minor-bidi;} p.MsoPlainText, li.MsoPlainText, div.MsoPlainText {mso-style-noshow:yes; mso-style-priority:99; mso-style-link:"Plain Text Char"; margin:0in; margin-bottom:.0001pt; mso-pagination:widow-orphan; font-size:11.0pt; mso-bidi-font-size:10.5pt; font-family:"Calibri",sans-serif; mso-fareast-font-family:Calibri; mso-fareast-theme-font:minor-latin; mso-bidi-font-family:"Times New Roman"; mso-bidi-theme-font:minor-bidi;} span.PlainTextChar {mso-style-name:"Plain Text Char"; mso-style-noshow:yes; mso-style-priority:99; mso-style-unhide:no; mso-style-locked:yes; mso-style-link:"Plain Text"; mso-bidi-font-size:10.5pt; font-family:"Calibri",sans-serif; mso-ascii-font-family:Calibri; mso-hansi-font-family:Calibri;} .MsoChpDefault {mso-style-type:export-only; mso-default-props:yes; font-family:"Calibri",sans-serif; mso-ascii-font-family:Calibri; mso-ascii-theme-font:minor-latin; mso-fareast-font-family:Calibri; mso-fareast-theme-font:minor-latin; mso-hansi-font-family:Calibri; mso-hansi-theme-font:minor-latin; mso-bidi-font-family:"Times New Roman"; mso-bidi-theme-font:minor-bidi;} @page WordSection1 {size:8.5in 11.0in; margin:1.0in 1.0in 1.0in 1.0in; mso-header-margin:.5in; mso-footer-margin:.5in; mso-paper-source:0;} div.WordSection1 {page:WordSection1;} --> </style><!--[if gte mso 10]> <style> /* Style Definitions */ table.MsoNormalTable {mso-style-name:"Table Normal"; mso-tstyle-rowband-size:0; mso-tstyle-colband-size:0; mso-style-noshow:yes; mso-style-priority:99; mso-style-parent:""; mso-padding-alt:0in 5.4pt 0in 5.4pt; mso-para-margin:0in; mso-para-margin-bottom:.0001pt; mso-pagination:widow-orphan; font-size:11.0pt; font-family:"Calibri",sans-serif; mso-ascii-font-family:Calibri; mso-ascii-theme-font:minor-latin; mso-hansi-font-family:Calibri; mso-hansi-theme-font:minor-latin;} </style> <![endif]--><br> <blockquote cite="mid:20180828123931.GV5565@intel.com" type="cite"> <pre wrap=""> </pre> <blockquote type="cite"> <pre wrap="">+ } else if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, + &intel_state->cdclk.actual)) { ret = intel_modeset_all_pipes(state); if (ret < 0) return ret; + + intel_state->cdclk.pipe = INVALID_PIPE; } DRM_DEBUG_KMS("New cdclk calculated to be logical %u kHz, actual %u kHz\n", </pre> </blockquote> <pre wrap=""> </pre> </blockquote> <br> </body> </html>
+Susanta From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On Behalf Of Kumar, Abhay Sent: Tuesday, August 28, 2018 5:55 PM To: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: intel-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] [PATCH v1] drm/i915: Skip modeset for cdclk changes if possible On 8/28/2018 5:39 AM, Ville Syrjälä wrote: On Mon, Aug 27, 2018 at 11:50:32AM -0700, Abhay Kumar wrote: From: Ville Syrjälä <ville.syrjala@linux.intel.com><mailto:ville.syrjala@linux.intel.com> If we have only a single active pipe and the cdclk change only requires the cd2x divider to be updated bxt+ can do the update with forcing a full modeset on the pipe. Try to hook that up. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com><mailto:ville.syrjala@linux.intel.com> Signed-off-by: Abhay Kumar <abhay.kumar@intel.com><mailto:abhay.kumar@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 3 +- drivers/gpu/drm/i915/i915_reg.h | 3 +- drivers/gpu/drm/i915/intel_cdclk.c | 105 +++++++++++++++++++++++++---------- drivers/gpu/drm/i915/intel_display.c | 20 ++++++- drivers/gpu/drm/i915/intel_drv.h | 9 ++- 5 files changed, 105 insertions(+), 35 deletions(-) <snip> @@ -12252,12 +12253,24 @@ static int intel_modeset_checks(struct drm_atomic_state *state) return ret; } + /* All pipes must be switched off while we change the cdclk. */ - if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, - &intel_state->cdclk.actual)) { + if (is_power_of_2(intel_state->active_crtcs) && + intel_cdclk_needs_cd2x_update(dev_priv, + &dev_priv->cdclk.actual, + &intel_state->cdclk.actual)) { + ret = intel_lock_all_pipes(state); + if (ret < 0) + return ret; + + intel_state->cdclk.pipe = ilog2(intel_state->active_crtcs); BTW on further reflection this probably isn't quite sufficient. Let's say we have a commit with allow_modeset=true, but we aren't actually required to do a modeset based on any of the state changes. If we still have to change cdclk we should actually be doing the cd2x update atomically with the plane updates, or we should do it before or after the plane updates depending on whether the cdclk freq is going up or down. Doing the update atomically with the plane updates might be nicer in the end, but for that we would likely need to split the .set_cdclk() hooks into three parts (pre+commit+post). Whereas just doing the update before or after the plane updates as needed would probably be a little simpler. Yeah. That might also get rid of cdclk mismatch warning during multiple suspend resume cycle. [ 280.600259] cdclk state doesn't match! [ 280.600270] calling 1-8+ @ 3110, parent: usb1, cb: usb_dev_resume [ 280.600276] WARNING: CPU: 3 PID: 5224 at /mnt/host/source/src/third_party/ker nel/v4.14/drivers/gpu/drm/i915/intel_cdclk.c:1867 intel_set_cdclk+0xaa/0xdb [ 280.600277] Modules linked in: cmac rfcomm uinput snd_soc_sst_bxt_da7219_max9 8357a snd_soc_hdac_hdmi snd_soc_dmic lzo lzo_compress snd_soc_skl snd_soc_skl_ip c snd_soc_sst_ipc snd_soc_sst_dsp snd_soc_acpi snd_hda_ext_core snd_hda_core [ 280.600307] calling phy0+ @ 3102, parent: 0000:00:0c.0, cb: wiphy_resume [cf g80211] [ 280.600308] zram snd_soc_max98357a acpi_als snd_soc_da7219 bridge stp llc ip t_MASQUERADE nf_nat_masquerade_ipv4 xt_mark fuse snd_seq_dummy snd_seq snd_seq_d evice btusb btrtl btbcm iio_trig_sysfs btintel uvcvideo bluetooth videobuf2_vmal loc videobuf2_memops videobuf2_v4l2 ecdh_generic videobuf2_core cros_ec_sensors cros_ec_sensors_ring cros_ec_sensors_core industrialio_triggered_buffer kfifo_bu [ 280.600346] RDX: ffffffffb8258dd0 RSI: 0000000000000002 RDI: ffffffffb8258db0 [ 280.600347] RBP: ffffbb9546b73aa0 R08: 0000000000000000 R09: 0000000000000000 [ 280.600348] R10: 0000000000000000 R11: ffffffffb86d8518 R12: ffffa22075790000 [ 280.600349] R13: ffffa22075793d24 R14: 00000000ffffffff R15: ffffa2202eec9800 00000000000 [ 280.600352] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 280.600353] CR2: 00007f43c04d0e50 CR3: 0000000224112000 CR4: 00000000003406e0 [ 280.600354] Call Trace: [ 280.600361] intel_atomic_commit_tail+0x20a/0xacb [ 280.600363] ? intel_atomic_commit_ready+0x44/0x4c [ 280.600365] intel_atomic_commit+0x227/0x238 [ 280.600368] glk_force_audio_cdclk+0x9f/0x119 [ 280.600370] i915_audio_component_get_power+0x3e/0x4d [ 280.600376] snd_hdac_display_power+0x53/0x97 [snd_hda_core] [ 280.600379] calling 1-9+ @ 3086, parent: usb1, cb: usb_dev_resume [ 280.600384] skl_resume+0x3a/0x17a [snd_soc_skl] [ 280.600387] ? pci_pm_suspend_noirq+0x1e9/0x1e9 [ 280.600391] dpm_run_callback+0x59/0xbf [ 280.600394] device_resume+0x192/0x1d4 [ 280.600396] dpm_resume+0x145/0x1da [ 280.600398] dpm_resume_end+0x11/0x1a [ 280.600403] suspend_devices_and_enter+0x354/0x5c2 [ 280.600407] ? remove_wait_queue+0x51/0x51 [ 280.600409] pm_suspend+0x29c/0x2e2 [ 280.600411] state_store+0xa2/0xcb [ 280.600415] kernfs_fop_write+0x103/0x14a [ 280.600420] __vfs_write+0x37/0xd0 [ 280.600424] ? inode_security+0x19/0x20 [ 280.600426] ? selinux_file_permission+0x78/0xad [ 280.600428] vfs_write+0xb9/0xfd [ 280.600430] SyS_write+0x5f/0xa3 [ 280.600434] do_syscall_64+0x64/0x72 [ 280.600438] entry_SYSCALL_64_after_hwframe+0x3d/0xa2 [ 280.600441] RIP: 0033:0x7f969d8af3b0 00001 [ 280.600443] RAX: ffffffffffffffda RBX: 00007f969ddf0000 RCX: 00007f969d8af3b0 [ 280.600444] RDX: 0000000000000006 RSI: 00007f969ddf0000 RDI: 0000000000000001 [ 280.600445] RBP: 00007ffd9e3f4880 R08: ffffffffffffffff R09: 0000000000000000 [ 280.600446] R10: 0000000000001000 R11: 0000000000000246 R12: 0000000000000006 [ 280.600447] R13: 0000000000020000 R14: 00007f969ddf0000 R15: 0000000000000001 + } else if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, + &intel_state->cdclk.actual)) { ret = intel_modeset_all_pipes(state); if (ret < 0) return ret; + + intel_state->cdclk.pipe = INVALID_PIPE; } DRM_DEBUG_KMS("New cdclk calculated to be logical %u kHz, actual %u kHz\n", <html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style><!-- /* Font Definitions */ @font-face {font-family:"Cambria Math"; panose-1:2 4 5 3 5 4 6 3 2 4;} @font-face {font-family:Calibri; panose-1:2 15 5 2 2 2 4 3 2 4;} @font-face {font-family:Consolas; panose-1:2 11 6 9 2 2 4 3 2 4;} /* Style Definitions */ p.MsoNormal, li.MsoNormal, div.MsoNormal {margin:0in; margin-bottom:.0001pt; font-size:11.0pt; font-family:"Calibri",sans-serif; color:black;} a:link, span.MsoHyperlink {mso-style-priority:99; color:blue; text-decoration:underline;} a:visited, span.MsoHyperlinkFollowed {mso-style-priority:99; color:purple; text-decoration:underline;} p.MsoPlainText, li.MsoPlainText, div.MsoPlainText {mso-style-priority:99; mso-style-link:"Plain Text Char"; margin:0in; margin-bottom:.0001pt; font-size:11.0pt; font-family:"Calibri",sans-serif; color:black;} p {mso-style-priority:99; mso-margin-top-alt:auto; margin-right:0in; mso-margin-bottom-alt:auto; margin-left:0in; font-size:11.0pt; font-family:"Calibri",sans-serif; color:black;} pre {mso-style-priority:99; mso-style-link:"HTML Preformatted Char"; margin:0in; margin-bottom:.0001pt; font-size:10.0pt; font-family:"Courier New"; color:black;} span.HTMLPreformattedChar {mso-style-name:"HTML Preformatted Char"; mso-style-priority:99; mso-style-link:"HTML Preformatted"; font-family:Consolas; color:black;} span.PlainTextChar {mso-style-name:"Plain Text Char"; mso-style-priority:99; mso-style-link:"Plain Text"; font-family:"Calibri",sans-serif;} span.EmailStyle22 {mso-style-type:personal-reply; font-family:"Calibri",sans-serif; color:#1F497D;} .MsoChpDefault {mso-style-type:export-only; font-size:10.0pt;} @page WordSection1 {size:8.5in 11.0in; margin:1.0in 1.0in 1.0in 1.0in;} div.WordSection1 {page:WordSection1;} --></style><!--[if gte mso 9]><xml> <o:shapedefaults v:ext="edit" spidmax="1026" /> </xml><![endif]--><!--[if gte mso 9]><xml> <o:shapelayout v:ext="edit"> <o:idmap v:ext="edit" data="1" /> </o:shapelayout></xml><![endif]--> </head> <body bgcolor="white" lang="EN-US" link="blue" vlink="purple"> <div class="WordSection1"> <p class="MsoNormal"><span style="color:#1F497D">+Susanta<o:p></o:p></span></p> <p class="MsoNormal"><a name="_MailEndCompose"><span style="color:#1F497D"><o:p> </o:p></span></a></p> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> <p class="MsoNormal"><a name="_____replyseparator"></a><b><span style="color:windowtext">From:</span></b><span style="color:windowtext"> Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] <b>On Behalf Of </b>Kumar, Abhay<br> <b>Sent:</b> Tuesday, August 28, 2018 5:55 PM<br> <b>To:</b> Ville Syrjälä <ville.syrjala@linux.intel.com><br> <b>Cc:</b> intel-gfx@lists.freedesktop.org<br> <b>Subject:</b> Re: [Intel-gfx] [PATCH v1] drm/i915: Skip modeset for cdclk changes if possible<o:p></o:p></span></p> </div> </div> <p class="MsoNormal"><o:p> </o:p></p> <p><span style="font-size:12.0pt"><o:p> </o:p></span></p> <p class="MsoNormal"><o:p> </o:p></p> <div> <p class="MsoNormal">On 8/28/2018 5:39 AM, Ville Syrjälä wrote:<o:p></o:p></p> </div> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre>On Mon, Aug 27, 2018 at 11:50:32AM -0700, Abhay Kumar wrote:<o:p></o:p></pre> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre>From: Ville Syrjälä <a href="mailto:ville.syrjala@linux.intel.com"><ville.syrjala@linux.intel.com></a><o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre>If we have only a single active pipe and the cdclk change only requires<o:p></o:p></pre> <pre>the cd2x divider to be updated bxt+ can do the update with forcing a full<o:p></o:p></pre> <pre>modeset on the pipe. Try to hook that up.<o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre>Signed-off-by: Ville Syrjälä <a href="mailto:ville.syrjala@linux.intel.com"><ville.syrjala@linux.intel.com></a><o:p></o:p></pre> <pre>Signed-off-by: Abhay Kumar <a href="mailto:abhay.kumar@intel.com"><abhay.kumar@intel.com></a><o:p></o:p></pre> <pre>---<o:p></o:p></pre> <pre> drivers/gpu/drm/i915/i915_drv.h | 3 +-<o:p></o:p></pre> <pre> drivers/gpu/drm/i915/i915_reg.h | 3 +-<o:p></o:p></pre> <pre> drivers/gpu/drm/i915/intel_cdclk.c | 105 +++++++++++++++++++++++++----------<o:p></o:p></pre> <pre> drivers/gpu/drm/i915/intel_display.c | 20 ++++++-<o:p></o:p></pre> <pre> drivers/gpu/drm/i915/intel_drv.h | 9 ++-<o:p></o:p></pre> <pre> 5 files changed, 105 insertions(+), 35 deletions(-)<o:p></o:p></pre> <pre><o:p> </o:p></pre> </blockquote> <pre><snip><o:p></o:p></pre> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre>@@ -12252,12 +12253,24 @@ static int intel_modeset_checks(struct drm_atomic_state *state)<o:p></o:p></pre> <pre> return ret;<o:p></o:p></pre> <pre> }<o:p></o:p></pre> <pre> <o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre> /* All pipes must be switched off while we change the cdclk. */<o:p></o:p></pre> <pre>- if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual,<o:p></o:p></pre> <pre>- &intel_state->cdclk.actual)) {<o:p></o:p></pre> <pre>+ if (is_power_of_2(intel_state->active_crtcs) &&<o:p></o:p></pre> <pre>+ intel_cdclk_needs_cd2x_update(dev_priv,<o:p></o:p></pre> <pre>+ &dev_priv->cdclk.actual,<o:p></o:p></pre> <pre>+ &intel_state->cdclk.actual)) {<o:p></o:p></pre> <pre>+ ret = intel_lock_all_pipes(state);<o:p></o:p></pre> <pre>+ if (ret < 0)<o:p></o:p></pre> <pre>+ return ret;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+ intel_state->cdclk.pipe = ilog2(intel_state->active_crtcs);<o:p></o:p></pre> </blockquote> <pre><o:p> </o:p></pre> <pre>BTW on further reflection this probably isn't quite sufficient. Let's<o:p></o:p></pre> <pre>say we have a commit with allow_modeset=true, but we aren't actually<o:p></o:p></pre> <pre>required to do a modeset based on any of the state changes. If we still<o:p></o:p></pre> <pre>have to change cdclk we should actually be doing the cd2x update<o:p></o:p></pre> <pre>atomically with the plane updates, or we should do it before or after<o:p></o:p></pre> <pre>the plane updates depending on whether the cdclk freq is going up or<o:p></o:p></pre> <pre>down.<o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre>Doing the update atomically with the plane updates might be nicer in the<o:p></o:p></pre> <pre>end, but for that we would likely need to split the .set_cdclk() hooks<o:p></o:p></pre> <pre>into three parts (pre+commit+post). Whereas just doing the update before<o:p></o:p></pre> <pre>or after the plane updates as needed would probably be a little simpler.<o:p></o:p></pre> </blockquote> <p class="MsoNormal" style="margin-bottom:12.0pt">Yeah. That might also get rid of cdclk mismatch warning during multiple suspend resume cycle.<o:p></o:p></p> <p class="MsoPlainText">[ 280.600259] cdclk state doesn't match!<o:p></o:p></p> <p class="MsoPlainText">[ 280.600270] calling 1-8+ @ 3110, parent: usb1, cb: usb_dev_resume<o:p></o:p></p> <p class="MsoPlainText">[ 280.600276] WARNING: CPU: 3 PID: 5224 at /mnt/host/source/src/third_party/ker<o:p></o:p></p> <p class="MsoPlainText">nel/v4.14/drivers/gpu/drm/i915/intel_cdclk.c:1867 intel_set_cdclk+0xaa/0xdb<o:p></o:p></p> <p class="MsoPlainText">[ 280.600277] Modules linked in: cmac rfcomm uinput snd_soc_sst_bxt_da7219_max9<o:p></o:p></p> <p class="MsoPlainText">8357a snd_soc_hdac_hdmi snd_soc_dmic lzo lzo_compress snd_soc_skl snd_soc_skl_ip<o:p></o:p></p> <p class="MsoPlainText">c snd_soc_sst_ipc snd_soc_sst_dsp snd_soc_acpi snd_hda_ext_core snd_hda_core<o:p></o:p></p> <p class="MsoPlainText">[ 280.600307] calling phy0+ @ 3102, parent: 0000:00:0c.0, cb: wiphy_resume [cf<o:p></o:p></p> <p class="MsoPlainText">g80211]<o:p></o:p></p> <p class="MsoPlainText">[ 280.600308] zram snd_soc_max98357a acpi_als snd_soc_da7219 bridge stp llc ip<o:p></o:p></p> <p class="MsoPlainText">t_MASQUERADE nf_nat_masquerade_ipv4 xt_mark fuse snd_seq_dummy snd_seq snd_seq_d<o:p></o:p></p> <p class="MsoPlainText">evice btusb btrtl btbcm iio_trig_sysfs btintel uvcvideo bluetooth videobuf2_vmal<o:p></o:p></p> <p class="MsoPlainText">loc videobuf2_memops videobuf2_v4l2 ecdh_generic videobuf2_core cros_ec_sensors<o:p></o:p></p> <p class="MsoPlainText">cros_ec_sensors_ring cros_ec_sensors_core industrialio_triggered_buffer kfifo_bu<o:p></o:p></p> <p class="MsoPlainText">[ 280.600346] RDX: ffffffffb8258dd0 RSI: 0000000000000002 RDI: ffffffffb8258db0<o:p></o:p></p> <p class="MsoPlainText">[ 280.600347] RBP: ffffbb9546b73aa0 R08: 0000000000000000 R09: 0000000000000000<o:p></o:p></p> <p class="MsoPlainText">[ 280.600348] R10: 0000000000000000 R11: ffffffffb86d8518 R12: ffffa22075790000<o:p></o:p></p> <p class="MsoPlainText">[ 280.600349] R13: ffffa22075793d24 R14: 00000000ffffffff R15: ffffa2202eec9800<o:p></o:p></p> <p class="MsoPlainText">00000000000<o:p></o:p></p> <p class="MsoPlainText">[ 280.600352] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033<o:p></o:p></p> <p class="MsoPlainText">[ 280.600353] CR2: 00007f43c04d0e50 CR3: 0000000224112000 CR4: 00000000003406e0<o:p></o:p></p> <p class="MsoPlainText">[ 280.600354] Call Trace:<o:p></o:p></p> <p class="MsoPlainText">[ 280.600361] intel_atomic_commit_tail+0x20a/0xacb<o:p></o:p></p> <p class="MsoPlainText">[ 280.600363] ? intel_atomic_commit_ready+0x44/0x4c<o:p></o:p></p> <p class="MsoPlainText">[ 280.600365] intel_atomic_commit+0x227/0x238<o:p></o:p></p> <p class="MsoPlainText">[ 280.600368] glk_force_audio_cdclk+0x9f/0x119<o:p></o:p></p> <p class="MsoPlainText">[ 280.600370] i915_audio_component_get_power+0x3e/0x4d<o:p></o:p></p> <p class="MsoPlainText">[ 280.600376] snd_hdac_display_power+0x53/0x97 [snd_hda_core]<o:p></o:p></p> <p class="MsoPlainText">[ 280.600379] calling 1-9+ @ 3086, parent: usb1, cb: usb_dev_resume<o:p></o:p></p> <p class="MsoPlainText">[ 280.600384] skl_resume+0x3a/0x17a [snd_soc_skl]<o:p></o:p></p> <p class="MsoPlainText">[ 280.600387] ? pci_pm_suspend_noirq+0x1e9/0x1e9<o:p></o:p></p> <p class="MsoPlainText">[ 280.600391] dpm_run_callback+0x59/0xbf<o:p></o:p></p> <p class="MsoPlainText">[ 280.600394] device_resume+0x192/0x1d4<o:p></o:p></p> <p class="MsoPlainText">[ 280.600396] dpm_resume+0x145/0x1da<o:p></o:p></p> <p class="MsoPlainText">[ 280.600398] dpm_resume_end+0x11/0x1a<o:p></o:p></p> <p class="MsoPlainText">[ 280.600403] suspend_devices_and_enter+0x354/0x5c2<o:p></o:p></p> <p class="MsoPlainText">[ 280.600407] ? remove_wait_queue+0x51/0x51<o:p></o:p></p> <p class="MsoPlainText">[ 280.600409] pm_suspend+0x29c/0x2e2<o:p></o:p></p> <p class="MsoPlainText">[ 280.600411] state_store+0xa2/0xcb<o:p></o:p></p> <p class="MsoPlainText">[ 280.600415] kernfs_fop_write+0x103/0x14a<o:p></o:p></p> <p class="MsoPlainText">[ 280.600420] __vfs_write+0x37/0xd0<o:p></o:p></p> <p class="MsoPlainText">[ 280.600424] ? inode_security+0x19/0x20<o:p></o:p></p> <p class="MsoPlainText">[ 280.600426] ? selinux_file_permission+0x78/0xad<o:p></o:p></p> <p class="MsoPlainText">[ 280.600428] vfs_write+0xb9/0xfd<o:p></o:p></p> <p class="MsoPlainText">[ 280.600430] SyS_write+0x5f/0xa3<o:p></o:p></p> <p class="MsoPlainText">[ 280.600434] do_syscall_64+0x64/0x72<o:p></o:p></p> <p class="MsoPlainText">[ 280.600438] entry_SYSCALL_64_after_hwframe+0x3d/0xa2<o:p></o:p></p> <p class="MsoPlainText">[ 280.600441] RIP: 0033:0x7f969d8af3b0<o:p></o:p></p> <p class="MsoPlainText">00001<o:p></o:p></p> <p class="MsoPlainText">[ 280.600443] RAX: ffffffffffffffda RBX: 00007f969ddf0000 RCX: 00007f969d8af3b0<o:p></o:p></p> <p class="MsoPlainText">[ 280.600444] RDX: 0000000000000006 RSI: 00007f969ddf0000 RDI: 0000000000000001<o:p></o:p></p> <p class="MsoPlainText">[ 280.600445] RBP: 00007ffd9e3f4880 R08: ffffffffffffffff R09: 0000000000000000<o:p></o:p></p> <p class="MsoPlainText">[ 280.600446] R10: 0000000000001000 R11: 0000000000000246 R12: 0000000000000006<o:p></o:p></p> <p class="MsoPlainText">[ 280.600447] R13: 0000000000020000 R14: 00007f969ddf0000 R15: 0000000000000001<o:p></o:p></p> <p class="MsoNormal"><span style="font-size:12.0pt;font-family:"Times New Roman",serif"><br> <br> <o:p></o:p></span></p> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre><o:p> </o:p></pre> <pre><o:p> </o:p></pre> <pre><o:p> </o:p></pre> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre>+ } else if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual,<o:p></o:p></pre> <pre>+ &intel_state->cdclk.actual)) {<o:p></o:p></pre> <pre> ret = intel_modeset_all_pipes(state);<o:p></o:p></pre> <pre> if (ret < 0)<o:p></o:p></pre> <pre> return ret;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+ intel_state->cdclk.pipe = INVALID_PIPE;<o:p></o:p></pre> <pre> }<o:p></o:p></pre> <pre> <o:p></o:p></pre> <pre> DRM_DEBUG_KMS("New cdclk calculated to be logical %u kHz, actual %u kHz\n",<o:p></o:p></pre> </blockquote> <pre><o:p> </o:p></pre> </blockquote> <p class="MsoNormal"><span style="font-size:12.0pt;font-family:"Times New Roman",serif"><o:p> </o:p></span></p> </div> </body> </html>
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index e5b9d3c77139..1f0a6427e76c 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -408,7 +408,8 @@ struct drm_i915_display_funcs { void (*get_cdclk)(struct drm_i915_private *dev_priv, struct intel_cdclk_state *cdclk_state); void (*set_cdclk)(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state); + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe); int (*get_fifo_size)(struct drm_i915_private *dev_priv, enum i9xx_plane_id i9xx_plane); int (*compute_pipe_wm)(struct intel_crtc_state *cstate); diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h index 8534f88a60f6..7702cec70b0d 100644 --- a/drivers/gpu/drm/i915/i915_reg.h +++ b/drivers/gpu/drm/i915/i915_reg.h @@ -9262,7 +9262,8 @@ enum skl_power_gate { #define BXT_CDCLK_CD2X_PIPE(pipe) ((pipe) << 20) #define CDCLK_DIVMUX_CD_OVERRIDE (1 << 19) #define BXT_CDCLK_CD2X_PIPE_NONE BXT_CDCLK_CD2X_PIPE(3) -#define ICL_CDCLK_CD2X_PIPE_NONE (7 << 19) +#define ICL_CDCLK_CD2X_PIPE(pipe) ((pipe) << 19) +#define ICL_CDCLK_CD2X_PIPE_NONE ICL_CDCLK_CD2X_PIPE(7) #define BXT_CDCLK_SSA_PRECHARGE_ENABLE (1 << 16) #define CDCLK_FREQ_DECIMAL_MASK (0x7ff) diff --git a/drivers/gpu/drm/i915/intel_cdclk.c b/drivers/gpu/drm/i915/intel_cdclk.c index 29075c763428..1955f6aa54e1 100644 --- a/drivers/gpu/drm/i915/intel_cdclk.c +++ b/drivers/gpu/drm/i915/intel_cdclk.c @@ -516,7 +516,8 @@ static void vlv_program_pfi_credits(struct drm_i915_private *dev_priv) } static void vlv_set_cdclk(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state) + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe) { int cdclk = cdclk_state->cdclk; u32 val, cmd = cdclk_state->voltage_level; @@ -597,7 +598,8 @@ static void vlv_set_cdclk(struct drm_i915_private *dev_priv, } static void chv_set_cdclk(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state) + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe) { int cdclk = cdclk_state->cdclk; u32 val, cmd = cdclk_state->voltage_level; @@ -695,7 +697,8 @@ static void bdw_get_cdclk(struct drm_i915_private *dev_priv, } static void bdw_set_cdclk(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state) + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe) { int cdclk = cdclk_state->cdclk; uint32_t val; @@ -985,7 +988,8 @@ static void skl_dpll0_disable(struct drm_i915_private *dev_priv) } static void skl_set_cdclk(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state) + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe) { int cdclk = cdclk_state->cdclk; int vco = cdclk_state->vco; @@ -1156,7 +1160,7 @@ void skl_init_cdclk(struct drm_i915_private *dev_priv) cdclk_state.cdclk = skl_calc_cdclk(0, cdclk_state.vco); cdclk_state.voltage_level = skl_calc_voltage_level(cdclk_state.cdclk); - skl_set_cdclk(dev_priv, &cdclk_state); + skl_set_cdclk(dev_priv, &cdclk_state, INVALID_PIPE); } /** @@ -1174,7 +1178,7 @@ void skl_uninit_cdclk(struct drm_i915_private *dev_priv) cdclk_state.vco = 0; cdclk_state.voltage_level = skl_calc_voltage_level(cdclk_state.cdclk); - skl_set_cdclk(dev_priv, &cdclk_state); + skl_set_cdclk(dev_priv, &cdclk_state, INVALID_PIPE); } static int bxt_calc_cdclk(int min_cdclk) @@ -1353,7 +1357,8 @@ static void bxt_de_pll_enable(struct drm_i915_private *dev_priv, int vco) } static void bxt_set_cdclk(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state) + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe) { int cdclk = cdclk_state->cdclk; int vco = cdclk_state->vco; @@ -1406,11 +1411,10 @@ static void bxt_set_cdclk(struct drm_i915_private *dev_priv, bxt_de_pll_enable(dev_priv, vco); val = divider | skl_cdclk_decimal(cdclk); - /* - * FIXME if only the cd2x divider needs changing, it could be done - * without shutting off the pipe (if only one pipe is active). - */ - val |= BXT_CDCLK_CD2X_PIPE_NONE; + if (pipe == INVALID_PIPE) + val |= BXT_CDCLK_CD2X_PIPE_NONE; + else + val |= BXT_CDCLK_CD2X_PIPE(pipe); /* * Disable SSA Precharge when CD clock frequency < 500 MHz, * enable otherwise. @@ -1523,7 +1527,7 @@ void bxt_init_cdclk(struct drm_i915_private *dev_priv) } cdclk_state.voltage_level = bxt_calc_voltage_level(cdclk_state.cdclk); - bxt_set_cdclk(dev_priv, &cdclk_state); + bxt_set_cdclk(dev_priv, &cdclk_state, INVALID_PIPE); } /** @@ -1541,7 +1545,7 @@ void bxt_uninit_cdclk(struct drm_i915_private *dev_priv) cdclk_state.vco = 0; cdclk_state.voltage_level = bxt_calc_voltage_level(cdclk_state.cdclk); - bxt_set_cdclk(dev_priv, &cdclk_state); + bxt_set_cdclk(dev_priv, &cdclk_state, INVALID_PIPE); } static int cnl_calc_cdclk(int min_cdclk) @@ -1661,7 +1665,8 @@ static void cnl_cdclk_pll_enable(struct drm_i915_private *dev_priv, int vco) } static void cnl_set_cdclk(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state) + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe) { int cdclk = cdclk_state->cdclk; int vco = cdclk_state->vco; @@ -1702,11 +1707,10 @@ static void cnl_set_cdclk(struct drm_i915_private *dev_priv, cnl_cdclk_pll_enable(dev_priv, vco); val = divider | skl_cdclk_decimal(cdclk); - /* - * FIXME if only the cd2x divider needs changing, it could be done - * without shutting off the pipe (if only one pipe is active). - */ - val |= BXT_CDCLK_CD2X_PIPE_NONE; + if (pipe == INVALID_PIPE) + val |= BXT_CDCLK_CD2X_PIPE_NONE; + else + val |= BXT_CDCLK_CD2X_PIPE(pipe); I915_WRITE(CDCLK_CTL, val); /* inform PCU of the change */ @@ -1845,10 +1849,12 @@ static int icl_calc_cdclk_pll_vco(struct drm_i915_private *dev_priv, int cdclk) } static void icl_set_cdclk(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state) + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe) { unsigned int cdclk = cdclk_state->cdclk; unsigned int vco = cdclk_state->vco; + u32 val; int ret; mutex_lock(&dev_priv->pcu_lock); @@ -1870,8 +1876,12 @@ static void icl_set_cdclk(struct drm_i915_private *dev_priv, if (dev_priv->cdclk.hw.vco != vco) cnl_cdclk_pll_enable(dev_priv, vco); - I915_WRITE(CDCLK_CTL, ICL_CDCLK_CD2X_PIPE_NONE | - skl_cdclk_decimal(cdclk)); + val = skl_cdclk_decimal(cdclk); + if (pipe == INVALID_PIPE) + val |= ICL_CDCLK_CD2X_PIPE_NONE; + else + val |= ICL_CDCLK_CD2X_PIPE(pipe); + I915_WRITE(CDCLK_CTL, val); mutex_lock(&dev_priv->pcu_lock); sandybridge_pcode_write(dev_priv, SKL_PCODE_CDCLK_CONTROL, @@ -2000,7 +2010,7 @@ void icl_init_cdclk(struct drm_i915_private *dev_priv) sanitized_state.voltage_level = icl_calc_voltage_level(sanitized_state.cdclk); - icl_set_cdclk(dev_priv, &sanitized_state); + icl_set_cdclk(dev_priv, &sanitized_state, INVALID_PIPE); } /** @@ -2018,7 +2028,7 @@ void icl_uninit_cdclk(struct drm_i915_private *dev_priv) cdclk_state.vco = 0; cdclk_state.voltage_level = icl_calc_voltage_level(cdclk_state.cdclk); - icl_set_cdclk(dev_priv, &cdclk_state); + icl_set_cdclk(dev_priv, &cdclk_state, INVALID_PIPE); } /** @@ -2046,7 +2056,7 @@ void cnl_init_cdclk(struct drm_i915_private *dev_priv) cdclk_state.vco = cnl_cdclk_pll_vco(dev_priv, cdclk_state.cdclk); cdclk_state.voltage_level = cnl_calc_voltage_level(cdclk_state.cdclk); - cnl_set_cdclk(dev_priv, &cdclk_state); + cnl_set_cdclk(dev_priv, &cdclk_state, INVALID_PIPE); } /** @@ -2064,7 +2074,7 @@ void cnl_uninit_cdclk(struct drm_i915_private *dev_priv) cdclk_state.vco = 0; cdclk_state.voltage_level = cnl_calc_voltage_level(cdclk_state.cdclk); - cnl_set_cdclk(dev_priv, &cdclk_state); + cnl_set_cdclk(dev_priv, &cdclk_state, INVALID_PIPE); } /** @@ -2084,6 +2094,41 @@ bool intel_cdclk_needs_modeset(const struct intel_cdclk_state *a, } /** + * intel_cdclk_needs_cd2x_update - Determine if two CDCLK states require a cd2x divider update + * @a: first CDCLK state + * @b: second CDCLK state + * + * Returns: + * True if the CDCLK states require just a cd2x divider update, false if not. + */ +bool intel_cdclk_needs_cd2x_update(struct drm_i915_private *dev_priv, + const struct intel_cdclk_state *a, + const struct intel_cdclk_state *b) +{ + /* Older hw doesn't have the capability */ + if (INTEL_GEN(dev_priv) < 10 && !IS_GEN9_LP(dev_priv)) + return false; + + return a->cdclk != b->cdclk && + a->vco == b->vco && + a->ref == b->ref; +} + +/** + * intel_cdclk_can_skip_modeset - Determine if two CDCLK states can a modeset on all pipes + * @a: first CDCLK state + * @b: second CDCLK state + * + * Returns: + * True if the CDCLK states require pipes to be off during reprogramming, false if not. + */ +bool intel_cdclk_can_skip_modeset(const struct intel_cdclk_state *a, + const struct intel_cdclk_state *b) +{ + return a->vco == b->vco && a->ref == b->ref; +} + +/** * intel_cdclk_changed - Determine if two CDCLK states are different * @a: first CDCLK state * @b: second CDCLK state @@ -2111,12 +2156,14 @@ void intel_dump_cdclk_state(const struct intel_cdclk_state *cdclk_state, * intel_set_cdclk - Push the CDCLK state to the hardware * @dev_priv: i915 device * @cdclk_state: new CDCLK state + * @pipe: pipe with which to synchronize the update * * Program the hardware based on the passed in CDCLK state, * if necessary. */ void intel_set_cdclk(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state) + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe) { if (!intel_cdclk_changed(&dev_priv->cdclk.hw, cdclk_state)) return; @@ -2126,7 +2173,7 @@ void intel_set_cdclk(struct drm_i915_private *dev_priv, intel_dump_cdclk_state(cdclk_state, "Changing CDCLK to"); - dev_priv->display.set_cdclk(dev_priv, cdclk_state); + dev_priv->display.set_cdclk(dev_priv, cdclk_state, pipe); if (WARN(intel_cdclk_changed(&dev_priv->cdclk.hw, cdclk_state), "cdclk state doesn't match!\n")) { diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index b0b6e1e9a294..5a67fb39300a 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -12217,6 +12217,7 @@ static int intel_modeset_checks(struct drm_atomic_state *state) intel_state->active_crtcs = dev_priv->active_crtcs; intel_state->cdclk.logical = dev_priv->cdclk.logical; intel_state->cdclk.actual = dev_priv->cdclk.actual; + intel_state->cdclk.pipe = INVALID_PIPE; for_each_oldnew_crtc_in_state(state, crtc, old_crtc_state, new_crtc_state, i) { if (new_crtc_state->active) @@ -12252,12 +12253,24 @@ static int intel_modeset_checks(struct drm_atomic_state *state) return ret; } + /* All pipes must be switched off while we change the cdclk. */ - if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, - &intel_state->cdclk.actual)) { + if (is_power_of_2(intel_state->active_crtcs) && + intel_cdclk_needs_cd2x_update(dev_priv, + &dev_priv->cdclk.actual, + &intel_state->cdclk.actual)) { + ret = intel_lock_all_pipes(state); + if (ret < 0) + return ret; + + intel_state->cdclk.pipe = ilog2(intel_state->active_crtcs); + } else if (intel_cdclk_needs_modeset(&dev_priv->cdclk.actual, + &intel_state->cdclk.actual)) { ret = intel_modeset_all_pipes(state); if (ret < 0) return ret; + + intel_state->cdclk.pipe = INVALID_PIPE; } DRM_DEBUG_KMS("New cdclk calculated to be logical %u kHz, actual %u kHz\n", @@ -12650,7 +12663,8 @@ static void intel_atomic_commit_tail(struct drm_atomic_state *state) if (intel_state->modeset) { drm_atomic_helper_update_legacy_modeset_state(state->dev, state); - intel_set_cdclk(dev_priv, &dev_priv->cdclk.actual); + intel_set_cdclk(dev_priv, &dev_priv->cdclk.actual, + intel_state->cdclk.pipe); /* * SKL workaround: bspec recommends we disable the SAGV when we diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h index 843eefaa0f0c..e5ffe42084a5 100644 --- a/drivers/gpu/drm/i915/intel_drv.h +++ b/drivers/gpu/drm/i915/intel_drv.h @@ -456,6 +456,9 @@ struct intel_atomic_state { * state only when all crtc's are DPMS off. */ struct intel_cdclk_state actual; + + /* pipe to which cd2x update is synchronized */ + enum pipe pipe; } cdclk; bool dpll_set, modeset; @@ -1464,12 +1467,16 @@ void intel_init_cdclk_hooks(struct drm_i915_private *dev_priv); void intel_update_max_cdclk(struct drm_i915_private *dev_priv); void intel_update_cdclk(struct drm_i915_private *dev_priv); void intel_update_rawclk(struct drm_i915_private *dev_priv); +bool intel_cdclk_needs_cd2x_update(struct drm_i915_private *dev_priv, + const struct intel_cdclk_state *a, + const struct intel_cdclk_state *b); bool intel_cdclk_needs_modeset(const struct intel_cdclk_state *a, const struct intel_cdclk_state *b); bool intel_cdclk_changed(const struct intel_cdclk_state *a, const struct intel_cdclk_state *b); void intel_set_cdclk(struct drm_i915_private *dev_priv, - const struct intel_cdclk_state *cdclk_state); + const struct intel_cdclk_state *cdclk_state, + enum pipe pipe); void intel_dump_cdclk_state(const struct intel_cdclk_state *cdclk_state, const char *context);