Re: [vbox-dev] [PATCH][resend] Fix shortcuts like alt-tab not working under [X]wayland

2020-06-10 Thread Hans de Goede
#19650 XWayland: Fix shortcuts like alt-tab not working under [X]wayland https://www.virtualbox.org/ticket/19650 cheers frankB On Mon, 08 Jun 2020 17:18:57 +0200, Hans de Goede wrote: Hi, Note this is a resend since my first submission was ignored. This patch makes virtualbox work much better

Re: [vbox-dev] malinline vboxsf vs bundled vboxsf mount options differences (was Removing suid root bit from VBoxDRMClient)

2020-06-09 Thread Hans de Goede
the workflow and packaging rules I would be happy to give you direct commit rights. On Mon, 08 Jun 2020 17:46:58 +0200, Hans de Goede wrote: Hi All, While looking into upgrading the Fedora virtualbox-guest-additions packages to 6.0.10 I noticed that the  "VBoxClient --vmsvga-x11&q

[vbox-dev] Removing suid root bit from VBoxDRMClient

2020-06-08 Thread Hans de Goede
Hi All, While looking into upgrading the Fedora virtualbox-guest-additions packages to 6.0.10 I noticed that the "VBoxClient --vmsvga-x11" call in VBoxClient-all has been replaced with "VBoxClient --vmsvga" and that that one will either behave as the old --vmsvga-x11 version (when running

[vbox-dev] [PATCH][resend] Fix shortcuts like alt-tab not working under [X]wayland

2020-06-08 Thread Hans de Goede
Hi, Note this is a resend since my first submission was ignored. This patch makes virtualbox work much better (things like alt+tab will work in the VM) when run inside a Wayland session. Can we please get this merged? The attached patch tackles a long-standing issue when running VirtualBox

Re: [vbox-dev] [ PATCH] Fix crash on startup under Wayland

2020-06-08 Thread Hans de Goede
ticket/19583 the possible fix for this is still being discussed internally. On Tue, 07 Apr 2020 14:00:47 +0200, Hans de Goede < hdego...@redhat.com> wrote: Hi, I noticed that with a default Fedora 32 workstation install (default GNOME3 on Wayland session) VirtualBox crashes immediately at

Re: [vbox-dev] [ PATCH] Fix crash on startup under Wayland

2020-05-18 Thread Hans de Goede
for this is still being discussed internally. On Tue, 07 Apr 2020 14:00:47 +0200, Hans de Goede wrote: Hi, I noticed that with a default Fedora 32 workstation install (default GNOME3 on Wayland session) VirtualBox crashes immediately at startup. The problem is that QT5 now defaults to the Wayland

Re: [vbox-dev] [ PATCH] Fix crash on startup under Wayland

2020-04-07 Thread Hans de Goede
workaround but properly fixing this will take a significant (huge even) amount of effort. Regards, Hans On Tue, 07 Apr 2020 14:00:47 +0200, Hans de Goede wrote: I noticed that with a default Fedora 32 workstation install (default GNOME3 on Wayland session) VirtualBox crashes immediately

[vbox-dev] [PATCH] Fix shortcuts like alt-tab not working under [X]wayland

2020-04-07 Thread Hans de Goede
Hi, The attached patch tackles a long-standing issue when running VirtualBox under [X]wayland. By default keyboard-grabs are denied to X11 apps under Xwayland, this causes e.g. alt-tab to not be send to the VM, but to go to the host window-environment. The attached patch modifies VirtualBox to

[vbox-dev] [ PATCH] Fix crash on startup under Wayland

2020-04-07 Thread Hans de Goede
Hi, I noticed that with a default Fedora 32 workstation install (default GNOME3 on Wayland session) VirtualBox crashes immediately at startup. The problem is that QT5 now defaults to the Wayland backend and VirtualBox assumes the X11 backend is used and makes a bunch of unchecked X11 calls which

Re: [vbox-dev] [PATCH 2nd resend] VBoxServiceAutoMount: Change Linux mount code to use

2019-05-07 Thread Hans de Goede
Hi, On 04-05-19 16:44, Michael Thayer wrote: 03.05.19 23:40, Knut St. Osmundsen wrote: [...] Finally, does the libc or kernel mount() implementation somehow invoke the /sbin/mount.vboxsf helper tool?  Please point me to the code causing this, because our vboxsf sources does not parse any

[vbox-dev] Linux guests (with vboxvideo kernel driver) sometimes showing screen corruption when going fullscreen

2018-01-19 Thread Hans de Goede
Hi, After the following bug was filed against Fedora: https://bugzilla.redhat.com/show_bug.cgi?id=1536264 I've tried to reproduce this and I can reproduce it sometimes: This is a Fedora 27 x86_64 host running gnome3 on Xorg running virtualbox 5.2.4 from the official RPMS with a Fedora 27

[vbox-dev] Changing vboxsf linux mount data argument from vbsf_mount_info_new to a normal string

2018-01-15 Thread Hans de Goede
Hi Michael, Now a days the mount syscall usually takes a c-string with "," separated key=value pair of arguments in plain text. The upstream kernel folks have requested that I change vboxsf to also work this way. My plan for this is for me to not only modify vboxsf for this, but for me to also

[vbox-dev] Seamless mode is broken on 5.2.x (X11 guest on X11 host)

2018-01-15 Thread Hans de Goede
Hi All, It seems that seamless mode is broken when using 5.2.x as a host, I've tested this with both 5.2.2 host + guest as well as with a 5.2.4 host + guest. This is with the official vbox host RPM packages on top of F27 x86_64 running a GNOME3 session in Xorg mode, with a F27 x86_64 guest also

[vbox-dev] [PATCH 0/1] vboxsf: Fix various issues with force_restat

2018-01-09 Thread Hans de Goede
Hi Michael, et. al, Here is a patch to vboxsf fixing various issues with force_restat. Note this is against the version I'm preparing for mainline submission (now that vboxguest is queued for merging into mainline for 4.16), but it should be easy to port these to the VirtualBox svn code.

[vbox-dev] [PATCH] vboxsf: Fix various issues with force_restat

2018-01-09 Thread Hans de Goede
gave us (so uninitialized) and wrongly clearing it on the parent directory inode -In sf_read_super() explictly initialize force_restat to 0, rather then leaving force_restat on the new sf_inode_info to whatever kmalloc gave us Signed-off-by: Hans de Goede <hdego...@redhat.com> --- fs/

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

2017-10-12 Thread Hans de Goede
HI, On 12-10-17 14:44, Hans de Goede wrote: Hi, On 12-10-17 11:17, Michael Thayer wrote: 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

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

2017-10-12 Thread Hans de Goede
Hi, On 12-10-17 11:17, Michael Thayer wrote: 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

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

2017-10-11 Thread Hans de Goede
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

[vbox-dev] [PATCH resend] VBoxGuest.h Fix VBGL_IOCTL_WAIT_FOR_EVENTS definition

2017-10-03 Thread Hans de Goede
The size argument to VBGL_IOCTL_CODE_SIZE should be VBGL_IOCTL_WAIT_FOR_EVENTS_SIZE. This fixes VBGL_IOCTL_WAIT_FOR_EVENTS always failing with EINVAL. Which in turn fixes seamless mode not working under Linux. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- include/VBox/VBoxGuest

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

2017-09-30 Thread Hans de Goede
Hi, On 29-09-17 19:38, Sérgio Basto wrote: On Fri, 2017-09-29 at 19:27 +0200, Hans de Goede wrote: Hi, On 28-09-17 15:00, Sérgio Basto wrote: On Wed, 2017-09-27 at 18:16 +0200, Hans de Goede wrote: I just tried Fedora kernel 4.13.1-303.fc27, which has the vboxvideo driver enabled

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

2017-09-29 Thread Hans de Goede
Hi, On 28-09-17 15:00, Sérgio Basto wrote: On Wed, 2017-09-27 at 18:16 +0200, Hans de Goede wrote: I just tried Fedora kernel 4.13.1-303.fc27, which has the vboxvideo driver enabled, This is enabled in Fedora kernels or vanilla kernels , do you mean since 4.13.1 vbox guestaditionals drives

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

2017-09-27 Thread Hans de Goede
Hi, On 27-09-17 08:46, Gianfranco Costamagna wrote: Hello, we tried in Ubuntu to use the new shiny vboxvideo mainline kernel driver with the new shiny CONFIG_DRM_VBOXVIDEO config key, and we had to quickly disable it to use the old working kernel module. Reason: it breaks with wayland. see:

[vbox-dev] [PATCH] VBoxGuest.h Fix VBGL_IOCTL_WAIT_FOR_EVENTS definition

2017-09-21 Thread Hans de Goede
The size argument to VBGL_IOCTL_CODE_SIZE should be VBGL_IOCTL_WAIT_FOR_EVENTS_SIZE. This fixes VBGL_IOCTL_WAIT_FOR_EVENTS always failing with EINVAL. Which in turn fixes seamless mode not working under Linux. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- include/VBox/VBoxGuest

Re: [vbox-dev] Removing code only used by no longer supported hosts (was Opportunity to significantly shrink the vboxsf Linux driver)

2017-08-25 Thread Hans de Goede
Hi, On 25-08-17 10:29, Michael Thayer wrote: Hello Hans, 21.08.2017 14:49, Hans de Goede wrote: [...] First of all thank you for the answers to the above questions (see archives). I've just found one more piece of code where I wonder if that should be kept. VbglR0SfMapFolder first tries

Re: [vbox-dev] Removing code only used by no longer supported hosts (was Opportunity to significantly shrink the vboxsf Linux driver)

2017-08-21 Thread Hans de Goede
Hi, On 10-08-17 16:11, Hans de Goede wrote: Hi, On 08-08-17 16:23, Hans de Goede wrote: Hi, On 07/25/2017 11:06 AM, Michael Thayer wrote: > 13.07.2017 16:42, Michael Thayer wrote: >> 10.07.2017 12:29, Hans de Goede wrote: >> [Patch to shared folder code proposed by Hans to

[vbox-dev] Modifying the vboxguest ioctl API (Was: Re: [RFC] VGDrvCommonIoCtl: Add f32bit flag argument)

2017-08-21 Thread Hans de Goede
Hi, On 14-08-17 14:19, Hans de Goede wrote: Hi, On 14-08-17 13:43, Knut St. Osmundsen wrote: Hi Hans, the other platforms also have KPIs or similar constructs for figuring out whether the client process issuing the I/O controls is a 32-bit or 64-bit one. However, using

[vbox-dev] [PATCH 0/3] VBoxGuest.h: ioctl definitions cleanups

2017-08-16 Thread Hans de Goede
Hi Knut, Michael, While taking another look at the ioctl number definitions I noticed that most ioctls were defined using VBOXGUEST_IOCTL_CODE_ (note the _ at the end). So I started out writing a patch to make all ioctls which do not have a specific 32 bit version use VBOXGUEST_IOCTL_CODE_

[vbox-dev] [PATCH 1/3] VBoxGuest.h: Be consistent in our VBOXGUEST_IOCTL_CODE_ usage

2017-08-16 Thread Hans de Goede
and use VBOXGUEST_IOCTL_CODE_ for all ioctls which do not have a 32 bit compat counter-part. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- include/VBox/VBoxGuest.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/include/VBox/VBoxGuest.h b/includ

[vbox-dev] [PATCH 3/3] VBoxGuest.h: Use separate ioctl numbers for 32 and 64 bit HGCM calls

2017-08-16 Thread Hans de Goede
VBOXGUEST_IOCTL_CODE_ directly for all ioctls. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- include/VBox/VBoxGuest.h | 50 1 file changed, 17 insertions(+), 33 deletions(-) diff --git a/include/VBox/VBoxGuest.h b/include/VBox/VBoxGuest.h

[vbox-dev] [PATCH 2/3] VBoxGuest.h: Remove separate definitions of HGCM_[DIS]CONNECT and HGCM_[DIS]CONNECT_32

2017-08-16 Thread Hans de Goede
bit builds. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- include/VBox/VBoxGuest.h | 6 ++ src/VBox/Additions/common/VBoxGuest/VBoxGuest.cpp | 6 -- 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/include/VBox/VBoxGuest.h b/includ

Re: [vbox-dev] [RFC] VGDrvCommonIoCtl: Add f32bit flag argument

2017-08-14 Thread Hans de Goede
On 14-08-17 09:27, Hans de Goede wrote: Note to linux-kernel readers: This Cc-ed to linux-kernel because it is relevant for the "[RFC 0/2] Add Virtual Box vboxguest and vboxsf guest drivers to the mainline kernel" thread. Hi Michael, Knut, My first submission of the vboxgu

[vbox-dev] [PATCH] VGDrvCommonIoCtl: Add f32bit flag argument

2017-08-14 Thread Hans de Goede
On some operating-systems the driver can tell whether an ioctl is a 32bit compat callback or not without looking at the ioctl-nr, at a flag argument to pass this info along and use it for Linux to not need separate ioctl codes for 32 and 64 bit. Signed-off-by: Hans de Goede <hdego...@redhat.

[vbox-dev] First RFC submission of vboxguest and vboxsf to the upstream linux kernel

2017-08-11 Thread Hans de Goede
Hi All, I've just submitted a first RFC version of the vboxguest and vboxsf to the upstream linux kernel: https://lkml.org/lkml/2017/8/11/400 https://lkml.org/lkml/2017/8/11/398 https://lkml.org/lkml/2017/8/11/399 I've Cc-ed Knut and Michael, but not vbox-dev because the mainline kernel devs

Re: [vbox-dev] Removing code only used by no longer supported hosts (was Opportunity to significantly shrink the vboxsf Linux driver)

2017-08-10 Thread Hans de Goede
Hi, On 08-08-17 16:23, Hans de Goede wrote: Hi, On 07/25/2017 11:06 AM, Michael Thayer wrote: > 13.07.2017 16:42, Michael Thayer wrote: >> 10.07.2017 12:29, Hans de Goede wrote: >> [Patch to shared folder code proposed by Hans to reduce the line count >> of the Lin

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

2017-08-08 Thread Hans de Goede
Hi, On 08/07/2017 07:06 PM, Knut St. Osmundsen wrote: 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. No problem, I've already reverted the changes in the vboxguest version I'm

Re: [vbox-dev] Recent changes to vboxsf have introduced a bug? (was Opportunity to significantly shrink the vboxsf Linux driver)

2017-08-08 Thread Hans de Goede
Hi, On 07/25/2017 11:06 AM, Michael Thayer wrote: 13.07.2017 16:42, Michael Thayer wrote: 10.07.2017 12:29, Hans de Goede wrote: [Patch to shared folder code proposed by Hans to reduce the line count of the Linux driver.] I redid the patch again, after realising that the non-physical-page

Re: [vbox-dev] Removing code only used by no longer supported hosts (was Opportunity to significantly shrink the vboxsf Linux driver)

2017-08-08 Thread Hans de Goede
Hi, On 07/25/2017 11:06 AM, Michael Thayer wrote: > 13.07.2017 16:42, Michael Thayer wrote: >> 10.07.2017 12:29, Hans de Goede wrote: >> [Patch to shared folder code proposed by Hans to reduce the line count >> of the Linux driver.] >> >> I reviewed the patch

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-06 Thread Hans de Goede
. Right, I would prefer not to do this, note that HGCM_CALL which seems to be one of the most used ioctls already does this. Regards, Hans On 2017-08-01 5:19 PM, Hans de Goede wrote: Returning VERR_* style vbox-runtime codes as positive values is an ugly hack and unlike any other ioctl call

Re: [vbox-dev] How to test memory ballloon ?

2017-08-06 Thread Hans de Goede
Hi, On 25-07-17 12:01, Andreas Löffler wrote: Hello Hans, please have a look at the manual, section 4.9.1 "Memory ballooning" and commands like VBoxManage controlvm "VM name" guestmemoryballoon VBoxManage modifyvm "VM name" --guestmemoryballoon There is also a frontend called

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

2017-08-05 Thread Hans de Goede
Hi, On 04-08-17 12:45, 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 <p...@axentia.se> Thank you, patch looks good to me: Acked-by: H

[vbox-dev] Interesting patch for vbox upstream

2017-08-01 Thread Hans de Goede
Hi, You may want to do something similar to this for vbox upstream: https://github.com/jwrdegoede/vboxguest/commit/8a79961c661e3fba88c0e50447b2344e0c811d1a Note in my tree this applies on top of: https://github.com/jwrdegoede/vboxguest/commit/8c232c972f7b2dfbfc863a1aaeb0b44fdfcf82d0 Which I

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

2017-08-01 Thread Hans de Goede
as is. This commit removes the positive VERR_* style vbox-runtime codes and makes vgdrvLinuxIOCtl always return standard Linux errno values. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/common/VBoxGuest/VBoxGuest-linux.c | 7 ++- src/VBox/Additions/common/VBoxGu

[vbox-dev] [PATCH 1/2] RTErrConvertTo/FromErrno: Add translation for some extra codes

2017-08-01 Thread Hans de Goede
To RTErrConvertToErrno and RTErrConvertFromErrno. Signed-off-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Runtime/common/err/RTErrConvertFromErrno.cpp | 9 + src/VBox/Runtime/common/err/RTErrConvertToErrno.cpp | 9 + 2 files changed, 10 insertions(+), 8 deletions(-) diff

Re: [vbox-dev] New kernel error message when running 4.13 as guest

2017-07-26 Thread Hans de Goede
Hi Michal, On 26-07-17 17:18, Michal Necasek wrote: Hi Hans, The message is harmless. It's triggered because VirtualBox currently does not expose the host CPU microcode revision to guest operating systems. I don't think we want to do that without fully understanding what sorts of

[vbox-dev] How to test memory ballloon ?

2017-07-25 Thread Hans de Goede
Hi, I would like to test the (Linux guest) memory balloon code. Is there anything I can do to explicitly make the host request the guest to give up some memory through the balloon ? Regards, Hans ___ vbox-dev mailing list vbox-dev@virtualbox.org

Re: [vbox-dev] vgdrvIoCtl_CancelAllWaitEvents inconsistent behavior ?

2017-07-24 Thread Hans de Goede
Hi, On 05-07-17 17:57, Michael Thayer wrote: Hello Hans, 05.07.2017 15:55, Hans de Goede wrote: [...] On 05-07-17 12:14, Michael Thayer wrote: [Discussion of CANCEL_ALL_WAITEVENTS use in seamless.cpp.] I believe your reading of the code was correct and mine not. As per my last message

[vbox-dev] New kernel error message when running 4.13 as guest

2017-07-24 Thread Hans de Goede
Hi all, I just upgraded a vbox guest to 4.13, the good news is everything works fine, but one thing which I noticed, which would be nice to fix is this new error message: [0.00] [Firmware Bug]: TSC_DEADLINE disabled due to Errata; please update microcode to version: 0xb2 (or later)

Re: [vbox-dev] Opportunity to significantly shrink the vboxsf Linux driver

2017-07-11 Thread Hans de Goede
Hi Larry, On 10-07-17 21:37, Larry Finger wrote: On 07/10/2017 10:12 AM, Michael Thayer wrote: Hello Hans, 10.07.2017 12:29, Hans de Goede wrote: Hi, On 10-07-17 12:08, Hans de Goede wrote: Hi all, During my vboxguest Linux cleanup work I made a small detour to the vboxsf driver. I

Re: [vbox-dev] Opportunity to significantly shrink the vboxsf Linux driver

2017-07-10 Thread Hans de Goede
Hi, On 10-07-17 12:08, Hans de Goede wrote: Hi all, During my vboxguest Linux cleanup work I made a small detour to the vboxsf driver. I noticed that although it depends on vboxguest to make hgcm calls, it still comes with its own copy of GenericRequest, Physheap, Init and VMMDev.c It seems

[vbox-dev] Opportunity to significantly shrink the vboxsf Linux driver

2017-07-10 Thread Hans de Goede
that it should be easy to adopt this patch for upstream. Regards, Hans >From 996996a67913eb5e2cd2f4711cb3db565cd26e9b Mon Sep 17 00:00:00 2001 From: Hans de Goede <hdego...@redhat.com> Date: Mon, 10 Jul 2017 12:01:55 +0200 Subject: [PATCH 1/2] Stop calling VbglGRPerform ourselves just to get

Re: [vbox-dev] Replacing VbglPhysHeapAlloc with kmalloc for linux vboxguest module ?

2017-07-06 Thread Hans de Goede
Hi, On 06-07-17 11:44, Knut St. Osmundsen wrote: Hi Hans. On 2017-07-06 11:13 AM, Hans de Goede wrote: Hi, VbglPhysHeapAlloc seems to be yet another mem-allocator which chunks up memory returned by RTMemContAlloc, which returns physical contiguous memory below 4GB. Need for keeping the code

[vbox-dev] Replacing VbglPhysHeapAlloc with kmalloc for linux vboxguest module ?

2017-07-06 Thread Hans de Goede
Hi, VbglPhysHeapAlloc seems to be yet another mem-allocator which chunks up memory returned by RTMemContAlloc, which returns physical contiguous memory below 4GB. I don't see any reason why we cannot use kmalloc directly for that with a GFP_DMA32 flag. One other thing which RTMemContAlloc

Re: [vbox-dev] vgdrvIoCtl_CancelAllWaitEvents inconsistent behavior ?

2017-07-06 Thread Hans de Goede
Hi, On 06-07-17 08:12, Michael Thayer wrote: Hello Hans, 05.07.2017 21:47, Hans de Goede wrote: [Discussion of CANCEL_ALL_WAITEVENTS use in seamless.cpp and behaviour changes in r67796, r67798 and r67802.] Looks good, thanks I'm still going to go for compatibility with the old behavior though

Re: [vbox-dev] vgdrvIoCtl_CancelAllWaitEvents inconsistent behavior ?

2017-07-05 Thread Hans de Goede
Hi, On 05-07-17 17:57, Michael Thayer wrote: Hello Hans, 05.07.2017 15:55, Hans de Goede wrote: [...] On 05-07-17 12:14, Michael Thayer wrote: [Discussion of CANCEL_ALL_WAITEVENTS use in seamless.cpp.] I believe your reading of the code was correct and mine not. As per my last message

Re: [vbox-dev] vgdrvIoCtl_CancelAllWaitEvents inconsistent behavior ?

2017-07-05 Thread Hans de Goede
Hi, On 05-07-17 12:14, Michael Thayer wrote: Hello Hans, 05.07.2017 11:18, Hans de Goede wrote: [Discussion of inconsistent wait event cancelling code in VBoxGuest.] I've been looking at the userspace code myself and this bit: src/VBox/Additions/x11/VBoxClient/seamless.cpp: int SeamlessMain

Re: [vbox-dev] vgdrvIoCtl_CancelAllWaitEvents inconsistent behavior ?

2017-07-05 Thread Hans de Goede
Hi, On 04-07-17 16:57, Michael Thayer wrote: Hello Hans, 04.07.2017 14:24, Hans de Goede wrote: Hi, On 04-07-17 11:46, Michael Thayer wrote: Hello Hans, 04.07.2017 08:55, Hans de Goede wrote: [Discussion of inconsistent wait event cancelling code in VBoxGuest.] So some remarks about

Re: [vbox-dev] 3d pass-through and gnome-shell ?

2017-07-04 Thread Hans de Goede
Hi, On 01-07-17 21:53, Michael Thayer wrote: Hello Hans, 27.06.2017 20:46, Hans de Goede wrote: [...] On 06/27/2017 05:50 PM, Michael Thayer wrote: [...] 27.06.2017 15:17, Hans de Goede wrote: [Discussion of 3D problems with Fedora 25 and 26 guests.] Back to 3d passthrough any idea why

Re: [vbox-dev] vgdrvIoCtl_CancelAllWaitEvents inconsistent behavior ?

2017-07-04 Thread Hans de Goede
Hi, On 04-07-17 11:46, Michael Thayer wrote: Hello Hans, 04.07.2017 08:55, Hans de Goede wrote: [Discussion of inconsistent wait event cancelling code in VBoxGuest.] So some remarks about this, I was thinking that *maybe* the old behavior is on purpose, imagine the following: 1) cancel

Re: [vbox-dev] vgdrvIoCtl_CancelAllWaitEvents inconsistent behavior ?

2017-07-04 Thread Hans de Goede
Hi Michael, Thank you for looking into this. On 04-07-17 08:14, Michael Thayer wrote: Hello Hans, 03.07.2017 20:49, Michael Thayer wrote: [Discussion of incorrect looking wait code in VBoxGuest.cpp.]> I did not get any response when I asked around about this. Reading the code (quite a bit

[vbox-dev] vgdrvIoCtl_CancelAllWaitEvents inconsistent behavior ?

2017-07-03 Thread Hans de Goede
Hi, So while working on cleaning up the vboxguest driver for mainline linux kernel submission (*) I noticed some inconsistent behavior wrt vgdrvIoCtl_CancelAllWaitEvents vs vgdrvIoCtl_WaitEvent. If there are no waiters then vgdrvIoCtl_CancelAllWaitEvents sets pSession->fPendingCancelWaitEvents

Re: [vbox-dev] [PATCH v4] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-28 Thread Hans de Goede
://github.com/0day-ci/linux/commits/Hans-de-Goede/staging-vboxvideo-Add-vboxvideo-to-drivers-staging/20170627-035353 config: x86_64-randconfig-n0-06270721 (attached as .config) compiler: gcc-4.8 (Debian 4.8.4-1) 4.8.4 reproduce: # save the attached .config to linux build tree make ARCH=x86_64

Re: [vbox-dev] 3d pass-through and gnome-shell ?

2017-06-27 Thread Hans de Goede
Hi, On 06/27/2017 05:50 PM, Michael Thayer wrote: Hello Hans, 27.06.2017 15:17, Hans de Goede wrote: [...] On 06/27/2017 01:46 PM, Michael Thayer wrote: [Discussion of 3D problems in Fedora VirtualBox guests, of ld.so.conf trickery used to put 3D libraries in place, but not completing

Re: [vbox-dev] 3d pass-through and gnome-shell ?

2017-06-27 Thread Hans de Goede
Hi, On 06/27/2017 01:46 PM, Michael Thayer wrote: Hello Hans, 27.06.2017 11:38, Hans de Goede wrote: Hi, On 27-06-17 11:34, Michael Thayer wrote: Hello Hans, 26.06.2017 18:12, Hans de Goede wrote: [Discussion of 3D problems in Fedora VirtualBox guests and ld.so.conf trickery used to put 3D

Re: [vbox-dev] 3d pass-through and gnome-shell ?

2017-06-27 Thread Hans de Goede
Hi, On 27-06-17 11:34, Michael Thayer wrote: Hello Hans, 26.06.2017 18:12, Hans de Goede wrote: [Discussion of 3D problems in Fedora VirtualBox guests and ld.so.conf trickery used to put 3D libraries in place, but not completing in time for the display manager.] Ok, I understand, this is easy

Re: [vbox-dev] [PATCH v3] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-26 Thread Hans de Goede
Hi, On 26-06-17 18:24, Daniel Vetter wrote: On Mon, Jun 26, 2017 at 06:06:19PM +0200, Hans de Goede wrote: Hi, On 23-06-17 11:31, Daniel Vetter wrote: On Thu, Jun 22, 2017 at 11:11:37AM +0200, Hans de Goede wrote: This commit adds the vboxvideo drm/kms driver for the virtual graphics card

Re: [vbox-dev] vboxguest [Was: [PATCH v2] staging: vboxvideo: Add vboxvideo to drivers/staging]

2017-06-26 Thread Hans de Goede
Hi, On 15-06-17 12:29, Michael Thayer wrote: Hello Hans, Continuing this in a new thread without all the other recipients. 14.06.2017 17:03, Hans de Goede wrote: Hi, On 14-06-17 15:40, Michael Thayer wrote: Hello Hans, 14.06.2017 15:30, Hans de Goede wrote: [Discussion of vboxvideo

Re: [vbox-dev] 3d pass-through and gnome-shell ?

2017-06-26 Thread Hans de Goede
Hi, On 26-06-17 11:24, Michael Thayer wrote: Hello Hans, 23.06.2017 11:52, Hans de Goede wrote: Hi, On 23-06-17 10:44, Michael Thayer wrote: Hello Hans, 22.06.2017 11:49, Hans de Goede wrote: [...] I've been testing vbox 3d pass-through, and if I use metacity as a window manger

Re: [vbox-dev] [PATCH v3] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-26 Thread Hans de Goede
Hi, On 22-06-17 14:03, Dan Carpenter wrote: > This is obviously much better and easier to review. I ended up > reviewing it like I would normal code instead of staging code. My main > thing is could you please re-write the module init error handling so > that each function does its own error

Re: [vbox-dev] 3d pass-through and gnome-shell ?

2017-06-23 Thread Hans de Goede
Hi, On 23-06-17 10:44, Michael Thayer wrote: Hello Hans, 22.06.2017 11:49, Hans de Goede wrote: [...] I've been testing vbox 3d pass-through, and if I use metacity as a window manger then running glxgears -info shows: GL_RENDERER = Chromium GL_VERSION= 2.1 Chromium 1.9 GL_VENDOR

[vbox-dev] 3d pass-through and gnome-shell ?

2017-06-22 Thread Hans de Goede
Hi, I've been testing vbox 3d pass-through, and if I use metacity as a window manger then running glxgears -info shows: GL_RENDERER = Chromium GL_VERSION= 2.1 Chromium 1.9 GL_VENDOR = Humper So things seem to be working, but gnome3 / gnome-shell does not work ? Regards, Hans

Re: [vbox-dev] Additions: linux/drm: Change vbox_*.c to kernel coding style set 2

2017-06-22 Thread Hans de Goede
Hi, On 22-06-17 07:46, Michael Thayer wrote: Hello Hans, Sorry for the long delay. I am working on too many things in parallel (though I suspect you have a few too!) No problem. 14.06.2017 16:52, Michael Thayer wrote: Hello Hans, 14.06.2017 15:36, Hans de Goede wrote: [Discussion

Re: [vbox-dev] [PATCH v2] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-14 Thread Hans de Goede
Hi, On 14-06-17 15:40, Michael Thayer wrote: Hello Hans, 14.06.2017 15:30, Hans de Goede wrote: [Discussion of vboxvideo and vboxguest driver clean-up.] As I already mentioned in previous mails on this, for the vboxguest driver my plan is to simply do a fork and remove anything related

Re: [vbox-dev] Additions: linux/drm: Change vbox_*.c to kernel coding style set 2

2017-06-14 Thread Hans de Goede
Hi, On 14-06-17 15:20, Michael Thayer wrote: Hello Hans, 09.06.2017 09:48, Hans de Goede wrote: [...] Here are the last 2 patches to complete the conversion of the linux drm/kms driver files to the kernel coding style. You hereby have my and Red Hat's permission to use these under the MIT

Re: [vbox-dev] [PATCH v2] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-14 Thread Hans de Goede
Hi all, On 14-06-17 11:34, Michael Thayer wrote: Hello Sean, 13.06.2017 20:03, Sean Paul wrote: [Discussion of vboxvideo driver clean-up.] First, thank you for your submission. Thank you for your feedback. [Discussion of the OS-independent code in the driver submission.] I took a quick

Re: [vbox-dev] [PATCH v2] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-13 Thread Hans de Goede
Hi, On 12-06-17 18:03, Greg Kroah-Hartman wrote: On Mon, Jun 12, 2017 at 05:40:21PM +0200, Hans de Goede wrote: Hi, On 12-06-17 13:44, Greg Kroah-Hartman wrote: On Mon, Jun 12, 2017 at 12:07:41PM +0200, Hans de Goede wrote: The most important thing is for the driver to be atomic if it's KMS

Re: [vbox-dev] [PATCH v2] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-13 Thread Hans de Goede
Hi, On 12-06-17 13:44, Greg Kroah-Hartman wrote: On Mon, Jun 12, 2017 at 12:07:41PM +0200, Hans de Goede wrote: The most important thing is for the driver to be atomic if it's KMS only, and it would be good to have someone review that properly. I believe it does not use the atomic APIs atm

Re: [vbox-dev] [PATCH] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-10 Thread Hans de Goede
Hi, On 09-06-17 11:58, Hans de Goede wrote: This commit adds the vboxvideo drm/kms driver for the virtual graphics card used in Virtual Box virtual machines to drivers/staging. Why drivers/staging? This driver is already being patched into the kernel by several distros, thus it is good to get

Re: [vbox-dev] [PATCH] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-09 Thread Hans de Goede
Hi, On 09-06-17 12:40, Michael Thayer wrote: Hello, 09.06.2017 12:21, Hans de Goede wrote: Hi, On 09-06-17 12:07, Greg Kroah-Hartman wrote: On Fri, Jun 09, 2017 at 11:58:31AM +0200, Hans de Goede wrote: This commit adds the vboxvideo drm/kms driver for the virtual graphics card used

[vbox-dev] 4.12 REGRESSION, 4.12 does not boot as VirtualBox guest

2017-06-09 Thread Hans de Goede
Hi Linus, Sorry for escalating this right up to you, but this issue does not seem to be getting any attention and I'm not sure which subsys maintainer to poke about this. The problem is that the 4.12-rc# kernels fail to boot on VirtualBox guests hosted on any machine with a Sky Lake (and

Re: [vbox-dev] [PATCH] staging: vboxvideo: Add vboxvideo to drivers/staging

2017-06-09 Thread Hans de Goede
Hi, On 09-06-17 12:07, Greg Kroah-Hartman wrote: On Fri, Jun 09, 2017 at 11:58:31AM +0200, Hans de Goede wrote: This commit adds the vboxvideo drm/kms driver for the virtual graphics card used in Virtual Box virtual machines to drivers/staging. Why drivers/staging? This driver is already

[vbox-dev] [PATCH 1/2] Additions: linux/drm: Change vbox_prime.c to kernel coding style

2017-06-09 Thread Hans de Goede
-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/linux/drm/vbox_prime.c | 44 +++ 1 file changed, 22 insertions(+), 22 deletions(-) diff --git a/src/VBox/Additions/linux/drm/vbox_prime.c b/src/VBox/Additions/linux/drm/vbox_prime.c index 5c47ec6a..7216b642

[vbox-dev] [PATCH 2/2] Additions: linux/drm: Change vbox_ttm.c to kernel coding style

2017-06-09 Thread Hans de Goede
-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/linux/drm/vbox_ttm.c | 627 1 file changed, 321 insertions(+), 306 deletions(-) diff --git a/src/VBox/Additions/linux/drm/vbox_ttm.c b/src/VBox/Additions/linux/drm/vbox_ttm.c index 5099ba06..186da83c

[vbox-dev] [PATCH 6/7] Additions: linux/drm: Change vbox_main.c to kernel coding style

2017-06-08 Thread Hans de Goede
-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/linux/drm/vbox_main.c | 705 --- 1 file changed, 370 insertions(+), 335 deletions(-) diff --git a/src/VBox/Additions/linux/drm/vbox_main.c b/src/VBox/Additions/linux/drm/vbox_main.c index aea5ddb9..2bf6bc2c

[vbox-dev] [PATCH 3/7] Additions: linux/drm: Change vbox_fb.c to kernel coding style

2017-06-08 Thread Hans de Goede
-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/linux/drm/vbox_fb.c | 618 + 1 file changed, 315 insertions(+), 303 deletions(-) diff --git a/src/VBox/Additions/linux/drm/vbox_fb.c b/src/VBox/Additions/linux/drm/vbox_fb.c index cf538c1a..0519bd61

[vbox-dev] [PATCH 4/7] Additions: linux/drm: Fix checkpatch warning in vbox_hgsmi.c

2017-06-08 Thread Hans de Goede
Fix: CHECK: Alignment should match open parenthesis #123: FILE: drivers/staging/vboxvideo/vbox_hgsmi.c:123: offset = gen_pool_virt_to_phys(guest_pool, (unsigned long)buf - sizeof(HGSMIBUFFERHEADER)); Signed-off-by: Hans de Goede <hdego...@redhat.com> --

[vbox-dev] [PATCH 1/7] Additions: linux/drm: Change vbox_drv.c to kernel coding style

2017-06-08 Thread Hans de Goede
-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/linux/drm/vbox_drv.c | 318 1 file changed, 161 insertions(+), 157 deletions(-) diff --git a/src/VBox/Additions/linux/drm/vbox_drv.c b/src/VBox/Additions/linux/drm/vbox_drv.c index 00924fe8..b7af0061

[vbox-dev] [PATCH 5/7] Additions: linux/drm: Change vbox_irq.c to kernel coding style

2017-06-08 Thread Hans de Goede
-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/linux/drm/vbox_irq.c | 244 +--- 1 file changed, 133 insertions(+), 111 deletions(-) diff --git a/src/VBox/Additions/linux/drm/vbox_irq.c b/src/VBox/Additions/linux/drm/vbox_irq.c index 7fdf8660..c9b4079f

[vbox-dev] [PATCH 2/7] Additions: linux/drm: Change vbox_drv.h to kernel coding style

2017-06-08 Thread Hans de Goede
-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/linux/drm/vbox_drv.h | 267 1 file changed, 133 insertions(+), 134 deletions(-) diff --git a/src/VBox/Additions/linux/drm/vbox_drv.h b/src/VBox/Additions/linux/drm/vbox_drv.h index d21cded6..0d411b18

[vbox-dev] [PATCH 0/7] Additions: linux/drm: Change vbox_*.? files to kernel coding style

2017-06-08 Thread Hans de Goede
Hi, Here is the first batch of patches to change the various vbox_*.? files in the linux drm/kms driver to the kernel coding style. You hereby have my and Red Hat's permission to use these patches under the MIT license. Regards, Hans ___ vbox-dev

[vbox-dev] [PATCH 1/2] Additions: VboxVideo: Fix compiler warning when building Linux vboxvideo driver

2017-06-06 Thread Hans de Goede
] void *p = VBoxHGSMIBufferAlloc(pCtx, sizeof(VBVAQUERYMODEHINTS) ^~~~ Signed-off-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/common/VBoxVideo/Modesetting.cpp | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/VBox/Additions/common/VBo

[vbox-dev] [PATCH 2/2] Additions: linux/drm: Fix compiler warning

2017-06-06 Thread Hans de Goede
Fix: drivers/staging/vboxvideo/vbox_main.c: In function ‘vbox_hw_init’: drivers/staging/vboxvideo/vbox_main.c:278:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] int ret; ^~~ Signed-off-by: Hans de Goede <hdego...@redhat.com> --- sr

Re: [vbox-dev] Linux-4.12-rc1 and rc2 not working as vbox guest ?

2017-06-01 Thread Hans de Goede
Hi, On 01-06-17 15:42, Frank Mehnert wrote: Hi Hans, On Donnerstag, 1. Juni 2017 14:22:29 CEST Hans de Goede wrote: Again, this is not a VirtualBox bug although this panic will probably not happen on real hardware. [...] No as for the disagree-ing with this not being a VirtualBox bug, even

Re: [vbox-dev] Linux-4.12-rc1 and rc2 not working as vbox guest ?

2017-05-29 Thread Hans de Goede
Hi, On 29-05-17 15:12, Frank Mehnert wrote: Hi Hans, On Montag, 29. Mai 2017 13:27:33 CEST Hans de Goede wrote: On 29-05-17 12:23, Hans de Goede wrote: On 25-05-17 11:57, Hans de Goede wrote: On 24-05-17 18:13, Michael Thayer wrote: 24.05.2017 16:02, Michael Thayer wrote: 24.05.2017 15:16

Re: [vbox-dev] Linux-4.12-rc1 and rc2 not working as vbox guest ?

2017-05-29 Thread Hans de Goede
Hi, On 29-05-17 12:23, Hans de Goede wrote: Hi, On 25-05-17 11:57, Hans de Goede wrote: Hi, On 24-05-17 18:13, Michael Thayer wrote: 24.05.2017 16:02, Michael Thayer wrote: 24.05.2017 15:16, Hans de Goede wrote: [...] I cannot get a Fedora 26 guest to boot with either distro-provided

Re: [vbox-dev] Linux-4.12-rc1 and rc2 not working as vbox guest ?

2017-05-29 Thread Hans de Goede
Hi, On 25-05-17 11:57, Hans de Goede wrote: Hi, On 24-05-17 18:13, Michael Thayer wrote: 24.05.2017 16:02, Michael Thayer wrote: 24.05.2017 15:16, Hans de Goede wrote: [...] I cannot get a Fedora 26 guest to boot with either distro-provided or self-built 4.12-rc# kernels, the distro

Re: [vbox-dev] Linux-4.12-rc1 and rc2 not working as vbox guest ?

2017-05-24 Thread Hans de Goede
Hi, On 24-05-17 16:02, Michael Thayer wrote: 24.05.2017 15:16, Hans de Goede wrote: [...] I cannot get a Fedora 26 guest to boot with either distro-provided or self-built 4.12-rc# kernels, the distro-provided 4.11 kernel works fine. After selecting a 4.12# kernel in the grub menu the vm shows

[vbox-dev] Linux-4.12-rc1 and rc2 not working as vbox guest ?

2017-05-24 Thread Hans de Goede
Hi, I cannot get a Fedora 26 guest to boot with either distro-provided or self-built 4.12-rc# kernels, the distro-provided 4.11 kernel works fine. After selecting a 4.12# kernel in the grub menu the vm shows an empty text screen with a cursor in the top left corner and then just hangs.

[vbox-dev] guest-ad install bug causing: "sf_read_super_aux error=-22" errors on Linux + fix

2017-05-17 Thread Hans de Goede
Hi, I could not get manual mounting of vboxsf mounts to work under Linux (Fedora) the problem turned out to be that the guest-additions installer (the .run file) creates a /sbin/mount.vboxsf symlink to /opt/VBox.../other/mount.vboxsf which does not exist correcting this to point to:

[vbox-dev] [PATCH 3/3] additions/linux/drm: Remove dependency on iprt/cdefs, stdarg, stdint and types.h

2017-05-17 Thread Hans de Goede
Signed-off-by: Hans de Goede <hdego...@redhat.com> --- src/VBox/Additions/linux/drm/VBoxVideoIPRT.h | 28 +++- src/VBox/Additions/linux/drm/files_vboxvideo_drv | 4 2 files changed, 23 insertions(+), 9 deletions(-) diff --git a/src/VBox/Additions/lin

  1   2   >