mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2026-05-25 23:28:12 +02:00
222b85328e
The standard way to query options in mesa is `os_get_option()` which
abstracts platform-specific mechanisms to get config variables.
However in quite a few places `getenv()` is still used and this may
preclude controlling some options on some systems.
For instance it is not generally possible to use `MESA_DEBUG` on
Android.
So replace most `getenv()` occurrences with `os_get_option()` to
support configuration options more consistently across different
platforms.
Do the same with `secure_getenv()` replacing it with
`os_get_option_secure()`.
The bulk of the proposed changes are mechanically performed by the
following script:
-----------------------------------------------------------------------
#!/bin/sh
set -e
replace() {
# Don't replace in some files, for example where `os_get_option` is defined,
# or in external files
EXCLUDE_FILES_PATTERN='(src/util/os_misc.c|src/util/u_debug.h|src/gtest/include/gtest/internal/gtest-port.h)'
# Don't replace some "system" variables
EXCLUDE_VARS_PATTERN='("XDG|"DISPLAY|"HOME|"TMPDIR|"POSIXLY_CORRECT)'
git grep "[=!( ]$1(" -- src/ | cut -d ':' -f 1 | sort | uniq | \
grep -v -E "$EXCLUDE_FILES_PATTERN" | \
while read -r file;
do
# Don't replace usages of XDG_* variables or HOME
sed -E -e "/$EXCLUDE_VARS_PATTERN/!s/([=!\( ])$1\(/\1$2\(/g" -i "$file";
done
}
# Add const to os_get_option results, to avoid warning about discarded qualifier:
# warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
# but also errors in some cases:
# error: invalid conversion from ‘const char*’ to ‘char*’ [-fpermissive]
add_const_results() {
git grep -l -P '(?<!const )char.*os_get_option' | \
while read -r file;
do
sed -e '/^\s*const/! s/\(char.*os_get_option\)/const \1/g' -i "$file"
done
}
replace 'secure_getenv' 'os_get_option_secure'
replace 'getenv' 'os_get_option'
add_const_results
-----------------------------------------------------------------------
After this, the `#include "util/os_misc.h"` is also added in files where
`os_get_option()` was not used before.
And since the replacements from the script above generated some new
`-Wdiscarded-qualifiers` warnings, those have been addressed as well,
generally by declaring `os_get_option()` results as `const char *` and
adjusting some function declarations.
Finally some replacements caused new errors like:
-----------------------------------------------------------------------
../src/gallium/auxiliary/gallivm/lp_bld_misc.cpp:127:31: error: no matching function for call to 'strtok'
127 | for (n = 0, option = strtok(env_llc_options, " "); option; n++, option = strtok(NULL, " ")) {
| ^~~~~~
/android-ndk-r27c/toolchains/llvm/prebuilt/linux-x86_64/bin/../sysroot/usr/include/string.h:124:17: note: candidate function not viable: 1st argument ('const char *') would lose const qualifier
124 | char* _Nullable strtok(char* _Nullable __s, const char* _Nonnull __delimiter);
| ^ ~~~~~~~~~~~~~~~~~~~
-----------------------------------------------------------------------
Those have been addressed too, copying the const string returned by
`os_get_option()` so that it could be modified.
In particular, the error above has been fixed by copying the `const
char *env_llc_options` variable in
`src/gallium/auxiliary/gallivm/lp_bld_misc.cpp` to a `char *` which can
be tokenized using `strtok()`.
Reviewed-by: Eric Engestrom <eric@igalia.com>
Reviewed-by: Yonggang Luo <luoyonggang@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/38128>
70 lines
1.5 KiB
Meson
70 lines
1.5 KiB
Meson
# Copyright © 2017 Intel Corporation
|
|
# SPDX-License-Identifier: MIT
|
|
|
|
inc_gbm = include_directories('.', 'main', 'backends/dri')
|
|
|
|
files_gbm = files(
|
|
'main/backend.c',
|
|
'main/backend.h',
|
|
'main/gbm.c',
|
|
'main/gbm.h',
|
|
'main/gbmint.h',
|
|
)
|
|
args_gbm = [
|
|
'-DDEFAULT_BACKENDS_PATH="@0@"'.format(gbm_backends_path),
|
|
]
|
|
incs_gbm = [include_directories('main'), inc_loader, inc_gallium, inc_util]
|
|
|
|
libgbm_name = 'gbm'
|
|
|
|
if with_platform_android and get_option('platform-sdk-version') >= 30
|
|
libgbm_name = 'gbm_mesa'
|
|
endif
|
|
|
|
libgbm = shared_library(
|
|
libgbm_name,
|
|
files_gbm,
|
|
include_directories : incs_gbm,
|
|
c_args : [args_gbm],
|
|
link_args : [ld_args_gc_sections],
|
|
link_with : [libloader],
|
|
dependencies : [dep_libdrm, idep_xmlconfig],
|
|
gnu_symbol_visibility : 'hidden',
|
|
version : '1.0.0',
|
|
install : true,
|
|
)
|
|
|
|
if with_tests
|
|
abi_check = executable('gbm_abi_check', 'main/gbm_abi_check.c')
|
|
test('gbm-abi-check', abi_check, suite : ['gbm'])
|
|
endif
|
|
|
|
install_headers('main/gbm.h', 'main/gbm_backend_abi.h')
|
|
|
|
pkg.generate(
|
|
name : 'gbm',
|
|
filebase : 'gbm',
|
|
description : 'Mesa gbm library',
|
|
version : meson.project_version(),
|
|
libraries : libgbm,
|
|
libraries_private : gbm_priv_libs,
|
|
variables : ['gbmbackendspath=' + gbm_backends_path],
|
|
)
|
|
|
|
if with_symbols_check
|
|
test(
|
|
'gbm-symbols-check',
|
|
symbols_check,
|
|
args : [
|
|
'--lib', libgbm,
|
|
'--symbols-file', files('gbm-symbols.txt'),
|
|
symbols_check_args,
|
|
],
|
|
suite : ['gbm'],
|
|
)
|
|
endif
|
|
|
|
dep_gbm = declare_dependency(
|
|
link_with : libgbm,
|
|
include_directories : inc_gbm,
|
|
)
|