[OpenWrt-Devel] [PATCH 4/4] build: add urandom-seed and urngd to default packages set

Yousong Zhou yszhou4tech at gmail.com
Tue May 28 22:25:52 EDT 2019


On Tue, 28 May 2019 at 05:30, Petr Štetiar <ynezz at true.cz> wrote:
>
> urandom-seed content was split from base-files into separate package so
> in order to preserve the current functionality and to provide some
> fallback mechanism in case jent-rng initialization fails in urngd we
> need to add it back.
>
> urngd is OpenWrt's micro non-physical true random number generator based
> on timing jitter.
>

If I understand the patch correctly it seems the urandom-seed will run
along side with urngd, not a fallback.  Is urandom-seed a must, or
only serve as a precaution just in case?

If urngd can solve the problem what urandom-seed is for, I would
suggest we make urandom-seed an opt-in option, not included by
default.

Regards,
                yousong


> Signed-off-by: Petr Štetiar <ynezz at true.cz>
> ---
>  include/target.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/target.mk b/include/target.mk
> index 4f3bd43e6cb6..a813ba2d2d87 100644
> --- a/include/target.mk
> +++ b/include/target.mk
> @@ -13,7 +13,7 @@ __target_inc=1
>  DEVICE_TYPE?=router
>
>  # Default packages - the really basic set
> -DEFAULT_PACKAGES:=base-files libc libgcc busybox dropbear mtd uci opkg netifd fstools uclient-fetch logd
> +DEFAULT_PACKAGES:=base-files libc libgcc busybox dropbear mtd uci opkg netifd fstools uclient-fetch logd urandom-seed urngd
>  # For nas targets
>  DEFAULT_PACKAGES.nas:=block-mount fdisk lsblk mdadm
>  # For router targets
> --
> 1.9.1
>
>
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel

_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


More information about the openwrt-devel mailing list