Re: [devel] Q: postinst hook for firmware-*

["Dmitry V. Levin" <ldv@altlinux.org>]

[-- Attachment #1: Type: text/plain, Size: 3152 bytes --]

On Wed, Sep 06, 2017 at 02:09:09PM +0200, Konstantin Lepikhov wrote:
> Hi Dmitry!
> 
> On 09/05/2017, at 12:35:09 AM you wrote:
> 
> <skip>
> > > > Пакеты kernel-image не дёргают make-initrd самостоятельно, полагаясь на
> > > > /usr/lib/rpm/boot_kernel.filetrigger; если эти файлы склонны расползаться
> > > > по пакетам, может быть, лучше подойдёт файлтриггер?
> > > Так если мне нужно перегенерить initrd, то я не против дернуть тот же
> > > boot_kernel.filetrigger. Где это задается? Внутри rpm?
> > 
> > Прямо в boot_kernel.filetrigger; это пакет bootloader-utils.
> Подойдет ли такое изменение?
> 
> -- 
> WBR et al.

> >From c974d0d02c1275c600929b3590e6ef644a4e8159 Mon Sep 17 00:00:00 2001
> From: "Konstantin A. Lepikhov" <lakostis@altlinux.ru>
> Date: Wed, 6 Sep 2017 13:49:26 +0200
> Subject: [PATCH] kernel.filetrigger: ucode support
> 
> We need to rebuild initramfs on CPU ucode changes otherwise everything
> will continue use outdated version till next kernel update.
> ---
>  kernel.filetrigger | 27 ++++++++++++++++++++++++++-
>  1 file changed, 26 insertions(+), 1 deletion(-)
> 
> diff --git a/kernel.filetrigger b/kernel.filetrigger
> index 9cdf01e..38ea249 100755
> --- a/kernel.filetrigger
> +++ b/kernel.filetrigger
> @@ -9,6 +9,7 @@
>  BOOTDIR=/boot
>  VMLINUZ_PREFIX=/boot/vmlinuz
>  MODULES_PREFIX=/lib/modules
> +UCODE_PREFIX=/lib/firmware
>  INITRD_AUTOUPDATE=
>  
>  . /etc/sysconfig/installkernel
> @@ -99,6 +100,27 @@ $VERSION"
>  	last_added="$VERSION"
>  }
>  
> +ucode_handled=
> +handle_ucode()
> +{
> +	local f VENDOR VERSION
> +	f="$1"; shift
> +	# filename format: $UCODE_PREFIX/<vendor>-ucode/*
> +	VENDOR=${f#$UCODE_PREFIX/}
> +	VENDOR=${VENDOR%%-ucode/*}
> +	ucode_handled="$ucode_handled $VENDOR"

содержимое ucode_handled дальше только проверяется на непустоту,
можно записывать туда что-то более простое, например, ucode_handled=1

> +	set +f

set +f дальше не используется и, видимо, не нужен.

> +	f="$(readlink -e /boot/vmlinuz)"
> +	if [ -n "$f" -a -n "$kernel_versions_handled" ]; then
> +		VERSION=${f#$VMLINUZ_PREFIX-}
> +		case "$kernel_version_handled" in
> +			"* $VERSION*")
> +			ucode_handled=
> +			;;
> +		esac
> +	fi

Это лучше делать всего один раз после обработки цикла по файлам.
Хотя почему это исключение сделано именно для /boot/vmlinuz, неочевидно.

> +}
> +
>  while read f; do
>  	case "$f" in
>  		$VMLINUZ_PREFIX-[0-9].*-*-*)
> @@ -107,10 +129,13 @@ while read f; do
>  		$MODULES_PREFIX/*-*-*/*/*.ko*)
>  			handle_module "$f"
>  			;;
> +		$UCODE_PREFIX/*-ucode/*)
> +			handle_ucode "$f"

Получается, что здесь достаточно написать ucode_detected=1,

> +			;;
>  	esac
>  done

А сюда поместить сброс ucode_detected в случае, если он уже был обработан.

> -if [ -n "$kernel_versions_handled" -a "$INITRD_AUTOUPDATE" = all ]; then
> +if [ -n "$kernel_versions_handled" -o -n "$ucode_handled" ] && [ "$INITRD_AUTOUPDATE" = all ]; then
>  	set +f
>  	ls -c $VMLINUZ_PREFIX-[0-9].*-*-* 2> /dev/null | while read f; do
>  		[ "$f" != "$(readlink -e /boot/vmlinuz)" ] ||


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]