qemu/scripts/make-config-poison.sh

#! /bin/sh

if test $# = 0; then
  exit 0
fi

# Create list of config switches that should be poisoned in common code,
# but filter out several which are handled manually.
exec sed -n \
  -e' /CONFIG_TCG/d' \
  -e '/CONFIG_USER_ONLY/d' \
  -e '/CONFIG_SOFTMMU/d' \
  -e '/TARGET_NOT_USING_LEGACY_LDST_PHYS_API/d' \
  -e '/TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API/d' \
  -e '/^#define / {' \
  -e    's///' \
  -e    's/ .*//' \
  -e    's/^/#pragma GCC poison /p' \
  -e '}' "$@" | sort -u
configure, meson: move config-poison.h to meson This ensures that the file is regenerated properly whenever config-target.h or config-devices.h files change. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> 5 years ago			`#! /bin/sh`

			`if test $# = 0; then`
			`exit 0`
			`fi`

exec/poison: Do not poison CONFIG_SOFTMMU If CONFIG_USER_ONLY is ok generically, so is CONFIG_SOFTMMU, because they are exactly opposite. Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> 3 years ago			`# Create list of config switches that should be poisoned in common code,`
			`# but filter out several which are handled manually.`
configure, meson: move config-poison.h to meson This ensures that the file is regenerated properly whenever config-target.h or config-devices.h files change. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> 5 years ago			`exec sed -n \`
			`-e' /CONFIG_TCG/d' \`
			`-e '/CONFIG_USER_ONLY/d' \`
exec/poison: Do not poison CONFIG_SOFTMMU If CONFIG_USER_ONLY is ok generically, so is CONFIG_SOFTMMU, because they are exactly opposite. Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> 3 years ago			`-e '/CONFIG_SOFTMMU/d' \`
system/memory: Allow restricting legacy ldst_phys() API usage Commit 500131154d6 ("exec.c: Add new address_space_ld/st functions") added a new API to fix a shortcoming of the ld/st_phys() API, which does blind bus access, not reporting failure (and it also allow to provide transaction attributes). Later commit 42874d3a8c6 ("Switch non-CPU callers from ld/st_phys to address_space_ld/st*") automatically converted the legacy uses to the new API, not precising transaction attributes (MEMTXATTRS_UNSPECIFIED) and ignoring the transation result (passing NULL pointer as MemTxResult). While this is a faithful replacement, without any logical change, we later realized better is to not use MEMTXATTRS_UNSPECIFIED or NULL MemTxResult, and adapt each call site on a pair basis, looking at the device model datasheet to do the correct behavior (which is unlikely to ignore transaction failures). Since this is quite some work, we defer that to device model maintainers. Meanwhile we introduce a definition, to allow a target which removed all legacy API call to prohibit further legacy API uses, named "TARGET_NOT_USING_LEGACY_LDST_PHYS_API". Since all targets should be able to check this definition, we take care to not poison it. Suggested-by: Pierrick Bouvier <pierrick.bouvier@linaro.org> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Message-Id: <20251224151351.86733-6-philmd@linaro.org> 8 months ago			`-e '/TARGET_NOT_USING_LEGACY_LDST_PHYS_API/d' \`
system: Allow restricting the legacy ld/st_phys() 'native-endian' API Guard the native endian APIs we want to remove by surrounding them with TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API #ifdef'ry. Since all targets can check the definition, do not poison it. Once a target gets cleaned we'll set the definition in the target config, then the target won't be able to use the legacy API anymore. Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Message-ID: <20260109165058.59144-15-philmd@linaro.org> 8 months ago			`-e '/TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API/d' \`
configure, meson: move config-poison.h to meson This ensures that the file is regenerated properly whenever config-target.h or config-devices.h files change. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> 5 years ago			`-e '/^#define / {' \`
			`-e 's///' \`
			`-e 's/ .*//' \`
			`-e 's/^/#pragma GCC poison /p' \`
meson: Avoid duplicates in generated config-poison.h again Commit eed56e9a89f "configure, meson: move config-poison.h to meson" lost a "\| sort -u". Restore it. config-poison shrinks from ~4500 to ~700 lines when all targets are enabled. Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Tested-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20230206102037.3621709-1-armbru@redhat.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> 4 years ago			`-e '}' "$@" \| sort -u`