[RFC PATCH 0/2] drm: make drm_get_{connector, encoder}_name thread safe
Jani Nikula
jani.nikula at intel.com
Wed May 14 06:58:18 PDT 2014
Hi all -
This series stores connector/encoder names in the relevant structs to
make the name getters thread safe.
What say you, is the wasted memory too high a price to pay for the
thread safety and implementation simplicity of this approach? I think
making drm_get_connector_name and drm_get_encoder_name return allocated
buffers makes a lot of code really ugly and error prone.
I am assuming connector_type, connector_type_id, encoder_type, and
encoder.base.id remain unchanged for the lifetime of the
connector/encoder - is that a correct assumption?
BR,
Jani.
N.B. I did not actually test this...
Jani Nikula (2):
drm: store connector name in connector struct
drm: store encoder name in encoder struct
drivers/gpu/drm/drm_crtc.c | 67 +++++++++++++++++++++++++---------------------
include/drm/drm_crtc.h | 4 +++
2 files changed, 41 insertions(+), 30 deletions(-)
--
1.9.1
More information about the dri-devel
mailing list