linux-kernel - Re: [PATCH] ACPI: remove unused ACPI

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 28 Oct 2013 14:13:01 +0800
From:	Lan Tianyu <lantianyu1986@...il.com>
To:	Michael Opdenacker <michael.opdenacker@...e-electrons.com>
Cc:	Len Brown <lenb@...nel.org>, rjw@...ysocki.net,
	"linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>,
	"linux-kernel@...r kernel org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] ACPI: remove unused ACPI_PROCFS Kconfig param

2013/10/27 Michael Opdenacker <michael.opdenacker@...e-electrons.com>:
> This patch removes the ACPI_PROCFS parameter
> (support for legacy /proc/acpi), which is defined
> but no longer used anywhere in the makefiles and source code.
>

I think this should be not removed now. Since there are still interfaces
under /proc/acpi (E.G, /proc/acpi/wakeup and /proc/acpi/lid/*).

The origin commit which introduces the kconfig is to make some
interfaces that have backup interfaces in the sysfs configurable before
removing them actually.

So we still need this kconfig when try to remove remaining /proc/acpi
interfaces.

commit b981c591891dc8885de36498d38fa8d8a5481069
Author: Zhang Rui <rui.zhang@...el.com>
Date:   Mon Jan 29 11:02:30 2007 +0800

    ACPI: add a Kconfig option for ACPI procfs interface

    Add a kconfig option CONFIG_ACPI_PROCFS to make procfs interface
    a configurable attribute of ACPI.

    No procfs interface is actually deprecated,
    and no sysfs interface is added in this patch.

    CONGI_ACPI_PROCFS is used to mark procfs interface as deprecated
    once the same function is duplicated in sysfs.


> Signed-off-by: Michael Opdenacker <michael.opdenacker@...e-electrons.com>
> ---
>  drivers/acpi/Kconfig | 13 -------------
>  1 file changed, 13 deletions(-)
>
> diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig
> index 6efe2ac..201cfb7 100644
> --- a/drivers/acpi/Kconfig
> +++ b/drivers/acpi/Kconfig
> @@ -43,19 +43,6 @@ config ACPI_SLEEP
>         depends on SUSPEND || HIBERNATION
>         default y
>
> -config ACPI_PROCFS
> -       bool "Deprecated /proc/acpi files"
> -       depends on PROC_FS
> -       help
> -         For backwards compatibility, this option allows
> -         deprecated /proc/acpi/ files to exist, even when
> -         they have been replaced by functions in /sys.
> -
> -         This option has no effect on /proc/acpi/ files
> -         and functions which do not yet exist in /sys.
> -
> -         Say N to delete /proc/acpi/ files that have moved to /sys/
> -
>  config ACPI_PROCFS_POWER
>         bool "Deprecated power /proc/acpi directories"
>         depends on PROC_FS
> --
> 1.8.1.2
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Best regards
Tianyu Lan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/