2008-11-07 22:05:41 +00:00
|
|
|
/*
|
|
|
|
* Copyright (c) 2006-2008 Intel Corporation
|
|
|
|
* Copyright (c) 2007 Dave Airlie <airlied@linux.ie>
|
|
|
|
*
|
|
|
|
* DRM core CRTC related functions
|
|
|
|
*
|
|
|
|
* Permission to use, copy, modify, distribute, and sell this software and its
|
|
|
|
* documentation for any purpose is hereby granted without fee, provided that
|
|
|
|
* the above copyright notice appear in all copies and that both that copyright
|
|
|
|
* notice and this permission notice appear in supporting documentation, and
|
|
|
|
* that the name of the copyright holders not be used in advertising or
|
|
|
|
* publicity pertaining to distribution of the software without specific,
|
|
|
|
* written prior permission. The copyright holders make no representations
|
|
|
|
* about the suitability of this software for any purpose. It is provided "as
|
|
|
|
* is" without express or implied warranty.
|
|
|
|
*
|
|
|
|
* THE COPYRIGHT HOLDERS DISCLAIM ALL WARRANTIES WITH REGARD TO THIS SOFTWARE,
|
|
|
|
* INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS, IN NO
|
|
|
|
* EVENT SHALL THE COPYRIGHT HOLDERS BE LIABLE FOR ANY SPECIAL, INDIRECT OR
|
|
|
|
* CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE,
|
|
|
|
* DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER
|
|
|
|
* TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE
|
|
|
|
* OF THIS SOFTWARE.
|
|
|
|
*
|
|
|
|
* Authors:
|
|
|
|
* Keith Packard
|
|
|
|
* Eric Anholt <eric@anholt.net>
|
|
|
|
* Dave Airlie <airlied@linux.ie>
|
|
|
|
* Jesse Barnes <jesse.barnes@intel.com>
|
|
|
|
*/
|
|
|
|
|
2011-08-30 22:16:33 +00:00
|
|
|
#include <linux/export.h>
|
2019-05-26 17:35:35 +00:00
|
|
|
#include <linux/kernel.h>
|
2011-08-31 15:29:09 +00:00
|
|
|
#include <linux/moduleparam.h>
|
2011-08-30 22:16:33 +00:00
|
|
|
|
2014-11-04 21:57:27 +00:00
|
|
|
#include <drm/drm_atomic.h>
|
2019-05-26 17:35:35 +00:00
|
|
|
#include <drm/drm_atomic_helper.h>
|
2018-09-05 13:57:11 +00:00
|
|
|
#include <drm/drm_atomic_uapi.h>
|
2019-08-26 15:26:29 +00:00
|
|
|
#include <drm/drm_bridge.h>
|
2012-10-02 17:01:07 +00:00
|
|
|
#include <drm/drm_crtc.h>
|
|
|
|
#include <drm/drm_crtc_helper.h>
|
2019-05-26 17:35:35 +00:00
|
|
|
#include <drm/drm_drv.h>
|
|
|
|
#include <drm/drm_edid.h>
|
|
|
|
#include <drm/drm_encoder.h>
|
2012-10-02 17:01:07 +00:00
|
|
|
#include <drm/drm_fb_helper.h>
|
2019-05-26 17:35:35 +00:00
|
|
|
#include <drm/drm_fourcc.h>
|
2014-10-29 10:13:47 +00:00
|
|
|
#include <drm/drm_plane_helper.h>
|
2019-05-26 17:35:35 +00:00
|
|
|
#include <drm/drm_print.h>
|
|
|
|
#include <drm/drm_vblank.h>
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2019-11-19 12:58:05 +00:00
|
|
|
#include "drm_crtc_helper_internal.h"
|
|
|
|
|
2014-11-06 19:53:29 +00:00
|
|
|
/**
|
|
|
|
* DOC: overview
|
|
|
|
*
|
|
|
|
* The CRTC modeset helper library provides a default set_config implementation
|
|
|
|
* in drm_crtc_helper_set_config(). Plus a few other convenience functions using
|
|
|
|
* the same callbacks which drivers can use to e.g. restore the modeset
|
|
|
|
* configuration on resume with drm_helper_resume_force_mode().
|
|
|
|
*
|
2015-12-04 08:45:46 +00:00
|
|
|
* Note that this helper library doesn't track the current power state of CRTCs
|
2017-01-25 06:26:43 +00:00
|
|
|
* and encoders. It can call callbacks like &drm_encoder_helper_funcs.dpms even
|
|
|
|
* though the hardware is already in the desired state. This deficiency has been
|
|
|
|
* fixed in the atomic helpers.
|
2015-12-04 08:45:46 +00:00
|
|
|
*
|
2014-11-06 19:53:29 +00:00
|
|
|
* The driver callbacks are mostly compatible with the atomic modeset helpers,
|
|
|
|
* except for the handling of the primary plane: Atomic helpers require that the
|
|
|
|
* primary plane is implemented as a real standalone plane and not directly tied
|
|
|
|
* to the CRTC state. For easier transition this library provides functions to
|
|
|
|
* implement the old semantics required by the CRTC helpers using the new plane
|
|
|
|
* and atomic helper callbacks.
|
|
|
|
*
|
|
|
|
* Drivers are strongly urged to convert to the atomic helpers (by way of first
|
|
|
|
* converting to the plane helpers). New drivers must not use these functions
|
|
|
|
* but need to implement the atomic interface instead, potentially using the
|
|
|
|
* atomic helpers for that.
|
2015-12-04 08:45:44 +00:00
|
|
|
*
|
|
|
|
* These legacy modeset helpers use the same function table structures as
|
|
|
|
* all other modesetting helpers. See the documentation for struct
|
2016-12-29 20:48:26 +00:00
|
|
|
* &drm_crtc_helper_funcs, &struct drm_encoder_helper_funcs and struct
|
2015-12-04 08:45:44 +00:00
|
|
|
* &drm_connector_helper_funcs.
|
2014-11-06 19:53:29 +00:00
|
|
|
*/
|
2013-10-08 15:44:47 +00:00
|
|
|
|
2009-05-31 03:42:28 +00:00
|
|
|
/**
|
|
|
|
* drm_helper_encoder_in_use - check if a given encoder is in use
|
|
|
|
* @encoder: encoder to check
|
|
|
|
*
|
2014-01-23 21:58:26 +00:00
|
|
|
* Checks whether @encoder is with the current mode setting output configuration
|
|
|
|
* in use by any connector. This doesn't mean that it is actually enabled since
|
|
|
|
* the DPMS state is tracked separately.
|
2009-05-31 03:42:28 +00:00
|
|
|
*
|
2014-01-23 21:58:26 +00:00
|
|
|
* Returns:
|
|
|
|
* True if @encoder is used, false otherwise.
|
2009-05-31 03:42:28 +00:00
|
|
|
*/
|
|
|
|
bool drm_helper_encoder_in_use(struct drm_encoder *encoder)
|
|
|
|
{
|
|
|
|
struct drm_connector *connector;
|
2016-12-15 15:58:43 +00:00
|
|
|
struct drm_connector_list_iter conn_iter;
|
2009-05-31 03:42:28 +00:00
|
|
|
struct drm_device *dev = encoder->dev;
|
2014-01-23 21:18:47 +00:00
|
|
|
|
2019-01-10 10:30:45 +00:00
|
|
|
WARN_ON(drm_drv_uses_atomic_modeset(dev));
|
|
|
|
|
2014-05-11 22:30:48 +00:00
|
|
|
/*
|
|
|
|
* We can expect this mutex to be locked if we are not panicking.
|
|
|
|
* Locking is currently fubar in the panic handler.
|
|
|
|
*/
|
2014-06-05 10:28:59 +00:00
|
|
|
if (!oops_in_progress) {
|
2014-05-11 22:30:48 +00:00
|
|
|
WARN_ON(!mutex_is_locked(&dev->mode_config.mutex));
|
2014-06-05 10:28:59 +00:00
|
|
|
WARN_ON(!drm_modeset_is_locked(&dev->mode_config.connection_mutex));
|
|
|
|
}
|
2014-05-11 22:30:48 +00:00
|
|
|
|
2016-12-15 15:58:43 +00:00
|
|
|
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_begin(dev, &conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
drm_for_each_connector_iter(connector, &conn_iter) {
|
|
|
|
if (connector->encoder == encoder) {
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2009-05-31 03:42:28 +00:00
|
|
|
return true;
|
2016-12-15 15:58:43 +00:00
|
|
|
}
|
|
|
|
}
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2009-05-31 03:42:28 +00:00
|
|
|
return false;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_helper_encoder_in_use);
|
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
/**
|
|
|
|
* drm_helper_crtc_in_use - check if a given CRTC is in a mode_config
|
|
|
|
* @crtc: CRTC to check
|
|
|
|
*
|
2014-01-23 21:58:26 +00:00
|
|
|
* Checks whether @crtc is with the current mode setting output configuration
|
|
|
|
* in use by any connector. This doesn't mean that it is actually enabled since
|
|
|
|
* the DPMS state is tracked separately.
|
2008-11-07 22:05:41 +00:00
|
|
|
*
|
2014-01-23 21:58:26 +00:00
|
|
|
* Returns:
|
|
|
|
* True if @crtc is used, false otherwise.
|
2008-11-07 22:05:41 +00:00
|
|
|
*/
|
|
|
|
bool drm_helper_crtc_in_use(struct drm_crtc *crtc)
|
|
|
|
{
|
|
|
|
struct drm_encoder *encoder;
|
|
|
|
struct drm_device *dev = crtc->dev;
|
2014-01-23 21:18:47 +00:00
|
|
|
|
2019-01-10 10:30:45 +00:00
|
|
|
WARN_ON(drm_drv_uses_atomic_modeset(dev));
|
|
|
|
|
2014-05-11 22:30:48 +00:00
|
|
|
/*
|
|
|
|
* We can expect this mutex to be locked if we are not panicking.
|
|
|
|
* Locking is currently fubar in the panic handler.
|
|
|
|
*/
|
|
|
|
if (!oops_in_progress)
|
|
|
|
WARN_ON(!mutex_is_locked(&dev->mode_config.mutex));
|
|
|
|
|
2015-07-09 21:44:35 +00:00
|
|
|
drm_for_each_encoder(encoder, dev)
|
2009-05-31 03:42:28 +00:00
|
|
|
if (encoder->crtc == crtc && drm_helper_encoder_in_use(encoder))
|
2008-11-07 22:05:41 +00:00
|
|
|
return true;
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_helper_crtc_in_use);
|
|
|
|
|
2010-07-01 06:49:57 +00:00
|
|
|
static void
|
|
|
|
drm_encoder_disable(struct drm_encoder *encoder)
|
|
|
|
{
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_encoder_helper_funcs *encoder_funcs = encoder->helper_private;
|
2010-07-01 06:49:57 +00:00
|
|
|
|
2016-05-05 13:24:32 +00:00
|
|
|
if (!encoder_funcs)
|
|
|
|
return;
|
|
|
|
|
2010-07-01 06:49:57 +00:00
|
|
|
if (encoder_funcs->disable)
|
|
|
|
(*encoder_funcs->disable)(encoder);
|
2016-05-05 13:24:32 +00:00
|
|
|
else if (encoder_funcs->dpms)
|
2010-07-01 06:49:57 +00:00
|
|
|
(*encoder_funcs->dpms)(encoder, DRM_MODE_DPMS_OFF);
|
|
|
|
}
|
|
|
|
|
2014-03-20 13:26:34 +00:00
|
|
|
static void __drm_helper_disable_unused_functions(struct drm_device *dev)
|
2008-11-07 22:05:41 +00:00
|
|
|
{
|
|
|
|
struct drm_encoder *encoder;
|
|
|
|
struct drm_crtc *crtc;
|
|
|
|
|
2014-01-23 21:18:47 +00:00
|
|
|
drm_warn_on_modeset_not_all_locked(dev);
|
|
|
|
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2010-12-21 02:47:56 +00:00
|
|
|
if (!drm_helper_encoder_in_use(encoder)) {
|
2010-07-01 06:49:57 +00:00
|
|
|
drm_encoder_disable(encoder);
|
2014-05-24 18:30:10 +00:00
|
|
|
/* disconnect encoder from any connector */
|
2009-09-02 04:00:11 +00:00
|
|
|
encoder->crtc = NULL;
|
2009-08-31 05:16:30 +00:00
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_crtc(crtc, dev) {
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
|
2008-11-07 22:05:41 +00:00
|
|
|
crtc->enabled = drm_helper_crtc_in_use(crtc);
|
|
|
|
if (!crtc->enabled) {
|
2010-06-11 21:04:35 +00:00
|
|
|
if (crtc_funcs->disable)
|
|
|
|
(*crtc_funcs->disable)(crtc);
|
|
|
|
else
|
|
|
|
(*crtc_funcs->dpms)(crtc, DRM_MODE_DPMS_OFF);
|
2014-04-01 22:22:40 +00:00
|
|
|
crtc->primary->fb = NULL;
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2014-03-20 13:26:34 +00:00
|
|
|
|
|
|
|
/**
|
|
|
|
* drm_helper_disable_unused_functions - disable unused objects
|
|
|
|
* @dev: DRM device
|
|
|
|
*
|
|
|
|
* This function walks through the entire mode setting configuration of @dev. It
|
2015-12-04 08:45:46 +00:00
|
|
|
* will remove any CRTC links of unused encoders and encoder links of
|
|
|
|
* disconnected connectors. Then it will disable all unused encoders and CRTCs
|
2014-03-20 13:26:34 +00:00
|
|
|
* either by calling their disable callback if available or by calling their
|
|
|
|
* dpms callback with DRM_MODE_DPMS_OFF.
|
2016-01-13 14:31:16 +00:00
|
|
|
*
|
|
|
|
* NOTE:
|
|
|
|
*
|
|
|
|
* This function is part of the legacy modeset helper library and will cause
|
|
|
|
* major confusion with atomic drivers. This is because atomic helpers guarantee
|
|
|
|
* to never call ->disable() hooks on a disabled function, or ->enable() hooks
|
|
|
|
* on an enabled functions. drm_helper_disable_unused_functions() on the other
|
|
|
|
* hand throws such guarantees into the wind and calls disable hooks
|
|
|
|
* unconditionally on unused functions.
|
2014-03-20 13:26:34 +00:00
|
|
|
*/
|
|
|
|
void drm_helper_disable_unused_functions(struct drm_device *dev)
|
|
|
|
{
|
2019-01-10 10:30:45 +00:00
|
|
|
WARN_ON(drm_drv_uses_atomic_modeset(dev));
|
2016-06-08 12:19:18 +00:00
|
|
|
|
2014-03-20 13:26:34 +00:00
|
|
|
drm_modeset_lock_all(dev);
|
|
|
|
__drm_helper_disable_unused_functions(dev);
|
|
|
|
drm_modeset_unlock_all(dev);
|
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
EXPORT_SYMBOL(drm_helper_disable_unused_functions);
|
|
|
|
|
2009-02-24 00:09:34 +00:00
|
|
|
/*
|
|
|
|
* Check the CRTC we're going to map each output to vs. its current
|
|
|
|
* CRTC. If they don't match, we have to disable the output and the CRTC
|
|
|
|
* since the driver will have to re-route things.
|
|
|
|
*/
|
|
|
|
static void
|
|
|
|
drm_crtc_prepare_encoders(struct drm_device *dev)
|
|
|
|
{
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_encoder_helper_funcs *encoder_funcs;
|
2009-02-24 00:09:34 +00:00
|
|
|
struct drm_encoder *encoder;
|
|
|
|
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2009-02-24 00:09:34 +00:00
|
|
|
encoder_funcs = encoder->helper_private;
|
2016-05-05 13:24:32 +00:00
|
|
|
if (!encoder_funcs)
|
|
|
|
continue;
|
|
|
|
|
2009-02-24 00:09:34 +00:00
|
|
|
/* Disable unused encoders */
|
|
|
|
if (encoder->crtc == NULL)
|
2010-07-01 06:49:57 +00:00
|
|
|
drm_encoder_disable(encoder);
|
2009-02-24 00:09:34 +00:00
|
|
|
/* Disable encoders whose CRTC is about to change */
|
|
|
|
if (encoder_funcs->get_crtc &&
|
|
|
|
encoder->crtc != (*encoder_funcs->get_crtc)(encoder))
|
2010-07-01 06:49:57 +00:00
|
|
|
drm_encoder_disable(encoder);
|
2009-02-24 00:09:34 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
/**
|
2012-11-01 13:45:16 +00:00
|
|
|
* drm_crtc_helper_set_mode - internal helper to set a mode
|
2008-11-07 22:05:41 +00:00
|
|
|
* @crtc: CRTC to program
|
|
|
|
* @mode: mode to use
|
2012-11-09 17:26:32 +00:00
|
|
|
* @x: horizontal offset into the surface
|
|
|
|
* @y: vertical offset into the surface
|
2012-11-01 13:45:16 +00:00
|
|
|
* @old_fb: old framebuffer, for cleanup
|
2008-11-07 22:05:41 +00:00
|
|
|
*
|
|
|
|
* Try to set @mode on @crtc. Give @crtc and its associated connectors a chance
|
2012-11-01 13:45:16 +00:00
|
|
|
* to fixup or reject the mode prior to trying to set it. This is an internal
|
|
|
|
* helper that drivers could e.g. use to update properties that require the
|
|
|
|
* entire output pipe to be disabled and re-enabled in a new configuration. For
|
|
|
|
* example for changing whether audio is enabled on a hdmi link or for changing
|
|
|
|
* panel fitter or dither attributes. It is also called by the
|
|
|
|
* drm_crtc_helper_set_config() helper function to drive the mode setting
|
|
|
|
* sequence.
|
2008-11-07 22:05:41 +00:00
|
|
|
*
|
2014-01-23 21:58:26 +00:00
|
|
|
* Returns:
|
|
|
|
* True if the mode was set successfully, false otherwise.
|
2008-11-07 22:05:41 +00:00
|
|
|
*/
|
|
|
|
bool drm_crtc_helper_set_mode(struct drm_crtc *crtc,
|
|
|
|
struct drm_display_mode *mode,
|
2008-12-18 03:14:46 +00:00
|
|
|
int x, int y,
|
|
|
|
struct drm_framebuffer *old_fb)
|
2008-11-07 22:05:41 +00:00
|
|
|
{
|
|
|
|
struct drm_device *dev = crtc->dev;
|
2015-03-19 04:33:03 +00:00
|
|
|
struct drm_display_mode *adjusted_mode, saved_mode, saved_hwmode;
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
|
|
|
|
const struct drm_encoder_helper_funcs *encoder_funcs;
|
2008-11-07 22:05:41 +00:00
|
|
|
int saved_x, saved_y;
|
2013-10-29 15:09:44 +00:00
|
|
|
bool saved_enabled;
|
2008-11-07 22:05:41 +00:00
|
|
|
struct drm_encoder *encoder;
|
|
|
|
bool ret = true;
|
|
|
|
|
2019-01-10 10:30:45 +00:00
|
|
|
WARN_ON(drm_drv_uses_atomic_modeset(dev));
|
|
|
|
|
2014-01-23 21:18:47 +00:00
|
|
|
drm_warn_on_modeset_not_all_locked(dev);
|
|
|
|
|
2013-10-29 15:09:44 +00:00
|
|
|
saved_enabled = crtc->enabled;
|
2008-11-07 22:05:41 +00:00
|
|
|
crtc->enabled = drm_helper_crtc_in_use(crtc);
|
|
|
|
if (!crtc->enabled)
|
|
|
|
return true;
|
|
|
|
|
2011-01-28 11:31:56 +00:00
|
|
|
adjusted_mode = drm_mode_duplicate(dev, mode);
|
2013-10-29 15:09:44 +00:00
|
|
|
if (!adjusted_mode) {
|
|
|
|
crtc->enabled = saved_enabled;
|
2012-03-13 10:35:48 +00:00
|
|
|
return false;
|
2013-10-29 15:09:44 +00:00
|
|
|
}
|
2011-01-28 11:31:56 +00:00
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
saved_mode = crtc->mode;
|
2015-03-19 04:33:03 +00:00
|
|
|
saved_hwmode = crtc->hwmode;
|
2008-11-07 22:05:41 +00:00
|
|
|
saved_x = crtc->x;
|
|
|
|
saved_y = crtc->y;
|
|
|
|
|
|
|
|
/* Update crtc values up front so the driver can rely on them for mode
|
|
|
|
* setting.
|
|
|
|
*/
|
|
|
|
crtc->mode = *mode;
|
|
|
|
crtc->x = x;
|
|
|
|
crtc->y = y;
|
|
|
|
|
|
|
|
/* Pass our mode to the connectors and the CRTC to give them a chance to
|
|
|
|
* adjust it according to limitations or connector properties, and also
|
|
|
|
* a chance to reject the mode entirely.
|
|
|
|
*/
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2008-11-07 22:05:41 +00:00
|
|
|
|
|
|
|
if (encoder->crtc != crtc)
|
|
|
|
continue;
|
2013-08-14 20:47:37 +00:00
|
|
|
|
2016-05-05 13:24:32 +00:00
|
|
|
encoder_funcs = encoder->helper_private;
|
|
|
|
if (!encoder_funcs)
|
|
|
|
continue;
|
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
encoder_funcs = encoder->helper_private;
|
2016-02-10 12:15:22 +00:00
|
|
|
if (encoder_funcs->mode_fixup) {
|
|
|
|
if (!(ret = encoder_funcs->mode_fixup(encoder, mode,
|
|
|
|
adjusted_mode))) {
|
|
|
|
DRM_DEBUG_KMS("Encoder fixup failed\n");
|
|
|
|
goto done;
|
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-02-16 14:10:03 +00:00
|
|
|
if (crtc_funcs->mode_fixup) {
|
|
|
|
if (!(ret = crtc_funcs->mode_fixup(crtc, mode,
|
|
|
|
adjusted_mode))) {
|
|
|
|
DRM_DEBUG_KMS("CRTC fixup failed\n");
|
|
|
|
goto done;
|
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
2015-12-08 16:41:53 +00:00
|
|
|
DRM_DEBUG_KMS("[CRTC:%d:%s]\n", crtc->base.id, crtc->name);
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2015-03-19 04:33:03 +00:00
|
|
|
crtc->hwmode = *adjusted_mode;
|
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
/* Prepare the encoders and CRTCs before setting the mode. */
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2008-11-07 22:05:41 +00:00
|
|
|
|
|
|
|
if (encoder->crtc != crtc)
|
|
|
|
continue;
|
2013-08-14 20:47:37 +00:00
|
|
|
|
2016-05-05 13:24:32 +00:00
|
|
|
encoder_funcs = encoder->helper_private;
|
|
|
|
if (!encoder_funcs)
|
|
|
|
continue;
|
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
/* Disable the encoders as the first thing we do. */
|
2016-05-05 13:24:32 +00:00
|
|
|
if (encoder_funcs->prepare)
|
|
|
|
encoder_funcs->prepare(encoder);
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
|
2009-02-24 00:09:34 +00:00
|
|
|
drm_crtc_prepare_encoders(dev);
|
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
crtc_funcs->prepare(crtc);
|
|
|
|
|
|
|
|
/* Set up the DPLL and any encoders state that needs to adjust or depend
|
|
|
|
* on the DPLL.
|
|
|
|
*/
|
2009-02-11 13:25:09 +00:00
|
|
|
ret = !crtc_funcs->mode_set(crtc, mode, adjusted_mode, x, y, old_fb);
|
|
|
|
if (!ret)
|
|
|
|
goto done;
|
2008-11-07 22:05:41 +00:00
|
|
|
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2008-11-07 22:05:41 +00:00
|
|
|
|
|
|
|
if (encoder->crtc != crtc)
|
|
|
|
continue;
|
|
|
|
|
2016-05-05 13:24:32 +00:00
|
|
|
encoder_funcs = encoder->helper_private;
|
|
|
|
if (!encoder_funcs)
|
|
|
|
continue;
|
|
|
|
|
2019-01-11 14:45:48 +00:00
|
|
|
DRM_DEBUG_KMS("[ENCODER:%d:%s] set [MODE:%s]\n",
|
|
|
|
encoder->base.id, encoder->name, mode->name);
|
2016-05-05 13:24:32 +00:00
|
|
|
if (encoder_funcs->mode_set)
|
|
|
|
encoder_funcs->mode_set(encoder, mode, adjusted_mode);
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
/* Now enable the clocks, plane, pipe, and connectors that we set up. */
|
|
|
|
crtc_funcs->commit(crtc);
|
|
|
|
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2008-11-07 22:05:41 +00:00
|
|
|
|
|
|
|
if (encoder->crtc != crtc)
|
|
|
|
continue;
|
|
|
|
|
2016-05-05 13:24:32 +00:00
|
|
|
encoder_funcs = encoder->helper_private;
|
|
|
|
if (!encoder_funcs)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
if (encoder_funcs->commit)
|
|
|
|
encoder_funcs->commit(encoder);
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
|
drm/vblank: Add support for precise vblank timestamping.
The DRI2 swap & sync implementation needs precise
vblank counts and precise timestamps corresponding
to those vblank counts. For conformance to the OpenML
OML_sync_control extension specification the DRM
timestamp associated with a vblank count should
correspond to the start of video scanout of the first
scanline of the video frame following the vblank
interval for that vblank count.
Therefore we need to carry around precise timestamps
for vblanks. Currently the DRM and KMS drivers generate
timestamps ad-hoc via do_gettimeofday() in some
places. The resulting timestamps are sometimes not
very precise due to interrupt handling delays, they
don't conform to OML_sync_control and some are wrong,
as they aren't taken synchronized to the vblank.
This patch implements support inside the drm core
for precise and robust timestamping. It consists
of the following interrelated pieces.
1. Vblank timestamp caching:
A per-crtc ringbuffer stores the most recent vblank
timestamps corresponding to vblank counts.
The ringbuffer can be read out lock-free via the
accessor function:
struct timeval timestamp;
vblankcount = drm_vblank_count_and_time(dev, crtcid, ×tamp).
The function returns the current vblank count and
the corresponding timestamp for start of video
scanout following the vblank interval. It can be
used anywhere between enclosing drm_vblank_get(dev, crtcid)
and drm_vblank_put(dev,crtcid) statements. It is used
inside the drmWaitVblank ioctl and in the vblank event
queueing and handling. It should be used by kms drivers for
timestamping of bufferswap completion.
The timestamp ringbuffer is reinitialized each time
vblank irq's get reenabled in drm_vblank_get()/
drm_update_vblank_count(). It is invalidated when
vblank irq's get disabled.
The ringbuffer is updated inside drm_handle_vblank()
at each vblank irq.
2. Calculation of precise vblank timestamps:
drm_get_last_vbltimestamp() is used to compute the
timestamp for the end of the most recent vblank (if
inside active scanout), or the expected end of the
current vblank interval (if called inside a vblank
interval). The function calls into a new optional kms
driver entry point dev->driver->get_vblank_timestamp()
which is supposed to provide the precise timestamp.
If a kms driver doesn't implement the entry point or
if the call fails, a simple do_gettimeofday() timestamp
is returned as crude approximation of the true vblank time.
A new drm module parameter drm.timestamp_precision_usec
allows to disable high precision timestamps (if set to
zero) or to specify the maximum acceptable error in
the timestamps in microseconds.
Kms drivers could implement their get_vblank_timestamp()
function in a gpu specific way, as long as returned
timestamps conform to OML_sync_control, e.g., by use
of gpu specific hardware timestamps.
Optionally, kms drivers can simply wrap and use the new
utility function drm_calc_vbltimestamp_from_scanoutpos().
This function calls a new optional kms driver function
dev->driver->get_scanout_position() which returns the
current horizontal and vertical video scanout position
of the crtc. The scanout position together with the
drm_display_timing of the current video mode is used
to calculate elapsed time relative to start of active scanout
for the current video frame. This elapsed time is subtracted
from the current do_gettimeofday() time to get the timestamp
corresponding to start of video scanout. Currently
non-interlaced, non-doublescan video modes, with or
without panel scaling are handled correctly. Interlaced/
doublescan modes are tbd in a future patch.
3. Filtering of redundant vblank irq's and removal of
some race-conditions in the vblank irq enable/disable path:
Some gpu's (e.g., Radeon R500/R600) send spurious vblank
irq's outside the vblank if vblank irq's get reenabled.
These get detected by use of the vblank timestamps and
filtered out to avoid miscounting of vblanks.
Some race-conditions between the vblank irq enable/disable
functions, the vblank irq handler and the gpu itself (updating
its hardware vblank counter in the "wrong" moment) are
fixed inside vblank_disable_and_save() and
drm_update_vblank_count() by use of the vblank timestamps and
a new spinlock dev->vblank_time_lock.
The time until vblank irq disable is now configurable via
a new drm module parameter drm.vblankoffdelay to allow
experimentation with timeouts that are much shorter than
the current 5 seconds and should allow longer vblank off
periods for better power savings.
Followup patches will use these new functions to
implement precise timestamping for the intel and radeon
kms drivers.
Signed-off-by: Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
Signed-off-by: Dave Airlie <airlied@redhat.com>
2010-10-23 02:20:23 +00:00
|
|
|
/* Calculate and store various constants which
|
|
|
|
* are later needed by vblank and swap-completion
|
|
|
|
* timestamping. They are derived from true hwmode.
|
|
|
|
*/
|
2013-10-26 14:16:30 +00:00
|
|
|
drm_calc_timestamping_constants(crtc, &crtc->hwmode);
|
drm/vblank: Add support for precise vblank timestamping.
The DRI2 swap & sync implementation needs precise
vblank counts and precise timestamps corresponding
to those vblank counts. For conformance to the OpenML
OML_sync_control extension specification the DRM
timestamp associated with a vblank count should
correspond to the start of video scanout of the first
scanline of the video frame following the vblank
interval for that vblank count.
Therefore we need to carry around precise timestamps
for vblanks. Currently the DRM and KMS drivers generate
timestamps ad-hoc via do_gettimeofday() in some
places. The resulting timestamps are sometimes not
very precise due to interrupt handling delays, they
don't conform to OML_sync_control and some are wrong,
as they aren't taken synchronized to the vblank.
This patch implements support inside the drm core
for precise and robust timestamping. It consists
of the following interrelated pieces.
1. Vblank timestamp caching:
A per-crtc ringbuffer stores the most recent vblank
timestamps corresponding to vblank counts.
The ringbuffer can be read out lock-free via the
accessor function:
struct timeval timestamp;
vblankcount = drm_vblank_count_and_time(dev, crtcid, ×tamp).
The function returns the current vblank count and
the corresponding timestamp for start of video
scanout following the vblank interval. It can be
used anywhere between enclosing drm_vblank_get(dev, crtcid)
and drm_vblank_put(dev,crtcid) statements. It is used
inside the drmWaitVblank ioctl and in the vblank event
queueing and handling. It should be used by kms drivers for
timestamping of bufferswap completion.
The timestamp ringbuffer is reinitialized each time
vblank irq's get reenabled in drm_vblank_get()/
drm_update_vblank_count(). It is invalidated when
vblank irq's get disabled.
The ringbuffer is updated inside drm_handle_vblank()
at each vblank irq.
2. Calculation of precise vblank timestamps:
drm_get_last_vbltimestamp() is used to compute the
timestamp for the end of the most recent vblank (if
inside active scanout), or the expected end of the
current vblank interval (if called inside a vblank
interval). The function calls into a new optional kms
driver entry point dev->driver->get_vblank_timestamp()
which is supposed to provide the precise timestamp.
If a kms driver doesn't implement the entry point or
if the call fails, a simple do_gettimeofday() timestamp
is returned as crude approximation of the true vblank time.
A new drm module parameter drm.timestamp_precision_usec
allows to disable high precision timestamps (if set to
zero) or to specify the maximum acceptable error in
the timestamps in microseconds.
Kms drivers could implement their get_vblank_timestamp()
function in a gpu specific way, as long as returned
timestamps conform to OML_sync_control, e.g., by use
of gpu specific hardware timestamps.
Optionally, kms drivers can simply wrap and use the new
utility function drm_calc_vbltimestamp_from_scanoutpos().
This function calls a new optional kms driver function
dev->driver->get_scanout_position() which returns the
current horizontal and vertical video scanout position
of the crtc. The scanout position together with the
drm_display_timing of the current video mode is used
to calculate elapsed time relative to start of active scanout
for the current video frame. This elapsed time is subtracted
from the current do_gettimeofday() time to get the timestamp
corresponding to start of video scanout. Currently
non-interlaced, non-doublescan video modes, with or
without panel scaling are handled correctly. Interlaced/
doublescan modes are tbd in a future patch.
3. Filtering of redundant vblank irq's and removal of
some race-conditions in the vblank irq enable/disable path:
Some gpu's (e.g., Radeon R500/R600) send spurious vblank
irq's outside the vblank if vblank irq's get reenabled.
These get detected by use of the vblank timestamps and
filtered out to avoid miscounting of vblanks.
Some race-conditions between the vblank irq enable/disable
functions, the vblank irq handler and the gpu itself (updating
its hardware vblank counter in the "wrong" moment) are
fixed inside vblank_disable_and_save() and
drm_update_vblank_count() by use of the vblank timestamps and
a new spinlock dev->vblank_time_lock.
The time until vblank irq disable is now configurable via
a new drm module parameter drm.vblankoffdelay to allow
experimentation with timeouts that are much shorter than
the current 5 seconds and should allow longer vblank off
periods for better power savings.
Followup patches will use these new functions to
implement precise timestamping for the intel and radeon
kms drivers.
Signed-off-by: Mario Kleiner <mario.kleiner@tuebingen.mpg.de>
Signed-off-by: Dave Airlie <airlied@redhat.com>
2010-10-23 02:20:23 +00:00
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
/* FIXME: add subpixel order */
|
|
|
|
done:
|
2011-01-28 11:31:56 +00:00
|
|
|
drm_mode_destroy(dev, adjusted_mode);
|
2008-11-07 22:05:41 +00:00
|
|
|
if (!ret) {
|
2013-10-29 15:09:44 +00:00
|
|
|
crtc->enabled = saved_enabled;
|
2008-11-07 22:05:41 +00:00
|
|
|
crtc->mode = saved_mode;
|
2015-03-19 04:33:03 +00:00
|
|
|
crtc->hwmode = saved_hwmode;
|
2008-11-07 22:05:41 +00:00
|
|
|
crtc->x = saved_x;
|
|
|
|
crtc->y = saved_y;
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_crtc_helper_set_mode);
|
|
|
|
|
2014-04-29 09:44:39 +00:00
|
|
|
static void
|
2011-11-28 21:10:05 +00:00
|
|
|
drm_crtc_helper_disable(struct drm_crtc *crtc)
|
|
|
|
{
|
|
|
|
struct drm_device *dev = crtc->dev;
|
|
|
|
struct drm_connector *connector;
|
|
|
|
struct drm_encoder *encoder;
|
|
|
|
|
|
|
|
/* Decouple all encoders and their attached connectors from this crtc */
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2016-12-15 15:58:43 +00:00
|
|
|
struct drm_connector_list_iter conn_iter;
|
|
|
|
|
2011-11-28 21:10:05 +00:00
|
|
|
if (encoder->crtc != crtc)
|
|
|
|
continue;
|
|
|
|
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_begin(dev, &conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
drm_for_each_connector_iter(connector, &conn_iter) {
|
2011-11-28 21:10:05 +00:00
|
|
|
if (connector->encoder != encoder)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
connector->encoder = NULL;
|
drm: Track the proper DPMS mode of connectors
When userspace removes the active framebuffer using DRM_IOCTL_MODE_RMFB,
or explicitly disables the CRTC (by calling drmModeSetCrtc(..., NULL)
for example), a NULL framebuffer will be passed to the .set_config()
implementation of a CRTC. The drm_crtc_helper_set_config() helper will
decide to disable a CRTC when that happens.
To do so, it calls drm_crtc_helper_disable(), which in turn will iterate
over all encoders and decouple them from their connectors and finally
call drm_helper_disable_unused_functions() to clean up and call the
.disable() or .dpms() implementation for each encoder. However, at no
point during this sequence does it track the DPMS mode of a connector,
so it will usually remain on after this.
When a connector is enabled again, drm_helper_connector_dpms() will not
notice that the DPMS mode actually changed and won't do anything, which
causes the connector to stay disabled indefinitely.
To prevent this from happening, explicitly set the connector's DPMS mode
to off when the CRTC is disabled. That way it reflects the correct state
and can be enabled again.
This solves an issue observed when terminating an X server running on
the xf86-video-modesetting driver. Without this patch, the connector
would not be enabled properly and the screen would stay dark.
Acked-by: David Airlie <airlied@linux.ie>
Signed-off-by: Thierry Reding <treding@nvidia.com>
2013-10-02 13:50:06 +00:00
|
|
|
|
|
|
|
/*
|
|
|
|
* drm_helper_disable_unused_functions() ought to be
|
|
|
|
* doing this, but since we've decoupled the encoder
|
|
|
|
* from the connector above, the required connection
|
|
|
|
* between them is henceforth no longer available.
|
|
|
|
*/
|
|
|
|
connector->dpms = DRM_MODE_DPMS_OFF;
|
2016-04-27 01:27:54 +00:00
|
|
|
|
|
|
|
/* we keep a reference while the encoder is bound */
|
2017-02-28 14:46:39 +00:00
|
|
|
drm_connector_put(connector);
|
2011-11-28 21:10:05 +00:00
|
|
|
}
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2011-11-28 21:10:05 +00:00
|
|
|
}
|
|
|
|
|
2014-03-20 13:26:34 +00:00
|
|
|
__drm_helper_disable_unused_functions(dev);
|
2011-11-28 21:10:05 +00:00
|
|
|
}
|
|
|
|
|
2019-09-13 23:28:56 +00:00
|
|
|
/*
|
|
|
|
* For connectors that support multiple encoders, either the
|
|
|
|
* .atomic_best_encoder() or .best_encoder() operation must be implemented.
|
|
|
|
*/
|
|
|
|
struct drm_encoder *
|
|
|
|
drm_connector_get_single_encoder(struct drm_connector *connector)
|
|
|
|
{
|
2019-09-13 23:28:57 +00:00
|
|
|
struct drm_encoder *encoder;
|
|
|
|
|
|
|
|
WARN_ON(hweight32(connector->possible_encoders) > 1);
|
|
|
|
drm_connector_for_each_possible_encoder(connector, encoder)
|
|
|
|
return encoder;
|
|
|
|
|
|
|
|
return NULL;
|
2019-09-13 23:28:56 +00:00
|
|
|
}
|
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
/**
|
|
|
|
* drm_crtc_helper_set_config - set a new config from userspace
|
2012-11-01 13:45:16 +00:00
|
|
|
* @set: mode set configuration
|
2017-03-22 21:50:57 +00:00
|
|
|
* @ctx: lock acquire context, not used here
|
2008-11-07 22:05:41 +00:00
|
|
|
*
|
2017-01-25 06:26:43 +00:00
|
|
|
* The drm_crtc_helper_set_config() helper function implements the of
|
|
|
|
* &drm_crtc_funcs.set_config callback for drivers using the legacy CRTC
|
|
|
|
* helpers.
|
2015-12-04 08:45:46 +00:00
|
|
|
*
|
|
|
|
* It first tries to locate the best encoder for each connector by calling the
|
2017-01-25 06:26:43 +00:00
|
|
|
* connector @drm_connector_helper_funcs.best_encoder helper operation.
|
2015-12-04 08:45:46 +00:00
|
|
|
*
|
|
|
|
* After locating the appropriate encoders, the helper function will call the
|
|
|
|
* mode_fixup encoder and CRTC helper operations to adjust the requested mode,
|
|
|
|
* or reject it completely in which case an error will be returned to the
|
|
|
|
* application. If the new configuration after mode adjustment is identical to
|
|
|
|
* the current configuration the helper function will return without performing
|
|
|
|
* any other operation.
|
|
|
|
*
|
|
|
|
* If the adjusted mode is identical to the current mode but changes to the
|
|
|
|
* frame buffer need to be applied, the drm_crtc_helper_set_config() function
|
2017-01-25 06:26:43 +00:00
|
|
|
* will call the CRTC &drm_crtc_helper_funcs.mode_set_base helper operation.
|
2015-12-04 08:45:46 +00:00
|
|
|
*
|
|
|
|
* If the adjusted mode differs from the current mode, or if the
|
|
|
|
* ->mode_set_base() helper operation is not provided, the helper function
|
|
|
|
* performs a full mode set sequence by calling the ->prepare(), ->mode_set()
|
|
|
|
* and ->commit() CRTC and encoder helper operations, in that order.
|
|
|
|
* Alternatively it can also use the dpms and disable helper operations. For
|
2016-12-29 20:48:26 +00:00
|
|
|
* details see &struct drm_crtc_helper_funcs and struct
|
2015-12-04 08:45:46 +00:00
|
|
|
* &drm_encoder_helper_funcs.
|
|
|
|
*
|
|
|
|
* This function is deprecated. New drivers must implement atomic modeset
|
|
|
|
* support, for which this function is unsuitable. Instead drivers should use
|
|
|
|
* drm_atomic_helper_set_config().
|
2008-11-07 22:05:41 +00:00
|
|
|
*
|
2014-01-23 21:58:26 +00:00
|
|
|
* Returns:
|
|
|
|
* Returns 0 on success, negative errno numbers on failure.
|
2008-11-07 22:05:41 +00:00
|
|
|
*/
|
2017-03-22 21:50:57 +00:00
|
|
|
int drm_crtc_helper_set_config(struct drm_mode_set *set,
|
|
|
|
struct drm_modeset_acquire_ctx *ctx)
|
2008-11-07 22:05:41 +00:00
|
|
|
{
|
|
|
|
struct drm_device *dev;
|
2016-06-02 17:27:52 +00:00
|
|
|
struct drm_crtc **save_encoder_crtcs, *new_crtc;
|
|
|
|
struct drm_encoder **save_connector_encoders, *new_encoder, *encoder;
|
2009-08-03 12:43:59 +00:00
|
|
|
bool mode_changed = false; /* if true do a full mode set */
|
|
|
|
bool fb_changed = false; /* if true and !mode_changed just do a flip */
|
2016-06-02 17:27:52 +00:00
|
|
|
struct drm_connector *connector;
|
2016-12-15 15:58:43 +00:00
|
|
|
struct drm_connector_list_iter conn_iter;
|
2008-11-07 22:05:41 +00:00
|
|
|
int count = 0, ro, fail = 0;
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_crtc_helper_funcs *crtc_funcs;
|
2011-11-07 18:39:57 +00:00
|
|
|
struct drm_mode_set save_set;
|
2012-05-17 11:27:21 +00:00
|
|
|
int ret;
|
2010-11-26 18:45:58 +00:00
|
|
|
int i;
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2009-07-20 05:48:07 +00:00
|
|
|
DRM_DEBUG_KMS("\n");
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2013-06-14 22:13:11 +00:00
|
|
|
BUG_ON(!set);
|
|
|
|
BUG_ON(!set->crtc);
|
|
|
|
BUG_ON(!set->crtc->helper_private);
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2013-06-14 22:13:11 +00:00
|
|
|
/* Enforce sane interface api - has been abused by the fb helper. */
|
|
|
|
BUG_ON(!set->mode && set->fb);
|
|
|
|
BUG_ON(set->fb && set->num_connectors == 0);
|
2008-11-07 22:05:41 +00:00
|
|
|
|
|
|
|
crtc_funcs = set->crtc->helper_private;
|
|
|
|
|
2019-01-10 10:30:45 +00:00
|
|
|
dev = set->crtc->dev;
|
|
|
|
WARN_ON(drm_drv_uses_atomic_modeset(dev));
|
|
|
|
|
2011-01-31 11:16:33 +00:00
|
|
|
if (!set->mode)
|
|
|
|
set->fb = NULL;
|
|
|
|
|
2010-07-15 19:43:25 +00:00
|
|
|
if (set->fb) {
|
2015-12-08 16:41:53 +00:00
|
|
|
DRM_DEBUG_KMS("[CRTC:%d:%s] [FB:%d] #connectors=%d (x y) (%i %i)\n",
|
|
|
|
set->crtc->base.id, set->crtc->name,
|
|
|
|
set->fb->base.id,
|
|
|
|
(int)set->num_connectors, set->x, set->y);
|
2010-07-15 19:43:25 +00:00
|
|
|
} else {
|
2015-12-08 16:41:53 +00:00
|
|
|
DRM_DEBUG_KMS("[CRTC:%d:%s] [NOFB]\n",
|
|
|
|
set->crtc->base.id, set->crtc->name);
|
2014-04-29 09:44:39 +00:00
|
|
|
drm_crtc_helper_disable(set->crtc);
|
|
|
|
return 0;
|
2010-07-15 19:43:25 +00:00
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2014-01-23 21:18:47 +00:00
|
|
|
drm_warn_on_modeset_not_all_locked(dev);
|
|
|
|
|
2013-10-29 15:09:46 +00:00
|
|
|
/*
|
|
|
|
* Allocate space for the backup of all (non-pointer) encoder and
|
|
|
|
* connector data.
|
|
|
|
*/
|
2017-10-13 07:37:47 +00:00
|
|
|
save_encoder_crtcs = kcalloc(dev->mode_config.num_encoder,
|
2016-06-02 17:27:52 +00:00
|
|
|
sizeof(struct drm_crtc *), GFP_KERNEL);
|
|
|
|
if (!save_encoder_crtcs)
|
2008-11-07 22:05:41 +00:00
|
|
|
return -ENOMEM;
|
|
|
|
|
2017-10-13 07:37:47 +00:00
|
|
|
save_connector_encoders = kcalloc(dev->mode_config.num_connector,
|
2016-06-02 17:27:52 +00:00
|
|
|
sizeof(struct drm_encoder *), GFP_KERNEL);
|
|
|
|
if (!save_connector_encoders) {
|
|
|
|
kfree(save_encoder_crtcs);
|
2009-08-27 08:18:29 +00:00
|
|
|
return -ENOMEM;
|
|
|
|
}
|
|
|
|
|
2013-10-29 15:09:46 +00:00
|
|
|
/*
|
|
|
|
* Copy data. Note that driver private data is not affected.
|
2009-08-27 08:18:29 +00:00
|
|
|
* Should anything bad happen only the expected state is
|
|
|
|
* restored, not the drivers personal bookkeeping.
|
|
|
|
*/
|
|
|
|
count = 0;
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2016-06-02 17:27:52 +00:00
|
|
|
save_encoder_crtcs[count++] = encoder->crtc;
|
2009-08-27 08:18:29 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
count = 0;
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_begin(dev, &conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
drm_for_each_connector_iter(connector, &conn_iter)
|
2016-06-02 17:27:52 +00:00
|
|
|
save_connector_encoders[count++] = connector->encoder;
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2009-08-27 08:18:29 +00:00
|
|
|
|
2011-11-07 18:39:57 +00:00
|
|
|
save_set.crtc = set->crtc;
|
|
|
|
save_set.mode = &set->crtc->mode;
|
|
|
|
save_set.x = set->crtc->x;
|
|
|
|
save_set.y = set->crtc->y;
|
2014-04-01 22:22:40 +00:00
|
|
|
save_set.fb = set->crtc->primary->fb;
|
2011-11-07 18:39:57 +00:00
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
/* We should be able to check here if the fb has the same properties
|
|
|
|
* and then just flip_or_move it */
|
2014-04-01 22:22:40 +00:00
|
|
|
if (set->crtc->primary->fb != set->fb) {
|
2009-01-09 21:56:14 +00:00
|
|
|
/* If we have no fb then treat it as a full mode set */
|
2014-04-01 22:22:40 +00:00
|
|
|
if (set->crtc->primary->fb == NULL) {
|
2009-07-20 05:48:07 +00:00
|
|
|
DRM_DEBUG_KMS("crtc has no fb, full mode set\n");
|
2009-01-09 21:56:14 +00:00
|
|
|
mode_changed = true;
|
2016-11-18 19:53:10 +00:00
|
|
|
} else if (set->fb->format != set->crtc->primary->fb->format) {
|
2013-04-21 23:38:47 +00:00
|
|
|
mode_changed = true;
|
2010-02-11 04:28:58 +00:00
|
|
|
} else
|
2009-01-09 21:56:14 +00:00
|
|
|
fb_changed = true;
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
if (set->x != set->crtc->x || set->y != set->crtc->y)
|
2009-01-09 21:56:14 +00:00
|
|
|
fb_changed = true;
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2016-01-14 06:00:10 +00:00
|
|
|
if (!drm_mode_equal(set->mode, &set->crtc->mode)) {
|
2009-07-20 05:48:07 +00:00
|
|
|
DRM_DEBUG_KMS("modes are different, full mode set\n");
|
2008-11-07 22:05:41 +00:00
|
|
|
drm_mode_debug_printmodeline(&set->crtc->mode);
|
|
|
|
drm_mode_debug_printmodeline(set->mode);
|
2009-01-09 21:56:14 +00:00
|
|
|
mode_changed = true;
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
|
drm/crtc: fix connector reference counting mismatch in drm_crtc_helper_set_config
Since commit 0955c1250e96 ("drm/crtc: take references to connectors used
in a modeset. (v2)"), the reference counts of all connectors in the
drm_mode_set given to drm_crtc_helper_set_config are incremented, and then
the reference counts of all connectors are decremented on success, but in a
temporary copy of the connector structure. This leads to the following
error after the first modeset on imx-drm:
Unable to handle kernel NULL pointer dereference at virtual address 00000004
pgd = ad8c4000
[00000004] *pgd=3d9c5831, *pte=00000000, *ppte=00000000
Internal error: Oops: 817 [#1] PREEMPT SMP ARM
Modules linked in:
CPU: 1 PID: 190 Comm: kmsfb-manage Not tainted 4.7.0-rc1+ #657
Hardware name: Freescale i.MX6 Quad/DualLit: [<80506098>] lr : [<80252e94>] psr: 200c0013
sp : adca7ca8 ip : adca7b90 fp : adca7cd4
r10: 00000000 r9 : 00000100 r8 : 00000200
r7 : af3c9800 r6 : aded7848 r5 : aded7800 r4 : 00000000
r3 : af3ca058 r2 : 00000200 r1 : af3ca058 r0 : 00000000
Flags: nzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none
Control: 10c5387d Table: 3d8c404a DAC: 00000051
Process kmsfb-manage (pid: 190, stack limit = 0xadca6210)
Stack: (0xadca7ca8 to 0xadca8000)
7ca0: 805190e0 aded7800 aded7820 80501a88 8155a290 af3c9c6c
7cc0: adca7ddc 0000000f adca7cec adca7cd8 80519104 80506044 805190e0 aded7800
7ce0: adca7d04 adca7cf0 80501ac0 805190ec aded7820 aded7814 adca7d24 adca7d08
7d00: 804fdb80 80501a94 aded7800 af3ca010 aded7afc af3c9c60 adca7d94 adca7d28
7d20: 804e3518 804fdb20 00000000 af3c9b1c adca7d50 81506f44 00000000 8093c500
7d40: af3c9c6c ae4f2ca8 ae4f2c18 00000000 00000000 ae637f00 00000000 aded7800
7d60: 00000001 af3c9800 af23c300 ae77fcc0 ae4f2c18 00000001 af3c9800 8155a290
7d80: af1af700 adca6000 adca7db4 adca7d98 804fea6c 804e2de4 adca7e50 adb3d940
7da0: 00000001 af3c9800 adca7e24 adca7db8 8050440c 804fea0c ae77fcc0 00000003
7dc0: adca7e24 adb3d940 af1af700 ae77fcc0 ae77fccc ae4f2c18 8083d44c ae77fcc0
7de0: ae4002 80d03040 adca7e64 adca7e40 adca7e50 80503f08
7e40: 7ebd5630 adca7e50 00000068 c06864a2 7ebd5be8 00000000 00000001 00000018
7e60: 00000026 00000000 00000000 00000000 00000001 000115bc 05010500 05a0059f
7e80: 03200000 03360321 00000337 0000003c 00000000 00000040 30383231 30303878
7ea0: 00000000 00000000 00000000 00000000 00000000 00000000 80173058 80172e30
7ec0: 80d77d32 00004000 adf7d900 00000003 00000000 7ebd5630 af342bb0 adfe3b80
7ee0: 80272f50 00000003 adca6000 00000000 adca7f7c adca7f00 802725ec 804f52cc
7f00: 802809cc 80178450 00000000 00000000 80280880 80145904 adb3d8c0 adf7d990
7f20: ffffffff 00000003 00004000 01614c10 c06864a2 00000003 adca6000 00000000
7f40: adca7f6c adca7f50 80280b04 8028088c 000115bc adfe3b81 7ebd5630 adfe3b80
7f60: c06864a2 00000003 adca6000 00000000 adca7fa4 adca7f80 80272f50 80272548
7f80: 000115bc 00017050 00000001 01614c10 00000036 801089e4 00000000 adca7fa8
7fa0: 80108840 80272f18 00017050 00000001 00000003 c06864a2 7ebd5630 000115bc
7fc0: 00017050 00000001 01614c10 00000036 00000003 00000000 00000026 00000018
7fe0: 00016f38 7ebd562c 0000b5e9 76ef31e6 400c0030 00000003 ff5f37db bfe7dd4d
Backtrace:
[<80506038>] (drm_connector_cleanup) from [<80519104>] (dw_hdmi_connector_destroy+0x24/0x28)
r10:0000000f r9:adca7ddc r8:af3c9c6c r7:8155a290 r6:80501a88 r5:aded7820
r4:aded7800 r3:805190e0
[<805190e0>] (dw_hdmi_connector_destroy) from [<80501ac0>] (drm_connector_free+0x38/0x3c)
r4:aded7800 nreference) from [<804e3518>] (drm_crtc_helper_set_config+0x740/0xbf4)
r6:af3c9c60 r5:aded7afc r4:af3ca010 r3:aded7800
[<804e2dd8>] (drm_crtc_helper_set_config) from [<804fea6c>] (drm_mode_set_config_internal+0x6c/0xf4)
r10:adca6000 r9:af1af700 r8:8155a290 r7:af3c9800 r6:00000001 r5:ae4f2c18
r4:ae77fcc0
[<804fea00>] (drm_mode_set_config_internal) from [<8050440c>] (drm_mode_setcrtc+0x504/0x57c)
r7:af3c9800 r6:00000001 r5:adb3d940 r4:adca7e50
[<80503f08>] (drm_mode_setcrtc) from [<804f5404>] (drm_ioctl+0x144/0x4dc)
r10:ada2e000 r9:000000a2 r8:af3c9800 r7:8155a290 r6:809320b4 r5:00000051
r4:adca7e50
[<804f52c0>] (drm_ioctl) from [<802725ec>] (do_vfs_ioctl+0xb0/0x9d0)
r10:00000000 r9:adca6000 r8:00000003 r7:80272f50 r6:adfe3b80 r5:af342bb0
r4:7ebd5630
[<8027253c>] (do_vfs_ioctl) from [<80272f50>] (SyS_ioctl+0x44/0x6c)
r10:00000000 r9:adca6000 r8:00000003 r7:c06864a2 r6:adfe3b80 r5:7ebd5630
r4:adfe3b81
[<80272f0c>] (SyS_ioctl) from [<80108840>] (ret_fast_syscall+0x0/0x1c)
r8:801089e4 r7:00000036 r6:01614c10 r5:00000001 r4:00017050 r3:000115bc
Code: 0a00000c e5932004 e1a01003 e1a0a004 (e5842004)
---[ end trace 9a7257572ccacb16 ]---
Only the reference count of connectors that weren't previously bound to
an encoder should be incremented after a call to drm_crtc_helper_set_config.
And only the reference count of connectors that were previously bound to
an encoder and are unbound afterwards should ever be decremented.
The reference counts of the temporary copies in the save_connectors
should not be touched at all.
This patch fixes the above error by only incrementing the reference count
of those connectors in the set that are initially not bound to any encoder,
and also by restoring the reference count of only those connectors in the
set in the failure case.
"Note that this can only be hit when fbdev emulation is disabled, since
then the refcount drops from 1 to 0 and we call the connector destroy
functions on the backup copy, which eventually results in tears. With
fbdev emulation the refcount only goes down from 2 to 1 ever. And since we
unconditionally increment the refcount on the real object, the refcount of
that will slowly increase. The backup connector's refcount doesn't matter,
since we kfree() that either way in the end of
drm_crtc_helper_set_config()."
Fixes: 0955c1250e96 ("drm/crtc: take references to connectors used in a modeset. (v2)")
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
2016-06-02 17:27:51 +00:00
|
|
|
/* take a reference on all unbound connectors in set, reuse the
|
|
|
|
* already taken reference for bound connectors
|
|
|
|
*/
|
2016-04-27 01:27:54 +00:00
|
|
|
for (ro = 0; ro < set->num_connectors; ro++) {
|
drm/crtc: fix connector reference counting mismatch in drm_crtc_helper_set_config
Since commit 0955c1250e96 ("drm/crtc: take references to connectors used
in a modeset. (v2)"), the reference counts of all connectors in the
drm_mode_set given to drm_crtc_helper_set_config are incremented, and then
the reference counts of all connectors are decremented on success, but in a
temporary copy of the connector structure. This leads to the following
error after the first modeset on imx-drm:
Unable to handle kernel NULL pointer dereference at virtual address 00000004
pgd = ad8c4000
[00000004] *pgd=3d9c5831, *pte=00000000, *ppte=00000000
Internal error: Oops: 817 [#1] PREEMPT SMP ARM
Modules linked in:
CPU: 1 PID: 190 Comm: kmsfb-manage Not tainted 4.7.0-rc1+ #657
Hardware name: Freescale i.MX6 Quad/DualLit: [<80506098>] lr : [<80252e94>] psr: 200c0013
sp : adca7ca8 ip : adca7b90 fp : adca7cd4
r10: 00000000 r9 : 00000100 r8 : 00000200
r7 : af3c9800 r6 : aded7848 r5 : aded7800 r4 : 00000000
r3 : af3ca058 r2 : 00000200 r1 : af3ca058 r0 : 00000000
Flags: nzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none
Control: 10c5387d Table: 3d8c404a DAC: 00000051
Process kmsfb-manage (pid: 190, stack limit = 0xadca6210)
Stack: (0xadca7ca8 to 0xadca8000)
7ca0: 805190e0 aded7800 aded7820 80501a88 8155a290 af3c9c6c
7cc0: adca7ddc 0000000f adca7cec adca7cd8 80519104 80506044 805190e0 aded7800
7ce0: adca7d04 adca7cf0 80501ac0 805190ec aded7820 aded7814 adca7d24 adca7d08
7d00: 804fdb80 80501a94 aded7800 af3ca010 aded7afc af3c9c60 adca7d94 adca7d28
7d20: 804e3518 804fdb20 00000000 af3c9b1c adca7d50 81506f44 00000000 8093c500
7d40: af3c9c6c ae4f2ca8 ae4f2c18 00000000 00000000 ae637f00 00000000 aded7800
7d60: 00000001 af3c9800 af23c300 ae77fcc0 ae4f2c18 00000001 af3c9800 8155a290
7d80: af1af700 adca6000 adca7db4 adca7d98 804fea6c 804e2de4 adca7e50 adb3d940
7da0: 00000001 af3c9800 adca7e24 adca7db8 8050440c 804fea0c ae77fcc0 00000003
7dc0: adca7e24 adb3d940 af1af700 ae77fcc0 ae77fccc ae4f2c18 8083d44c ae77fcc0
7de0: ae4002 80d03040 adca7e64 adca7e40 adca7e50 80503f08
7e40: 7ebd5630 adca7e50 00000068 c06864a2 7ebd5be8 00000000 00000001 00000018
7e60: 00000026 00000000 00000000 00000000 00000001 000115bc 05010500 05a0059f
7e80: 03200000 03360321 00000337 0000003c 00000000 00000040 30383231 30303878
7ea0: 00000000 00000000 00000000 00000000 00000000 00000000 80173058 80172e30
7ec0: 80d77d32 00004000 adf7d900 00000003 00000000 7ebd5630 af342bb0 adfe3b80
7ee0: 80272f50 00000003 adca6000 00000000 adca7f7c adca7f00 802725ec 804f52cc
7f00: 802809cc 80178450 00000000 00000000 80280880 80145904 adb3d8c0 adf7d990
7f20: ffffffff 00000003 00004000 01614c10 c06864a2 00000003 adca6000 00000000
7f40: adca7f6c adca7f50 80280b04 8028088c 000115bc adfe3b81 7ebd5630 adfe3b80
7f60: c06864a2 00000003 adca6000 00000000 adca7fa4 adca7f80 80272f50 80272548
7f80: 000115bc 00017050 00000001 01614c10 00000036 801089e4 00000000 adca7fa8
7fa0: 80108840 80272f18 00017050 00000001 00000003 c06864a2 7ebd5630 000115bc
7fc0: 00017050 00000001 01614c10 00000036 00000003 00000000 00000026 00000018
7fe0: 00016f38 7ebd562c 0000b5e9 76ef31e6 400c0030 00000003 ff5f37db bfe7dd4d
Backtrace:
[<80506038>] (drm_connector_cleanup) from [<80519104>] (dw_hdmi_connector_destroy+0x24/0x28)
r10:0000000f r9:adca7ddc r8:af3c9c6c r7:8155a290 r6:80501a88 r5:aded7820
r4:aded7800 r3:805190e0
[<805190e0>] (dw_hdmi_connector_destroy) from [<80501ac0>] (drm_connector_free+0x38/0x3c)
r4:aded7800 nreference) from [<804e3518>] (drm_crtc_helper_set_config+0x740/0xbf4)
r6:af3c9c60 r5:aded7afc r4:af3ca010 r3:aded7800
[<804e2dd8>] (drm_crtc_helper_set_config) from [<804fea6c>] (drm_mode_set_config_internal+0x6c/0xf4)
r10:adca6000 r9:af1af700 r8:8155a290 r7:af3c9800 r6:00000001 r5:ae4f2c18
r4:ae77fcc0
[<804fea00>] (drm_mode_set_config_internal) from [<8050440c>] (drm_mode_setcrtc+0x504/0x57c)
r7:af3c9800 r6:00000001 r5:adb3d940 r4:adca7e50
[<80503f08>] (drm_mode_setcrtc) from [<804f5404>] (drm_ioctl+0x144/0x4dc)
r10:ada2e000 r9:000000a2 r8:af3c9800 r7:8155a290 r6:809320b4 r5:00000051
r4:adca7e50
[<804f52c0>] (drm_ioctl) from [<802725ec>] (do_vfs_ioctl+0xb0/0x9d0)
r10:00000000 r9:adca6000 r8:00000003 r7:80272f50 r6:adfe3b80 r5:af342bb0
r4:7ebd5630
[<8027253c>] (do_vfs_ioctl) from [<80272f50>] (SyS_ioctl+0x44/0x6c)
r10:00000000 r9:adca6000 r8:00000003 r7:c06864a2 r6:adfe3b80 r5:7ebd5630
r4:adfe3b81
[<80272f0c>] (SyS_ioctl) from [<80108840>] (ret_fast_syscall+0x0/0x1c)
r8:801089e4 r7:00000036 r6:01614c10 r5:00000001 r4:00017050 r3:000115bc
Code: 0a00000c e5932004 e1a01003 e1a0a004 (e5842004)
---[ end trace 9a7257572ccacb16 ]---
Only the reference count of connectors that weren't previously bound to
an encoder should be incremented after a call to drm_crtc_helper_set_config.
And only the reference count of connectors that were previously bound to
an encoder and are unbound afterwards should ever be decremented.
The reference counts of the temporary copies in the save_connectors
should not be touched at all.
This patch fixes the above error by only incrementing the reference count
of those connectors in the set that are initially not bound to any encoder,
and also by restoring the reference count of only those connectors in the
set in the failure case.
"Note that this can only be hit when fbdev emulation is disabled, since
then the refcount drops from 1 to 0 and we call the connector destroy
functions on the backup copy, which eventually results in tears. With
fbdev emulation the refcount only goes down from 2 to 1 ever. And since we
unconditionally increment the refcount on the real object, the refcount of
that will slowly increase. The backup connector's refcount doesn't matter,
since we kfree() that either way in the end of
drm_crtc_helper_set_config()."
Fixes: 0955c1250e96 ("drm/crtc: take references to connectors used in a modeset. (v2)")
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
2016-06-02 17:27:51 +00:00
|
|
|
if (set->connectors[ro]->encoder)
|
|
|
|
continue;
|
2017-02-28 14:46:39 +00:00
|
|
|
drm_connector_get(set->connectors[ro]);
|
2016-04-27 01:27:54 +00:00
|
|
|
}
|
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
/* a) traverse passed in connector list and get encoders for them */
|
|
|
|
count = 0;
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_begin(dev, &conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
drm_for_each_connector_iter(connector, &conn_iter) {
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_connector_helper_funcs *connector_funcs =
|
2008-11-07 22:05:41 +00:00
|
|
|
connector->helper_private;
|
|
|
|
new_encoder = connector->encoder;
|
|
|
|
for (ro = 0; ro < set->num_connectors; ro++) {
|
|
|
|
if (set->connectors[ro] == connector) {
|
2019-09-13 23:28:56 +00:00
|
|
|
if (connector_funcs->best_encoder)
|
|
|
|
new_encoder = connector_funcs->best_encoder(connector);
|
|
|
|
else
|
|
|
|
new_encoder = drm_connector_get_single_encoder(connector);
|
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
/* if we can't get an encoder for a connector
|
|
|
|
we are setting now - then fail */
|
|
|
|
if (new_encoder == NULL)
|
|
|
|
/* don't break so fail path works correct */
|
|
|
|
fail = 1;
|
2013-07-19 16:57:11 +00:00
|
|
|
|
|
|
|
if (connector->dpms != DRM_MODE_DPMS_ON) {
|
|
|
|
DRM_DEBUG_KMS("connector dpms not on, full mode switch\n");
|
|
|
|
mode_changed = true;
|
|
|
|
}
|
2014-04-01 20:14:59 +00:00
|
|
|
|
|
|
|
break;
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if (new_encoder != connector->encoder) {
|
2009-07-20 05:48:07 +00:00
|
|
|
DRM_DEBUG_KMS("encoder changed, full mode switch\n");
|
2009-01-09 21:56:14 +00:00
|
|
|
mode_changed = true;
|
2009-08-18 22:56:45 +00:00
|
|
|
/* If the encoder is reused for another connector, then
|
|
|
|
* the appropriate crtc will be set later.
|
|
|
|
*/
|
2009-09-01 01:39:04 +00:00
|
|
|
if (connector->encoder)
|
|
|
|
connector->encoder->crtc = NULL;
|
2008-11-07 22:05:41 +00:00
|
|
|
connector->encoder = new_encoder;
|
|
|
|
}
|
|
|
|
}
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2008-11-07 22:05:41 +00:00
|
|
|
|
|
|
|
if (fail) {
|
|
|
|
ret = -EINVAL;
|
2009-08-27 08:18:29 +00:00
|
|
|
goto fail;
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
count = 0;
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_begin(dev, &conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
drm_for_each_connector_iter(connector, &conn_iter) {
|
2008-11-07 22:05:41 +00:00
|
|
|
if (!connector->encoder)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
if (connector->encoder->crtc == set->crtc)
|
|
|
|
new_crtc = NULL;
|
|
|
|
else
|
|
|
|
new_crtc = connector->encoder->crtc;
|
|
|
|
|
|
|
|
for (ro = 0; ro < set->num_connectors; ro++) {
|
|
|
|
if (set->connectors[ro] == connector)
|
|
|
|
new_crtc = set->crtc;
|
|
|
|
}
|
2009-02-24 00:09:34 +00:00
|
|
|
|
|
|
|
/* Make sure the new CRTC will work with the encoder */
|
|
|
|
if (new_crtc &&
|
|
|
|
!drm_encoder_crtc_ok(connector->encoder, new_crtc)) {
|
|
|
|
ret = -EINVAL;
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2009-08-27 08:18:29 +00:00
|
|
|
goto fail;
|
2009-02-24 00:09:34 +00:00
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
if (new_crtc != connector->encoder->crtc) {
|
2009-07-20 05:48:07 +00:00
|
|
|
DRM_DEBUG_KMS("crtc changed, full mode switch\n");
|
2009-01-09 21:56:14 +00:00
|
|
|
mode_changed = true;
|
2008-11-07 22:05:41 +00:00
|
|
|
connector->encoder->crtc = new_crtc;
|
|
|
|
}
|
2010-07-15 19:43:25 +00:00
|
|
|
if (new_crtc) {
|
2015-12-08 16:41:53 +00:00
|
|
|
DRM_DEBUG_KMS("[CONNECTOR:%d:%s] to [CRTC:%d:%s]\n",
|
|
|
|
connector->base.id, connector->name,
|
|
|
|
new_crtc->base.id, new_crtc->name);
|
2010-07-15 19:43:25 +00:00
|
|
|
} else {
|
|
|
|
DRM_DEBUG_KMS("[CONNECTOR:%d:%s] to [NOCRTC]\n",
|
2015-12-08 16:41:53 +00:00
|
|
|
connector->base.id, connector->name);
|
2010-07-15 19:43:25 +00:00
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2008-11-07 22:05:41 +00:00
|
|
|
|
|
|
|
/* mode_set_base is not a required function */
|
2009-01-09 21:56:14 +00:00
|
|
|
if (fb_changed && !crtc_funcs->mode_set_base)
|
|
|
|
mode_changed = true;
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2009-01-09 21:56:14 +00:00
|
|
|
if (mode_changed) {
|
2013-10-29 15:09:45 +00:00
|
|
|
if (drm_helper_crtc_in_use(set->crtc)) {
|
2009-07-20 05:48:07 +00:00
|
|
|
DRM_DEBUG_KMS("attempting to set mode from"
|
|
|
|
" userspace\n");
|
2008-11-07 22:05:41 +00:00
|
|
|
drm_mode_debug_printmodeline(set->mode);
|
2014-04-01 22:22:40 +00:00
|
|
|
set->crtc->primary->fb = set->fb;
|
2008-11-07 22:05:41 +00:00
|
|
|
if (!drm_crtc_helper_set_mode(set->crtc, set->mode,
|
2008-12-18 03:14:46 +00:00
|
|
|
set->x, set->y,
|
2013-10-29 15:09:42 +00:00
|
|
|
save_set.fb)) {
|
2015-12-08 16:41:53 +00:00
|
|
|
DRM_ERROR("failed to set mode on [CRTC:%d:%s]\n",
|
|
|
|
set->crtc->base.id, set->crtc->name);
|
2014-04-01 22:22:40 +00:00
|
|
|
set->crtc->primary->fb = save_set.fb;
|
2008-11-07 22:05:41 +00:00
|
|
|
ret = -EINVAL;
|
2009-08-27 08:18:29 +00:00
|
|
|
goto fail;
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
2013-07-19 16:57:11 +00:00
|
|
|
DRM_DEBUG_KMS("Setting connector DPMS state to on\n");
|
|
|
|
for (i = 0; i < set->num_connectors; i++) {
|
|
|
|
DRM_DEBUG_KMS("\t[CONNECTOR:%d:%s] set DPMS on\n", set->connectors[i]->base.id,
|
2014-06-03 11:56:20 +00:00
|
|
|
set->connectors[i]->name);
|
2013-07-19 16:57:11 +00:00
|
|
|
set->connectors[i]->funcs->dpms(set->connectors[i], DRM_MODE_DPMS_ON);
|
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
2014-03-20 13:26:34 +00:00
|
|
|
__drm_helper_disable_unused_functions(dev);
|
2009-01-09 21:56:14 +00:00
|
|
|
} else if (fb_changed) {
|
2009-09-18 00:43:52 +00:00
|
|
|
set->crtc->x = set->x;
|
|
|
|
set->crtc->y = set->y;
|
2014-04-01 22:22:40 +00:00
|
|
|
set->crtc->primary->fb = set->fb;
|
2009-02-11 13:25:09 +00:00
|
|
|
ret = crtc_funcs->mode_set_base(set->crtc,
|
2013-10-29 15:09:42 +00:00
|
|
|
set->x, set->y, save_set.fb);
|
2011-01-08 15:10:41 +00:00
|
|
|
if (ret != 0) {
|
2013-10-29 15:09:43 +00:00
|
|
|
set->crtc->x = save_set.x;
|
|
|
|
set->crtc->y = save_set.y;
|
2014-04-01 22:22:40 +00:00
|
|
|
set->crtc->primary->fb = save_set.fb;
|
2009-08-27 08:18:29 +00:00
|
|
|
goto fail;
|
2011-01-08 15:10:41 +00:00
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
|
2016-06-02 17:27:52 +00:00
|
|
|
kfree(save_connector_encoders);
|
|
|
|
kfree(save_encoder_crtcs);
|
2008-11-07 22:05:41 +00:00
|
|
|
return 0;
|
|
|
|
|
2009-08-27 08:18:29 +00:00
|
|
|
fail:
|
|
|
|
/* Restore all previous data. */
|
|
|
|
count = 0;
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2016-06-02 17:27:52 +00:00
|
|
|
encoder->crtc = save_encoder_crtcs[count++];
|
2009-02-11 16:39:21 +00:00
|
|
|
}
|
2009-08-27 08:18:29 +00:00
|
|
|
|
2008-11-07 22:05:41 +00:00
|
|
|
count = 0;
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_begin(dev, &conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
drm_for_each_connector_iter(connector, &conn_iter)
|
2016-06-02 17:27:52 +00:00
|
|
|
connector->encoder = save_connector_encoders[count++];
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2009-08-27 08:18:29 +00:00
|
|
|
|
drm/crtc: fix connector reference counting mismatch in drm_crtc_helper_set_config
Since commit 0955c1250e96 ("drm/crtc: take references to connectors used
in a modeset. (v2)"), the reference counts of all connectors in the
drm_mode_set given to drm_crtc_helper_set_config are incremented, and then
the reference counts of all connectors are decremented on success, but in a
temporary copy of the connector structure. This leads to the following
error after the first modeset on imx-drm:
Unable to handle kernel NULL pointer dereference at virtual address 00000004
pgd = ad8c4000
[00000004] *pgd=3d9c5831, *pte=00000000, *ppte=00000000
Internal error: Oops: 817 [#1] PREEMPT SMP ARM
Modules linked in:
CPU: 1 PID: 190 Comm: kmsfb-manage Not tainted 4.7.0-rc1+ #657
Hardware name: Freescale i.MX6 Quad/DualLit: [<80506098>] lr : [<80252e94>] psr: 200c0013
sp : adca7ca8 ip : adca7b90 fp : adca7cd4
r10: 00000000 r9 : 00000100 r8 : 00000200
r7 : af3c9800 r6 : aded7848 r5 : aded7800 r4 : 00000000
r3 : af3ca058 r2 : 00000200 r1 : af3ca058 r0 : 00000000
Flags: nzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none
Control: 10c5387d Table: 3d8c404a DAC: 00000051
Process kmsfb-manage (pid: 190, stack limit = 0xadca6210)
Stack: (0xadca7ca8 to 0xadca8000)
7ca0: 805190e0 aded7800 aded7820 80501a88 8155a290 af3c9c6c
7cc0: adca7ddc 0000000f adca7cec adca7cd8 80519104 80506044 805190e0 aded7800
7ce0: adca7d04 adca7cf0 80501ac0 805190ec aded7820 aded7814 adca7d24 adca7d08
7d00: 804fdb80 80501a94 aded7800 af3ca010 aded7afc af3c9c60 adca7d94 adca7d28
7d20: 804e3518 804fdb20 00000000 af3c9b1c adca7d50 81506f44 00000000 8093c500
7d40: af3c9c6c ae4f2ca8 ae4f2c18 00000000 00000000 ae637f00 00000000 aded7800
7d60: 00000001 af3c9800 af23c300 ae77fcc0 ae4f2c18 00000001 af3c9800 8155a290
7d80: af1af700 adca6000 adca7db4 adca7d98 804fea6c 804e2de4 adca7e50 adb3d940
7da0: 00000001 af3c9800 adca7e24 adca7db8 8050440c 804fea0c ae77fcc0 00000003
7dc0: adca7e24 adb3d940 af1af700 ae77fcc0 ae77fccc ae4f2c18 8083d44c ae77fcc0
7de0: ae4002 80d03040 adca7e64 adca7e40 adca7e50 80503f08
7e40: 7ebd5630 adca7e50 00000068 c06864a2 7ebd5be8 00000000 00000001 00000018
7e60: 00000026 00000000 00000000 00000000 00000001 000115bc 05010500 05a0059f
7e80: 03200000 03360321 00000337 0000003c 00000000 00000040 30383231 30303878
7ea0: 00000000 00000000 00000000 00000000 00000000 00000000 80173058 80172e30
7ec0: 80d77d32 00004000 adf7d900 00000003 00000000 7ebd5630 af342bb0 adfe3b80
7ee0: 80272f50 00000003 adca6000 00000000 adca7f7c adca7f00 802725ec 804f52cc
7f00: 802809cc 80178450 00000000 00000000 80280880 80145904 adb3d8c0 adf7d990
7f20: ffffffff 00000003 00004000 01614c10 c06864a2 00000003 adca6000 00000000
7f40: adca7f6c adca7f50 80280b04 8028088c 000115bc adfe3b81 7ebd5630 adfe3b80
7f60: c06864a2 00000003 adca6000 00000000 adca7fa4 adca7f80 80272f50 80272548
7f80: 000115bc 00017050 00000001 01614c10 00000036 801089e4 00000000 adca7fa8
7fa0: 80108840 80272f18 00017050 00000001 00000003 c06864a2 7ebd5630 000115bc
7fc0: 00017050 00000001 01614c10 00000036 00000003 00000000 00000026 00000018
7fe0: 00016f38 7ebd562c 0000b5e9 76ef31e6 400c0030 00000003 ff5f37db bfe7dd4d
Backtrace:
[<80506038>] (drm_connector_cleanup) from [<80519104>] (dw_hdmi_connector_destroy+0x24/0x28)
r10:0000000f r9:adca7ddc r8:af3c9c6c r7:8155a290 r6:80501a88 r5:aded7820
r4:aded7800 r3:805190e0
[<805190e0>] (dw_hdmi_connector_destroy) from [<80501ac0>] (drm_connector_free+0x38/0x3c)
r4:aded7800 nreference) from [<804e3518>] (drm_crtc_helper_set_config+0x740/0xbf4)
r6:af3c9c60 r5:aded7afc r4:af3ca010 r3:aded7800
[<804e2dd8>] (drm_crtc_helper_set_config) from [<804fea6c>] (drm_mode_set_config_internal+0x6c/0xf4)
r10:adca6000 r9:af1af700 r8:8155a290 r7:af3c9800 r6:00000001 r5:ae4f2c18
r4:ae77fcc0
[<804fea00>] (drm_mode_set_config_internal) from [<8050440c>] (drm_mode_setcrtc+0x504/0x57c)
r7:af3c9800 r6:00000001 r5:adb3d940 r4:adca7e50
[<80503f08>] (drm_mode_setcrtc) from [<804f5404>] (drm_ioctl+0x144/0x4dc)
r10:ada2e000 r9:000000a2 r8:af3c9800 r7:8155a290 r6:809320b4 r5:00000051
r4:adca7e50
[<804f52c0>] (drm_ioctl) from [<802725ec>] (do_vfs_ioctl+0xb0/0x9d0)
r10:00000000 r9:adca6000 r8:00000003 r7:80272f50 r6:adfe3b80 r5:af342bb0
r4:7ebd5630
[<8027253c>] (do_vfs_ioctl) from [<80272f50>] (SyS_ioctl+0x44/0x6c)
r10:00000000 r9:adca6000 r8:00000003 r7:c06864a2 r6:adfe3b80 r5:7ebd5630
r4:adfe3b81
[<80272f0c>] (SyS_ioctl) from [<80108840>] (ret_fast_syscall+0x0/0x1c)
r8:801089e4 r7:00000036 r6:01614c10 r5:00000001 r4:00017050 r3:000115bc
Code: 0a00000c e5932004 e1a01003 e1a0a004 (e5842004)
---[ end trace 9a7257572ccacb16 ]---
Only the reference count of connectors that weren't previously bound to
an encoder should be incremented after a call to drm_crtc_helper_set_config.
And only the reference count of connectors that were previously bound to
an encoder and are unbound afterwards should ever be decremented.
The reference counts of the temporary copies in the save_connectors
should not be touched at all.
This patch fixes the above error by only incrementing the reference count
of those connectors in the set that are initially not bound to any encoder,
and also by restoring the reference count of only those connectors in the
set in the failure case.
"Note that this can only be hit when fbdev emulation is disabled, since
then the refcount drops from 1 to 0 and we call the connector destroy
functions on the backup copy, which eventually results in tears. With
fbdev emulation the refcount only goes down from 2 to 1 ever. And since we
unconditionally increment the refcount on the real object, the refcount of
that will slowly increase. The backup connector's refcount doesn't matter,
since we kfree() that either way in the end of
drm_crtc_helper_set_config()."
Fixes: 0955c1250e96 ("drm/crtc: take references to connectors used in a modeset. (v2)")
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
2016-06-02 17:27:51 +00:00
|
|
|
/* after fail drop reference on all unbound connectors in set, let
|
|
|
|
* bound connectors keep their reference
|
|
|
|
*/
|
2016-04-27 01:27:54 +00:00
|
|
|
for (ro = 0; ro < set->num_connectors; ro++) {
|
drm/crtc: fix connector reference counting mismatch in drm_crtc_helper_set_config
Since commit 0955c1250e96 ("drm/crtc: take references to connectors used
in a modeset. (v2)"), the reference counts of all connectors in the
drm_mode_set given to drm_crtc_helper_set_config are incremented, and then
the reference counts of all connectors are decremented on success, but in a
temporary copy of the connector structure. This leads to the following
error after the first modeset on imx-drm:
Unable to handle kernel NULL pointer dereference at virtual address 00000004
pgd = ad8c4000
[00000004] *pgd=3d9c5831, *pte=00000000, *ppte=00000000
Internal error: Oops: 817 [#1] PREEMPT SMP ARM
Modules linked in:
CPU: 1 PID: 190 Comm: kmsfb-manage Not tainted 4.7.0-rc1+ #657
Hardware name: Freescale i.MX6 Quad/DualLit: [<80506098>] lr : [<80252e94>] psr: 200c0013
sp : adca7ca8 ip : adca7b90 fp : adca7cd4
r10: 00000000 r9 : 00000100 r8 : 00000200
r7 : af3c9800 r6 : aded7848 r5 : aded7800 r4 : 00000000
r3 : af3ca058 r2 : 00000200 r1 : af3ca058 r0 : 00000000
Flags: nzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none
Control: 10c5387d Table: 3d8c404a DAC: 00000051
Process kmsfb-manage (pid: 190, stack limit = 0xadca6210)
Stack: (0xadca7ca8 to 0xadca8000)
7ca0: 805190e0 aded7800 aded7820 80501a88 8155a290 af3c9c6c
7cc0: adca7ddc 0000000f adca7cec adca7cd8 80519104 80506044 805190e0 aded7800
7ce0: adca7d04 adca7cf0 80501ac0 805190ec aded7820 aded7814 adca7d24 adca7d08
7d00: 804fdb80 80501a94 aded7800 af3ca010 aded7afc af3c9c60 adca7d94 adca7d28
7d20: 804e3518 804fdb20 00000000 af3c9b1c adca7d50 81506f44 00000000 8093c500
7d40: af3c9c6c ae4f2ca8 ae4f2c18 00000000 00000000 ae637f00 00000000 aded7800
7d60: 00000001 af3c9800 af23c300 ae77fcc0 ae4f2c18 00000001 af3c9800 8155a290
7d80: af1af700 adca6000 adca7db4 adca7d98 804fea6c 804e2de4 adca7e50 adb3d940
7da0: 00000001 af3c9800 adca7e24 adca7db8 8050440c 804fea0c ae77fcc0 00000003
7dc0: adca7e24 adb3d940 af1af700 ae77fcc0 ae77fccc ae4f2c18 8083d44c ae77fcc0
7de0: ae4002 80d03040 adca7e64 adca7e40 adca7e50 80503f08
7e40: 7ebd5630 adca7e50 00000068 c06864a2 7ebd5be8 00000000 00000001 00000018
7e60: 00000026 00000000 00000000 00000000 00000001 000115bc 05010500 05a0059f
7e80: 03200000 03360321 00000337 0000003c 00000000 00000040 30383231 30303878
7ea0: 00000000 00000000 00000000 00000000 00000000 00000000 80173058 80172e30
7ec0: 80d77d32 00004000 adf7d900 00000003 00000000 7ebd5630 af342bb0 adfe3b80
7ee0: 80272f50 00000003 adca6000 00000000 adca7f7c adca7f00 802725ec 804f52cc
7f00: 802809cc 80178450 00000000 00000000 80280880 80145904 adb3d8c0 adf7d990
7f20: ffffffff 00000003 00004000 01614c10 c06864a2 00000003 adca6000 00000000
7f40: adca7f6c adca7f50 80280b04 8028088c 000115bc adfe3b81 7ebd5630 adfe3b80
7f60: c06864a2 00000003 adca6000 00000000 adca7fa4 adca7f80 80272f50 80272548
7f80: 000115bc 00017050 00000001 01614c10 00000036 801089e4 00000000 adca7fa8
7fa0: 80108840 80272f18 00017050 00000001 00000003 c06864a2 7ebd5630 000115bc
7fc0: 00017050 00000001 01614c10 00000036 00000003 00000000 00000026 00000018
7fe0: 00016f38 7ebd562c 0000b5e9 76ef31e6 400c0030 00000003 ff5f37db bfe7dd4d
Backtrace:
[<80506038>] (drm_connector_cleanup) from [<80519104>] (dw_hdmi_connector_destroy+0x24/0x28)
r10:0000000f r9:adca7ddc r8:af3c9c6c r7:8155a290 r6:80501a88 r5:aded7820
r4:aded7800 r3:805190e0
[<805190e0>] (dw_hdmi_connector_destroy) from [<80501ac0>] (drm_connector_free+0x38/0x3c)
r4:aded7800 nreference) from [<804e3518>] (drm_crtc_helper_set_config+0x740/0xbf4)
r6:af3c9c60 r5:aded7afc r4:af3ca010 r3:aded7800
[<804e2dd8>] (drm_crtc_helper_set_config) from [<804fea6c>] (drm_mode_set_config_internal+0x6c/0xf4)
r10:adca6000 r9:af1af700 r8:8155a290 r7:af3c9800 r6:00000001 r5:ae4f2c18
r4:ae77fcc0
[<804fea00>] (drm_mode_set_config_internal) from [<8050440c>] (drm_mode_setcrtc+0x504/0x57c)
r7:af3c9800 r6:00000001 r5:adb3d940 r4:adca7e50
[<80503f08>] (drm_mode_setcrtc) from [<804f5404>] (drm_ioctl+0x144/0x4dc)
r10:ada2e000 r9:000000a2 r8:af3c9800 r7:8155a290 r6:809320b4 r5:00000051
r4:adca7e50
[<804f52c0>] (drm_ioctl) from [<802725ec>] (do_vfs_ioctl+0xb0/0x9d0)
r10:00000000 r9:adca6000 r8:00000003 r7:80272f50 r6:adfe3b80 r5:af342bb0
r4:7ebd5630
[<8027253c>] (do_vfs_ioctl) from [<80272f50>] (SyS_ioctl+0x44/0x6c)
r10:00000000 r9:adca6000 r8:00000003 r7:c06864a2 r6:adfe3b80 r5:7ebd5630
r4:adfe3b81
[<80272f0c>] (SyS_ioctl) from [<80108840>] (ret_fast_syscall+0x0/0x1c)
r8:801089e4 r7:00000036 r6:01614c10 r5:00000001 r4:00017050 r3:000115bc
Code: 0a00000c e5932004 e1a01003 e1a0a004 (e5842004)
---[ end trace 9a7257572ccacb16 ]---
Only the reference count of connectors that weren't previously bound to
an encoder should be incremented after a call to drm_crtc_helper_set_config.
And only the reference count of connectors that were previously bound to
an encoder and are unbound afterwards should ever be decremented.
The reference counts of the temporary copies in the save_connectors
should not be touched at all.
This patch fixes the above error by only incrementing the reference count
of those connectors in the set that are initially not bound to any encoder,
and also by restoring the reference count of only those connectors in the
set in the failure case.
"Note that this can only be hit when fbdev emulation is disabled, since
then the refcount drops from 1 to 0 and we call the connector destroy
functions on the backup copy, which eventually results in tears. With
fbdev emulation the refcount only goes down from 2 to 1 ever. And since we
unconditionally increment the refcount on the real object, the refcount of
that will slowly increase. The backup connector's refcount doesn't matter,
since we kfree() that either way in the end of
drm_crtc_helper_set_config()."
Fixes: 0955c1250e96 ("drm/crtc: take references to connectors used in a modeset. (v2)")
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
2016-06-02 17:27:51 +00:00
|
|
|
if (set->connectors[ro]->encoder)
|
|
|
|
continue;
|
2017-02-28 14:46:39 +00:00
|
|
|
drm_connector_put(set->connectors[ro]);
|
2016-04-27 01:27:54 +00:00
|
|
|
}
|
|
|
|
|
2011-11-07 18:39:57 +00:00
|
|
|
/* Try to restore the config */
|
|
|
|
if (mode_changed &&
|
|
|
|
!drm_crtc_helper_set_mode(save_set.crtc, save_set.mode, save_set.x,
|
|
|
|
save_set.y, save_set.fb))
|
|
|
|
DRM_ERROR("failed to restore config after modeset failure\n");
|
|
|
|
|
2016-06-02 17:27:52 +00:00
|
|
|
kfree(save_connector_encoders);
|
|
|
|
kfree(save_encoder_crtcs);
|
2008-11-07 22:05:41 +00:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_crtc_helper_set_config);
|
|
|
|
|
2009-05-31 03:42:28 +00:00
|
|
|
static int drm_helper_choose_encoder_dpms(struct drm_encoder *encoder)
|
|
|
|
{
|
|
|
|
int dpms = DRM_MODE_DPMS_OFF;
|
|
|
|
struct drm_connector *connector;
|
2016-12-15 15:58:43 +00:00
|
|
|
struct drm_connector_list_iter conn_iter;
|
2009-05-31 03:42:28 +00:00
|
|
|
struct drm_device *dev = encoder->dev;
|
|
|
|
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_begin(dev, &conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
drm_for_each_connector_iter(connector, &conn_iter)
|
2009-05-31 03:42:28 +00:00
|
|
|
if (connector->encoder == encoder)
|
|
|
|
if (connector->dpms < dpms)
|
|
|
|
dpms = connector->dpms;
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
|
2009-05-31 03:42:28 +00:00
|
|
|
return dpms;
|
|
|
|
}
|
|
|
|
|
2013-08-14 20:47:37 +00:00
|
|
|
/* Helper which handles bridge ordering around encoder dpms */
|
|
|
|
static void drm_helper_encoder_dpms(struct drm_encoder *encoder, int mode)
|
|
|
|
{
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_encoder_helper_funcs *encoder_funcs;
|
2013-08-14 20:47:37 +00:00
|
|
|
|
2016-05-05 13:24:32 +00:00
|
|
|
encoder_funcs = encoder->helper_private;
|
|
|
|
if (!encoder_funcs)
|
|
|
|
return;
|
|
|
|
|
2013-08-14 20:47:37 +00:00
|
|
|
if (encoder_funcs->dpms)
|
|
|
|
encoder_funcs->dpms(encoder, mode);
|
|
|
|
}
|
|
|
|
|
2009-05-31 03:42:28 +00:00
|
|
|
static int drm_helper_choose_crtc_dpms(struct drm_crtc *crtc)
|
|
|
|
{
|
|
|
|
int dpms = DRM_MODE_DPMS_OFF;
|
|
|
|
struct drm_connector *connector;
|
2016-12-15 15:58:43 +00:00
|
|
|
struct drm_connector_list_iter conn_iter;
|
2009-05-31 03:42:28 +00:00
|
|
|
struct drm_device *dev = crtc->dev;
|
|
|
|
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_begin(dev, &conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
drm_for_each_connector_iter(connector, &conn_iter)
|
2009-05-31 03:42:28 +00:00
|
|
|
if (connector->encoder && connector->encoder->crtc == crtc)
|
|
|
|
if (connector->dpms < dpms)
|
|
|
|
dpms = connector->dpms;
|
2017-02-28 14:46:43 +00:00
|
|
|
drm_connector_list_iter_end(&conn_iter);
|
2016-12-15 15:58:43 +00:00
|
|
|
|
2009-05-31 03:42:28 +00:00
|
|
|
return dpms;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2012-11-01 13:45:16 +00:00
|
|
|
* drm_helper_connector_dpms() - connector dpms helper implementation
|
|
|
|
* @connector: affected connector
|
|
|
|
* @mode: DPMS mode
|
2009-05-31 03:42:28 +00:00
|
|
|
*
|
2017-01-25 06:26:43 +00:00
|
|
|
* The drm_helper_connector_dpms() helper function implements the
|
|
|
|
* &drm_connector_funcs.dpms callback for drivers using the legacy CRTC
|
|
|
|
* helpers.
|
2015-12-04 08:45:46 +00:00
|
|
|
*
|
|
|
|
* This is the main helper function provided by the CRTC helper framework for
|
2012-11-01 13:45:16 +00:00
|
|
|
* implementing the DPMS connector attribute. It computes the new desired DPMS
|
2017-01-25 06:26:43 +00:00
|
|
|
* state for all encoders and CRTCs in the output mesh and calls the
|
|
|
|
* &drm_crtc_helper_funcs.dpms and &drm_encoder_helper_funcs.dpms callbacks
|
|
|
|
* provided by the driver.
|
2015-12-04 08:45:46 +00:00
|
|
|
*
|
|
|
|
* This function is deprecated. New drivers must implement atomic modeset
|
2017-07-25 12:02:04 +00:00
|
|
|
* support, where DPMS is handled in the DRM core.
|
2015-07-21 09:34:55 +00:00
|
|
|
*
|
|
|
|
* Returns:
|
|
|
|
* Always returns 0.
|
2009-05-31 03:42:28 +00:00
|
|
|
*/
|
2015-07-21 09:34:55 +00:00
|
|
|
int drm_helper_connector_dpms(struct drm_connector *connector, int mode)
|
2009-05-31 03:42:28 +00:00
|
|
|
{
|
|
|
|
struct drm_encoder *encoder = connector->encoder;
|
|
|
|
struct drm_crtc *crtc = encoder ? encoder->crtc : NULL;
|
2013-08-14 20:47:37 +00:00
|
|
|
int old_dpms, encoder_dpms = DRM_MODE_DPMS_OFF;
|
2009-05-31 03:42:28 +00:00
|
|
|
|
2019-01-10 10:30:45 +00:00
|
|
|
WARN_ON(drm_drv_uses_atomic_modeset(connector->dev));
|
|
|
|
|
2009-05-31 03:42:28 +00:00
|
|
|
if (mode == connector->dpms)
|
2015-07-21 09:34:55 +00:00
|
|
|
return 0;
|
2009-05-31 03:42:28 +00:00
|
|
|
|
|
|
|
old_dpms = connector->dpms;
|
|
|
|
connector->dpms = mode;
|
|
|
|
|
2013-08-14 20:47:37 +00:00
|
|
|
if (encoder)
|
|
|
|
encoder_dpms = drm_helper_choose_encoder_dpms(encoder);
|
|
|
|
|
2009-05-31 03:42:28 +00:00
|
|
|
/* from off to on, do crtc then encoder */
|
|
|
|
if (mode < old_dpms) {
|
|
|
|
if (crtc) {
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
|
2009-05-31 03:42:28 +00:00
|
|
|
if (crtc_funcs->dpms)
|
|
|
|
(*crtc_funcs->dpms) (crtc,
|
|
|
|
drm_helper_choose_crtc_dpms(crtc));
|
|
|
|
}
|
2013-08-14 20:47:37 +00:00
|
|
|
if (encoder)
|
|
|
|
drm_helper_encoder_dpms(encoder, encoder_dpms);
|
2009-05-31 03:42:28 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
/* from on to off, do encoder then crtc */
|
|
|
|
if (mode > old_dpms) {
|
2013-08-14 20:47:37 +00:00
|
|
|
if (encoder)
|
|
|
|
drm_helper_encoder_dpms(encoder, encoder_dpms);
|
2009-05-31 03:42:28 +00:00
|
|
|
if (crtc) {
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_crtc_helper_funcs *crtc_funcs = crtc->helper_private;
|
2009-05-31 03:42:28 +00:00
|
|
|
if (crtc_funcs->dpms)
|
|
|
|
(*crtc_funcs->dpms) (crtc,
|
|
|
|
drm_helper_choose_crtc_dpms(crtc));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2015-07-21 09:34:55 +00:00
|
|
|
return 0;
|
2009-05-31 03:42:28 +00:00
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_helper_connector_dpms);
|
|
|
|
|
2014-01-22 15:42:02 +00:00
|
|
|
/**
|
|
|
|
* drm_helper_resume_force_mode - force-restore mode setting configuration
|
|
|
|
* @dev: drm_device which should be restored
|
|
|
|
*
|
|
|
|
* Drivers which use the mode setting helpers can use this function to
|
|
|
|
* force-restore the mode setting configuration e.g. on resume or when something
|
|
|
|
* else might have trampled over the hw state (like some overzealous old BIOSen
|
|
|
|
* tended to do).
|
2014-01-23 21:28:30 +00:00
|
|
|
*
|
|
|
|
* This helper doesn't provide a error return value since restoring the old
|
|
|
|
* config should never fail due to resource allocation issues since the driver
|
|
|
|
* has successfully set the restored configuration already. Hence this should
|
|
|
|
* boil down to the equivalent of a few dpms on calls, which also don't provide
|
|
|
|
* an error code.
|
|
|
|
*
|
|
|
|
* Drivers where simply restoring an old configuration again might fail (e.g.
|
|
|
|
* due to slight differences in allocating shared resources when the
|
|
|
|
* configuration is restored in a different order than when userspace set it up)
|
|
|
|
* need to use their own restore logic.
|
2015-12-02 16:50:04 +00:00
|
|
|
*
|
|
|
|
* This function is deprecated. New drivers should implement atomic mode-
|
|
|
|
* setting and use the atomic suspend/resume helpers.
|
|
|
|
*
|
|
|
|
* See also:
|
|
|
|
* drm_atomic_helper_suspend(), drm_atomic_helper_resume()
|
2014-01-22 15:42:02 +00:00
|
|
|
*/
|
2014-01-23 21:28:30 +00:00
|
|
|
void drm_helper_resume_force_mode(struct drm_device *dev)
|
2008-11-07 22:05:41 +00:00
|
|
|
{
|
|
|
|
struct drm_crtc *crtc;
|
2009-12-31 06:30:46 +00:00
|
|
|
struct drm_encoder *encoder;
|
2015-03-11 09:51:06 +00:00
|
|
|
const struct drm_crtc_helper_funcs *crtc_funcs;
|
2014-01-23 21:28:30 +00:00
|
|
|
int encoder_dpms;
|
|
|
|
bool ret;
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2019-01-10 10:30:45 +00:00
|
|
|
WARN_ON(drm_drv_uses_atomic_modeset(dev));
|
|
|
|
|
2014-03-21 00:45:40 +00:00
|
|
|
drm_modeset_lock_all(dev);
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_crtc(crtc, dev) {
|
2008-11-07 22:05:41 +00:00
|
|
|
|
|
|
|
if (!crtc->enabled)
|
|
|
|
continue;
|
|
|
|
|
2008-12-18 03:14:46 +00:00
|
|
|
ret = drm_crtc_helper_set_mode(crtc, &crtc->mode,
|
2014-04-01 22:22:40 +00:00
|
|
|
crtc->x, crtc->y, crtc->primary->fb);
|
2008-11-07 22:05:41 +00:00
|
|
|
|
2014-01-23 21:28:30 +00:00
|
|
|
/* Restoring the old config should never fail! */
|
2008-11-07 22:05:41 +00:00
|
|
|
if (ret == false)
|
|
|
|
DRM_ERROR("failed to set mode on crtc %p\n", crtc);
|
2009-12-31 06:30:46 +00:00
|
|
|
|
|
|
|
/* Turn off outputs that were already powered off */
|
|
|
|
if (drm_helper_choose_crtc_dpms(crtc)) {
|
drm: Add modeset object iterators
And roll them out across drm_* files. The point here isn't code
prettification (it helps with that too) but that some of these lists
aren't static any more. And having macros will gives us a convenient
place to put locking checks into.
I didn't add an iterator for props since that's only used by a
list_for_each_entry_safe in the driver teardown code.
Search&replace was done with the below cocci spatch. Note that there's
a bunch more places that didn't match and which would need some manual
changes, but I've intentially left these out for this mostly automated
patch.
iterator name drm_for_each_crtc;
struct drm_crtc *crtc;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
+ drm_for_each_crtc (crtc, dev) {
...
}
@@
iterator name drm_for_each_encoder;
struct drm_encoder *encoder;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(encoder, &dev->mode_config.encoder_list, head) {
+ drm_for_each_encoder (encoder, dev) {
...
}
@@
iterator name drm_for_each_fb;
struct drm_framebuffer *fb;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(fb, &dev->mode_config.fb_list, head) {
+ drm_for_each_fb (fb, dev) {
...
}
@@
iterator name drm_for_each_connector;
struct drm_connector *connector;
struct drm_device *dev;
expression head;
@@
- list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
+ drm_for_each_connector (connector, dev) {
...
}
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2015-07-09 21:44:25 +00:00
|
|
|
drm_for_each_encoder(encoder, dev) {
|
2009-12-31 06:30:46 +00:00
|
|
|
|
|
|
|
if(encoder->crtc != crtc)
|
|
|
|
continue;
|
|
|
|
|
2013-08-14 20:47:37 +00:00
|
|
|
encoder_dpms = drm_helper_choose_encoder_dpms(
|
|
|
|
encoder);
|
|
|
|
|
|
|
|
drm_helper_encoder_dpms(encoder, encoder_dpms);
|
2009-12-31 06:30:46 +00:00
|
|
|
}
|
2010-08-06 14:03:31 +00:00
|
|
|
|
|
|
|
crtc_funcs = crtc->helper_private;
|
|
|
|
if (crtc_funcs->dpms)
|
|
|
|
(*crtc_funcs->dpms) (crtc,
|
|
|
|
drm_helper_choose_crtc_dpms(crtc));
|
2009-12-31 06:30:46 +00:00
|
|
|
}
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
2014-01-23 21:28:30 +00:00
|
|
|
|
2009-07-08 06:13:13 +00:00
|
|
|
/* disable the unused connectors while restoring the modesetting */
|
2014-03-20 13:26:34 +00:00
|
|
|
__drm_helper_disable_unused_functions(dev);
|
2014-03-21 00:45:40 +00:00
|
|
|
drm_modeset_unlock_all(dev);
|
2008-11-07 22:05:41 +00:00
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_helper_resume_force_mode);
|
2018-12-17 19:43:00 +00:00
|
|
|
|
|
|
|
/**
|
|
|
|
* drm_helper_force_disable_all - Forcibly turn off all enabled CRTCs
|
|
|
|
* @dev: DRM device whose CRTCs to turn off
|
|
|
|
*
|
|
|
|
* Drivers may want to call this on unload to ensure that all displays are
|
|
|
|
* unlit and the GPU is in a consistent, low power state. Takes modeset locks.
|
|
|
|
*
|
|
|
|
* Note: This should only be used by non-atomic legacy drivers. For an atomic
|
|
|
|
* version look at drm_atomic_helper_shutdown().
|
|
|
|
*
|
|
|
|
* Returns:
|
|
|
|
* Zero on success, error code on failure.
|
|
|
|
*/
|
|
|
|
int drm_helper_force_disable_all(struct drm_device *dev)
|
|
|
|
{
|
|
|
|
struct drm_crtc *crtc;
|
|
|
|
int ret = 0;
|
|
|
|
|
|
|
|
drm_modeset_lock_all(dev);
|
|
|
|
drm_for_each_crtc(crtc, dev)
|
|
|
|
if (crtc->enabled) {
|
|
|
|
struct drm_mode_set set = {
|
|
|
|
.crtc = crtc,
|
|
|
|
};
|
|
|
|
|
|
|
|
ret = drm_mode_set_config_internal(&set);
|
|
|
|
if (ret)
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
out:
|
|
|
|
drm_modeset_unlock_all(dev);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_helper_force_disable_all);
|