Re: [PATCH v3] ARM: configs: Remove useless UEVENT_HELPER_PATH
On Thu, Jun 13, 2019 at 04:53:53PM +0200, Krzysztof Kozlowski wrote:
> On Tue, 4 Jun 2019 at 20:52, 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
> > Acked-by: Geert Uytterhoeven
> > Acked-by: Alexandre Belloni
> > For vexpress:
> > Acked-by: Sudeep Holla
> >
> > ---
> >
> > Changes since v2:
> > 1. Remove unrelated files.
> > 2. Add Geert's ack.
> >
> > Changes sice v3:
> > 1. Change also mini2440_defconfig.
> > 2. Add more acks.
>
> Hi Arnd and Olof,
>
> Do you want to apply it directly or maybe I can send it along with
> other my defconfig pull request?
Oh, just saw v3, since it didn't thread with v2.
Applied already with the equivalent fixups, so we're good.
-Olof
Re: [PATCH v3] ARM: configs: Remove useless UEVENT_HELPER_PATH
On Tue, 4 Jun 2019 at 20:52, 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
> Acked-by: Geert Uytterhoeven
> Acked-by: Alexandre Belloni
> For vexpress:
> Acked-by: Sudeep Holla
>
> ---
>
> Changes since v2:
> 1. Remove unrelated files.
> 2. Add Geert's ack.
>
> Changes sice v3:
> 1. Change also mini2440_defconfig.
> 2. Add more acks.
Hi Arnd and Olof,
Do you want to apply it directly or maybe I can send it along with
other my defconfig pull request?
Best regards,
Krzysztof
[PATCH v3] ARM: configs: Remove useless UEVENT_HELPER_PATH
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
Acked-by: Geert Uytterhoeven
Acked-by: Alexandre Belloni
For vexpress:
Acked-by: Sudeep Holla
---
Changes since v2:
1. Remove unrelated files.
2. Add Geert's ack.
Changes sice v3:
1. Change also mini2440_defconfig.
2. Add more acks.
---
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/axm55xx_defconfig| 1 -
arch/arm/configs/cm_x2xx_defconfig| 1 -
arch/arm/configs/cm_x300_defconfig| 1 -
arch/arm/configs/cns3420vb_defconfig | 1 -
arch/arm/configs/colibri_pxa270_defconfig | 1 -
arch/arm/configs/colibri_pxa300_defconfig | 1 -
arch/arm/configs/corgi_defconfig | 1 -
arch/arm/configs/dove_defconfig | 1 -
arch/arm/configs/em_x270_defconfig| 1 -
arch/arm/configs/ep93xx_defconfig | 1 -
arch/arm/configs/eseries_pxa_defconfig| 1 -
arch/arm/configs/ezx_defconfig| 1 -
arch/arm/configs/gemini_defconfig | 1 -
arch/arm/configs/h3600_defconfig | 1 -
arch/arm/configs/h5000_defconfig | 1 -
arch/arm/configs/imote2_defconfig | 1 -
arch/arm/configs/imx_v4_v5_defconfig | 1 -
arch/arm/configs/iop13xx_defconfig| 1 -
arch/arm/configs/iop32x_defconfig | 1 -
arch/arm/configs/iop33x_defconfig | 1 -
arch/arm/configs/ixp4xx_defconfig | 1 -
arch/arm/configs/jornada720_defconfig | 1 -
arch/arm/configs/keystone_defconfig | 1 -
arch/arm/configs/ks8695_defconfig | 1 -
arch/arm/configs/lpc32xx_defconfig| 1 -
arch/arm/configs/magician_defconfig | 1 -
arch/arm/configs/mini2440_defconfig | 1 -
arch/arm/configs/moxart_defconfig | 1 -
arch/arm/configs/multi_v5_defconfig | 1 -
arch/arm/configs/mv78xx0_defconfig| 1 -
arch/arm/configs/mvebu_v5_defconfig | 1 -
arch/arm/configs/mvebu_v7_defconfig | 1 -
arch/arm/configs/nhk8815_defconfig| 1 -
arch/arm/configs/nuc910_defconfig | 1 -
arch/arm/configs/nuc950_defconfig | 1 -
arch/arm/configs/nuc960_defconfig | 1 -
arch/arm/configs/omap1_defconfig | 1 -
arch/arm/configs/orion5x_defconfig| 1 -
arch/arm/configs/palmz72_defconfig| 1 -
arch/arm/configs/pcm027_defconfig | 1 -
arch/arm/configs/prima2_defconfig | 1 -
arch/arm/configs/pxa168_defconfig | 1 -
arch/arm/configs/pxa3xx_defconfig | 1 -
arch/arm/configs/pxa910_defconfig | 1 -
arch/arm/configs/pxa_defconfig| 1 -
arch/arm/configs/realview_defconfig | 1 -
arch/arm/configs/s3c2410_defconfig| 1 -
arch/arm/configs/s3c6400_defconfig| 1 -
arch/arm/configs/s5pv210_defconfig| 1 -
arch/arm/configs/sama5_defconfig | 1 -
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 -
70 files changed, 70 deletions(-)
diff --git a/arch/arm/configs/acs5k_defconfig b/arch/arm/configs/acs5k_defconfig
index d04ee19e5b75..bcb8bda09158 100644
--- a/arch/arm/configs/acs5k_defconfig
+++ b/arch/arm/configs/acs5k_defconfig
@@ -30,7 +30,6 @@ CONFIG_INET=y
CONFIG_IP_PNP=y
CONFIG_IP_PNP_DHCP=y
# CONFIG_IPV6 is not set
-CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
CONFIG_MTD=y
CONFIG_MTD_BLOCK=y
CONFIG_MTD_CFI=y
diff --git a/arch/arm/configs/acs5k_tiny_defconfig
b/arch/arm/configs/acs5k_tiny_defconfig
index 25c593df41d1..e802cdebfd0b 100644
--- a/arch/arm/configs/acs5k_tiny_defconfig
+++ b/arch/arm/configs/acs5k_tiny_defconfig
@@ -25,7 +25,6 @@ CONFIG_INET=y
# CONFIG_INET_XFRM_MODE_TUNNEL is not set
# CONFIG_INET_XFRM_MODE_BEET is not set
# CONFIG_IPV6 is not
