GLVND: AARCH64 : Fix address passed to clear cache

Bug #1816004 reported by Sai Kiran Korwar on 2019-02-15
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
libglvnd (Ubuntu)
Undecided
Timo Aaltonen
Bionic
Undecided
Timo Aaltonen

Bug Description

[Impact]

A new fix has been merged to NVIDIA libglvnd repo at https://github.com/NVIDIA/libglvnd/pull/171/commits/9948df636708e2915e5c2daa61b318e6252bc02e
We need to include it in bionic.
It would be ok to just include this fix even without updating to the latest glvnd version.

This change fixes a crash seen in some GLX apps while calling glXGetProcAddress.

Source Package : https://launchpad.net/ubuntu/+source/libglvnd/1.0.0-2ubuntu2.2

Can we get this fix merged in current LTS 18.04 as "-updates"

[Test app]

To reproduce this failure: Install python-pyglet on 18.04 ubuntu.
And run a small test app with this piece of code.
void *handle = dlopen("libGLX.so.0", 2);
glXGetProcAddressARB = (GLXextFuncPtr (*)(const GLubyte*))dlsym(handle, "glXGetProcAddressARB");
glXGetProcAddressARB("glXAllocateMemoryMESA");

[Regression potential]

Minimal, disco shipped with the fix.

affects: vulkan (Ubuntu) → libglvnd (Ubuntu)
description: updated
Timo Aaltonen (tjaalton) on 2019-02-15
Changed in libglvnd (Ubuntu):
assignee: nobody → Timo Aaltonen (tjaalton)
Timo Aaltonen (tjaalton) wrote :

would be nice to get a new upstream release, since this commit isn't in a tagged release

Kyle Brenneman (kyle-brenneman) wrote :

I just tagged a 1.1.1 release for libglvnd with the AARCH64 and PPC64LE fixes:
https://github.com/NVIDIA/libglvnd/releases/tag/v1.1.1

If you're switching from 1.0.0, note that the dispatch table order changed between v1.0.0 and v1.1.0, so you'll need matching builds of libGLdispatch.so, libOpenGL.so, libGL.so, and the libGLES*.so libraries.

Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in libglvnd (Ubuntu):
status: New → Confirmed
Sai Kiran Korwar (skirank) wrote :

Hi Timo,

I am hoping we can get the latest v1.1.1 release in Cosmic Cuttlefish and Disco Dingo as well.
Could you help update?

Timo Aaltonen (tjaalton) wrote :

disco will, it's on the queue and should be accepted after beta freeze is over

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libglvnd - 1.1.1-0ubuntu1

---------------
libglvnd (1.1.1-0ubuntu1) disco; urgency=medium

  * New upstream release. (LP: #1816004)

 -- Timo Aaltonen <email address hidden> Wed, 13 Mar 2019 20:05:49 +0200

Changed in libglvnd (Ubuntu):
status: Confirmed → Fix Released
Sai Kiran Korwar (skirank) wrote :

Thanks Timo.

Can you confirm if this will be available with bionic as well?

Sai Kiran Korwar (skirank) wrote :

Timo,

Can you give an update about the inclusion of this fix in bionic?
We still need this in bionic.

Timo Aaltonen (tjaalton) wrote :

depends on the nature of the backport, I'll ask if we can just update straight to 1.1.1

Changed in libglvnd (Ubuntu Bionic):
assignee: nobody → Timo Aaltonen (tjaalton)
Sai Kiran Korwar (skirank) wrote :

Updating to v1.1.1 would be best. Thanks.

Sai Kiran Korwar (skirank) wrote :

Any update here? Thanks!

Timo Aaltonen (tjaalton) wrote :

uploaded to bionic-proposed queue, with just the fix added

Changed in libglvnd (Ubuntu Bionic):
status: New → In Progress
description: updated
description: updated
Alka Mohite (amohite) wrote :

Is there an update on the inclusion of the fix in bionic-update?

Timo Aaltonen (tjaalton) wrote :

somehow the upload didn't end up in the queue earlier, but is there now..

Hello Sai, or anyone else affected,

Accepted libglvnd into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/libglvnd/1.0.0-2ubuntu2.3 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in libglvnd (Ubuntu Bionic):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-bionic
Sai Kiran Korwar (skirank) wrote :

I have verified that the libglvnd package libglvnd-1.0.0-2ubuntu2.3 fixes the bug for me.
Added the verification-done-bionic tag.

Thanks for including this fix.

tags: added: verification-done-bionic
removed: verification-needed verification-needed-bionic
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libglvnd - 1.0.0-2ubuntu2.3

---------------
libglvnd (1.0.0-2ubuntu2.3) bionic; urgency=medium

  * aarch64-fix-address-passed-to-clear-cache.diff: Fix a crash in some
    GLX apps. (LP: #1816004)

 -- Timo Aaltonen <email address hidden> Fri, 10 May 2019 15:17:37 +0300

Changed in libglvnd (Ubuntu Bionic):
status: Fix Committed → Fix Released

The verification of the Stable Release Update for libglvnd has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers