-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
xorg.xf86videovmware: fix 3d support with mesa #392492
base: master
Are you sure you want to change the base?
xorg.xf86videovmware: fix 3d support with mesa #392492
Conversation
enable 3d support.
No, this is very bad and we should not be shipping this. We should probably drop the DDX and just use |
@K900 Well for me there is currently no obvious best solution. While
Switching to Wayland is neither an option because it does not support copy-paste. So I would like to continue to support the xfx86videovmware for some time. What is the current disadvantage of supporting it at the moment? |
XA is an old, basically unmaintained, vmware-only code path. This DDX is the only driver that even uses it. Also, as far as I can tell, open-vm-tools should propagate resolution settings to the kernel: https://github.com/vmware/open-vm-tools/blob/739c5a2f4bfd4cdda491e6a6f6869d88c0bd6972/open-vm-tools/services/plugins/resolutionKMS/resolutionKMS.c and |
@K900 XA may be old but the xf86videovmware DDX works really well with the virtual SVGA 3D graphics cards (Do one thing but do it right). Discontinuing it only because it is old does not convince me. However, I think that we can agree that the current state of having a xf86videovmware driver without 3D support is a bad option. I exposed the gallium-xa option in a new top-level parameter with the newest commits so that it can stay disabled as default for mesa. Can you live with that? We can still discuss and decide to completely discontinue the vmware driver for 25.05 (with a warning that you should switch to the modesetting driver) in another pull request. |
Adding the option makes it worse, actually, because now there's multiple configurations that need to be supported and tested. I am going to try and get a VMWare install and see why auto-resize is not working with modesetting. |
d3913d8
to
dbb8e92
Compare
Alright. I dropped the commits again.
@K900 Screen resize in host -> (RPC) -> vmtoolsd -> (syscall) kernel creates udev screen resize event -> xserver with xfx86-video-vmmware driver reads event -> (kernel mode setting) resizes guest window. Now the question is why it does not work with the modesetting driver? |
@K900 Did you make some progress? I actually found another missing feature of the modesetting driver😄: it does not support to use multiple monitors (e.g. the cycle monitor feature or xinerama as the xf86videovmware driver calls it). So that brings me back to the original question: What is the downside of continuing supporting the DDX for now? Can you live with that? Or what do you need so that you could live with supporting this DDX? Do you want to bring more people in to discuss this? |
No, I was busy with other real life things. Do we know the closure size impact of readding XA? |
@K900 Yes, I used the following command: That means a closure size increase of only 1.57% which is in my opinion ok. |
Fixes #390737.
Context
The xf86videovmware driver needs the gallium-xa driver for 3d support. Otherwise the X server crashes if you have 3d acceleration enabled in vmware workstation/player:
@K900 Hope one more mesa rebuild is ok despite your hard work to reduce them 😃.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.