Re: [PATCH v2] trace: tracing_event_filter: fast path when no subsystem filters

2023-10-12 Thread Nick Lowell
On Wed, Oct 4, 2023 at 10:52 AM Steven Rostedt wrote: > > On Wed, 4 Oct 2023 10:39:33 -0400 > Nick Lowell wrote: > > > > [ cut here ] > > > WARNING: CPU: 5 PID: 944 at kernel/trace/trace_events_filter.c:2423 > > > apply_subsystem_

Re: [PATCH v2] trace: tracing_event_filter: fast path when no subsystem filters

2023-10-04 Thread Nick Lowell
On Tue, Oct 3, 2023 at 10:29 PM Steven Rostedt wrote: > > On Mon, 2 Oct 2023 10:41:48 -0400 > Nicholas Lowell wrote: > > > @@ -2411,7 +2418,12 @@ int apply_subsystem_event_filter(struct > > trace_subsystem_dir *dir, > > } > > > > if (!strcmp(strstrip(filter_string), "0")) { > > -

Re: [PATCH] trace: tracing_event_filter: fast path when no subsystem filters

2023-10-02 Thread Nick Lowell
Sending again in plain text mode. Thanks for the great feedback! Hopefully my inline comments/questions aren't garbled. On Sat, Sep 30, 2023 at 4:04 AM Steven Rostedt wrote: > > On Tue, 26 Sep 2023 10:20:58 -0400 > Nicholas Lowell wrote: > > > From: Nicholas Lowell > > > > If there are no

Re: [PATCH v2 0/2] get_maintainer: add patch-only keyword matching

2023-09-28 Thread Nick Desaulniers
mention it, I see it, and it does seem consistent in that regard. Justin, is your formatter configurable to match that convention? Maybe it's still useful, as long as you configure it to stick to the pre-existing convention. -- Thanks, ~Nick Desaulniers

Re: [PATCH 0/3] get_maintainer: add patch-only keyword matching

2023-09-27 Thread Nick Desaulniers
|3 + > scripts/get_maintainer.pl | 3334 > +++-- > 2 files changed, 1718 insertions(+), 1619 deletions(-) > --- > base-commit: 6465e260f48790807eef06b583b38ca9789b6072 > change-id: 20230926-get_maintainer_add_d-07424a814e72 > > Best regards, > -- > Justin Stitt > -- Thanks, ~Nick Desaulniers

Re: [PATCH 01/15] objtool: Find a destination for jumps beyond the section end

2021-04-20 Thread Nick Desaulniers
lls through to the next function" > > which is what we normally do in this type of situation. > > It may be caused by UB, or a compiler bug, but either way we should > figure out the root cause. We probably want to creduce or cvise this. IIRC we still have outstanding issues with switch statements with user-annotated unreachable branches not getting eliminated. -- Thanks, ~Nick Desaulniers

[PATCH v3] arm64: vdso32: drop -no-integrated-as flag

2021-04-20 Thread Nick Desaulniers
- \ CROSS_COMPILE_COMPAT=arm-linux-gnueabi- make LLVM=1 LLVM_IAS=1 \ defconfig arch/arm64/kernel/vdso32/ Suggested-by: Nathan Chancellor Signed-off-by: Nick Desaulniers Reviewed-by: Nathan Chancellor Reviewed-by: Vincenzo Frascino --- Changes V2 -> V3: * Pick up reviewed by tags. Changes

Re: [PATCH 00/13] [RFC] Rust support

2021-04-19 Thread Nick Desaulniers
his component in Rust from the start, 51 (73.9%) of these bugs would not have been possible." -- Thanks, ~Nick Desaulniers

Re: [PATCH 00/13] [RFC] Rust support

2021-04-19 Thread Nick Desaulniers
On Fri, Apr 16, 2021 at 11:47 AM Paul E. McKenney wrote: > > On Thu, Apr 15, 2021 at 11:04:37PM -0700, Nick Desaulniers wrote: > > On Thu, Apr 15, 2021 at 9:27 PM Boqun Feng wrote: > > > > > > But I think the Rust Community still wants to have a good memory model,

[PATCH v4 4/5] RISC-V: Add kdump support

2021-04-18 Thread Nick Kossifidis
he ioresources tree, so that it can be used later on for implementing strict /dev/mem. * Minor cleanups and re-base Signed-off-by: Nick Kossifidis --- arch/riscv/include/asm/elf.h| 6 +++ arch/riscv/include/asm/kexec.h | 19 +--- arch/riscv/kernel/Makefile |

[PATCH v4 3/5] RISC-V: Improve init_resources

2021-04-18 Thread Nick Kossifidis
, messing things up, just re-use the previous pre-allocated resource and free any unused resources after both loops finish. Signed-off-by: Nick Kossifidis --- arch/riscv/kernel/setup.c | 91 --- 1 file changed, 46 insertions(+), 45 deletions(-) diff --git a/arch

[PATCH v4 2/5] RISC-V: Add kexec support

2021-04-18 Thread Nick Kossifidis
dt address of the included fdt. * Use SYM_* macros on kexec_relocate.S. * Compatibility with STRICT_KERNEL_RWX. * Compatibility with HOTPLUG_CPU for SMP * Small cleanups Signed-off-by: Nick Kossifidis --- arch/riscv/Kconfig | 15 +++ arch/riscv/include/asm/kexec.h

[PATCH v4 1/5] RISC-V: Add EM_RISCV to kexec UAPI header

2021-04-18 Thread Nick Kossifidis
Add RISC-V to the list of supported kexec architectures, we need to add the definition early-on so that later patches can use it. EM_RISCV is 243 as per ELF psABI specification here: https://github.com/riscv/riscv-elf-psabi-doc/blob/master/riscv-elf.md Signed-off-by: Nick Kossifidis

[PATCH v4 0/5] RISC-V: Add kexec/kdump support

2021-04-18 Thread Nick Kossifidis
used later on for implementing strict /dev/mem * Use linux,usable-memory on /memory instead of a new binding * USe a reserved-memory node for ELF core header Nick Kossifidis (5): RISC-V: Add EM_RISCV to kexec UAPI header RISC-V: Add kexec support RISC-V: Improve init_resources RISC-V:

[PATCH v4 5/5] RISC-V: Add crash kernel support

2021-04-18 Thread Nick Kossifidis
ous kernel through /proc/vmcore, using the vmcore-dmesg utility from kexec-tools. v4: * Rebase on top of "fixes" branch v3: * Rebase v2: * Use linux,usable-memory on /memory instead of a new binding * Use a reserved-memory node for ELF core header Signed-off-by: Nick Kossifidis ---

Re: [PATCH 00/13] [RFC] Rust support

2021-04-16 Thread Nick Desaulniers
On Thu, Apr 15, 2021 at 9:27 PM Boqun Feng wrote: > > [Copy LKMM people, Josh, Nick and Wedson] > > On Thu, Apr 15, 2021 at 08:58:16PM +0200, Peter Zijlstra wrote: > > On Wed, Apr 14, 2021 at 08:45:51PM +0200, oj...@kernel.org wrote: > > > > > Rust is a systems

Re: [PATCH 04/13] Kbuild: Rust support

2021-04-15 Thread Nick Desaulniers
On Wed, Apr 14, 2021 at 5:43 PM Miguel Ojeda wrote: > > On Thu, Apr 15, 2021 at 1:19 AM Nick Desaulniers > wrote: > > > > -Oz in clang typically generates larger kernel code than -Os; LLVM > > seems to aggressively emit libcalls even when the setup for a call > >

Re: [PATCH] X86: Makefile: Replace -pg with CC_FLAGS_FTRACE

2021-04-15 Thread Nick Desaulniers
o := n > KCSAN_SANITIZE_cmdline.o := n > > ifdef CONFIG_FUNCTION_TRACER > -CFLAGS_REMOVE_cmdline.o = -pg > +CFLAGS_REMOVE_cmdline.o = $(CC_FLAGS_FTRACE) > endif > > CFLAGS_cmdline.o := -fno-stack-protector -fno-jump-tables > -- > 2.20.1 > > > -- Thanks, ~Nick Desaulniers

Re: [PATCH v2] arm64: vdso32: drop -no-integrated-as flag

2021-04-15 Thread Nick Desaulniers
On Thu, Apr 15, 2021 at 6:31 AM Vincenzo Frascino wrote: > > > > On 4/14/21 10:45 PM, Nick Desaulniers wrote: > > Clang can assemble these files just fine; this is a relic from the top > > level Makefile conditionally adding this. We no longer need --prefix, > >

Re: [PATCH 09/13] Samples: Rust examples

2021-04-15 Thread Nick Desaulniers
On Thu, Apr 15, 2021 at 12:10 AM Greg Kroah-Hartman wrote: > > On Wed, Apr 14, 2021 at 04:24:45PM -0700, Nick Desaulniers wrote: > > On Wed, Apr 14, 2021 at 12:35 PM Linus Torvalds > > wrote: > > > > > > On Wed, Apr 14, 2021 at 11:47 AM wrote

Re: [PATCH 00/13] [RFC] Rust support

2021-04-14 Thread Nick Desaulniers
at: > > rust-for-li...@vger.kernel.org > > and take a look at the project itself at: > > https://github.com/Rust-for-Linux Looks like Wedson's writeup is now live. Nice job Wedson! https://security.googleblog.com/2021/04/rust-in-linux-kernel.html -- Thanks, ~Nick Desaulniers

Re: [PATCH 02/13] kallsyms: Increase maximum kernel symbol length to 512

2021-04-14 Thread Nick Desaulniers
SYM_NAME_LEN > -#define KSYM_NAME_LEN 256 > +#define KSYM_NAME_LEN 512 > #endif > > struct perf_record_ksymbol { > diff --git a/tools/lib/symbol/kallsyms.h b/tools/lib/symbol/kallsyms.h > index 72ab9870454b..542f9b059c3b 100644 > --- a/tools/lib/symbol/kallsyms.h > +++ b/tools/lib/symbol/kallsyms.h > @@ -7,7 +7,7 @@ > #include > > #ifndef KSYM_NAME_LEN > -#define KSYM_NAME_LEN 256 > +#define KSYM_NAME_LEN 512 > #endif > > static inline u8 kallsyms2elf_binding(char type) > -- > 2.17.1 > -- Thanks, ~Nick Desaulniers

Re: [PATCH 03/13] Makefile: Generate CLANG_FLAGS even in GCC builds

2021-04-14 Thread Nick Desaulniers
renced from Kconfig (so it needs export), > +# and from include/config/auto.conf.cmd to detect the compiler upgrade. > +CC_VERSION_TEXT = $(shell $(CC) --version 2>/dev/null | head -n 1 | sed > 's/\#//g') > + > +ifneq ($(findstring clang,$(CC_VERSION_TEXT)),) > +CLANG_FLAGS+= $(TENTATIVE_CLANG_FLAGS) > KBUILD_CFLAGS += $(CLANG_FLAGS) > KBUILD_AFLAGS += $(CLANG_FLAGS) > export CLANG_FLAGS > -- > 2.17.1 > -- Thanks, ~Nick Desaulniers

Re: [PATCH 09/13] Samples: Rust examples

2021-04-14 Thread Nick Desaulniers
ld > use as a example of a real piece of code that actually does something > meaningful? Are you suggesting that they "rewrite it in Rust?" :^P *ducks* (sorry, I couldn't help myself) Perhaps it would be a good exercise to demonstrate some of the benefits of using Rust for driver work? -- Thanks, ~Nick Desaulniers

Re: [PATCH 04/13] Kbuild: Rust support

2021-04-14 Thread Nick Desaulniers
\ > $(_a_flags) $(modkern_aflags) > > diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c > index 2568dbe16ed6..a83d646ecef5 100644 > --- a/scripts/kconfig/confdata.c > +++ b/scripts/kconfig/confdata.c > @@ -637,6 +637,56 @@ static struct conf_printer kconfig_printer_cb = > .print_comment = kconfig_print_comment, > }; > > +/* > + * rustc cfg printer > + * > + * This printer is used when generating the resulting rustc configuration > + * after kconfig invocation and `defconfig` files. > + */ > +static void rustc_cfg_print_symbol(FILE *fp, struct symbol *sym, const char > *value, void *arg) > +{ > + const char *str; > + > + switch (sym->type) { > + case S_INT: > + case S_HEX: > + case S_BOOLEAN: > + case S_TRISTATE: > + str = sym_escape_string_value(value); > + > + /* > +* We don't care about disabled ones, i.e. no need for > +* what otherwise are "comments" in other printers. > +*/ > + if (*value == 'n') > + return; > + > + /* > +* To have similar functionality to the C macro `IS_ENABLED()` > +* we provide an empty `--cfg CONFIG_X` here in both `y` > +* and `m` cases. > +* > +* Then, the common `fprintf()` below will also give us > +* a `--cfg CONFIG_X="y"` or `--cfg CONFIG_X="m"`, which can > +* be used as the equivalent of `IS_BUILTIN()`/`IS_MODULE()`. > +*/ > + if (*value == 'y' || *value == 'm') > + fprintf(fp, "--cfg=%s%s\n", CONFIG_, sym->name); > + > + break; > + default: > + str = value; > + break; > + } > + > + fprintf(fp, "--cfg=%s%s=%s\n", CONFIG_, sym->name, str); > +} > + > +static struct conf_printer rustc_cfg_printer_cb = > +{ > + .print_symbol = rustc_cfg_print_symbol, > +}; > + > /* > * Header printer > * > @@ -1044,7 +1094,7 @@ int conf_write_autoconf(int overwrite) > struct symbol *sym; > const char *name; > const char *autoconf_name = conf_get_autoconfig_name(); > - FILE *out, *out_h; > + FILE *out, *out_h, *out_rustc_cfg; > int i; > > if (!overwrite && is_present(autoconf_name)) > @@ -1065,6 +1115,13 @@ int conf_write_autoconf(int overwrite) > return 1; > } > > + out_rustc_cfg = fopen(".tmp_rustc_cfg", "w"); > + if (!out_rustc_cfg) { > + fclose(out); > + fclose(out_h); > + return 1; > + } > + > conf_write_heading(out, _printer_cb, NULL); > conf_write_heading(out_h, _printer_cb, NULL); > > @@ -1076,9 +1133,11 @@ int conf_write_autoconf(int overwrite) > /* write symbols to auto.conf and autoconf.h */ > conf_write_symbol(out, sym, _printer_cb, (void *)1); > conf_write_symbol(out_h, sym, _printer_cb, NULL); > + conf_write_symbol(out_rustc_cfg, sym, _cfg_printer_cb, > NULL); > } > fclose(out); > fclose(out_h); > + fclose(out_rustc_cfg); > > name = getenv("KCONFIG_AUTOHEADER"); > if (!name) > @@ -1097,6 +1156,12 @@ int conf_write_autoconf(int overwrite) > if (rename(".tmpconfig", autoconf_name)) > return 1; > > + name = "include/generated/rustc_cfg"; > + if (make_parent_dir(name)) > + return 1; > + if (rename(".tmp_rustc_cfg", name)) > + return 1; > + > return 0; > } > > diff --git a/scripts/rust-version.sh b/scripts/rust-version.sh > new file mode 100755 > index ..67b6d31688e2 > --- /dev/null > +++ b/scripts/rust-version.sh > @@ -0,0 +1,31 @@ > +#!/bin/sh > +# SPDX-License-Identifier: GPL-2.0 > +# > +# rust-version rust-command > +# > +# Print the compiler version of `rust-command' in a 5 or 6-digit form > +# such as `14502' for rustc-1.45.2 etc. > +# > +# Returns 0 if not found (so that Kconfig does not complain) > +compiler="$*" > + > +if [ ${#compiler} -eq 0 ]; then > + echo "Error: No compiler specified." >&2 > + printf "Usage:\n\t$0 \n" >&2 > + exit 1 > +fi > + > +if ! command -v $compiler >/dev/null 2>&1; then > + echo 0 > + exit 0 > +fi > + > +VERSION=$($compiler --version | cut -f2 -d' ') > + > +# Cut suffix if any (e.g. `-dev`) > +VERSION=$(echo $VERSION | cut -f1 -d'-') > + > +MAJOR=$(echo $VERSION | cut -f1 -d'.') > +MINOR=$(echo $VERSION | cut -f2 -d'.') > +PATCHLEVEL=$(echo $VERSION | cut -f3 -d'.') > +printf "%d%02d%02d\\n" $MAJOR $MINOR $PATCHLEVEL > -- > 2.17.1 > -- Thanks, ~Nick Desaulniers

Re: [PATCH 11/13] MAINTAINERS: Rust

2021-04-14 Thread Nick Desaulniers
: linux-r...@vger.kernel.org ... > +F: rust/ > +F: samples/rust/ > +F: Documentation/rust/ Probably some other files, too? Like the target.json files under arch/{arch}/rust/ ? -- Thanks, ~Nick Desaulniers

Re: [PATCH 10/13] Documentation: Rust general information

2021-04-14 Thread Nick Desaulniers
+Building > + > + > +Building a kernel with Clang or a complete LLVM toolchain is the best > supported > +setup at the moment. That is:: > + > +make ARCH=... CROSS_COMPILE=... CC=clang -j... > + > +or:: > + > +make ARCH=... CROSS_COMPILE=... LLVM=1 -j... Please reorder; prefer LLVM=1 to CC=clang. Probably worth another cross reference to :ref:`kbuild_llvm`. > + > +Using GCC also works for some configurations, but it is *very* experimental > at > +the moment. > + > + > +Hacking > +--- > + > +If you want to dive deeper, take a look at the source code of the samples > +at ``samples/rust/``, the Rust support code under ``rust/`` and > +the ``Rust hacking`` menu under ``Kernel hacking``. > + > +If you use GDB/Binutils and Rust symbols aren't getting demangled, the reason > +is your toolchain doesn't support Rust's new v0 mangling scheme yet. There > are "new" as in changed, or "new" as in Rust previously did not mangle symbols? > +a few ways out: > + > + - If you don't mind building your own tools, we provide the following fork > +with the support cherry-picked from GCC on top of very recent releases: > + > + > https://github.com/Rust-for-Linux/binutils-gdb/releases/tag/gdb-10.1-release-rust > + > https://github.com/Rust-for-Linux/binutils-gdb/releases/tag/binutils-2_35_1-rust > + > + - If you only need GDB and can enable ``CONFIG_DEBUG_INFO``, do so: > +some versions of GDB (e.g. vanilla GDB 10.1) are able to use > +the pre-demangled names embedded in the debug info. > + > + - If you don't need loadable module support, you may compile without > +the ``-Zsymbol-mangling-version=v0`` flag. However, we don't maintain > +support for that, so avoid it unless you are in a hurry. > -- > 2.17.1 > -- Thanks, ~Nick Desaulniers

Re: [PATCH 11/13] MAINTAINERS: Rust

2021-04-14 Thread Nick Desaulniers
least be part of Wedson's core responsibilities, shouldn't this be "Supported." Per Maintainers: 87 S: *Status*, one of the following: 88 Supported: Someone is actually paid to look after this. 89 Maintained: Someone actually looks after it. Either way, Acked-by: Nick Des

[PATCH v2] arm64: vdso32: drop -no-integrated-as flag

2021-04-14 Thread Nick Desaulniers
- \ CROSS_COMPILE_COMPAT=arm-linux-gnueabi- make LLVM=1 LLVM_IAS=1 \ defconfig arch/arm64/kernel/vdso32/ Suggested-by: Nathan Chancellor Signed-off-by: Nick Desaulniers --- Changes V1 -> V2: * Remove --prefix, --gcc-toolchain, COMPAT_GCC_TOOLCHAIN, and COMPAT_GCC_TOOLCHAIN_DIR as per Nat

Re: [GIT PULL][PATCH v9 0/3] Update to zstd-1.4.10

2021-04-14 Thread Nick Terrell
On Wed, Apr 14, 2021 at 12:04 PM Eric Biggers wrote: > > On Wed, Apr 14, 2021 at 11:53:51AM -0700, Nick Terrell wrote: > > On Wed, Apr 14, 2021 at 11:35 AM Eric Biggers wrote: > > > > > > On Wed, Apr 14, 2021 at 11:01:29AM -0700, Nick Terrell wrote: > > > &

Re: [PATCH] arm64: alternatives: Move length validation in alternative_{insn,endif}

2021-04-14 Thread Nick Desaulniers
lternative_endif > macro, shuffling the .org directives so that the length validation > happen will always happen in the same subsections. alternative_insn has > not shown any issue yet but it appears that it could have the same issue > in the future so just preemptively change it. Thanks Nath

Re: [GIT PULL][PATCH v9 0/3] Update to zstd-1.4.10

2021-04-14 Thread Nick Terrell
On Wed, Apr 14, 2021 at 11:35 AM Eric Biggers wrote: > > On Wed, Apr 14, 2021 at 11:01:29AM -0700, Nick Terrell wrote: > > Hi all, > > > > I would really like to make some progress on this and get it merged. > > This patchset offsers: > > * 15-30% better decom

Re: [GIT PULL][PATCH v9 0/3] Update to zstd-1.4.10

2021-04-14 Thread Nick Terrell
Hi all, I would really like to make some progress on this and get it merged. This patchset offsers: * 15-30% better decompression speed * 3 years of zstd bug fixes and code improvements * Allows us to import zstd directly from upstream so we don't fall 3 years out of date again Thanks, Nick

[PATCH] arm64: vdso32: drop -no-integrated-as flag

2021-04-13 Thread Nick Desaulniers
-by: Nick Desaulniers --- arch/arm64/kernel/vdso32/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/kernel/vdso32/Makefile b/arch/arm64/kernel/vdso32/Makefile index 789ad420f16b..7812717f8b79 100644 --- a/arch/arm64/kernel/vdso32/Makefile +++ b/arch/arm64

[PATCH v3] gcov: clang: drop support for clang-10 and older

2021-04-13 Thread Nick Desaulniers
/rGcdd683b516d147925212724b09ec6fb792a40041 Link: https://reviews.llvm.org/rG13a633b438b6500ecad9e4f936ebadf3411d0f44 Link: https://lkml.kernel.org/r/20210312224132.3413602-3-ndesaulni...@google.com Signed-off-by: Nick Desaulniers Suggested-by: Nathan Chancellor Acked-by: Peter Oberparleiter Reviewed-by: Nathan

Re: [PATCH] gcov: clang: fix clang-11+ build

2021-04-12 Thread Nick Desaulniers
had time to check this particular build! Reviewed-by: Nick Desaulniers Tested-by: Nick Desaulniers > and my kvmalloc() conversion patch is in akpm/linux-next. > I guess this should be folded into > > gcov: use kvmalloc() > > If desired, I can send a combined patch instead. >

Re: [PATCH] crypto: arm/curve25519 - Move '.fpu' after '.arch'

2021-04-09 Thread Nick Desaulniers
ested-by: Arnd Bergmann > Suggested-by: Jessica Clarke > Signed-off-by: Nathan Chancellor Great work tracking down that Debian was carrying patches! Thank you! I've run this through the same 3 assemblers. Reviewed-by: Nick Desaulniers Tested-by: Nick Desaulniers > --- > arch/arm/crypto

Re: [PATCH -next] lib: zstd: Make symbol 'HUF_compressWeights_wksp' static

2021-04-09 Thread Nick Terrell
> On Apr 8, 2021, at 8:09 PM, Miguel Ojeda > wrote: > > On Fri, Apr 9, 2021 at 2:20 AM Nick Desaulniers > wrote: >> >> Quite a few other functions are declared in a header, but I don't see >> any existing callers in tree. I wonder if the maintainer could

Re: [GIT PULL][PATCH v9 0/3] Update to zstd-1.4.10

2021-04-09 Thread Nick Terrell
What can I do to help get this merged? Cristoph, is this new patch series with the kernel wrapper API satisfactory? Best, Nick On Tue, Mar 30, 2021 at 3:45 PM Nick Terrell wrote: > > From: Nick Terrell > > Please pull from > > g...@github.com:terrelln/linux.git t

Re: static_branch/jump_label vs branch merging

2021-04-09 Thread Nick Desaulniers
ht help? Then if there were side effects but you forgot to inform the compiler that there were via an explicit volatile qualifier, and it performed the suggested merge, oh well. -- Thanks, ~Nick Desaulniers

Re: [PATCH v3 4/5] RISC-V: Add kdump support

2021-04-09 Thread Nick Kossifidis
Στις 2021-04-06 21:36, Alex Ghiti έγραψε: + /* Switch to physical addressing */ + la s4, 1f + sub s4, s4, s3 + csrwstvec, s4 + csrwsptbr, zero satp is used everywhere instead of sptbr. And maybe you could CSR_ naming, like you did in

Re: [PATCH v3 2/5] RISC-V: Add kexec support

2021-04-09 Thread Nick Kossifidis
Στις 2021-04-06 21:38, Alex Ghiti έγραψε: Le 4/5/21 à 4:57 AM, Nick Kossifidis a écrit : + +/* Reserve a page for the control code buffer */ +#define KEXEC_CONTROL_PAGE_SIZE 4096 PAGE_SIZE instead ? Yup, I'll change it. +obj-${CONFIG_KEXEC}+= kexec_relocate.o machine_kexec.o

Re: [PATCH v3 3/5] RISC-V: Improve init_resources

2021-04-09 Thread Nick Kossifidis
Στις 2021-04-06 11:22, Geert Uytterhoeven έγραψε: Hi Nick, On Tue, Apr 6, 2021 at 10:11 AM Nick Kossifidis wrote: Hello Geert, Στις 2021-04-06 10:19, Geert Uytterhoeven έγραψε: > On Mon, Apr 5, 2021 at 10:57 AM Nick Kossifidis > wrote: >> * Kernel region is always present and w

Re: [PATCH v3 0/5] RISC-V: Add kexec/kdump support

2021-04-09 Thread Nick Kossifidis
Στις 2021-04-07 19:29, Rob Herring έγραψε: On Mon, Apr 05, 2021 at 11:57:07AM +0300, Nick Kossifidis wrote: This patch series adds kexec/kdump and crash kernel support on RISC-V. For testing the patches a patched version of kexec-tools is needed (still a work in progress) which can be found

Re: [PATCH -next] lib: zstd: Make symbol 'HUF_compressWeights_wksp' static

2021-04-08 Thread Nick Desaulniers
On Thu, Apr 8, 2021 at 5:55 AM Zhao Xuehui wrote: > > The symbol 'HUF_compressWeights_wksp' is not used outside of > huf_compress.c, so this commit marks it static. Reviewed-by: Nick Desaulniers Quite a few other functions are declared in a header, but I don't see any existing caller

[PATCH v2] gcov: re-fix clang-11+ support

2021-04-08 Thread Nick Desaulniers
Cc: sta...@vger.kernel.org #5.4+ Reported-by: Prasad Sodagudi Tested-by: Prasad Sodagudi Signed-off-by: Nick Desaulniers Reviewed-by: Nathan Chancellor --- Changes V1 -> V2: * Carried Nathan's reviewed-by tag. * Rebased on mainline, as per Andrew. * Left off patch 2/2 from the series ht

Re: [PATCH v4] lib/string: Introduce sysfs_streqcase

2021-04-08 Thread Nick Desaulniers
me more cases using strncasecmp to check the entire string > > > such as rtrs-clt-sysfs.c does. drivers/pnp/interface.c checks > > > "disable" command with strncasecmp but it would also work if the > > > command is "disable-wrong". > > > >

Re: [PATCH] x86/kernel: remove unneeded dead-store initialization

2021-04-07 Thread Nick Desaulniers
On Wed, Apr 7, 2021 at 12:07 PM Borislav Petkov wrote: > > On Wed, Apr 07, 2021 at 09:03:28PM +0200, Borislav Petkov wrote: > > On Wed, Apr 07, 2021 at 10:41:26AM -0700, Nick Desaulniers wrote: > > > You do have clang-tidy installed right? `which clang-tidy`? > >

Re: [PATCH 1/2] gcov: re-fix clang-11+ support

2021-04-07 Thread Nick Desaulniers
On Wed, Apr 7, 2021 at 2:21 PM Andrew Morton wrote: > > On Wed, 7 Apr 2021 11:54:55 -0700 Nick Desaulniers > wrote: > > > LLVM changed the expected function signature for > > llvm_gcda_emit_function() in the clang-11 release. Users of clang-11 or > > newer

Re: [PATCH] mailmap: update Andrey Konovalov's email address

2021-04-07 Thread Nick Desaulniers
err..with Marco's real email address this time On Wed, Apr 7, 2021 at 1:14 PM Nick Desaulniers wrote: > > (replying to > https://lore.kernel.org/lkml/ead0e9c32a2f70e0bde6f63b3b9470e0ef13d2ee.1616107969.git.andreyk...@google.com/) > > Just got the bounceback, RIP. :( > > M

Re: [PATCH] mailmap: update Andrey Konovalov's email address

2021-04-07 Thread Nick Desaulniers
(replying to https://lore.kernel.org/lkml/ead0e9c32a2f70e0bde6f63b3b9470e0ef13d2ee.1616107969.git.andreyk...@google.com/) Just got the bounceback, RIP. :( Marco is updating your epitaph. Acked-by: Nick Desaulniers

Re: [PATCH] lib/string: Introduce sysfs_streqcase

2021-04-07 Thread Nick Desaulniers
uld > work even-if the input is "min-wrongcommand". > > I found some more cases using strncasecmp to check the entire string > such as rtrs-clt-sysfs.c does. drivers/pnp/interface.c checks > "disable" command with strncasecmp but it would also work if the > comman

[PATCH 2/2] gcov: re-drop support for clang-10

2021-04-07 Thread Nick Desaulniers
LLVM changed the expected function signatures for llvm_gcda_emit_function() in the clang-11 release. Drop the older implementations and require folks to upgrade their compiler if they're interested in GCOV support. Signed-off-by: Nick Desaulniers --- kernel/gcov/clang.c | 40

[PATCH 1/2] gcov: re-fix clang-11+ support

2021-04-07 Thread Nick Desaulniers
Cc: sta...@vger.kernel.org #5.4+ Reported-by: Prasad Sodagudi Tested-by: Prasad Sodagudi Signed-off-by: Nick Desaulniers --- kernel/gcov/clang.c | 38 +- 1 file changed, 33 insertions(+), 5 deletions(-) diff --git a/kernel/gcov/clang.c b/kernel/gcov/clang.c

[PATCH 0/2] gcov: re-fix clang-11 support

2021-04-07 Thread Nick Desaulniers
in userspace. Nick Desaulniers (2): gcov: re-fix clang-11+ support gcov: re-drop support for clang-10 kernel/gcov/clang.c | 20 1 file changed, 4 insertions(+), 16 deletions(-) -- 2.31.1.295.g9ea45b61b8-goog

Re: arm-linux-gnueabi-ld: warning: orphan section `__cpuidle_method_of_table' from `arch/arm/mach-omap2/pm33xx-core.o' being placed in section `__cpuidle_method_of_table'

2021-04-07 Thread Nick Desaulniers
On Wed, Apr 7, 2021 at 11:23 AM Arnd Bergmann wrote: > > On Wed, Apr 7, 2021 at 8:07 PM Nick Desaulniers > wrote: > > > > On Wed, Apr 7, 2021 at 3:52 AM kernel test robot wrote: > > > > > > Hi Kees, > > > > > > FYI, the erro

Re: arm-linux-gnueabi-ld: warning: orphan section `__cpuidle_method_of_table' from `arch/arm/mach-omap2/pm33xx-core.o' being placed in section `__cpuidle_method_of_table'

2021-04-07 Thread Nick Desaulniers
ETHOD_OF_DECLARE` to create a static struct in such a section. Only arch/arm/mach-omap2/pm33xx-core.c uses that macro. > > --- > 0-DAY CI Kernel Test Service, Intel Corporation > https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org -- Thanks, ~Nick Desaulniers

Re: [PATCH] init: add support for zstd compressed modules

2021-04-07 Thread Nick Terrell
> On Apr 7, 2021, at 6:53 AM, Masahiro Yamada wrote: > > On Thu, Apr 1, 2021 at 4:21 AM Nick Terrell wrote: >> >> >> >>> On Mar 31, 2021, at 10:48 AM, Oleksandr Natalenko >>> wrote: >>> >>> Hello. >&g

Re: [PATCH] x86/kernel: remove unneeded dead-store initialization

2021-04-07 Thread Nick Desaulniers
gt; I get all kinds of missing python scripts: > FileNotFoundError: [Errno 2] No such file or directory: 'clang-tidy' You do have clang-tidy installed right? `which clang-tidy`? -- Thanks, ~Nick Desaulniers

[PATCH] MIPS: select ARCH_KEEP_MEMBLOCK unconditionally

2021-04-07 Thread Nick Desaulniers
BLOCK if DEBUG_KERNEL to enable sysfs memblock debug") Cc: Masahiro Yamada Reported-by: Guenter Roeck Signed-off-by: Nick Desaulniers --- arch/mips/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig index e9893cd34992..702648f60

Re: Usage of CXX in tools directory

2021-04-06 Thread Nick Desaulniers
> If we have HOSTCXX why not have a CXX in toplevel Makefile? > > In "tools: Factor Clang, LLC and LLVM utils definitions" (see [3]) I > did some factor-ing. > > For the records: Here Linus Git is my base. > > Ideas? > > Thanks. > > Regards, > - Sedat - >

Re: [PATCH v3 3/5] RISC-V: Improve init_resources

2021-04-06 Thread Nick Kossifidis
Hello Geert, Στις 2021-04-06 10:19, Geert Uytterhoeven έγραψε: Hi Nick, Thanks for your patch! On Mon, Apr 5, 2021 at 10:57 AM Nick Kossifidis wrote: * Kernel region is always present and we know where it is, no need to look for it inside the loop, just ignore it like the rest

[PATCH v3 1/5] RISC-V: Add EM_RISCV to kexec UAPI header

2021-04-05 Thread Nick Kossifidis
Add RISC-V to the list of supported kexec architecturs, we need to add the definition early-on so that later patches can use it. EM_RISCV is 243 as per ELF psABI specification here: https://github.com/riscv/riscv-elf-psabi-doc/blob/master/riscv-elf.md Signed-off-by: Nick Kossifidis --- include

[PATCH v3 3/5] RISC-V: Improve init_resources

2021-04-05 Thread Nick Kossifidis
-off-by: Geert Uytterhoeven Signed-off-by: Nick Kossifidis --- arch/riscv/kernel/setup.c | 90 --- 1 file changed, 46 insertions(+), 44 deletions(-) diff --git a/arch/riscv/kernel/setup.c b/arch/riscv/kernel/setup.c index e85bacff1..030554bab 100644 --- a/arch

[PATCH v3 5/5] RISC-V: Add crash kernel support

2021-04-05 Thread Nick Kossifidis
From: Nick Kossifidis This patch allows Linux to act as a crash kernel for use with kdump. Userspace will let the crash kernel know about the memory region it can use through linux,usable-memory property on the /memory node (overriding its reg property), and about the memory region where the elf

[PATCH v3 2/5] RISC-V: Add kexec support

2021-04-05 Thread Nick Kossifidis
. * Compatibility with HOTPLUG_CPU for SMP * Small cleanups Signed-off-by: Nick Kossifidis --- arch/riscv/Kconfig | 15 +++ arch/riscv/include/asm/kexec.h | 47 arch/riscv/kernel/Makefile | 5 + arch/riscv/kernel/kexec_relocate.S | 156

[PATCH v3 4/5] RISC-V: Add kdump support

2021-04-05 Thread Nick Kossifidis
inor cleanups and re-base Signed-off-by: Nick Kossifidis --- arch/riscv/include/asm/elf.h| 6 +++ arch/riscv/include/asm/kexec.h | 19 --- arch/riscv/kernel/Makefile | 2 +- arch/riscv/kernel/crash_save_regs.S | 56 + arch/riscv/kernel/kexec_rel

[PATCH v3 0/5] RISC-V: Add kexec/kdump support

2021-04-05 Thread Nick Kossifidis
on /memory instead of a new binding * USe a reserved-memory node for ELF core header Nick Kossifidis (5): RISC-V: Add EM_RISCV to kexec UAPI header RISC-V: Add kexec support RISC-V: Improve init_resources RISC-V: Add kdump support RISC-V: Add crash kernel support arch/riscv/Kconfig

Re: [PATCH] lib/string: Introduce sysfs_streqcase

2021-04-02 Thread Nick Desaulniers
if (!*s1 && *s2 == '\n' && !s2[1]) > + return true; > + if (*s1 == '\n' && !s1[1] && !*s2) > + return true; > + return false; > +} > +EXPORT_SYMBOL(sysfs_streqcase); This should be declared in include/linux/string.h in order for others to use this (as 0day bot notes). > + > /** > * match_string - matches given string in an array > * @array: array of strings > -- > 2.25.1 > -- Thanks, ~Nick Desaulniers

Re: [PATCH] usb: isp1301-omap: Add missing gpiod_add_lookup_table function

2021-04-01 Thread Nick Desaulniers
d ("usb: isp1301-omap: Convert to use GPIO descriptors") > Link: https://github.com/ClangBuiltLinux/linux/issues/1325 Looks consistent to me with other callers of gpiod_add_lookup_table from .init_machine callbacks. Reviewed-by: Nick Desaulniers > --- > arch/arm/mach-omap1/board

Re: [PATCH] ARM: OMAP: Fix use of possibly uninitialized irq variable

2021-04-01 Thread Nick Desaulniers
^ > = 0 > 1 warning generated. Ooh, yeah if cpu_is_omap15xx() then irq is unused uninitialized; I don't see any INT_1610_WAKE_UP_REQ-equlivalent for INT_15XX_WAKE_UP_REQ. Ok, LGTM. Reviewed-by: Nick Desaulniers I agree with Nathan on the Fixes tag. > > The p

Re: [PATCH] ARM: OMAP1: ams-delta: remove unused function ams_delta_camera_power

2021-04-01 Thread Nick Desaulniers
[-Wunused-function] > static int ams_delta_camera_power(struct device *dev, int power) >^ > 1 warning generated. > > The soc_camera support was dropped without removing > ams_delta_camera_power() function, making it unused. > > Signed-off-by: Maciej Falkowski

Re: [PATCH] init: add support for zstd compressed modules

2021-04-01 Thread Nick Terrell
f threads used. `-T1` (or not specifying `-T`) will produce the same output as `-T0`. `-T0` will be faster for large files (at the default level, multiple jobs will be spawned for files > 8MB), and be just as fast as `-T1` for smaller files. Best, Nick > I also saw that M

Re: [PATCH v2 2/3] kbuild: check the minimum assembler version in Kconfig

2021-03-31 Thread Nick Desaulniers
# version here. It is the same as the clang > > version, and > > + # it has been already checked by > > scripts/cc-version.sh. > > + echo LLVM 0 > > + exit 0 > > + fi > > + shift > > + done > > +} > > + > > +check_integrated_as "$@" > > + > > +orig_args="$@" > > + > > +# Get the first line of the --version output. > > +IFS=' > > +' > > +set -- $(LC_ALL=C "$@" -Wa,--version -c -x assembler /dev/null -o > > /dev/null 2>&1) > > + > > +# Split the line on spaces. > > +IFS=' ' > > +set -- $1 > > + > > +min_tool_version=$(dirname $0)/min-tool-version.sh > > + > > +if [ "$1" = GNU -a "$2" = assembler ]; then > > + shift $(($# - 1)) > > + version=$1 > > + min_version=$($min_tool_version binutils) > > + name=GNU > > +else > > + echo "$orig_args: unknown assembler invoked" >&2 > > + exit 1 > > +fi > > + > > +# Some distributions append a package release number, as in 2.34-4.fc32 > > +# Trim the hyphen and any characters that follow. > > +version=${version%-*} > > + > > +cversion=$(get_canonical_version $version) > > +min_cversion=$(get_canonical_version $min_version) > > + > > +if [ "$cversion" -lt "$min_cversion" ]; then > > + echo >&2 "***" > > + echo >&2 "*** Assembler is too old." > > + echo >&2 "*** Your $name assembler version:$version" > > + echo >&2 "*** Minimum $name assembler version: $min_version" > > + echo >&2 "***" > > + exit 1 > > +fi > > + > > +echo $name $cversion > > -- > > 2.27.0 > > > > -- -- Thanks, ~Nick Desaulniers

Re: [PATCH] blk-mq: fix alignment mismatch.

2021-03-31 Thread Nick Desaulniers
fair but small amount of churn IMO; but if Jens is not opposed it seems fine? -- Thanks, ~Nick Desaulniers

Re: [PATCH] init: add support for zstd compressed modules

2021-03-31 Thread Nick Terrell
> On Mar 31, 2021, at 10:48 AM, Oleksandr Natalenko > wrote: > > Hello. > > On Wed, Mar 31, 2021 at 05:39:25PM +, Nick Terrell wrote: >> >> >>> On Mar 30, 2021, at 4:50 AM, Oleksandr Natalenko >>> wrote: >>> >>> On Tu

Re: [PATCH 11/13] kconfig: do not use allnoconfig_y option

2021-03-31 Thread Nick Desaulniers
> > --- a/scripts/kconfig/Makefile > > +++ b/scripts/kconfig/Makefile > > @@ -102,7 +102,8 @@ configfiles=$(wildcard $(srctree)/kernel/configs/$@ > > $(srctree)/arch/$(SRCARCH)/c > > > > PHONY += tinyconfig > > tinyconfig: > > - $(Q)$(MAKE) -f $(srctree)/Makefile allnoconfig tiny.config > > + $(Q)KCONFIG_ALLCONFIG=kernel/configs/tiny-base.config $(MAKE) -f > > $(srctree)/Makefile allnoconfig > > + $(Q)$(MAKE) -f $(srctree)/Makefile tiny.config > > > > # CHECK: -o cache_dir= working? > > PHONY += testconfig > > -- > > 2.27.0 > > -- Thanks, ~Nick Desaulniers

Re: [PATCH 9/9] kbuild: remove CONFIG_MODULE_COMPRESS

2021-03-31 Thread Nick Desaulniers
DULE_SIG_SHA512 > > -config MODULE_COMPRESS The top level Makefile has comments and code that refer to this choice which is now removed. I think you'll want to fix that up in this change as well? Ah, patch 7 in the series does that: https://lore.kernel.org/linux-kbuild/20210331133811.3221540-7-m

Re: [PATCH 5/9] kbuild: rename extmod-prefix to extmod_prefix

2021-03-31 Thread Nick Desaulniers
On Wed, Mar 31, 2021 at 6:38 AM Masahiro Yamada wrote: > > This seems to be useful in sub-make as well. As a preparation of > exporting it, rename extmod-prefix to extmod_prefix because exported > variables cannot contain hyphens. > > Signed-off-by: Masahiro Yamada Reviewed-by

Re: [PATCH] x86/kernel: remove unneeded dead-store initialization

2021-03-31 Thread Nick Desaulniers
is identical before and after this change. > > No functional change. No change to object code. Reviewed-by: Nick Desaulniers Looks like this is from when this code was introduced in commit 0d55ba46bfbe ("x86/cacheinfo: Move cacheinfo sysfs code to generic infrastructure") though this

Re: [PATCH] init: add support for zstd compressed modules

2021-03-31 Thread Nick Terrell
, which will use a window size of up to 2MB, to match the XZ command. Note that if the file is smaller than the window size, it will be shrunk to the smallest power of two at least as large as the file. Best, Nick Terrell >> + endif # CONFIG_MODULE_COMPRESS_ZSTD >> endif # CONFIG_MODUL

[PATCH v9 2/3] lib: zstd: Add decompress_sources.h for decompress_unzstd

2021-03-30 Thread Nick Terrell
From: Nick Terrell Adds decompress_sources.h which includes every .c file necessary for zstd decompression. This is used in decompress_unzstd.c so the internal structure of the library isn't exposed. This allows us to upgrade the zstd library version without modifying any callers. Instead we

[PATCH v9 1/3] lib: zstd: Add kernel-specific API

2021-03-30 Thread Nick Terrell
From: Nick Terrell This patch: - Moves `include/linux/zstd.h` -> `include/linux/zstd_lib.h` - Updates modified zstd headers to yearless copyright - Adds a new API in `include/linux/zstd.h` that is functionally equivalent to the in-use subset of the current API. Functions are renamed to av

[GIT PULL][PATCH v9 0/3] Update to zstd-1.4.10

2021-03-30 Thread Nick Terrell
From: Nick Terrell Please pull from g...@github.com:terrelln/linux.git tags/v9-zstd-1.4.10 to get these changes. Alternatively the patchset is included. This patchset upgrades the zstd library to the latest upstream release. The current zstd version in the kernel is a modified version

Re: [PATCH v3 11/17] riscv: Convert to GENERIC_CMDLINE

2021-03-29 Thread Nick Kossifidis
Στις 2021-03-26 17:26, Rob Herring έγραψε: On Fri, Mar 26, 2021 at 8:20 AM Christophe Leroy wrote: Le 26/03/2021 à 15:08, Andreas Schwab a écrit : > On Mär 26 2021, Christophe Leroy wrote: > >> diff --git a/arch/riscv/kernel/setup.c b/arch/riscv/kernel/setup.c >> index

[PATCH v2] ARM: kprobes: test-thumb: fix for LLVM_IAS=1

2021-03-29 Thread Nick Desaulniers
-by: Nick Desaulniers --- See: https://lore.kernel.org/linux-arm-kernel/CAMj1kXE5uw4+zV3JVpfA2drOD5TZVMs5a_E5wrrnzjEYc=e...@mail.gmail.com/ for what I'd consider V1. The previous issues with .w suffixes have been fixed or have fixes pending in LLVM: * BL+DBG: https://reviews.llvm.org/D97236 * ORN

Re: [PATCH v2] ARM: kprobes: rewrite test-[arm|thumb].c in UAL

2021-03-29 Thread Nick Desaulniers
On Fri, Jan 29, 2021 at 1:40 AM Ard Biesheuvel wrote: > > On Fri, 29 Jan 2021 at 01:22, Nick Desaulniers > wrote: > > > > > On Thu, 28 Jan 2021 at 20:34, Nick Desaulniers > > > wrote: > > > > + TEST_RX("tbh[pc, r",7, (9f-(1f

Re: [PATCH 2/3] riscv: Workaround mcount name prior to clang-13

2021-03-29 Thread Nick Desaulniers
ang in mount.S and recordmcount.pl. > > Cc: sta...@vger.kernel.org > Link: https://github.com/ClangBuiltLinux/linux/issues/1331 > Signed-off-by: Nathan Chancellor Thanks for keeping this alive on clang-10, and resolving it for future releases! Reviewed-by: Nick Desaulniers > ---

Re: [PATCH] lockdep: address clang -Wformat warning printing for %hd

2021-03-29 Thread Nick Desaulniers
er, there is really no point in printing the number as a 16-bit > 'short' rather than either an 8-bit or 32-bit number, so just change > it to a normal %d. Thanks for the patch! Reviewed-by: Nick Desaulniers > > Fixes: de8f5e4f2dc1 ("lockdep: Introduce wait-type checks") >

Re: [PATCH v8 1/3] lib: zstd: Add kernel-specific API

2021-03-28 Thread Nick Terrell
ollowing correction seems to > be needed: Thanks Oleksandr! Looks like f2fs has been updated to use ZSTD_maxCLevel() since the first version of these patches. I'll put up a new version shortly with the fix, and update my test suite to build f2fs and other users as modules, so it can catch this.

Re: [PATCH v8 3/3] lib: zstd: Upgrade to latest upstream zstd version 1.4.10

2021-03-26 Thread Nick Terrell
On Fri, Mar 26, 2021 at 3:02 PM kernel test robot wrote: > > Hi Nick, > > Thank you for the patch! Perhaps something to improve: > > [auto build test WARNING on cryptodev/master] > [also build test WARNING on kdave/for-next f2fs/dev-test linus/master > v5.12-rc4 next-2

[PATCH v8 2/3] lib: zstd: Add decompress_sources.h for decompress_unzstd

2021-03-26 Thread Nick Terrell
From: Nick Terrell Adds decompress_sources.h which includes every .c file necessary for zstd decompression. This is used in decompress_unzstd.c so the internal structure of the library isn't exposed. This allows us to upgrade the zstd library version without modifying any callers. Instead we

[PATCH v8 0/3] Update to zstd-1.4.10

2021-03-26 Thread Nick Terrell
From: Nick Terrell Please pull from g...@github.com:terrelln/linux.git tags/v8-zstd-1.4.10 to get these changes. Alternatively the patchset is included. This patchset upgrades the zstd library to the latest upstream release. The current zstd version in the kernel is a modified version

[PATCH v8 1/3] lib: zstd: Add kernel-specific API

2021-03-26 Thread Nick Terrell
From: Nick Terrell This patch: - Moves `include/linux/zstd.h` -> `include/linux/zstd_lib.h` - Adds a new API in `include/linux/zstd.h` that is functionally equivalent to the in-use subset of the current API. Functions are renamed to avoid symbol collisions with zstd, to make it cl

High-quality Original IELTS certificate

2021-03-26 Thread Nick
Greetings, We are the leading provider of IELTS CERTIFICATE ONLINE. You don’t have to worry if you can’t pass in the IELTS exam, for we are here to issue you this certificate without constrain. We make it possible for you to get the IELTS certificate even without appearing for the exam or even

Re: drivers/gpu/drm/i915/gvt/gtt.c:267:19: error: unused function 'get_pt_type'

2021-03-24 Thread Nick Desaulniers
On Wed, Mar 24, 2021 at 2:12 AM Zhenyu Wang wrote: > > On 2021.03.23 15:15:29 -0700, Nick Desaulniers wrote: > > On Fri, Mar 19, 2021 at 11:45 PM kernel test robot wrote: > > > > > > Hi Nick, > > > > > > FYI, the error/warning still remains. >

Re: drivers/gpu/drm/i915/gvt/gtt.c:267:19: error: unused function 'get_pt_type'

2021-03-23 Thread Nick Desaulniers
On Fri, Mar 19, 2021 at 11:45 PM kernel test robot wrote: > > Hi Nick, > > FYI, the error/warning still remains. > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: 1c273e10bc0cc7efb933e0ca1

Re: [PATCH] irqchip/gic-v3: fix OF_BAD_ADDR error handling

2021-03-23 Thread Nick Desaulniers
ys_base == (phys_addr_t)OF_BAD_ADDR) { > ret = -ENXIO; > goto err_free_mbi; > } > -- > 2.29.2 > -- Thanks, ~Nick Desaulniers

Re: [PATCH v3 09/17] treewide: Change list_sort to use const pointers

2021-03-23 Thread Nick Desaulniers
Integrity (CFI) checking. > > Instead of removing the consts, this change defines the > list_cmp_func_t type and changes the comparison function types of > all list_sort() callers to use const pointers, thus avoiding type > mismatches. > > Suggested-by: Nick Desaulniers > Signed-off

Re: [PATCH] scripts: stable: add script to validate backports

2021-03-23 Thread Nick Desaulniers
6578b963f $ Should it have found a7889c6320b9 and 773e0c402534? Perhaps `git log --grep=` should be used instead? I thought `git grep` only greps files in the archive, not commit history? -- Thanks, ~Nick Desaulniers

  1   2   3   4   5   6   7   8   9   10   >