[vbox-dev] [PATCH 0/3] additions/linux/drm: Drop dependency on HGSMICommon.c HGSMIMemMalloc.c and iprt/*.h

Hans de Goede hdegoede at redhat.com
Thu Jun 1 14:37:48 GMT 2017


Hi,

On 31-05-17 21:53, Michael Thayer wrote:
> Hello Hans,
> 
> 17.05.2017 16:59, Hans de Goede wrote:
>> Hi All,
>>
>> Here is a set of patches further cleaning up the vboxvideo driver in
>> preparation for upstream submission. Total line-count of files needed
>> to build the vboxvideo driver before these changes: 16763, after: 7837 :)
>>
>> You hereby have my and Red Hat's permission to use these patches under the
>> MIT license.
> 
> Thank you!  It took a while, but the patches are now committed and publicly visible.  You might want to take a look at the current state of the repository, as I did a bit of re-arranging of files to try to reduce the unneeded code in the Linux driver a bit further.

I've just rebased my master branch on top, everything looks good,
thank you for merging the patches and thank you for splitting HGSMIGuest.cpp
I was already hoping you would do that, I didn't want to do it myself since
I only have a Linux test setup.

I've one request for the coming week (starting coming Tuesday), can you
please not touch the src/VBox/Additions/linux/drm/* files for a week
(except for any emergencies). The reason I'm asking is because I want
to reformat all the non shared files to linux kernel coding style and
if you also make changes that is going to cause some ugly conflicts.

Regards,

Hans



More information about the vbox-dev mailing list