Re: [PATCH v3 28/28] drm: vboxvideo: switch to drm_*_get(), drm_*_put() helpers

From: Hans de Goede
Date: Mon Aug 14 2017 - 08:23:20 EST


Hi,

On 14-08-17 10:58, Daniel Vetter wrote:
On Fri, Aug 11, 2017 at 06:11:08PM +0200, Hans de Goede wrote:
Hi,

On 11-08-17 18:04, Sean Paul wrote:
On Fri, Aug 11, 2017 at 03:26:45PM +0200, Hans de Goede wrote:
Hi,

On 11-08-17 14:33, Cihangir Akturk wrote:
Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
and drm_*_unreference() helpers.

drm_*_reference() and drm_*_unreference() functions are just
compatibility alias for drm_*_get() and drm_*_put() and should not be
used by new code. So convert all users of compatibility functions to
use the new APIs.

Generated by: scripts/coccinelle/api/drm-get-put.cocci

Signed-off-by: Cihangir Akturk <cakturk@xxxxxxxxx>

Thank you for doing this, looks good to me:

Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>


Applied to drm-misc-next, thank you for the review!

Erm vboxvideo is in staging, does this mean all patches for
it will now go through drm-misc-next despite it being in
staging (*) ? Because if some patches get merged through
drm-misc-next and some through Greg's staging repo that
is not going to end well.

-staging doesn't work for drm. And yes this patch needs to go in through
drm-misc I think (I don't want to be at Greg's mercy for merging cleanups,
same way we don't wait for driver maintainers if they don't merge the
patch in a timely fashion).

I'd say if no one is actually working on vbox cleanup (i.e. porting to
atomic) we'll throw it out next cycle again.

Michael Thayer from vbox upstream has said repeatedly that he is
preparing a patch-set to move vboxvideo over to the atomic APIs).

ATM I'm focusing on the other 2 vbox-guest drivers (but that is almost
done) so if Michael does not follow through on this then I will take
care of this.

Either way please don't remove the driver, we do plan to clean it up
and having it ping pong into, out of and back into the mainline kernel
is really not helpful.

Regards,

Hans