drivers/gpu/drm/vkms/vkms_crtc.c | 14 ++++++++++++++ drivers/gpu/drm/vkms/vkms_drv.c | 9 --------- drivers/gpu/drm/vkms/vkms_output.c | 25 +++++++------------------ drivers/gpu/drm/vkms/vkms_writeback.c | 13 ++++++------- 4 files changed, 27 insertions(+), 34 deletions(-)
To simplify the memory managment this series replace all manual drm
object managment by drm-managed one. This way the VKMS code don't have to
manage it directly and the DRM core will handle the object destruction.
No functional changes are intended in this series. This series depends on
[1] (for writeback connector) and [2] (for cleaning code).
PATCH 1/4: Migrate connector managment to drmm
PATCH 2/4: Migrate encoder managment to drmm
PATCH 3/4: Migrate connector management to drm
PATCH 4/4: Migrate writeback connector management to drm
[1]: https://lore.kernel.org/all/20240906-writeback-drmm-v1-1-01ede328182c@bootlin.com/
[2]: https://lore.kernel.org/all/20240906-vkms-remove-index-v1-1-3cfedd8ccb2f@bootlin.com/
Signed-off-by: Louis Chauvet <louis.chauvet@bootlin.com>
---
Changes in v3:
- As suggested by Maxime, split the managed and the dynamic allocation
parts in different series
- To reduce the diff in this series, extract the "remove crtc index" part,
see https://lore.kernel.org/all/20240906-vkms-remove-index-v1-1-3cfedd8ccb2f@bootlin.com/
- Link to v2: https://lore.kernel.org/r/20240827-google-vkms-managed-v2-0-f41104553aeb@bootlin.com
Changes in v2:
- Applied comments from José
- Extract the rename vkms_output -> vkms_crtc to avoid useless changes in
the last commit
- Extract the rename to_vkms_crtc_state to
drm_crtc_state_to_vkms_crtc_state to avoid useless changes in last
commit
- Extract the drm_mode_crtc_set_gamma_size result check in its own commit
- Rebased on drm-misc/drm-misc-next
- Link to v1: https://lore.kernel.org/r/20240814-google-vkms-managed-v1-0-7ab8b8921103@bootlin.com
---
Louis Chauvet (4):
drm/vkms: Switch to managed for connector
drm/vkms: Switch to managed for encoder
drm/vkms: Switch to managed for crtc
drm/vkms: Switch to managed for writeback connector
drivers/gpu/drm/vkms/vkms_crtc.c | 14 ++++++++++++++
drivers/gpu/drm/vkms/vkms_drv.c | 9 ---------
drivers/gpu/drm/vkms/vkms_output.c | 25 +++++++------------------
drivers/gpu/drm/vkms/vkms_writeback.c | 13 ++++++-------
4 files changed, 27 insertions(+), 34 deletions(-)
---
base-commit: d2194256049910d286cd6c308c2689df521d8842
change-id: 20240521-google-vkms-managed-4aec99461a77
prerequisite-message-id: <20240906-writeback-drmm-v1-1-01ede328182c@bootlin.com>
prerequisite-patch-id: 93bfa5c36385932ea291789faa7356639d9e4bfc
prerequisite-message-id: <20240906-vkms-remove-index-v1-1-3cfedd8ccb2f@bootlin.com>
prerequisite-patch-id: 130816a16434cafa13b7a2b629398a20782be3a6
Best regards,
--
Louis Chauvet <louis.chauvet@bootlin.com>
Hi Louis, Thanks for making this change even more atomic. > To simplify the memory managment this series replace all manual drm > object managment by drm-managed one. This way the VKMS code don't have to > manage it directly and the DRM core will handle the object destruction. > > No functional changes are intended in this series. This series depends on > [1] (for writeback connector) and [2] (for cleaning code). > > PATCH 1/4: Migrate connector managment to drmm > PATCH 2/4: Migrate encoder managment to drmm > PATCH 3/4: Migrate connector management to drm > PATCH 4/4: Migrate writeback connector management to drm > > [1]: https://lore.kernel.org/all/20240906-writeback-drmm-v1-1-01ede328182c@bootlin.com/ > [2]: https://lore.kernel.org/all/20240906-vkms-remove-index-v1-1-3cfedd8ccb2f@bootlin.com/ > > Signed-off-by: Louis Chauvet <louis.chauvet@bootlin.com> The 4 patches: Reviewed-by: José Expósito <jose.exposito89@gmail.com> > --- > Changes in v3: > - As suggested by Maxime, split the managed and the dynamic allocation > parts in different series > - To reduce the diff in this series, extract the "remove crtc index" part, > see https://lore.kernel.org/all/20240906-vkms-remove-index-v1-1-3cfedd8ccb2f@bootlin.com/ > - Link to v2: https://lore.kernel.org/r/20240827-google-vkms-managed-v2-0-f41104553aeb@bootlin.com > > Changes in v2: > - Applied comments from José > - Extract the rename vkms_output -> vkms_crtc to avoid useless changes in > the last commit > - Extract the rename to_vkms_crtc_state to > drm_crtc_state_to_vkms_crtc_state to avoid useless changes in last > commit > - Extract the drm_mode_crtc_set_gamma_size result check in its own commit > - Rebased on drm-misc/drm-misc-next > - Link to v1: https://lore.kernel.org/r/20240814-google-vkms-managed-v1-0-7ab8b8921103@bootlin.com>
Le 17/09/24 - 18:02, José Expósito a écrit : > Hi Louis, > > Thanks for making this change even more atomic. > > > To simplify the memory managment this series replace all manual drm > > object managment by drm-managed one. This way the VKMS code don't have to > > manage it directly and the DRM core will handle the object destruction. > > > > No functional changes are intended in this series. This series depends on > > [1] (for writeback connector) and [2] (for cleaning code). > > > > PATCH 1/4: Migrate connector managment to drmm > > PATCH 2/4: Migrate encoder managment to drmm > > PATCH 3/4: Migrate connector management to drm > > PATCH 4/4: Migrate writeback connector management to drm > > > > [1]: https://lore.kernel.org/all/20240906-writeback-drmm-v1-1-01ede328182c@bootlin.com/ > > [2]: https://lore.kernel.org/all/20240906-vkms-remove-index-v1-1-3cfedd8ccb2f@bootlin.com/ > > > > Signed-off-by: Louis Chauvet <louis.chauvet@bootlin.com> > > The 4 patches: > Reviewed-by: José Expósito <jose.exposito89@gmail.com> Thanks! Louis Chauvet > > --- > > Changes in v3: > > - As suggested by Maxime, split the managed and the dynamic allocation > > parts in different series > > - To reduce the diff in this series, extract the "remove crtc index" part, > > see https://lore.kernel.org/all/20240906-vkms-remove-index-v1-1-3cfedd8ccb2f@bootlin.com/ > > - Link to v2: https://lore.kernel.org/r/20240827-google-vkms-managed-v2-0-f41104553aeb@bootlin.com > > > > Changes in v2: > > - Applied comments from José > > - Extract the rename vkms_output -> vkms_crtc to avoid useless changes in > > the last commit > > - Extract the rename to_vkms_crtc_state to > > drm_crtc_state_to_vkms_crtc_state to avoid useless changes in last > > commit > > - Extract the drm_mode_crtc_set_gamma_size result check in its own commit > > - Rebased on drm-misc/drm-misc-next > > - Link to v1: https://lore.kernel.org/r/20240814-google-vkms-managed-v1-0-7ab8b8921103@bootlin.com>
© 2016 - 2024 Red Hat, Inc.