[vbox-dev] windows 10 aborted

2017-08-07 Thread Reinoud Koornstra
Hi Everyone, With version 5.1.26 windows shuts off completely and show aborted: 00:03:18.932707 Changing the VM state from 'DESTROYING' to 'TERMINATED' 00:03:18.935063 Console: Machine state changed to 'PoweredOff' Any place where I can file a bug for this? I can reproduce at will. Thanks,

Re: [vbox-dev] [PATCH 2/2] vgdrvLinuxIOCtl: Always return standard Linux errno values

2017-08-07 Thread Knut St. Osmundsen
Just to clarify the design a little here. The I/O controls aren't part of any user space API. They are supposed (*) to be 104% internal and only used between the driver (VBoxGuest) and the ring-3 library (VBoxGuestLib). User land clients will call functions in VBoxGuestLib and not care at all

Re: [vbox-dev] [PATCH 2/2] vgdrvLinuxIOCtl: Always return standard Linux errno values

2017-08-07 Thread Knut St. Osmundsen
Hi Hans, sorry but I'm not convinced this is really useful and improving the code. We will fix this properly in a cross platform manner eventually, I hope. -bird > On 7 Aug 2017, at 16:46, Hans de Goede wrote: > > Hi, > >> On 07-08-17 16:16, Michael Thayer wrote: >>

Re: [vbox-dev] [PATCH 2/2] vgdrvLinuxIOCtl: Always return standard Linux errno values

2017-08-07 Thread Hans de Goede
Hi, On 07-08-17 16:16, Michael Thayer wrote: Hello Hans and Knut, Would a new ioctl to query the last IPRT code for an open session be an option in case the caller does need the extra information? Thank you for the suggestion, but IMHO that is worse (uglier) then returning the vbox status

Re: [vbox-dev] [PATCH 2/2] vgdrvLinuxIOCtl: Always return standard Linux errno values

2017-08-07 Thread Michael Thayer
Hello Hans and Knut, Would a new ioctl to query the last IPRT code for an open session be an option in case the caller does need the extra information? Regards Michael 06.08.2017 22:08, Knut St. Osmundsen wrote: > Hi Hans, > > I'm with you on this not being entirely optimal. However 'man

Re: [vbox-dev] compile patch for 5.1.6

2017-08-07 Thread Klaus Espenlaub
Hi Reinoud, On 07.08.2017 01:52, Reinoud Koornstra wrote: Hello Everyone, Currently virtualbox 5.1.6 fails to compile for kernel version 4.9 and up. This is due to a change in get_user_pages api. The ints for write and force have changed to gup_flags. Please find the patch attach to fix this

[vbox-dev] [RESEND PATCH] staging: vboxvideo: remove dead gamma lut code

2017-08-07 Thread Peter Rosin
The redundant fb helpers .load_lut, .gamma_set and .gamma_get are no longer used. Remove the dead code that was not doing anything sensible anyway. Signed-off-by: Peter Rosin --- drivers/staging/vboxvideo/vbox_fb.c | 15 --- drivers/staging/vboxvideo/vbox_mode.c |

[vbox-dev] Compiling dkms with linux 4.9 and up

2017-08-07 Thread Reinoud Koornstra
Hello Everyone, Currently virtualbox fails to compile for kernel version 4.9 and up. This is due to a change in get_user_pages api. The ints for write and force have changed to gup_flags. Please find the patch attach to fix this for linux 4.9 and up. Thanks, Reinoud. ---

Re: [vbox-dev] [RESEND PATCH] staging: vboxvideo: remove dead gamma lut code

2017-08-07 Thread Daniel Vetter
On Fri, Aug 04, 2017 at 12:45:06PM +0200, Peter Rosin wrote: > The redundant fb helpers .load_lut, .gamma_set and .gamma_get are > no longer used. Remove the dead code that was not doing anything > sensible anyway. > > Signed-off-by: Peter Rosin > --- >