futex needs __get_user_pages_fast implementation

Bug #1236949 reported by Zi Shen Lim
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linaro-networking
Fix Released
High
Gary S. Robertson

Bug Description

Futex uses GUP. Currently on ARM, the default __get_user_pages_fast being used always returns 0, leading to a forever loop in get_futex_key.

Revision history for this message
Zi Shen Lim (zlim) wrote :

Steve Capper (LEG) is working on GUP.

Changed in linaro-networking:
status: New → In Progress
Revision history for this message
Mike Holmes (mike-holmes) wrote :

Steve Cappers fix has been applied to the LNG kernel, but the fix has not been applied upstream yet.

Changed in linaro-networking:
importance: Undecided → High
Revision history for this message
Steve Capper (steve-capper) wrote :
Revision history for this message
Mike Holmes (mike-holmes) wrote :

Latest version of this patch should be applied to the LNG kernel when it is updated to the latest RT patch in the 13.11 cycle.

Changed in linaro-networking:
assignee: nobody → Gary S. Robertson (gary-robertson)
Revision history for this message
Gary S. Robertson (gary-robertson) wrote :

Steve Capper's patches to address this were applied to the LNG kernels in time for the 13.10 release cycle.

I compared the contents of these applied patches line for line with the RFC submissions in the link posted in comment #3 for this bug. Other than some minor differences in line numbering in some new files, the content of the already-applied patches is identical to the content of the patches submitted for comment.

AFAIK there have been no comments on the patches as submitted upstream, nor am I aware that those patches have been accepted and applied to the upstream kernel as yet. Consequently at this time I am disinclined to take any action to replace the patches already applied to the LNG kernels with their counterparts from the RFC link in comment #3.

At such time as the patches are applied upstream I will be happy to revert our local commits in order to avoid future merge conflicts with upstream kernel updates. Then we can either cherry-pick or simply merge the patches from upstream.

Changed in linaro-networking:
status: In Progress → Fix Committed
Changed in linaro-networking:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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