Subject: glmark2 for Mir

From: Daniel van Vugt <[email-masked]>
Date: Wed, 27 Mar 2013 09:50:25 +0800

I notice this is now done:
   https://blueprints.launchpad.net/ubuntu/+spec/client-1303-mir-glmark2
which is very cool.

Although it's unlikely to produce useful numbers until we fix:
   https://bugs.launchpad.net/mir/+bug/1130553

Still, can we package up a Mir-compatible build of glmark2 for:
   https://launchpad.net/~mir-team/+archive/staging
?

---

From: Thomi Richards <[email-masked]>
Date: Thu, 28 Mar 2013 09:51:46 +1300

Hi Daniel,

>    https://launchpad.net/~mir-team/+archive/staging

This sounds like a job for.... QE Man!

I'm working on running glmark2 on mir on a daily basis and publishing the results... 
so I'll add this to my TODO list.

Cheers,

-- 
Thomi Richards

---

From: Alexandros Frantzis <[email-masked]>
Date: Thu, 28 Mar 2013 09:53:50 +0200

> so I'll add this to my TODO list.

Hi Thomi,

lp:glmark2 contains a backend for mir, but with the latest changes in
lp:mir, it now fails to build. Since we haven't yet stabilized our
headers/API, and to avoid flooding upstream with changes, I will create
lp:~mir-team/glmark2/mir and continue development there.  When we
stabilize I will push our changes upstream.

To configure for mir, you need:

./waf configure --with-flavors=mir-gl,mir-glesv2 --prefix=/usr

FYI, the packaging branch for glmark2 daily packages (x11 only):

https://code.launchpad.net/~glmark2-dev/glmark2/glmark2-package

Thanks,
Alexandros

---

From: Thomi Richards <[email-masked]>
Date: Tue, 02 Apr 2013 16:52:57 +1300

Hi everyone,

> stabilize I will push our changes upstream.

I tried building lp:~mir-team/glmark2/mir and failed. It looks like this
branch hasn't been updated for a while? I went ahead and created
lp:~mir-team/glmark2/fix-mir-include
<https://code.launchpad.net/+branch/%7Emir-team/glmark2/fix-mir-include>
which contains the header file location fix required to build lp:glmark2
against the latest libmirclient-dev package sin the staging PPA. I also
had to hack the packaging branch a fair bit in order to get this to build.

There's a build recipe here:

https://code.launchpad.net/~mir-team/+recipe/glmark2-daily

That's now set to build daily. If/when it starts failing I'll try and
fix it up again. My next step is to get glmark2 running on top of mir on
the testing laptop, and then finally to automate the whole thing on the
jenkins infrastructure.

Cheers,

-- 
Thomi Richards

---

From: Alexandros Frantzis <[email-masked]>
Date: Tue, 2 Apr 2013 10:58:27 +0300

> had to hack the packaging branch a fair bit in order to get this to build.

There was a mix-up with the branches. Now lp:~mir-team/glmark2/mir
contains the latest version.

> the testing laptop, and then finally to automate the whole thing on the

Great, looking forward to that!

Thanks,
Alexandros

---

From: Thomi Richards <[email-masked]>
Date: Mon, 08 Apr 2013 08:55:59 +1200

Hi,

> contains the latest version. 

This branch fails to build this morning:

[ 56/124] cxx: src/native-state-mir.cpp -> build/src/native-state-mir.cpp.1.o
../src/native-state-mir.cpp: In member function 'virtual bool NativeStateMir::create_window(const NativeState::WindowProperties&)':
../src/native-state-mir.cpp:112:80: error: too few arguments to function 'MirSurface* mir_surface_create_sync(MirConnection*, const MirSurfaceParameters*, const MirEventDelegate*)'
In file included from ../src/native-state-mir.h:26:0,
                 from ../src/native-state-mir.cpp:22:
/usr/include/mirclient/mir_toolkit/mir_client_library.h:141:13: note: declared here
Waf: Leaving directory `/build/buildd/glmark2-2012.08+268+17~quantal1/build'
Build failed
 -> task in 'glmark2-mir' failed (exit status 1): 
	{task 1439278220: cxx native-state-mir.cpp -> native-state-mir.cpp.1.o}

When you get a moment could you please update that branch and let me
know when I can try the build again?

Cheers,

-- 
Thomi Richards

---

From: Daniel van Vugt <[email-masked]>
Date: Mon, 08 Apr 2013 09:31:24 +0800

I'm not yet convinced that the offending API change is exactly in a form 
that we want to keep. So please hold off changing glmark2 to suit.

...or if you do change glmark2 then just beware you might have to change 
it again later.

On 08/04/13 04:55, Thomi Richards wrote:
> ['/usr/bin/g++', '-g', '-O2', '-D_FORTIFY_SOURCE=2', '-g', '-O2', '-fstack-protector', '--param=ssp-buffer-size=4', '-Wformat', '-Werror=format-security', '-Werror', '-Wall', '-Wextra', '-Wnon-virtual-dtor', '-pthread', '-I/build/buildd/glmark2-2012.08+268+17~quantal1/build/src', '-I/build/buildd/glmark2-2012.08+268+17~quantal1/src', '-I/build/buildd/glmark2-2012.08+268+17~quantal1/build/src/scene-ideas', '-I/build/buildd/glmark2-2012.08+268+17~quantal1/src/scene-ideas', '-I/build/buildd/glmark2-2012.08+268+17~quantal1/build/src/scene-terrain', '-I/build/buildd/glmark2-2012.08+268+17~quantal1/src/scene-terrain', '-I/build/buildd/glmark2-2012.08+268+17~quantal1/build/src/libmatrix', '-I/build/buildd/glmark2-2012.08+268+17~quantal1/src/libmatrix', '-I/usr/include/libpng12', '-I/usr/include/mirclient', '-I/usr/include/mircommon', '-I/usr/include/libdrm', '-DHAVE_STDLIB_H=1', '-DHAVE_STRING_H=1', '-DHAVE_UNISTD_H=1', '-DHAVE_STDINT_H=1', '-DHAVE_STDIO_H=1', '-DHAVE_JPEGLIB_H=1', '-DHAVE
_MEMSET=1', '-DHAVE_SQRT=1', '-DHAVE_LIBPNG12=1', '-DHAVE_X11=1', '-DHAVE_GL=1', '-DHAVE_EGL=1', '-DHAVE_GLESV2=1', '-DHAVE_DRM=1', '-DHAVE_MIRCLIENT=1', '-DGLMARK_DATA_PATH="/usr/share/glmark2"', '-DGLMARK_VERSION="2012.12"', '-DUSE_EXCEPTIONS', '-DGLMARK2_USE_MIR', '-DGLMARK2_USE_GL', '-DGLMARK2_USE_EGL', '../src/native-state-mir.cpp', '-c', '-o', 'src/native-state-mir.cpp.1.o']

---

From: Alexandros Frantzis <[email-masked]>
Date: Mon, 8 Apr 2013 15:37:26 +0300

On Mon, Apr 08, 2013 at 09:31:24AM +0800, Daniel van Vugt wrote:
> I'm not yet convinced that the offending API change is exactly in a
> form that we want to keep. So please hold off changing glmark2 to
> suit.
> 
> ...or if you do change glmark2 then just beware you might have to
> change it again later.

No problem. That's the reason we chose to work in lp:~mir-team/glmark2/mir
for the time being. When the API stabilizes we will push the final changes
upstream.

Thanks,
Alexandros

---

From: Alexandros Frantzis <[email-masked]>
Date: Mon, 8 Apr 2013 15:37:52 +0300

> know when I can try the build again?

Should be fixed.

Thanks,
Alexandros