Mesa demos

[camako]

First cut snapified mesa demos showcasing opengl, opengles1, opengles2, egl based apps. Will be used as a tool to develop the missing/incomplete interfaces.

[dholbach]

daniel@daydream:/dev/snappy/snappy-playpen/mesa-demos$ mesa-demos.es2gears
/snap/mesa-demos/x1/es2gears: line 13: cd: /snap/mesa-demos/x1/egl/opengles2: No such file or directory
/snap/mesa-demos/x1/es2gears: line 14: /home/daniel/dev/snappy/snappy-playpen/mesa-demos/es2gears_x11: No such file or directory
daniel@daydream:/dev/snappy/snappy-playpen/mesa-demos$

[dholbach]

Could they all share the same wrapper file?

[dholbach]

Can you maybe explain why --devmode is still required?

[camako]

es2 apps should work now.

[camako]

'--devmode' is required because seccomp is denying execution due to a missing syscall (sendmsg) for all these apps. I suspect the helper libraries need them (glx, glut, etc). I'm investigating.

Also, the opengles apps shouldn't use the opengl interface. I'll create an opengles interface in snapd. There might also need to be an egl interface that's needed. Still investigating these and devmode will be removed once these concerns are addressed.

[camako]

They could possibly share the same wrapper, but I'm not sure how to express that in the yaml (i.e. copying one file to multiple destinations). I'll ask in the snapcraft list.

But better yet, the wrappers can be eliminated once the 'environment' functionality is merged in (see the readme).

[camako]

👍

[dholbach]

Sorry, when I said "explain" I meant "note it down in the mesa-demos/README.md file".

A way to express it in the yaml file could be something like

apps:
something:
command: bin/common-wrapper something
something-else:
command: bin/common-wrapper something-else

... if that's possible.

[dholbach]

es2gears works now! :-)

[camako]

Thanks for your suggestion. Now I use one wrapper. I've also added to the readme why strict mode isn't yet turned on.

[camako]

oops

[camako]

👍

[dholbach]

And i915.

[camako]

Added it to the readme. Thanks for testing.

[camako]

👍

[dholbach]

Is there a snapd/snap-confine bug open for this?

[camako]

I'm still not done investigating this. I'll raise a bug when done.

[camako]

👍

[dholbach]

Beautiful work, just added two small comments.

[camako]

👍

[camako]

My investigation has lead to this bug : https://bugs.launchpad.net/snappy/+bug/1611978

[camako]

Updated the readme to reflect the result of the investigation about the seccomp denial (hence keeping this snap in devmode).

[camako]

👍

[zyga]

Validate on Debian, Fedora, Arch, etc...

[camako]

I've tried to use an lxd container for Debian and Fedora but they failed for different reasons. I've added this task as a TO-DO to the readme and will get to it when I've completed my main task (which is implementing the opengles interface).

[camako]

👍

[zyga]

Validate on various versions of Nvidia proprietary GPU driver (different driver, different behavior from snap-confine!)

[camako]

Currently I don't have an NVidia based machine. I'll ask around if anyone has one and can help me. For now this task is in the TO-DO.

[camako]

👍

[zyga]

Looks good. I'd love to use this for my GPU support work in snap-confine. I added two comments that I see as additional TODOs. I would love if those could be included in the tree.

[albaguirre]

Validated on ubuntu 16.04 with nvidia-361 (361.42-0ubuntu2) proprietary driver. Most of the demos included worked fine except mesa-demos.es2tri but that seems to be due to a driver idiosyncrasy rather than the snap itself.

[camako]

So can we merge this now?

[dholbach]

It looks like it currently fails to build: https://travis-ci.org/ubuntu/snappy-playpen/builds/155385333

[camako]

I think it fails to build due to an infrastructure problem. I see others failing the same way. How do we rekick the build?

[dholbach]

I re-kicked the build. It ran "about 6 hours ago": https://travis-ci.org/ubuntu/snappy-playpen/builds/155385333

[camako]

@dholbach @zyga, I've updated the PR with the latest changes in yaml, and also added the missing {build|stage}_packages. Could you give it a quick review again? Thanks.

[dholbach]

Builds and runs nicely! Good work!

[dholbach]

I'll merge this now. From a Playpen perspective this is well-done and working. If we need to test and further validate this, let's do this in a separate thread.

[zyga]

Thank you for merging this. I will look at validating this against snap-confine 1.0.41 that is in yakkety and will soon enter xenial.

[camako]

Thanks. I'll create separate PRs for the rest of the demos.