Skip to content

Commit

Permalink
ramoops: Move to fs/pstore/ram.c
Browse files Browse the repository at this point in the history
Since ramoops was converted to pstore, it has nothing to do with character
devices nowadays. Instead, today it is just a RAM backend for pstore.

The patch just moves things around. There are a few changes were needed
because of the move:

1. Kconfig and Makefiles fixups, of course.

2. In pstore/ram.c we have to play a bit with MODULE_PARAM_PREFIX, this
   is needed to keep user experience the same as with ramoops driver
   (i.e. so that ramoops.foo kernel command line arguments would still
   work).

Signed-off-by: Anton Vorontsov <anton.vorontsov@linaro.org>
Acked-by: Marco Stornelli <marco.stornelli@gmail.com>
Acked-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Anton Vorontsov authored and gregkh committed May 16, 2012
1 parent d109a67 commit 1894a25
Show file tree
Hide file tree
Showing 7 changed files with 21 additions and 14 deletions.
2 changes: 1 addition & 1 deletion Documentation/ramoops.txt
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ Setting the ramoops parameters can be done in 2 different manners:
2. Use a platform device and set the platform data. The parameters can then
be set through that platform data. An example of doing that is:

#include <linux/ramoops.h>
#include <linux/pstore_ram.h>
[...]

static struct ramoops_platform_data ramoops_data = {
Expand Down
9 changes: 0 additions & 9 deletions drivers/char/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -585,15 +585,6 @@ config DEVPORT

source "drivers/s390/char/Kconfig"

config RAMOOPS
tristate "Log panic/oops to a RAM buffer"
depends on HAS_IOMEM
depends on PSTORE
default n
help
This enables panic and oops messages to be logged to a circular
buffer in RAM where it can be read back at some later point.

config MSM_SMD_PKT
bool "Enable device interface for some SMD packet ports"
default n
Expand Down
1 change: 0 additions & 1 deletion drivers/char/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,6 @@ obj-$(CONFIG_HANGCHECK_TIMER) += hangcheck-timer.o
obj-$(CONFIG_TCG_TPM) += tpm/

obj-$(CONFIG_PS3_FLASH) += ps3flash.o
obj-$(CONFIG_RAMOOPS) += ramoops.o

obj-$(CONFIG_JS_RTC) += js-rtc.o
js-rtc-y = rtc.o
Expand Down
14 changes: 14 additions & 0 deletions fs/pstore/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,17 @@ config PSTORE
(e.g. ACPI_APEI on X86) which will select this for you.
If you don't have a platform persistent store driver,
say N.

config PSTORE_RAM
tristate "Log panic/oops to a RAM buffer"
depends on HAS_IOMEM
depends on PSTORE
default n
help
This enables panic and oops messages to be logged to a circular
buffer in RAM where it can be read back at some later point.

Note that for historical reasons, the module will be named
"ramoops.ko".

For more information, see Documentation/ramoops.txt.
3 changes: 3 additions & 0 deletions fs/pstore/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,6 @@
obj-y += pstore.o

pstore-objs += inode.o platform.o

ramoops-objs += ram.o
obj-$(CONFIG_PSTORE_RAM) += ramoops.o
2 changes: 1 addition & 1 deletion drivers/char/ramoops.c → fs/pstore/ram.c
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
#include <linux/ioport.h>
#include <linux/platform_device.h>
#include <linux/slab.h>
#include <linux/ramoops.h>
#include <linux/pstore_ram.h>

#define RAMOOPS_KERNMSG_HDR "===="
#define MIN_MEM_SIZE 4096UL
Expand Down
4 changes: 2 additions & 2 deletions include/linux/ramoops.h → include/linux/pstore_ram.h
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#ifndef __RAMOOPS_H
#define __RAMOOPS_H
#ifndef __LINUX_PSTORE_RAM_H__
#define __LINUX_PSTORE_RAM_H__

/*
* Ramoops platform data
Expand Down

0 comments on commit 1894a25

Please sign in to comment.