* Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH
[not found] <1559636093-26005-1-git-send-email-krzk@kernel.org>
@ 2019-06-04 8:55 ` Sudeep Holla
2019-06-06 2:10 ` Shawn Guo
` (3 subsequent siblings)
4 siblings, 0 replies; 5+ messages in thread
From: Sudeep Holla @ 2019-06-04 8:55 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: arm, Arnd Bergmann, Olof Johansson, Russell King, Joel Stanley,
Andrew Jeffery, Nicolas Ferre, Alexandre Belloni,
Ludovic Desroches, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Vladimir Zapolskiy, Sylvain Lemieux, Jason Cooper, Andrew Lunn,
Gregory Clement, Sebastian Hesselbarth <sebastian.he>
On Tue, Jun 04, 2019 at 10:14:53AM +0200, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
> CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
> made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
> [...] creates a high system load") and was kept only for ancient
> userland,
> 3. Certain userland specifically requests it to be disabled (systemd
> README: "Legacy hotplug slows down the system and confuses udev").
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> ---
>
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.
> ---
[...]
> diff --git a/arch/arm/configs/vexpress_defconfig b/arch/arm/configs/vexpress_defconfig
> index d170da388389..25753552277a 100644
> --- a/arch/arm/configs/vexpress_defconfig
> +++ b/arch/arm/configs/vexpress_defconfig
> @@ -45,7 +45,6 @@ CONFIG_IP_PNP_BOOTP=y
> # CONFIG_WIRELESS is not set
> CONFIG_NET_9P=y
> CONFIG_NET_9P_VIRTIO=y
> -CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
> CONFIG_DEVTMPFS=y
> CONFIG_DMA_CMA=y
> CONFIG_MTD=y
(for vexpress)
Acked-by: Sudeep Holla <sudeep.holla@arm.com>
--
Regards,
Sudeep
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH
[not found] <1559636093-26005-1-git-send-email-krzk@kernel.org>
2019-06-04 8:55 ` [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH Sudeep Holla
@ 2019-06-06 2:10 ` Shawn Guo
2019-06-06 9:16 ` Nicolas.Ferre
` (2 subsequent siblings)
4 siblings, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2019-06-06 2:10 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: arm, Arnd Bergmann, Olof Johansson, Russell King, Joel Stanley,
Andrew Jeffery, Nicolas Ferre, Alexandre Belloni,
Ludovic Desroches, Sascha Hauer, Pengutronix Kernel Team,
Fabio Estevam, NXP Linux Team, Vladimir Zapolskiy,
Sylvain Lemieux, Jason Cooper, Andrew Lunn, Gregory Clement,
Sebastian Hesselbarth, Aaro
On Tue, Jun 04, 2019 at 10:14:53AM +0200, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
> CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
> made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
> [...] creates a high system load") and was kept only for ancient
> userland,
> 3. Certain userland specifically requests it to be disabled (systemd
> README: "Legacy hotplug slows down the system and confuses udev").
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> ---
>
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.
> ---
...
> arch/arm/configs/imx_v4_v5_defconfig | 1 -
Acked-by: Shawn Guo <shawnguo@kernel.org>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH
[not found] <1559636093-26005-1-git-send-email-krzk@kernel.org>
2019-06-04 8:55 ` [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH Sudeep Holla
2019-06-06 2:10 ` Shawn Guo
@ 2019-06-06 9:16 ` Nicolas.Ferre
2019-06-19 14:09 ` Olof Johansson
[not found] ` <15AEA3356BFC1A54.4080@linux.kernel.org>
4 siblings, 0 replies; 5+ messages in thread
From: Nicolas.Ferre @ 2019-06-06 9:16 UTC (permalink / raw)
To: krzk, arm, arnd, olof, linux, joel, andrew, alexandre.belloni,
Ludovic.Desroches, shawnguo, s.hauer, kernel, festevam, linux-imx,
vz, slemieux.tyco, jason, andrew, gregory.clement,
sebastian.hesselbarth, aaro.koskinen, tony, dinguyen,
marc.w.gonzalez, mans, liviu.dudau, sudeep.holla,
lorenzo.pieralisi, linux-arm-kernel, linux-kernel, linux-aspeed,
linux-omap
Cc: geert+renesas
On 04/06/2019 at 10:14, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
> CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
> made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
> [...] creates a high system load") and was kept only for ancient
> userland,
> 3. Certain userland specifically requests it to be disabled (systemd
> README: "Legacy hotplug slows down the system and confuses udev").
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> ---
>
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.
> ---
> arch/arm/configs/acs5k_defconfig | 1 -
> arch/arm/configs/acs5k_tiny_defconfig | 1 -
> arch/arm/configs/am200epdkit_defconfig | 1 -
> arch/arm/configs/aspeed_g4_defconfig | 1 -
> arch/arm/configs/aspeed_g5_defconfig | 1 -
> arch/arm/configs/at91_dt_defconfig | 1 -
[..]
> arch/arm/configs/sama5_defconfig | 1 -
for at91 (at91_dt_defconfig & sama5_defconfig):
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
> arch/arm/configs/socfpga_defconfig | 1 -
> arch/arm/configs/spear13xx_defconfig | 1 -
> arch/arm/configs/spear3xx_defconfig | 1 -
> arch/arm/configs/spear6xx_defconfig | 1 -
> arch/arm/configs/spitz_defconfig | 1 -
> arch/arm/configs/tango4_defconfig | 1 -
> arch/arm/configs/tct_hammer_defconfig | 1 -
> arch/arm/configs/u300_defconfig | 1 -
> arch/arm/configs/u8500_defconfig | 1 -
> arch/arm/configs/vexpress_defconfig | 1 -
> arch/arm/configs/viper_defconfig | 1 -
> arch/arm/configs/xcep_defconfig | 1 -
> arch/arm/configs/zeus_defconfig | 1 -
> arch/arm/configs/zx_defconfig | 1 -
> 69 files changed, 69 deletions(-)
[..]
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH
[not found] <1559636093-26005-1-git-send-email-krzk@kernel.org>
` (2 preceding siblings ...)
2019-06-06 9:16 ` Nicolas.Ferre
@ 2019-06-19 14:09 ` Olof Johansson
[not found] ` <15AEA3356BFC1A54.4080@linux.kernel.org>
4 siblings, 0 replies; 5+ messages in thread
From: Olof Johansson @ 2019-06-19 14:09 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: arm, Arnd Bergmann, Russell King, Joel Stanley, Andrew Jeffery,
Nicolas Ferre, Alexandre Belloni, Ludovic Desroches, Shawn Guo,
Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
NXP Linux Team, Vladimir Zapolskiy, Sylvain Lemieux, Jason Cooper,
Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Aaro
On Tue, Jun 04, 2019 at 10:14:53AM +0200, Krzysztof Kozlowski wrote:
> Remove the CONFIG_UEVENT_HELPER_PATH because:
> 1. It is disabled since commit 1be01d4a5714 ("driver: base: Disable
> CONFIG_UEVENT_HELPER by default") as its dependency (UEVENT_HELPER) was
> made default to 'n',
> 2. It is not recommended (help message: "This should not be used today
> [...] creates a high system load") and was kept only for ancient
> userland,
> 3. Certain userland specifically requests it to be disabled (systemd
> README: "Legacy hotplug slows down the system and confuses udev").
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> ---
>
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.
So your other patch added mini2440 in a follow-up patch, but this one doesn't?!
Applied with that fixup.
-Olof
^ permalink raw reply [flat|nested] 5+ messages in thread[parent not found: <15AEA3356BFC1A54.4080@linux.kernel.org>]
* Re: [PATCH v2] ARM: configs: Remove useless UEVENT_HELPER_PATH
[not found] ` <15AEA3356BFC1A54.4080@linux.kernel.org>
@ 2019-07-05 22:29 ` Olof Johansson
0 siblings, 0 replies; 5+ messages in thread
From: Olof Johansson @ 2019-07-05 22:29 UTC (permalink / raw)
To: patchwork-soc+owner
Cc: Krzysztof Kozlowski, ARM-SoC Maintainers, Arnd Bergmann,
Russell King, Joel Stanley, Andrew Jeffery, Nicolas Ferre,
Alexandre Belloni, Ludovic Desroches, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Vladimir Zapolskiy, Sylvain Lemieux, Jason Cooper, Andrew Lunn,
Gregory Clement
On Fri, Jul 5, 2019 at 3:26 PM Olof Johansson via Linux.Kernel.Org
<olof=lixom.net@linux.kernel.org> wrote:
This didn't work as I anticipated. Please ignore, apologies for the spam.
-Olof
^ permalink raw reply [flat|nested] 5+ messages in thread