edac-fake-inject: Check if the Kernel supports error injection

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Get rid of mc_event_error_type

Somehow, the tracing library is not finding it on some systems:
overriding event (710) ras:mc_event with new print handler
trace-cmd: File exists
  function mc_event_error_type not defined

Let's just get rid of it.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Better handle parser errors with MC events

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

edac-fake-inject: Make it more generic

The tool used to support only 2 or 3 layer memory controllers,
faling with edac_ghes driver. Make it more generic to also work
there.

Also, don't assume that the SYSFS is mounted at /sys/kernel/debug,
but look at its mount location via /proc/mounts.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Fix rasdaemon -d

We need to get the debugfs pointer in order to toggle the MC events.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Get rid of the remaining warnings

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

libtrace: get rid of breakpoint() function

This isn't used anywhere.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Get rid of most warnings at libtrace

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Fix usage of toggle_ras_mc_event() by -d parameter

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Enable gcc warnings

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

rasdaemon.spec: specify the root directory on a consistent way

As fedora-review tool complained:

- Package consistently uses macro is (instead of hard-coded directory names).
  Note: Using both %{buildroot} and $RPM_BUILD_ROOT
  See: http://fedoraproject.org/wiki/Packaging/Guidelines#macros

Let's just use %{buildroot}.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Update it to point it to fedorapeople

The tarball used to generate the src.rpm is the one produced
by "make dist-bz2", with doesn't contain .gitignore files,
while fedorahosted only generates an snapshot with them.

That makes its hash to not match the one used at .src.rpm.

Fix it by using the uploaded file.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a target to upload a new version

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Update the spec file to require autotools for building it

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

rasdaemon.spec: Don't install INSTALL file

rpmlint complains with that:
rasdaemon.x86_64: W: install-file-in-docs /usr/share/doc/rasdaemon-0.3.0/INSTALL

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

rpmlint: fix version compliant

rasdaemon.x86_64: W: incoherent-version-in-changelog 0.2.0-1 ['0.3.0-1.fc18', '0.3.0-1']

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-mc-ctl.8.in: fix rpmlint compliants

rasdaemon.x86_64: W: manual-page-warning /usr/share/man/man8/ras-mc-ctl.8.gz 79: a space character is not allowed in an escape name
rasdaemon.x86_64: W: manual-page-warning /usr/share/man/man8/ras-mc-ctl.8.gz 122: warning: macro `EL' not defined

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Whitespace cleanups

No functional changes here, just whitespacing cleanups.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

rpmlint target: RPMS files are wrong. Fix it.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Fix rpmlint check line

It should not be checking the .tar.bz2, but, instead, the generated
rpm files.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add missing header files to Makefile.am

This is needed, in order to generate the proper dist tar files.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Bump it to version 0.3.0

As we now have initial mcelog/PCEe AER parsing, bump version
to 0.3.0.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a rule to build a source rpm file

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Auto-fill the rasdaemon.spec version

Instead of keeping it static, let ./configure to fill the
version of the rasdaemon.spec. That makes it a little easier
to be used on rpm-based distros.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add decoder for Ivy Bridge

The code came from mcelog. For now, let's disable the part that
handles the memory controller.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add decoder for Sandy Bridge

The code came from mcelog. For now, let's disable the part that
handles the memory controller.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add decoder for Intel MCE tulsa

The code came almost as-is from mcelog.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add decoder for Intel Dunnington CPUs

The code came almost as-is from mcelog.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a decoder for Nehalem-specific types

Note: Memory Controller-specific decoding was excluded.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a parser for Intel P4/P6 specific CPU error messages

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a parser for Intel P4/P6 processors

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

mce-intel: Add support to decode MCI/MCA

As almost all mce decoding code, those code came from Andi Kleen's
mcelog application.

While the code added there came from p4.c and nehalem.c, they're
used by all Intel CPUs so far.

Intel CPU-specific code parsing is still not implemented.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

mce-intel: simplify code and add an user_action field

While for pure print messages, the user recommended action can be
together with the error message, having it in a separate field
helps to latter handle the error. So, split it.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

mce-amd-k8: Code cleanups

Instead of doing the error_msg buffer filling logic everywhere,
move it to a common routine.

That cleans up the code a lot, and makes easier to use the same
code to also handle other *_msg fields on latter patches.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

mce-intel: add support to decode termal bank and mcg

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Improve MCE parser for AMD k8

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

mce-amd-k8: add status decoding logic

Add the status decoding logic from mcelog's k8.c file.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add per-cpu-type handlers for MCE log

For now, only the bank information is handled.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a basic handler for MCE logs

For now, this handler just detects the CPU type and parses all
fields at the MCE event trace.

Latter patches will add decoding capabilities to the event.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-events: prepare to handle MCE events

Parsing MCE events is hard, as it requires per-cpu-type parsing.
We can at least get those events and send them to syslog/journald.

So, ask tracing to collect them as well and add a hook for the
future mcelog parsing code.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add support for PCIe AER events

The code is currently untested, as I'm missing a testing
system where I could inject PCIe AER events.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Fix dummy function arguments when compiled without sqlite3 support

That shuts up a warning.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-mc-handler: remove some unused headers

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

rasdaemon: Better handle error conditions

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Print cpu number at event records log

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-record: retry open if busy

As we'll have several concurrent opens at the same time, we
need to retry if race conditions happen.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-events: make the error patch to do the right thing

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

README: add project goals

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Update README

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-events: open database on each thread

sqlite3 is only able to prevent race issues between different
threads if each thread opens its own connection to the database.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Update tarball URL

That makes rpmlint happy

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Fix make dist-* targets

Those targets require to know what are the header files.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a RPM spec file to build it with rpmbuild

Add a rasdaemon.spec template file useful for compiling it with
Fedora. It may require changes to work with other distributions
that also use rpm files, as each distro has their own rules for
rpm's, but at least this file can be used as a reference.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a service to register EDAC labels

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a manpage for the rasdaemon

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Modify the ras-mc-ctl manpage to reflect the current tool

Several changes happened at the tool, since when it was
part of edac-utils. Also, a few new options got added there.

Add the missing parts and change it to reflect its new name.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a man page for ras-mc-ctl

This is currently the same as edac-utils, but needs to be
re-written.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Parse ras-mc-ctl via autoconf tools

Instead of using fixed directory prefixes, let the building
system to tell them, via ./configure.

This uses the very same solution as edac-utils do.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add more autotools stuff into .gitignore

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

rasdaemon: change it to version 0.2.0

There are too many changes there, and it is now on a more functional
state. So, bump it to version 0.2.0.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-daemon: make sqlite3 code optional

The sqlite3 code is experimental and currently having some issues
with multi-threading. Convert it into an optional feature that
it is only explicitly enabled.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Update TODO file

ras-mc-ctl is the tool that allows inspecting/modifying the EDAC
data on sysfs. Remove it from the TODO file.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-mc-ctl: update copyright file

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-mc-ctl: rename edac-ctl to ras-mc-ctl

In order to allow both EDAC and rasdaemon to be installed
at the same system, rename the control tool to ras-mc-ctl.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

edac-ctl: drop support for legacy EDAC nodes

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

edac-ctl: Convert it to use EDAC version 3.0.0 (HERM)

EDAC version 3.0.0 (HERM) added a new set of sysfs nodes
that provides the real location of the memory banks.

Use it, instead of the fake csrow/channel location.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

edac-ctl: move dimm label parsing routine to the beginning

This routine will be used later on other places. So, move it to
the beginning.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Remove the already solved issues from TODO.

For now, don't renumber it, as the numbering is still usefull.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-events: Use uptime clock if available

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-events: Use per-instance tracing ringbuffer, if supported

Kernel 3.10-rc1 and upper supports per-instance ringbuffers.
Use it if available.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-events: warn if something got wrong with threads

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Some fixes at the log function

1) use the do { } while (0) logic, as it helps to avoid
   troubles with the define (see Kernel's Documentation);

2) add a "\n" where missing;

3) add ';' after log(...) to make it to look as a
real function.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

move ras events generic code into a separate file

As we'll add other event types, move the ras events generic
code into a separate dir.

While here, get hid of the DEBUGFS define, replacing it by
a code that will discover where the debugfs got mounted on
the system.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

libtrace: sync with latest changeset

Sync it with the last upstream changeset(07f0eee).

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add edac-ctl perl script to contrib

edac-ctl is part of edac-utils and it was written by
Mark Grondona <mgrondona@llnl.gov>. While it uses the old
EDAC API most of the time, it offers some interesting
features for EDAC. So, add it here.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

add two test scripts to contrib

edac-fake-inject allows testing the EDAC daemon, if the
Kernel is compiled with EDAC_DEBUG.

edac-tests run a series of tests and produce a report for
a given EDAC driver.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

rasdaemon: logging via syslog enabled

Patch adds simple (so far) code for customizing of logging levels
and possible outputs. Macro log(..) were introduced and should be
called in all places where rasdaemon could send some output either
to console or syslog.

Systemd unit file was added, so systemd will guarantee that daemon
will be restarted immediately after stop or killing.

Signed-off-by: Petr Holasek <pholasek@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

README: fixed description of configure script call

Signed-off-by: Petr Holasek <pholasek@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

The m4 directory needs to exist for autoreconf to work

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

TODO: use a separate ring buffer

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

TODO: numerate items there to help tracking them

Let's numerate the numbers at the list, as it helps to
reference them.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

TODO: update it to reflect the current status

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-mc-event: fix field names

A previous name replacement broke the namespace of the strings

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-mc-event: use the default logic to display the parse error

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-mc-event: use the default logic to display the parse error

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Update NEWS to reflect the rationale behind the project

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Better comment about the sense of timestamp

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Split sqlite3 statement prepare into a separate function

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Wait forever at poll()

timeout = 0 doesn't wait. What we want there is to wait forever

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a README file for libtrace

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

rasdaemon: make record events optional

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

ras-record: don't try to record if database open failed

Check if ras->stmt is not null before writing it. That avoids
the risk of trying to write something if the database was not
opened.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

remove plugin-ras.c

This file is not used.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add ACLOCAL_AMFLAGS to Makefile.am

Fixes a warning, from libtool:

libtoolize: Consider adding `-I m4' to ACLOCAL_AMFLAGS in Makefile.am.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add information about the needed packages to compile it

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Update gitignore to not produce false reports

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add quick usage instructions at README

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Improve INSTALL procedure

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

Add a TODO list

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

rasdaemon: by default, start the daemon

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>