DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 2C02AC07F98E
Subject: Re: [PATCH v3 28/28] drm: vboxvideo: switch to drm_*_get(),
 drm_*_put() helpers
To: Sean Paul <seanpaul@chromium.org>, Cihangir Akturk <cakturk@gmail.com>,
        devel@driverdev.osuosl.org, Michael Thayer <michael.thayer@oracle.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
        Peter Rosin <peda@axentia.se>
References: <1502454794-28558-1-git-send-email-cakturk@gmail.com>
 <1502454794-28558-29-git-send-email-cakturk@gmail.com>
 <27c2b794-73b0-641f-6dcd-6cb7eeb47dff@redhat.com>
 <20170811160416.sdahhswjk6vcd3qr@art_vandelay>
 <8fb1da0c-33b1-814f-3385-0cdf74193cb1@redhat.com>
 <20170814085826.uja2poh7yjwzb2yq@phenom.ffwll.local>
From: Hans de Goede <hdegoede@redhat.com>
Message-ID: <bc7915ff-0c20-a6a1-e352-5d427a8bbe24@redhat.com>
Date: Mon, 14 Aug 2017 14:23:11 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.2.1
MIME-Version: 1.0
In-Reply-To: <20170814085826.uja2poh7yjwzb2yq@phenom.ffwll.local>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2079
Lines: 58

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@gmail.com>
>>>>
>>>> Thank you for doing this, looks good to me:
>>>>
>>>> Reviewed-by: Hans de Goede <hdegoede@redhat.com>
>>>>
>>>
>>> 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