Bug 622024
Summary: | 64-bit kernel unable to oprofile 32-bit processes | ||||||
---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 5 | Reporter: | Tom Marshall <tdm.rhbz> | ||||
Component: | kernel | Assignee: | Jiri Olsa <jolsa> | ||||
Status: | CLOSED ERRATA | QA Contact: | Chao Ye <cye> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | low | ||||||
Version: | 5.5 | CC: | anton, cye, phan, qcai | ||||
Target Milestone: | rc | ||||||
Target Release: | --- | ||||||
Hardware: | x86_64 | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2011-01-13 21:09:29 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
hi, I sent the patch upstream http://lkml.org/lkml/2010/9/13/169 backported RHEL5 kernel could be downloaded here: http://people.redhat.com/jolsa/622024/ will send it to review, once accepted upstream. Any test results appreciated ;) thanks, jirka Upstream had some comments and the patch seems to be dropped. Some of the comments are specific to the later kernel (stack_frame_ia32 does not seem to exist in 2.6.18) and some are not (prefer CONFIG_COMPAT over CONFIG_X86_64). If upstream acceptance is a prerequisite to inclusion in RHEL, please advise whether you would like to fix and resubmit or whether you would like me to resubmit. Thanks! (In reply to comment #2) > Upstream had some comments and the patch seems to be dropped. Some of the > comments are specific to the later kernel (stack_frame_ia32 does not seem to > exist in 2.6.18) and some are not (prefer CONFIG_COMPAT over CONFIG_X86_64). > > If upstream acceptance is a prerequisite to inclusion in RHEL, please advise > whether you would like to fix and resubmit or whether you would like me to > resubmit. The upstream acceptance is crucial.. and I guess you are more oprofile experienced.. if you can resubmit the patch for upstream, it'd be big help. Feel free to directly answer the upstream conversation. thanks, jirka Actually, I'm not an oprofile user, I'm just the RHEL maintainer and "Linux guy" here. :) But I'll respin the patch and toss it back out to lkml. Once this gets accepted, will it automagically be included in RHEL 6? If not, what is the procedure to ensure that happens? I can send out the patch for RHEL6 as well, np thanks, jirka hi, any update on the upstream patch? thanks, jirka I sent new version upstream: http://marc.info/?l=linux-kernel&m=128577164225720&w=2 jirka This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux maintenance release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux Update release for currently deployed products. This request is not yet committed for inclusion in an Update release. Thanks Jiri! in kernel-2.6.18-227.el5 You can download this test kernel (or newer) from http://people.redhat.com/jwilson/el5 Detailed testing feedback is always welcomed. Confirmed that oprofile-0.9.6 can be compiled under kernel-2.6.18-221.el5.x86_64 An advisory has been issued which should help the problem described in this bug report. This report is therefore being closed with a resolution of ERRATA. For more information on therefore solution and/or where to find the updated files, please follow the link below. You may reopen this bug report if the solution does not work for you. http://rhn.redhat.com/errata/RHSA-2011-0017.html |
Created attachment 437244 [details] Add logic to backtrace ia32 processes in x86_64 kernel Description of problem: The oprofile kernel module, when compiled under x86_64, is unable to obtain a proper backtrace on ia32 userspace processes because it lacks the logic to do so. Please find the attached patch to add this logic.