[vbox-dev] New virtualbox vboxvideo mainline kernel broken in Ubuntu

[Michael Thayer]

Hello Both,

11.10.2017 17:41, Hans de Goede wrote:
> Hi,
> 
> On 11-10-17 07:26, Michael Thayer wrote:
>> Hello Gianfranco,
>>
>> 02.10.2017 12:11, Gianfranco Costamagna wrote:
>>>> Thanks for the heads-up.  I am gradually adjusting our in-tree code to
>>>> match vboxvideo in staging, so at some point, when things stop turning
>>>> up to stop me, I will probably hit the crucial difference.  Gianfranco,
>>>> if you have time could you please give the Additions from the next beta
>>>> release a try, when it is available, since I have already started the
>>>> process?  I would be very grateful if you could.  I will too of course
>>>> when I find a moment, moments are just in short supply.
>>>
>>>
>>> RC1 is now broken too
>> I haven't taken a look at the in-kernel version, but in RC1 the reason
>> was that the arguments to out*() calls got swapped: the Linux kernel
>> reverses them compared to everyone else I know of, and I failed to do
>> that during clean-up to match kernel style.  Will be committing soon.
> 
> Ok, so I double checked and this is not the same problem as people using
> the driver from the upstream kernel's driver/staging are seeing, the
> outl/w/b operands there are all in the right order.

That does not surprise me greatly.  The question is who if anyone will
instrument the staging version with printk commands to see what is going
wrong.  Hans, if you can answer without wasting time researching, can
you think of a simple way to send printk output straight to the
VirtualBox debug I/O port?  That could be simpler than setting up a
virtual serial port.  (I wonder how much work it would be to send
virtual serial port output straight to the log file...)  I do not
immediately plan to do the instrumentation - it is lower priority for me
than getting our code in a state to be submitted to the kernel, and that
is already going slowly enough.

Regards
Michael
-- 
Michael Thayer | VirtualBox engineer
ORACLE Deutschland B.V. & Co. KG | Werkstr. 24 | D-71384 Weinstadt

ORACLE Deutschland B.V. & Co. KG
Hauptverwaltung: Riesstraße 25, D-80992 München
Registergericht: Amtsgericht München, HRA 95603

Komplementärin: ORACLE Deutschland Verwaltung B.V.
Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister
der Handelskammer Midden-Nederland, Nr. 30143697
Geschäftsführer: Alexander van der Ven, Jan Schultheiss, Val Maher