Re: [PATCH v2] Revert "mm/page_alloc: fix memmap_init_zone pageblo ...
Linus Torvalds writes: (Summary) On Thu, Mar 15, 2018 at 11:21 AM, Michal Hocko <mhocko@kernel.org> wrote: [...] has a valid struct page backing it. No, it really just says the latter. There...
View ArticleRe: [PATCH RFC rebase 2/9] powerpc: Use barrier_nospec in copy_fro ...
Linus Torvalds writes: (Summary) On Thu, Mar 15, 2018 at 12:15 PM, Michal Suchanek <msuchanek@suse.de> wrote: [...] __gu_err; \ Is the above really correct? The barrier is *before* the...
View ArticleRe: [PATCH v4 1/2] kernel.h: Introduce const_max() for VLA removal
Linus Torvalds writes: (Summary) On Thu, Mar 15, 2018 at 12:47 PM, Kees Cook <keescook@chromium.org> wrote: [...] explicitly cast to size_t. Ugh, I really hate this. Ugh, I really hate this. It...
View ArticleRe: [PATCH v4 1/2] kernel.h: Introduce const_max() for VLA removal
Linus Torvalds writes: (Summary) How does that look? Ok, that alleviates my worry about one class of insane behavior, but it does raise a few other questions: it does raise a few other questions: -...
View ArticleRe: [PATCH v4 1/2] kernel.h: Introduce const_max() for VLA removal
Linus Torvalds writes: (Summary) I think all the existing users are of type "max_t()" anyway due to the very same issue, no? very same issue, no? At least if there's an explicit type like 'size_t',...
View ArticleRe: [PATCH RFC rebase 3/9] powerpc/64: Use barrier_nospec in sysca ...
Linus Torvalds writes: (Summary) Even if you were to do an expensive serialization before the branch, the branch will still predict after the serialization. the serialization. The thing is, it doesn't...
View ArticleRe: [PATCH 8/9] x86/dumpstack: Save first regs set for the executi ...
Linus Torvalds writes: (Summary) On Fri, Mar 16, 2018 at 4:48 AM, Borislav Petkov <bp@alien8.de> The "address" register contains the CR2 value as it was read *very* early in the page fault case,...
View ArticleRe: [PATCH v5 0/2] Remove false-positive VLAs when using max()
Linus Torvalds writes: (Summary) And no, using "-Werror=3Dvla-larger-than=3D1" doesn't work either, because the moronic compiler continues to think that "vla" is about the _type_, not the code: _type_,...
View ArticleRe: [PATCH v5 0/2] Remove false-positive VLAs when using max()
Linus Torvalds writes: (Summary) - */ - -/** - * min_t - return minimum of two values, using the specified type - * @type: data type to use - * @x: first value - * @y: second value - */ -#define...
View ArticleRe: [PATCH -next 00/22] remove in-kernel syscall invocations (part ...
Linus Torvalds writes: (Summary) That sucks, because you have two choices: - either pointlessly generate extra code for the 200+ system calls that are *not* used by the kernel that are *not* used by...
View ArticleRe: [PATCH v5 0/2] Remove false-positive VLAs when using max()
Linus Torvalds writes: (Summary) On Fri, Mar 16, 2018 at 1:03 PM, Miguel Ojeda<miguel.ojeda.sandonis@gmail.com> wrote:[...]https://godbolt.org/ Well, my *test* code works on that one and -Wvla...
View ArticleRe: [PATCH v5 0/2] Remove false-positive VLAs when using max()
Linus Torvalds writes: (Summary) wrote: [...] That's C99, straight from N1256.pdf (C99-TC3)... I checked C90, since the error is I checked C90, since the error is ISO C90 forbids variable length array...
View ArticleRe: [PATCH v5 0/2] Remove false-positive VLAs when using max()
Linus Torvalds writes: On Fri, Mar 16, 2018 at 1:14 PM, Linus Torvalds<torvalds@linux-foundation.org> wrote:[...] __builtin_choose_expr() simply didn't exist back then. No, that goes further...
View ArticleRe: [PATCH v5 0/2] Remove false-positive VLAs when using max()
Linus Torvalds writes: (Summary) wrote: [...] That is probably missing optimisation. Looking at the code is definitely an option. Looking at the code is definitely an option. In fact, instead of...
View ArticleRe: [PATCH] mm: add config for readahead window
Linus Torvalds writes: (Summary) wrote:[...] IIRC, Linus was against it. I have never seen any valid situation that wasn't tuning for one odd machine, usually with a horribly crappy disk setup and very...
View ArticleRe: what the hell is compat_sys_x86_waitpid() for?
Linus Torvalds writes: (Summary) wrote: [...] it was (and always had been) 100% pointless. That long long predates Dominik's patches, though. That long long predates Dominik's patches, though. It...
View ArticleRe: [PATCH v5 0/2] Remove false-positive VLAs when using max()
Linus Torvalds writes: (Summary) wrote: [...] =E2=80=99:[...] =E2=80=98__builtin_choose_expr=E2=80=99 not a constant Ok, so it really looks like that same "__builtin_constant_p() doesn't return a...
View ArticleRe: [RFC PATCH 3/6] fs: provide generic compat_sys_p{read,write}64 ...
Linus Torvalds writes: (Summary) Honestly, I think the patches like this are disgusting: Honestly, I think the patches like this are disgusting: On Sun, Mar 18, 2018 at 9:10 AM, Dominik...
View ArticleRe: [RFC PATCH 4/6] mm: provide generic compat_sys_readahead() imp ...
Linus Torvalds writes: (Summary) This is still too ugly to live. What *may* be acceptable is if architectures defined something like this: What *may* be acceptable is if architectures defined something...
View ArticleRe: [RFC PATCH 2/6] fs: provide a generic compat_sys_truncate64() ...
Linus Torvalds writes: (Summary) On Sun, Mar 18, 2018 at 9:10 AM, Dominik Brodowski<linux@dominikbrodowski.net> wrote:[...] +#endif /* __ARCH_WANT_COMPAT_SYS_TRUNCATE64 */ This really screams out...
View Article