Canonical System Image

Preview generation should not rely on glReadPixels

Bug #1407640 reported by Thomas Voß on 2015-01-05

This bug affects 2 people

Affects		Status	Importance	Assigned to	Milestone
	Canonical System Image	Confirmed	High	Unassigned	Canonical System Image backlog
	qtvideo-node (Ubuntu)	Confirmed	High	Jim Hodapp

Bug Description

Right now, snapshot/preview generation relies on a call to QGlFramebufferObject::toImage to obtain a QImage in http://bazaar.launchpad.net/~phablet-team/qtvideo-node/trunk/view/head:/src/snapshotgenerator.cpp#L118. Ultimately, this results in a very expensive call to glReadPixels, blocking the buffer processing of the camera app. Instead, we should avoid mapping the pixels to main memory and just copy the texture.

Thomas Voß (thomas-voss) on 2015-01-05

Changed in qtvideo-node (Ubuntu):
importance:	Undecided → High

Pat McGowan (pat-mcgowan) on 2015-10-16

Changed in canonical-devices-system-image:
assignee:	nobody → John McAleely (john.mcaleely)
importance:	Undecided → High
milestone:	none → backlog

Pat McGowan (pat-mcgowan) on 2015-10-22

Changed in canonical-devices-system-image:
status:	New → Confirmed

John McAleely (john.mcaleely) on 2017-01-05

Changed in canonical-devices-system-image:
assignee:	John McAleely (john.mcaleely) → nobody

Revision history for this message

Launchpad Janitor (janitor) wrote on 2017-01-05:

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

Changed in qtvideo-node (Ubuntu):
status:	New → Confirmed

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.