x86: efi_loader: Ensure the SMBIOS tables are sent via EFI

The EFI-loader code has not been fully converted to use bloblist, so
relies on the SMBIOS-table address being set in global_data.

Set this up in write_tables() so that the SMBIOS tables are actually
available.

Enable the command for x86 QEMU so that the SMBIOS tests actually run.

Signed-off-by: Simon Glass <sjg@chromium.org>
Fixes: 83ce35d6eb ("emulation: Use bloblist to hold tables")
Reported-by: Niklas Sombert <niklas.sombert@uni-duesseldorf.de>

Kconfig

@@ -551,7 +551,7 @@ config SYS_LOAD_ADDR
 	default 0x12000000 if ARCH_MX6 && !(MX6SL || MX6SLL  || MX6SX || MX6UL || MX6ULL)
 	default 0x80800000 if ARCH_MX7
 	default 0x90000000 if FSL_LSCH2 || FSL_LSCH3
-	default 0 if ARCH_EFI
+	default 0x02000000 if ARCH_EFI
 	default 0x0 if ARCH_SC5XX
 	help
 	  Address in memory to use as the default safe load address.

arch/x86/lib/tables.c

@@ -126,6 +126,8 @@ int write_tables(void)
 			use_high = true;
 			if (!gd->arch.table_start_high)
 				gd->arch.table_start_high = rom_addr;
+			if (table->tag == BLOBLISTT_SMBIOS_TABLES)
+				gd_set_smbios_start(rom_addr);
 		}
 		rom_table_end = table->write(rom_addr);
 		if (!rom_table_end) {

board/armltd/vexpress64/Kconfig

@@ -28,7 +28,7 @@ choice
 config TARGET_VEXPRESS64_BASE_FVP
 	bool "Support Versatile Express ARMv8a FVP BASE model"
 	select VEXPRESS64_BASE_MODEL
-	imply OF_HAS_PRIOR_STAGE if !BLOBLIST
+	imply OF_HAS_PRIOR_STAGE
 
 config TARGET_VEXPRESS64_BASER_FVP
 	bool "Support Versatile Express ARMv8r64 FVP BASE model"

board/armltd/vexpress64/Makefile

@@ -3,8 +3,5 @@
 # (C) Copyright 2000-2004
 # Wolfgang Denk, DENX Software Engineering, wd@denx.de.
 
-obj-y	:= vexpress64.o
-
-obj-$(CONFIG_OF_HAS_PRIOR_STAGE)	+= lowlevel_init.o
-
+obj-y	:= vexpress64.o lowlevel_init.o
 obj-$(CONFIG_TARGET_VEXPRESS64_JUNO)	+= pcie.o

board/armltd/vexpress64/vexpress64.c

@@ -100,9 +100,7 @@ int dram_init_banksize(void)
  * Push the variable into the .data section so that it
  * does not get cleared later.
  */
-#ifdef CONFIG_OF_HAS_PRIOR_STAGE
 unsigned long __section(".data") prior_stage_fdt_address[2];
-#endif
 
 #ifdef CONFIG_OF_BOARD
 
@@ -153,7 +151,6 @@ static phys_addr_t find_dtb_in_nor_flash(const char *partname)
 }
 #endif
 
-#ifdef CONFIG_OF_HAS_PRIOR_STAGE
 /*
  * Filter for a valid DTB, as TF-A happens to provide a pointer to some
  * data structure using the DTB format, which we cannot use.
@@ -204,7 +201,6 @@ int board_fdt_blob_setup(void **fdtp)
 	return -ENXIO;
 }
 #endif
-#endif
 
 /* Actual reset is done via PSCI. */
 void reset_cpu(void)

board/efi/Kconfig

@@ -17,7 +17,7 @@ config TARGET_EFI_X86_APP32
 	help
 	  This target is used for running U-Boot on top of EFI. In
 	  this case EFI does the early initialisation, and U-Boot
-	  starts once the RAM, video and CPU are fully running.
+	  takes over once the RAM, video and CPU are fully running.
 	  U-Boot is loaded as an application from EFI.
 
 config TARGET_EFI_X86_APP64
@@ -27,7 +27,7 @@ config TARGET_EFI_X86_APP64
 	help
 	  This target is used for running U-Boot on top of EFI in 64-bit mode.
 	  In this case EFI does the early initialisation, and U-Boot
-	  starts once the RAM, video and CPU are fully running.
+	  takes over once the RAM, video and CPU are fully running.
 	  U-Boot is loaded as an application from EFI.
 
 config TARGET_EFI_X86_PAYLOAD
@@ -59,7 +59,7 @@ config TARGET_EFI_ARM_APP64
 	help
 	  This target is used for running U-Boot on top of EFI in 64-bit mode.
 	  In this case EFI does the early initialisation, and U-Boot
-	  starts once the RAM, video and CPU are fully running.
+	  takes over once the RAM, video and CPU are fully running.
 	  U-Boot is loaded as an application from EFI.
 
 endchoice

board/efi/efi-arm_app/MAINTAINERS

@@ -1,3 +1,10 @@
+EFI-ARM_APP32 BOARD
+M:	Simon Glass <sjg@chromium.org>
+S:	Maintained
+F:	board/efi/Kconfig
+F:	board/efi/efi-arm_app/
+F:	configs/efi-arm_app32_defconfig
+
 EFI-ARM_APP64 BOARD
 M:	Simon Glass <sjg@chromium.org>
 S:	Maintained

board/emulation/qemu-x86/Kconfig

@@ -23,5 +23,6 @@ config BOARD_SPECIFIC_OPTIONS # dummy
 	imply VIRTIO_PCI
 	imply VIRTIO_NET
 	imply VIRTIO_BLK
+	imply CMD_SMBIOS
 
 endif

common/Kconfig

@@ -1055,8 +1055,8 @@ choice
 config BLOBLIST_FIXED
 	bool "Place bloblist at a fixed address in memory"
 	help
-	  Select this to use a fixed memory address for the bloblist. If the
-	  bloblist exists at this address from a previous phase, it is used as is.
+	  Select this to used a fixed memory address for the bloblist. If the
+	  bloblist exists at this address from a previous phase, it used as is.
 	  If not it is created at this address in U-Boot.
 
 config BLOBLIST_ALLOC
@@ -1066,12 +1066,6 @@ config BLOBLIST_ALLOC
 	  specify a fixed address on systems where this is unknown or can
 	  change at runtime.
 
-config BLOBLIST_PASSAGE
-	bool "Use bloblist in-place"
-	help
-	  Use a bloblist in the incoming standard passage. The size is detected
-	  automatically so CONFIG_BLOBLIST_SIZE can be 0.
-
 endchoice
 
 config BLOBLIST_ADDR
@@ -1086,17 +1080,17 @@ config BLOBLIST_ADDR
 
 config BLOBLIST_SIZE
 	hex "Size of bloblist"
-	default 0x0 if BLOBLIST_PASSAGE
 	default 0x400
 	help
 	  Sets the size of the bloblist in bytes. This must include all
 	  overhead (alignment, bloblist header, record header). The bloblist
 	  is set up in the first part of U-Boot to run (TPL, SPL or U-Boot
-	  proper), and this same bloblist is used for subsequent phases.
+	  proper), and this sane bloblist is used for subsequent phases.
 
 config BLOBLIST_SIZE_RELOC
 	hex "Size of bloblist after relocation"
 	default BLOBLIST_SIZE if BLOBLIST_FIXED || BLOBLIST_ALLOC
+	default 0x0 if BLOBLIST_PASSAGE
 	default 0x20000 if (ARM && EFI_LOADER && GENERATE_ACPI_TABLE)
 	help
 	  Sets the size of the bloblist in bytes after relocation. Since U-Boot

common/bloblist.c

@@ -1,4 +1,4 @@
-// SPDX-License-Identifier: GPL-2.0+
+// SPDX-License-Identifier: GPL-2.0+ BSD-3-Clause
 /*
  * Copyright 2018 Google, Inc
  * Written by Simon Glass <sjg@chromium.org>
@@ -489,9 +489,6 @@ int bloblist_reloc(void *to, uint to_size)
 {
 	struct bloblist_hdr *hdr;
 
-	if (!to_size)
-		return 0;
-
 	if (to_size < gd->bloblist->total_size)
 		return -ENOSPC;
 
@@ -522,6 +519,13 @@ int bloblist_init(void)
 	 * at a fixed address.
 	 */
 	bool from_addr = fixed && !xpl_is_first_phase();
+	/*
+	 * If U-Boot is in the first phase that an arch custom routine should
+	 * install the bloblist passed from previous loader to this fixed
+	 * address.
+	 */
+	bool from_boot_arg = fixed && xpl_is_first_phase();
+
 	if (xpl_prev_phase() == PHASE_TPL && !IS_ENABLED(CONFIG_TPL_BLOBLIST))
 		from_addr = false;
 	if (fixed)
@@ -529,13 +533,7 @@ int bloblist_init(void)
 				      CONFIG_BLOBLIST_ADDR);
 	size = CONFIG_BLOBLIST_SIZE;
 
-
-	/*
-	 * If the current boot stage is the first phase of U-Boot, then an
-	 * architecture-specific routine should be used to handle the bloblist
-	 * passed from the previous boot loader
-	 */
-	if (xpl_is_first_phase() && !IS_ENABLED(CONFIG_BLOBLIST_ALLOC))
+	if (from_boot_arg)
 		ret = xferlist_from_boot_arg(addr, size);
 	else if (from_addr)
 		ret = bloblist_check(addr, size);

configs/vexpress_fvp_bloblist_defconfig

@@ -1,5 +0,0 @@
-#include <configs/vexpress_fvp_defconfig>
-
-CONFIG_BLOBLIST=y
-CONFIG_BLOBLIST_PASSAGE=y
-CONFIG_BLOBLIST_SIZE_RELOC=0x10000

doc/board/armltd/vexpress64.rst

@@ -43,22 +43,6 @@ Juno is an Arm development board with the following features:
 
 More details can be found in the board documentation [3]_.
 
-Bloblist Support
-----------------
-
-The ``vexpress_fvp_bloblist_defconfig`` configures U-Boot to be compiled for
-Vexpress64 with Bloblist as the primary method for information handoff between
-boot stages. U-Boot offers three methods to set up a bloblist: using a
-predefined bloblist at a specified address, dynamically allocating memory for a
-bloblist, or utilizing a standard passage-provided bloblist with automatic size
-detection.
-
-By default, ``vexpress_fvp_bloblist_defconfig`` uses the standard passage method
-(CONFIG_BLOBLIST_PASSAGE) because TF-A provides a Transfer List in non-secure
-memory that U-Boot can utilise. This Bloblist, which is referred to as a Transfer List in
-TF-A, contains all necessary data for the handoff process, including DT and ACPI
-tables.
-
 References
 ----------
 

fs/ext4/ext4_write.c

@@ -25,7 +25,6 @@
 #include <malloc.h>
 #include <memalign.h>
 #include <part.h>
-#include <linux/overflow.h>
 #include <linux/stat.h>
 #include <div64.h>
 #include "ext4_common.h"
@@ -109,15 +108,8 @@ int ext4fs_get_bgdtable(void)
 {
 	int status;
 	struct ext_filesystem *fs = get_fs();
-	size_t alloc_size;
-	int gdsize_total;
-
-	if (check_mul_overflow(fs->no_blkgrp, fs->gdsize, &alloc_size))
-		return -1;
-	gdsize_total = ROUND(alloc_size, fs->blksz);
+	int gdsize_total = ROUND(fs->no_blkgrp * fs->gdsize, fs->blksz);
 	fs->no_blk_pergdt = gdsize_total / fs->blksz;
-	if (!fs->no_blk_pergdt)
-		return -1;
 
 	/* allocate memory for gdtable */
 	fs->gdtable = zalloc(gdsize_total);
@@ -125,7 +117,7 @@ int ext4fs_get_bgdtable(void)
 		return -ENOMEM;
 	/* read the group descriptor table */
 	status = ext4fs_devread((lbaint_t)fs->gdtable_blkno * fs->sect_perblk,
-				0, gdsize_total, fs->gdtable);
+				0, fs->blksz * fs->no_blk_pergdt, fs->gdtable);
 	if (status == 0)
 		goto fail;
 
@@ -607,17 +599,10 @@ int ext4fs_init(void)
 	int i;
 	uint32_t real_free_blocks = 0;
 	struct ext_filesystem *fs = get_fs();
-	size_t alloc_size;
-
-	/* check for a reasonable block size, no more than 64K */
-	if (LOG2_BLOCK_SIZE(ext4fs_root) > 16)
-		goto fail;
 
 	/* populate fs */
 	fs->blksz = EXT2_BLOCK_SIZE(ext4fs_root);
 	fs->sect_perblk = fs->blksz >> fs->dev_desc->log2blksz;
-	if (!fs->sect_perblk)
-		goto fail;
 
 	/* get the superblock */
 	fs->sb = zalloc(SUPERBLOCK_SIZE);
@@ -644,9 +629,7 @@ int ext4fs_init(void)
 	}
 
 	/* load all the available bitmap block of the partition */
-	if (check_mul_overflow(fs->no_blkgrp, sizeof(char *), &alloc_size))
-		goto fail;
-	fs->blk_bmaps = zalloc(alloc_size);
+	fs->blk_bmaps = zalloc(fs->no_blkgrp * sizeof(char *));
 	if (!fs->blk_bmaps)
 		goto fail;
 	for (i = 0; i < fs->no_blkgrp; i++) {
@@ -666,7 +649,7 @@ int ext4fs_init(void)
 	}
 
 	/* load all the available inode bitmap of the partition */
-	fs->inode_bmaps = zalloc(alloc_size);
+	fs->inode_bmaps = zalloc(fs->no_blkgrp * sizeof(unsigned char *));
 	if (!fs->inode_bmaps)
 		goto fail;
 	for (i = 0; i < fs->no_blkgrp; i++) {

include/bloblist.h

@@ -1,4 +1,4 @@
-/* SPDX-License-Identifier: GPL-2.0+ */
+/* SPDX-License-Identifier: GPL-2.0+ BSD-3-Clause */
 /*
  * This provides a standard way of passing information between boot phases
  * (TPL -> SPL -> U-Boot proper.)
@@ -475,7 +475,7 @@ int bloblist_init(void);
 /**
  * bloblist_maybe_init() - Init the bloblist system if not already done
  *
- * Calls bloblist_init() if the GD_FLG_BLOBLIST_READY flag is not set
+ * Calls bloblist_init() if the GD_FLG_BLOBLIST_READY flag is not et
  *
  * Return: 0 if OK, -ve on error
  */

include/configs/vexpress_aemv8.h

@@ -169,14 +169,12 @@
 		"  if load hostfs - ${kernel_addr_r} ${kernel_name}; then"	\
 		"    setenv fdt_high 0xffffffffffffffff;"		\
 		"    setenv initrd_high 0xffffffffffffffff;"		\
-		"    if test -n load hostfs - ${fdt_addr_r} ${fdtfile}; then"			\
-		"        fdt move $fdtcontroladdr $fdt_addr_r;"			\
-		"    fi;"			\
+		"    load hostfs - ${fdt_addr_r} ${fdtfile};"			\
 		"    load hostfs - ${ramdisk_addr_r} ${ramdisk_name};" \
 		"    fdt addr ${fdt_addr_r};"				\
 		"    fdt resize;"					\
 		"    fdt chosen ${ramdisk_addr_r} ${filesize};"	\
-		"    booti $kernel_addr_r - ${fdt_addr_r};"		\
+		"    booti $kernel_addr_r - $fdt_addr_r;"		\
 		"  fi;"							\
 		"fi\0"
 #define BOOTENV_DEV_NAME_SMH(devtypeu, devtypel, instance) "smh "

include/ext_common.h

@@ -67,7 +67,8 @@ struct cmd_tbl;
 #define EXT2_BLOCK_SIZE(data)	   (1 << LOG2_BLOCK_SIZE(data))
 
 /* Log2 size of ext2 block in bytes.  */
-#define LOG2_BLOCK_SIZE(data)	(le32_to_cpu((data)->sblock.log2_block_size) \
+#define LOG2_BLOCK_SIZE(data)	   (le32_to_cpu		   \
+				    (data->sblock.log2_block_size) \
 				    + EXT2_MIN_BLOCK_LOG_SIZE)
 
 #define EXT2_FT_DIR	2

include/limits.h

@@ -3,8 +3,6 @@
 #ifndef _LIMITS_H
 #define _LIMITS_H
 
-#include <linux/limits.h>
-
 #define INT_MAX     0x7fffffff
 #define UINT_MAX    0xffffffffU
 #define CHAR_BIT    8

include/linux/kernel.h

@@ -16,6 +16,18 @@
 #define LLONG_MIN	(-LLONG_MAX - 1)
 #define ULLONG_MAX	(~0ULL)
 
+#define U8_MAX		((u8)~0U)
+#define S8_MAX		((s8)(U8_MAX>>1))
+#define S8_MIN		((s8)(-S8_MAX - 1))
+#define U16_MAX		((u16)~0U)
+#define S16_MAX		((s16)(U16_MAX>>1))
+#define S16_MIN		((s16)(-S16_MAX - 1))
+#define U32_MAX		((u32)~0U)
+#define S32_MAX		((s32)(U32_MAX>>1))
+#define S32_MIN		((s32)(-S32_MAX - 1))
+#define U64_MAX		((u64)~0ULL)
+#define S64_MAX		((s64)(U64_MAX>>1))
+#define S64_MIN		((s64)(-S64_MAX - 1))
 
 #define INT32_MAX	S32_MAX
 

include/linux/limits.h

@@ -1,21 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0 */
-#ifndef _LINUX_LIMITS_H
-#define _LINUX_LIMITS_H
-
-#include <linux/types.h>
-
-#define U8_MAX		((u8)~0U)
-#define S8_MAX		((s8)(U8_MAX >> 1))
-#define S8_MIN		((s8)(-S8_MAX - 1))
-#define U16_MAX		((u16)~0U)
-#define S16_MAX		((s16)(U16_MAX >> 1))
-#define S16_MIN		((s16)(-S16_MAX - 1))
-#define U32_MAX		((u32)~0U)
-#define U32_MIN		((u32)0)
-#define S32_MAX		((s32)(U32_MAX >> 1))
-#define S32_MIN		((s32)(-S32_MAX - 1))
-#define U64_MAX		((u64)~0ULL)
-#define S64_MAX		((s64)(U64_MAX >> 1))
-#define S64_MIN		((s64)(-S64_MAX - 1))
-
-#endif /* _LINUX_LIMITS_H */

include/linux/overflow.h

@@ -1,443 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0 OR MIT */
-#ifndef __LINUX_OVERFLOW_H
-#define __LINUX_OVERFLOW_H
-
-#include <linux/compiler.h>
-#include <linux/limits.h>
-#include <linux/const.h>
-
-/*
- * We need to compute the minimum and maximum values representable in a given
- * type. These macros may also be useful elsewhere. It would seem more obvious
- * to do something like:
- *
- * #define type_min(T) (T)(is_signed_type(T) ? (T)1 << (8*sizeof(T)-1) : 0)
- * #define type_max(T) (T)(is_signed_type(T) ? ((T)1 << (8*sizeof(T)-1)) - 1 : ~(T)0)
- *
- * Unfortunately, the middle expressions, strictly speaking, have
- * undefined behaviour, and at least some versions of gcc warn about
- * the type_max expression (but not if -fsanitize=undefined is in
- * effect; in that case, the warning is deferred to runtime...).
- *
- * The slightly excessive casting in type_min is to make sure the
- * macros also produce sensible values for the exotic type _Bool. [The
- * overflow checkers only almost work for _Bool, but that's
- * a-feature-not-a-bug, since people shouldn't be doing arithmetic on
- * _Bools. Besides, the gcc builtins don't allow _Bool* as third
- * argument.]
- *
- * Idea stolen from
- * https://mail-index.netbsd.org/tech-misc/2007/02/05/0000.html -
- * credit to Christian Biere.
- */
-#define __type_half_max(type) ((type)1 << (8*sizeof(type) - 1 - is_signed_type(type)))
-#define __type_max(T) ((T)((__type_half_max(T) - 1) + __type_half_max(T)))
-#define type_max(t)	__type_max(typeof(t))
-#define __type_min(T) ((T)((T)-type_max(T)-(T)1))
-#define type_min(t)	__type_min(typeof(t))
-
-/*
- * Avoids triggering -Wtype-limits compilation warning,
- * while using unsigned data types to check a < 0.
- */
-#define is_non_negative(a) ((a) > 0 || (a) == 0)
-#define is_negative(a) (!(is_non_negative(a)))
-
-/*
- * Allows for effectively applying __must_check to a macro so we can have
- * both the type-agnostic benefits of the macros while also being able to
- * enforce that the return value is, in fact, checked.
- */
-static inline bool __must_check __must_check_overflow(bool overflow)
-{
-	return unlikely(overflow);
-}
-
-/**
- * check_add_overflow() - Calculate addition with overflow checking
- * @a: first addend
- * @b: second addend
- * @d: pointer to store sum
- *
- * Returns true on wrap-around, false otherwise.
- *
- * *@d holds the results of the attempted addition, regardless of whether
- * wrap-around occurred.
- */
-#define check_add_overflow(a, b, d)	\
-	__must_check_overflow(__builtin_add_overflow(a, b, d))
-
-/**
- * wrapping_add() - Intentionally perform a wrapping addition
- * @type: type for result of calculation
- * @a: first addend
- * @b: second addend
- *
- * Return the potentially wrapped-around addition without
- * tripping any wrap-around sanitizers that may be enabled.
- */
-#define wrapping_add(type, a, b)				\
-	({							\
-		type __val;					\
-		__builtin_add_overflow(a, b, &__val);		\
-		__val;						\
-	})
-
-/**
- * wrapping_assign_add() - Intentionally perform a wrapping increment assignment
- * @var: variable to be incremented
- * @offset: amount to add
- *
- * Increments @var by @offset with wrap-around. Returns the resulting
- * value of @var. Will not trip any wrap-around sanitizers.
- *
- * Returns the new value of @var.
- */
-#define wrapping_assign_add(var, offset)				\
-	({								\
-		typeof(var) *__ptr = &(var);				\
-		*__ptr = wrapping_add(typeof(var), *__ptr, offset);	\
-	})
-
-/**
- * check_sub_overflow() - Calculate subtraction with overflow checking
- * @a: minuend; value to subtract from
- * @b: subtrahend; value to subtract from @a
- * @d: pointer to store difference
- *
- * Returns true on wrap-around, false otherwise.
- *
- * *@d holds the results of the attempted subtraction, regardless of whether
- * wrap-around occurred.
- */
-#define check_sub_overflow(a, b, d)	\
-	__must_check_overflow(__builtin_sub_overflow(a, b, d))
-
-/**
- * wrapping_sub() - Intentionally perform a wrapping subtraction
- * @type: type for result of calculation
- * @a: minuend; value to subtract from
- * @b: subtrahend; value to subtract from @a
- *
- * Return the potentially wrapped-around subtraction without
- * tripping any wrap-around sanitizers that may be enabled.
- */
-#define wrapping_sub(type, a, b)				\
-	({							\
-		type __val;					\
-		__builtin_sub_overflow(a, b, &__val);		\
-		__val;						\
-	})
-
-/**
- * wrapping_assign_sub() - Intentionally perform a wrapping decrement assign
- * @var: variable to be decremented
- * @offset: amount to subtract
- *
- * Decrements @var by @offset with wrap-around. Returns the resulting
- * value of @var. Will not trip any wrap-around sanitizers.
- *
- * Returns the new value of @var.
- */
-#define wrapping_assign_sub(var, offset)				\
-	({								\
-		typeof(var) *__ptr = &(var);				\
-		*__ptr = wrapping_sub(typeof(var), *__ptr, offset);	\
-	})
-
-/**
- * check_mul_overflow() - Calculate multiplication with overflow checking
- * @a: first factor
- * @b: second factor
- * @d: pointer to store product
- *
- * Returns true on wrap-around, false otherwise.
- *
- * *@d holds the results of the attempted multiplication, regardless of whether
- * wrap-around occurred.
- */
-#define check_mul_overflow(a, b, d)	\
-	__must_check_overflow(__builtin_mul_overflow(a, b, d))
-
-/**
- * wrapping_mul() - Intentionally perform a wrapping multiplication
- * @type: type for result of calculation
- * @a: first factor
- * @b: second factor
- *
- * Return the potentially wrapped-around multiplication without
- * tripping any wrap-around sanitizers that may be enabled.
- */
-#define wrapping_mul(type, a, b)				\
-	({							\
-		type __val;					\
-		__builtin_mul_overflow(a, b, &__val);		\
-		__val;						\
-	})
-
-/**
- * check_shl_overflow() - Calculate a left-shifted value and check overflow
- * @a: Value to be shifted
- * @s: How many bits left to shift
- * @d: Pointer to where to store the result
- *
- * Computes *@d = (@a << @s)
- *
- * Returns true if '*@d' cannot hold the result or when '@a << @s' doesn't
- * make sense. Example conditions:
- *
- * - '@a << @s' causes bits to be lost when stored in *@d.
- * - '@s' is garbage (e.g. negative) or so large that the result of
- *   '@a << @s' is guaranteed to be 0.
- * - '@a' is negative.
- * - '@a << @s' sets the sign bit, if any, in '*@d'.
- *
- * '*@d' will hold the results of the attempted shift, but is not
- * considered "safe for use" if true is returned.
- */
-#define check_shl_overflow(a, s, d) __must_check_overflow(({		\
-	typeof(a) _a = a;						\
-	typeof(s) _s = s;						\
-	typeof(d) _d = d;						\
-	unsigned long long _a_full = _a;				\
-	unsigned int _to_shift =					\
-		is_non_negative(_s) && _s < 8 * sizeof(*d) ? _s : 0;	\
-	*_d = (_a_full << _to_shift);					\
-	(_to_shift != _s || is_negative(*_d) || is_negative(_a) ||	\
-	(*_d >> _to_shift) != _a);					\
-}))
-
-#define __overflows_type_constexpr(x, T) (			\
-	is_unsigned_type(typeof(x)) ?				\
-		(x) > type_max(T) :				\
-	is_unsigned_type(typeof(T)) ?				\
-		(x) < 0 || (x) > type_max(T) :			\
-	(x) < type_min(T) || (x) > type_max(T))
-
-#define __overflows_type(x, T)		({	\
-	typeof(T) v = 0;			\
-	check_add_overflow((x), v, &v);		\
-})
-
-/**
- * overflows_type - helper for checking the overflows between value, variables,
- *		    or data type
- *
- * @n: source constant value or variable to be checked
- * @T: destination variable or data type proposed to store @x
- *
- * Compares the @x expression for whether or not it can safely fit in
- * the storage of the type in @T. @x and @T can have different types.
- * If @x is a constant expression, this will also resolve to a constant
- * expression.
- *
- * Returns: true if overflow can occur, false otherwise.
- */
-#define overflows_type(n, T)					\
-	__builtin_choose_expr(__is_constexpr(n),		\
-			      __overflows_type_constexpr(n, T),	\
-			      __overflows_type(n, T))
-
-/**
- * castable_to_type - like __same_type(), but also allows for casted literals
- *
- * @n: variable or constant value
- * @T: variable or data type
- *
- * Unlike the __same_type() macro, this allows a constant value as the
- * first argument. If this value would not overflow into an assignment
- * of the second argument's type, it returns true. Otherwise, this falls
- * back to __same_type().
- */
-#define castable_to_type(n, T)						\
-	__builtin_choose_expr(__is_constexpr(n),			\
-			      !__overflows_type_constexpr(n, T),	\
-			      __same_type(n, T))
-
-/**
- * size_mul() - Calculate size_t multiplication with saturation at SIZE_MAX
- * @factor1: first factor
- * @factor2: second factor
- *
- * Returns: calculate @factor1 * @factor2, both promoted to size_t,
- * with any overflow causing the return value to be SIZE_MAX. The
- * lvalue must be size_t to avoid implicit type conversion.
- */
-static inline size_t __must_check size_mul(size_t factor1, size_t factor2)
-{
-	size_t bytes;
-
-	if (check_mul_overflow(factor1, factor2, &bytes))
-		return SIZE_MAX;
-
-	return bytes;
-}
-
-/**
- * size_add() - Calculate size_t addition with saturation at SIZE_MAX
- * @addend1: first addend
- * @addend2: second addend
- *
- * Returns: calculate @addend1 + @addend2, both promoted to size_t,
- * with any overflow causing the return value to be SIZE_MAX. The
- * lvalue must be size_t to avoid implicit type conversion.
- */
-static inline size_t __must_check size_add(size_t addend1, size_t addend2)
-{
-	size_t bytes;
-
-	if (check_add_overflow(addend1, addend2, &bytes))
-		return SIZE_MAX;
-
-	return bytes;
-}
-
-/**
- * size_sub() - Calculate size_t subtraction with saturation at SIZE_MAX
- * @minuend: value to subtract from
- * @subtrahend: value to subtract from @minuend
- *
- * Returns: calculate @minuend - @subtrahend, both promoted to size_t,
- * with any overflow causing the return value to be SIZE_MAX. For
- * composition with the size_add() and size_mul() helpers, neither
- * argument may be SIZE_MAX (or the result with be forced to SIZE_MAX).
- * The lvalue must be size_t to avoid implicit type conversion.
- */
-static inline size_t __must_check size_sub(size_t minuend, size_t subtrahend)
-{
-	size_t bytes;
-
-	if (minuend == SIZE_MAX || subtrahend == SIZE_MAX ||
-	    check_sub_overflow(minuend, subtrahend, &bytes))
-		return SIZE_MAX;
-
-	return bytes;
-}
-
-/**
- * array_size() - Calculate size of 2-dimensional array.
- * @a: dimension one
- * @b: dimension two
- *
- * Calculates size of 2-dimensional array: @a * @b.
- *
- * Returns: number of bytes needed to represent the array or SIZE_MAX on
- * overflow.
- */
-#define array_size(a, b)	size_mul(a, b)
-
-/**
- * array3_size() - Calculate size of 3-dimensional array.
- * @a: dimension one
- * @b: dimension two
- * @c: dimension three
- *
- * Calculates size of 3-dimensional array: @a * @b * @c.
- *
- * Returns: number of bytes needed to represent the array or SIZE_MAX on
- * overflow.
- */
-#define array3_size(a, b, c)	size_mul(size_mul(a, b), c)
-
-/**
- * flex_array_size() - Calculate size of a flexible array member
- *                     within an enclosing structure.
- * @p: Pointer to the structure.
- * @member: Name of the flexible array member.
- * @count: Number of elements in the array.
- *
- * Calculates size of a flexible array of @count number of @member
- * elements, at the end of structure @p.
- *
- * Return: number of bytes needed or SIZE_MAX on overflow.
- */
-#define flex_array_size(p, member, count)				\
-	__builtin_choose_expr(__is_constexpr(count),			\
-		(count) * sizeof(*(p)->member) + __must_be_array((p)->member),	\
-		size_mul(count, sizeof(*(p)->member) + __must_be_array((p)->member)))
-
-/**
- * struct_size() - Calculate size of structure with trailing flexible array.
- * @p: Pointer to the structure.
- * @member: Name of the array member.
- * @count: Number of elements in the array.
- *
- * Calculates size of memory needed for structure of @p followed by an
- * array of @count number of @member elements.
- *
- * Return: number of bytes needed or SIZE_MAX on overflow.
- */
-#define struct_size(p, member, count)					\
-	__builtin_choose_expr(__is_constexpr(count),			\
-		sizeof(*(p)) + flex_array_size(p, member, count),	\
-		size_add(sizeof(*(p)), flex_array_size(p, member, count)))
-
-/**
- * struct_size_t() - Calculate size of structure with trailing flexible array
- * @type: structure type name.
- * @member: Name of the array member.
- * @count: Number of elements in the array.
- *
- * Calculates size of memory needed for structure @type followed by an
- * array of @count number of @member elements. Prefer using struct_size()
- * when possible instead, to keep calculations associated with a specific
- * instance variable of type @type.
- *
- * Return: number of bytes needed or SIZE_MAX on overflow.
- */
-#define struct_size_t(type, member, count)					\
-	struct_size((type *)NULL, member, count)
-
-/**
- * _DEFINE_FLEX() - helper macro for DEFINE_FLEX() family.
- * Enables caller macro to pass (different) initializer.
- *
- * @type: structure type name, including "struct" keyword.
- * @name: Name for a variable to define.
- * @member: Name of the array member.
- * @count: Number of elements in the array; must be compile-time const.
- * @initializer: initializer expression (could be empty for no init).
- */
-#define _DEFINE_FLEX(type, name, member, count, initializer...)			\
-	_Static_assert(__builtin_constant_p(count),				\
-		       "onstack flex array members require compile-time const count"); \
-	union {									\
-		u8 bytes[struct_size_t(type, member, count)];			\
-		type obj;							\
-	} name##_u initializer;							\
-	type *name = (type *)&name##_u
-
-/**
- * DEFINE_RAW_FLEX() - Define an on-stack instance of structure with a trailing
- * flexible array member, when it does not have a __counted_by annotation.
- *
- * @type: structure type name, including "struct" keyword.
- * @name: Name for a variable to define.
- * @member: Name of the array member.
- * @count: Number of elements in the array; must be compile-time const.
- *
- * Define a zeroed, on-stack, instance of @type structure with a trailing
- * flexible array member.
- * Use __struct_size(@name) to get compile-time size of it afterwards.
- */
-#define DEFINE_RAW_FLEX(type, name, member, count)	\
-	_DEFINE_FLEX(type, name, member, count, = {})
-
-/**
- * DEFINE_FLEX() - Define an on-stack instance of structure with a trailing
- * flexible array member.
- *
- * @TYPE: structure type name, including "struct" keyword.
- * @NAME: Name for a variable to define.
- * @MEMBER: Name of the array member.
- * @COUNTER: Name of the __counted_by member.
- * @COUNT: Number of elements in the array; must be compile-time const.
- *
- * Define a zeroed, on-stack, instance of @TYPE structure with a trailing
- * flexible array member.
- * Use __struct_size(@NAME) to get compile-time size of it afterwards.
- */
-#define DEFINE_FLEX(TYPE, NAME, MEMBER, COUNTER, COUNT)	\
-	_DEFINE_FLEX(TYPE, NAME, MEMBER, COUNT, = { .obj.COUNTER = COUNT, })
-
-#endif /* __LINUX_OVERFLOW_H */

lib/efi_loader/efi_smbios.c

@@ -35,6 +35,7 @@ efi_status_t efi_smbios_register(void)
 	void *buf;
 
 	addr = gd_smbios_start();
+	printf("register %lx\n", addr);
 	if (!addr) {
 		log_err("No SMBIOS tables to install\n");
 		return EFI_NOT_FOUND;

tools/patman/control.py

@@ -63,8 +63,7 @@ def prepare_patches(col, branch, count, start, end, ignore_binary, signoff,
         branch, start, to_do, ignore_binary, series, signoff)
 
     # Fix up the patch files to our liking, and insert the cover letter
-    patchstream.fix_patches(series, patch_files, keep_change_id,
-                            insert_base_commit=not cover_fname)
+    patchstream.fix_patches(series, patch_files, keep_change_id)
     if cover_fname and series.get('cover'):
         patchstream.insert_cover_letter(cover_fname, series, to_do)
     return series, cover_fname, patch_files

tools/patman/func_test.py

@@ -357,31 +357,6 @@ Changes in v2:
                 expected = expected.splitlines()
                 self.assertEqual(expected, lines[start:(start+len(expected))])
 
-    def test_base_commit(self):
-        """Test adding a base commit with no cover letter"""
-        orig_text = self._get_text('test01.txt')
-        pos = orig_text.index('commit 5ab48490f03051875ab13d288a4bf32b507d76fd')
-        text = orig_text[:pos]
-        series = patchstream.get_metadata_for_test(text)
-        series.base_commit = Commit('1a44532')
-        series.branch = 'mybranch'
-        cover_fname, args = self._create_patches_for_test(series)
-        self.assertFalse(cover_fname)
-        with capture_sys_output() as out:
-            patchstream.fix_patches(series, args, insert_base_commit=True)
-        self.assertEqual('Cleaned 1 patch\n', out[0].getvalue())
-        lines = tools.read_file(args[0], binary=False).splitlines()
-        pos = lines.index('-- ')
-
-        # We expect these lines at the end:
-        # -- (with trailing space)
-        # 2.7.4
-        # (empty)
-        # base-commit: xxx
-        # branch: xxx
-        self.assertEqual('base-commit: 1a44532', lines[pos + 3])
-        self.assertEqual('branch: mybranch', lines[pos + 4])
-
     def make_commit_with_file(self, subject, body, fname, text):
         """Create a file and add it to the git repo with a new commit
 
@@ -552,11 +527,6 @@ complicated as possible''')
             self.assertEqual(f'base-commit: {base}', lines[0])
             self.assertEqual('branch: second', lines[1])
 
-            # Make sure that the base-commit is not present when it is in the
-            # cover letter
-            for fname in patch_files:
-                self.assertNotIn(b'base-commit:', tools.read_file(fname))
-
             # Check that it can skip patches at the end
             with capture_sys_output() as _:
                 _, cover_fname, patch_files = control.prepare_patches(

tools/patman/patchstream.py

@@ -76,13 +76,8 @@ class PatchStream:
     are interested in. We can also process a patch file in order to remove
     unwanted tags or inject additional ones. These correspond to the two
     phases of processing.
-
-    Args:
-        keep_change_id (bool): Keep the Change-Id tag
-        insert_base_commit (bool): True to add the base commit to the end
     """
-    def __init__(self, series, is_log=False, keep_change_id=False,
-                 insert_base_commit=False):
+    def __init__(self, series, is_log=False, keep_change_id=False):
         self.skip_blank = False          # True to skip a single blank line
         self.found_test = False          # Found a TEST= line
         self.lines_after_test = 0        # Number of lines found after TEST=
@@ -108,7 +103,6 @@ class PatchStream:
         self.recent_quoted = collections.deque([], 5)
         self.recent_unquoted = queue.Queue()
         self.was_quoted = None
-        self.insert_base_commit = insert_base_commit
 
     @staticmethod
     def process_text(text, is_comment=False):
@@ -664,13 +658,6 @@ class PatchStream:
                     outfd.write(line + '\n')
                     self.blank_count = 0
         self.finalise()
-        if self.insert_base_commit:
-            if self.series.base_commit:
-                print(f'base-commit: {self.series.base_commit.hash}',
-                      file=outfd)
-            if self.series.branch:
-                print(f'branch: {self.series.branch}', file=outfd)
-
 
 def insert_tags(msg, tags_to_emit):
     """Add extra tags to a commit message
@@ -791,8 +778,7 @@ def get_metadata_for_test(text):
     pst.finalise()
     return series
 
-def fix_patch(backup_dir, fname, series, cmt, keep_change_id=False,
-              insert_base_commit=False):
+def fix_patch(backup_dir, fname, series, cmt, keep_change_id=False):
     """Fix up a patch file, by adding/removing as required.
 
     We remove our tags from the patch file, insert changes lists, etc.
@@ -806,7 +792,6 @@ def fix_patch(backup_dir, fname, series, cmt, keep_change_id=False,
         series (Series): Series information about this patch set
         cmt (Commit): Commit object for this patch file
         keep_change_id (bool): Keep the Change-Id tag.
-        insert_base_commit (bool): True to add the base commit to the end
 
     Return:
         list: A list of errors, each str, or [] if all ok.
@@ -814,8 +799,7 @@ def fix_patch(backup_dir, fname, series, cmt, keep_change_id=False,
     handle, tmpname = tempfile.mkstemp()
     outfd = os.fdopen(handle, 'w', encoding='utf-8')
     infd = open(fname, 'r', encoding='utf-8')
-    pst = PatchStream(series, keep_change_id=keep_change_id,
-                      insert_base_commit=insert_base_commit)
+    pst = PatchStream(series, keep_change_id=keep_change_id)
     pst.commit = cmt
     pst.process_stream(infd, outfd)
     infd.close()
@@ -827,7 +811,7 @@ def fix_patch(backup_dir, fname, series, cmt, keep_change_id=False,
     shutil.move(tmpname, fname)
     return cmt.warn
 
-def fix_patches(series, fnames, keep_change_id=False, insert_base_commit=False):
+def fix_patches(series, fnames, keep_change_id=False):
     """Fix up a list of patches identified by filenames
 
     The patch files are processed in place, and overwritten.
@@ -836,7 +820,6 @@ def fix_patches(series, fnames, keep_change_id=False, insert_base_commit=False):
         series (Series): The Series object
         fnames (:type: list of str): List of patch files to process
         keep_change_id (bool): Keep the Change-Id tag.
-        insert_base_commit (bool): True to add the base commit to the end
     """
     # Current workflow creates patches, so we shouldn't need a backup
     backup_dir = None  #tempfile.mkdtemp('clean-patch')
@@ -846,8 +829,7 @@ def fix_patches(series, fnames, keep_change_id=False, insert_base_commit=False):
         cmt.patch = fname
         cmt.count = count
         result = fix_patch(backup_dir, fname, series, cmt,
-                           keep_change_id=keep_change_id,
-                           insert_base_commit=insert_base_commit)
+                           keep_change_id=keep_change_id)
         if result:
             print('%d warning%s for %s:' %
                   (len(result), 's' if len(result) > 1 else '', fname))