Hi Thomas,

On Tuesday, 25 February 2020, Thomas Zimmermann <tzimmermann@suse.de> wrote:

Non-KMS drivers store state in struct drm_driver. This bloats the
structure for KMS drivers and prevents it from being declared with
'static const' qualifiers. Moving the non-KMS state into a separate
data structure resolves this.

Have you considered subclassing the const drm_driver for legacy drivers. This way the legacy (pointer) data does not need to be exposed to non-legacy drivers.

Not to mention that const data (drm_driver), which has (const) pointer to mutable data (legacy bits) is a trap IMHO

Emil