Re: [PATCH] drm/kms-helper: disable hpd_irq handling when poll=0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Apr 6, 2013 at 12:05 PM, Daniel Vetter <daniel.vetter@xxxxxxxx> wrote:
> When inlining the actual hpd output probing in
>
> commit 69787f7da6b2adc4054357a661aaa1701a9ca76f
> Author: Daniel Vetter <daniel.vetter@xxxxxxxx>
> Date:   Tue Oct 23 18:23:34 2012 +0000
>
>     drm: run the hpd irq event code directly
>
> the check for the drm_kms_hlper.poll module option was lost. This
> regressed systems where this option is used to work-around output
> probing issues (like irq storms). Restore the old behaviour.

NAK.  It's been a while since I looked at it, but the whole point of
this patch set was to be able to disabling polling independently of
hpd.  If you add this check back, setting poll=0 disables both polling
and hpd.  I'd suggest we add a separate hpd option to disable hpd.

Alex

>
> Reported-by: George Amanakis <g_amanakis@xxxxxxxxx>
> Cc: George Amanakis <g_amanakis@xxxxxxxxx>
> Cc: Dave Airlie <airlied@xxxxxxxxxx>
> Cc: stable@xxxxxxxxxxxxxxx (for 3.8)
> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
> ---
>  drivers/gpu/drm/drm_crtc_helper.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_crtc_helper.c b/drivers/gpu/drm/drm_crtc_helper.c
> index 7b2d378..3260736 100644
> --- a/drivers/gpu/drm/drm_crtc_helper.c
> +++ b/drivers/gpu/drm/drm_crtc_helper.c
> @@ -1067,7 +1067,7 @@ void drm_helper_hpd_irq_event(struct drm_device *dev)
>         enum drm_connector_status old_status;
>         bool changed = false;
>
> -       if (!dev->mode_config.poll_enabled)
> +       if (!dev->mode_config.poll_enabled || !drm_kms_helper_poll)
>                 return;
>
>         mutex_lock(&dev->mode_config.mutex);
> --
> 1.7.10.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux