Re: PATCH: EDAC, core EDAC support code

[Arjan van de Ven <arjan@infradead.org>]

> +
> +static ctl_table mc_table[] = {
> +	{-1, "panic_on_ue", &panic_on_ue,
> +	 sizeof(int), 0644, NULL, proc_dointvec},
> +	{-2, "log_ue", &log_ue,
> +	 sizeof(int), 0644, NULL, proc_dointvec},
> +	{-3, "log_ce", &log_ce,
> +	 sizeof(int), 0644, NULL, proc_dointvec},
> +	{-4, "poll_msec", &poll_msec,
> +	 sizeof(int), 0644, NULL, proc_dointvec},
> +	{-7, "panic_on_pci_parity", &panic_on_pci_parity,
> +	 sizeof(int), 0644, NULL, proc_dointvec},
> +	{-8, "check_pci_parity", &check_pci_parity,
> +	 sizeof(int), 0644, NULL, proc_dointvec},
> +#ifdef CONFIG_EDAC_DEBUG
> +	{-9, "debug_level", &edac_debug_level,
> +	 sizeof(int), 0644, NULL, proc_dointvec},
> +#endif
> +	{0}
> +};

why aren't these module parameters? Nowadays you can have sysfs-runtime
changable module parameters... sounds a lot nicer than adding a chunk of
sysctl stuff

> +
> +#ifdef CONFIG_PROC_FS
> +static const char *mem_types[] = {
> +	[MEM_EMPTY] = "Empty",
> +	[MEM_RESERVED] = "Reserved",
> +	[MEM_UNKNOWN] = "Unknown",
> +	[MEM_FPM] = "FPM",
> +	[MEM_EDO] = "EDO",
> +	[MEM_BEDO] = "BEDO",
> +	[MEM_SDR] = "Unbuffered-SDR",
> +	[MEM_RDR] = "Registered-SDR",
> +	[MEM_DDR] = "Unbuffered-DDR",
> +	[MEM_RDDR] = "Registered-DDR",
> +	[MEM_RMBS] = "RMBS"
> +};


why proc? Again this sounds like a platform sysfs thing...