Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 2 years ago

Closed 22 months ago

Last modified 22 months ago

#10292 closed Patch - Bug Fix (fixed)

Fix for compilation and segfault on Nokia N9/N950

Reported by: jflatt@… Owned by: beirdo
Priority: minor Milestone: 0.26
Component: MythTV - General Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no

Description

This includes ffmpeg upstream commit 8b454c352f49c2a61db37793d838b553db3da734, which fixes compilation on arm.

Also, QGLContext::getProcAddress is returning 0, but calls to dlsym succeed. I'm not sure if dlfcn.h is on other platforms, it probably needs ifdef'd, but this is a start.

Attachments (2)

diff.txt (1.2 KB) - added by jflatt@… 2 years ago.
git diff
diff2.txt (679 bytes) - added by jflatt@… 22 months ago.
diff using QLibrary

Download all attachments as: .zip

Change History (8)

Changed 2 years ago by jflatt@…

git diff

comment:1 Changed 2 years ago by markk

For the proc address issue, can you check whether using QLibary works. So something like:-

result = QLibrary::resolve("libGL", (proc + exts[i]).toLatin1().data());

instead of using dlsym

comment:2 Changed 2 years ago by jflatt@…

Yes, QLibrary works. For at least this particular device, the library name is libGLESv2. I still get an all-green video, possibly YUV2RGB conversion problem. UI is good, OSD is good, but CPU is just as high as xv-blit.

comment:3 Changed 22 months ago by beirdo

  • Owner set to beirdo
  • Status changed from new to assigned

comment:4 Changed 22 months ago by beirdo

  • Status changed from assigned to infoneeded

The ffmpeg part of this is in master (will be 0.26) already.

As for the dlsym, do you have a new patch using the QLibrary call that would work for you? I think Mark's concern was with the portability to platforms such as Windows.

Changed 22 months ago by jflatt@…

diff using QLibrary

comment:5 Changed 22 months ago by jflatt <jflatt@…>

  • Resolution set to fixed
  • Status changed from infoneeded to closed

In 8211f9ddf1c1bbb6c1fe6a888044072e8f87aa54/mythtv:

Use QLibrary to find libGLESv2 for ARM

Please let us know if this causes regressions in other GLES setups.

Fixes #10292

Signed-off-by: Gavin Hurlbut <ghurlbut@…>

comment:6 Changed 22 months ago by beirdo

  • Milestone changed from unknown to 0.26

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.