Bug 652321 - jbd2_stats_proc_init has wrong location.
Summary: jbd2_stats_proc_init has wrong location.
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: kernel
Version: 5.7
Hardware: Unspecified
OS: Linux
low
medium
Target Milestone: rc
: ---
Assignee: Lukáš Czerner
QA Contact: Jian Li
URL:
Whiteboard:
Depends On:
Blocks: 655875
TreeView+ depends on / blocked
 
Reported: 2010-11-11 16:13 UTC by sheng.yang
Modified: 2014-03-04 00:06 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 655875 (view as bug list)
Environment:
Last Closed: 2011-07-21 10:26:29 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
fix patch (706 bytes, patch)
2010-11-11 16:13 UTC, sheng.yang
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2011:1065 0 normal SHIPPED_LIVE Important: Red Hat Enterprise Linux 5.7 kernel security and bug fix update 2011-07-21 09:21:37 UTC

Description sheng.yang 2010-11-11 16:13:34 UTC
Created attachment 459791 [details]
fix patch

Description of problem:

jbd2_stats_proc_init() was located on a wrong place. The /proc/fs/jbd2/<dev>/ cannot be create when use a external journal device. Note that this problem exist in rhel6 also.

Version-Release number of selected component (if applicable):
2.6.18-* 2.6.32-*

Comment 1 Eric Sandeen 2010-11-11 16:48:45 UTC
It'd be great if you could send this patch upstream as well, thanks.  That way it'll be fixed upstream and you'll get credit for the fix.

After that it looks quite reasonable for a RHEL6 fix.

Thanks,
-Eric

Comment 2 Ric Wheeler 2010-11-11 17:35:23 UTC
It would also be great to have you report this issue through your official support channel if you have a RHEL subscription or via your partner engineering contact. That way, we can track it properly as we put together issues that make a release.

Thanks!

Comment 3 Lukáš Czerner 2010-11-22 15:56:48 UTC
This has been fixed upstream with commit 0587aa3d11f9769a301b21bff2c3ed8365606b8d.

For RHEL6 there is the same fix and bz#655875

Comment 4 RHEL Program Management 2011-02-01 16:52:26 UTC
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.

Comment 11 Jarod Wilson 2011-03-23 21:44:09 UTC
Patch(es) available in kernel-2.6.18-250.el5
Detailed testing feedback is always welcomed.

Comment 13 Jian Li 2011-04-06 06:12:43 UTC
Bug occur in 2.6.18-246.el5, be varified in 2.6.18-250.el5, output of testcase (/kernel/filesystems/ext4/655875-jbd2_stats_proc_init-wrong) is as follows:
<NOTICE: e4fsprogs package is need on RHEL5.7>

Linux dell-pe1950-02.rhts.englab.brq.redhat.com 2.6.18-246.el5 #1 SMP Tue Mar 1 15:17:48 EST 2011 x86_64 x86_64 x86_64 GNU/Linux
[root@dell-pe1950-02 655875-jbd2_stats_proc_init-wrong]# make run
chmod a+x ./runtest.sh
./runtest.sh
prepare image files
8+0 records in
8+0 records out
......
mount fs
ls /proc/fs/jbd2

=====================================================
Linux dell-pe1950-02.rhts.englab.brq.redhat.com 2.6.18-250.el5 #1 SMP Mon Mar 21 19:15:18 EDT 2011 x86_64 x86_64 x86_64 GNU/Linux
[root@dell-pe1950-02 655875-jbd2_stats_proc_init-wrong]# make run
chmod a+x ./runtest.sh
./runtest.sh
prepare image files
8+0 records in
.........
mount fs
ls /proc/fs/jbd2
loop3

Comment 14 errata-xmlrpc 2011-07-21 10:26:29 UTC
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-1065.html


Note You need to log in before you can comment on or make changes to this bug.