Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 288781 - app-admin/mcelog needs patch to work with kernels >=2.6.31
Summary: app-admin/mcelog needs patch to work with kernels >=2.6.31
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: AMD64 Project
URL: http://www.kernel.org/pub/linux/utils...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-10-12 23:19 UTC by Jim Faulkner
Modified: 2010-03-31 17:42 UTC (History)
15 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch from https://bugzilla.redhat.com/show_bug.cgi?id=507026 (mcelog-0.7-record-length.patch,1.56 KB, patch)
2009-12-10 14:10 UTC, Paolo Pedroni
Details | Diff
Patch for mcelog ebuild (mcelog-0.7-r1.patch,710 bytes, patch)
2009-12-10 14:12 UTC, Paolo Pedroni
Details | Diff
mcelog-1.0_pre2.ebuild (mcelog-1.0_pre2.ebuild,1.05 KB, text/plain)
2010-01-12 20:45 UTC, Hanno Böck
Details
add eutils to inherit line. (mcelog-0.7-r1.patch,644 bytes, patch)
2010-02-10 05:34 UTC, Allen Parker
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jim Faulkner 2009-10-12 23:19:49 UTC
I discovered the following error in my cron output recently:
mcelog: warning: record length longer than expected. Consider update.

This problem appears to only occur when the running kernel is >=2.6.31.  This is because 2.6.31 included changes to the data structure which mcelog reads.  More information is here:
https://bugzilla.redhat.com/show_bug.cgi?id=507026

A patch to fix the issue is also included in the redhat bug report.
Comment 1 Sylvain BERTRAND 2009-10-14 11:26:42 UTC
they are readying 0.9. Seems there is a source RPM but no official source tar archive.
Comment 2 Paul B. Henson 2009-12-07 21:38:57 UTC
Not much progress from upstream. Any chance of getting the patch added in a rev bump pending the new release?
Comment 3 David Holl 2009-12-09 12:49:45 UTC
I am also interested in this fix.  May I contribute or help in any way?
Comment 4 Paolo Pedroni 2009-12-10 14:10:13 UTC
Created attachment 212621 [details, diff]
Patch from https://bugzilla.redhat.com/show_bug.cgi?id=507026
Comment 5 Paolo Pedroni 2009-12-10 14:12:27 UTC
Created attachment 212622 [details, diff]
Patch for mcelog ebuild
Comment 6 Hanno Böck gentoo-dev 2010-01-06 22:42:06 UTC
FYI, new mcelog pre-releases are at
http://www.kernel.org/pub/linux/utils/cpu/mce/
Comment 7 Hanno Böck gentoo-dev 2010-01-12 20:45:02 UTC
Created attachment 216280 [details]
mcelog-1.0_pre2.ebuild

This takes the latest 1.0_pre2 version (only change is the patch version and dbquery no longer exists).
Comment 8 Jim Faulkner 2010-01-24 16:42:03 UTC
pre3 is out:
http://www.kernel.org/pub/linux/utils/cpu/mce/mcelog-1.0pre3.tar.bz2

only changes from pre2 are:
> Add Xeon75xx support
> Add Intel architectural mca fallback for new family 6 Intel CPUs
Comment 9 Soren Harward 2010-01-31 05:39:49 UTC
(In reply to comment #8)
> pre3 is out:
> http://www.kernel.org/pub/linux/utils/cpu/mce/mcelog-1.0pre3.tar.bz2

Is the epatch command on line 27 wrong?  After emerge failed when it couldn't find the patch, I commented out the line and the program runs totally fine without it.
Comment 10 Allen Parker 2010-02-10 05:32:34 UTC
(In reply to comment #9)
> (In reply to comment #8)
> > pre3 is out:
> > http://www.kernel.org/pub/linux/utils/cpu/mce/mcelog-1.0pre3.tar.bz2
> 
> Is the epatch command on line 27 wrong?  After emerge failed when it couldn't
> find the patch, I commented out the line and the program runs totally fine
> without it.
> 

You also need to add 'eutils' to the inherit on line 5.
Comment 11 Allen Parker 2010-02-10 05:34:47 UTC
Created attachment 219061 [details, diff]
add eutils to inherit line.
Comment 12 Hanno Böck gentoo-dev 2010-02-13 00:00:12 UTC
I've committed now 1.0_pre3.
Comment 13 Matthew Gregory Sr. 2010-03-31 17:42:28 UTC
This has been in here for a while.  I think it should probably be taken care of.

I know that the _pre builds aren't ready to be marked stable, but I see no reason whatever that the 0.7-r1 patch can't be added to a new stable mcelog-0.7_r1.ebuild.

Is there a problem with not knowing which kernel the users have?

If so, I would think that the ebuild could easily check the kernel version and apply the epatch or not based on >= 2.6.31?