linux-kernel - Re: + mm-vmalloc-convert-vread-to-vread_iter-v8.patch added to mm-unstable branch

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:   Thu, 23 Mar 2023 22:01:11 +0000
From:   Lorenzo Stoakes <lstoakes@...il.com>
To:     linux-kernel@...r.kernel.org
Cc:     mm-commits@...r.kernel.org, akpm@...ux-foundation.org,
        sfr@...b.auug.org.au
Subject: Re: + mm-vmalloc-convert-vread-to-vread_iter-v8.patch added to
 mm-unstable branch

On Thu, Mar 23, 2023 at 02:55:10PM -0700, Andrew Morton wrote:
>
> The patch titled
>      Subject: mm-vmalloc-convert-vread-to-vread_iter-v8
> has been added to the -mm mm-unstable branch.  Its filename is
>      mm-vmalloc-convert-vread-to-vread_iter-v8.patch
>
> This patch will shortly appear at
>      https://git.kernel.org/pub/scm/linux/kernel/git/akpm/25-new.git/tree/patches/mm-vmalloc-convert-vread-to-vread_iter-v8.patch
>
> This patch will later appear in the mm-unstable branch at
>     git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm
>
> Before you just go and hit "reply", please:
>    a) Consider who else should be cc'ed
>    b) Prefer to cc a suitable mailing list as well
>    c) Ideally: find the original patch on the mailing list and do a
>       reply-to-all to that, adding suitable additional cc's
>
> *** Remember to use Documentation/process/submit-checklist.rst when testing your code ***
>
> The -mm tree is included into linux-next via the mm-everything
> branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm
> and is updated there every 2-3 working days
>
> ------------------------------------------------------
> From: Lorenzo Stoakes <lstoakes@...il.com>
> Subject: mm-vmalloc-convert-vread-to-vread_iter-v8
> Date: Thu, 23 Mar 2023 10:15:19 +0000
>
> make zero_iter() static, move uio.h inclusion to vmalloc.h
>
> Link: https://lkml.kernel.org/r/8506cbc667c39205e65a323f750ff9c11a463798.1679566220.git.lstoakes@gmail.com
> Signed-off-by: Lorenzo Stoakes <lstoakes@...il.com>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> ---
>
>  include/linux/vmalloc.h |    2 +-
>  mm/vmalloc.c            |    3 +--
>  2 files changed, 2 insertions(+), 3 deletions(-)
>
> --- a/include/linux/vmalloc.h~mm-vmalloc-convert-vread-to-vread_iter-v8
> +++ a/include/linux/vmalloc.h
> @@ -9,12 +9,12 @@
>  #include <asm/page.h>		/* pgprot_t */
>  #include <linux/rbtree.h>
>  #include <linux/overflow.h>
> +#include <linux/uio.h>
>
>  #include <asm/vmalloc.h>
>
>  struct vm_area_struct;		/* vma defining user mapping in mm_types.h */
>  struct notifier_block;		/* in notifier.h */
> -struct iov_iter;		/* in uio.h */
>
>  /* bits in flags of vmalloc's vm_struct below */
>  #define VM_IOREMAP		0x00000001	/* ioremap() and friends */
> --- a/mm/vmalloc.c~mm-vmalloc-convert-vread-to-vread_iter-v8
> +++ a/mm/vmalloc.c
> @@ -33,7 +33,6 @@
>  #include <linux/compiler.h>
>  #include <linux/memcontrol.h>
>  #include <linux/llist.h>
> -#include <linux/uio.h>

The above changes accidentally undoes the fix for sparc64 that Stephen
Rothwell provided (I have incorporated that in my local repo so if there
are any future respins they will include it going forward).

Could we revert the above but keep the static declaration change below?
Thanks!

>  #include <linux/bitops.h>
>  #include <linux/rbtree_augmented.h>
>  #include <linux/overflow.h>
> @@ -3446,7 +3445,7 @@ EXPORT_SYMBOL(vmalloc_32_user);
>   *
>   * Returns the number of zeroed bytes.
>   */
> -size_t zero_iter(struct iov_iter *iter, size_t count)
> +static size_t zero_iter(struct iov_iter *iter, size_t count)
>  {
>  	size_t remains = count;
>
> _
>
> Patches currently in -mm which might be from lstoakes@...il.com are
>
> mm-prefer-xxx_page-alloc-free-functions-for-order-0-pages.patch
> mm-refactor-do_fault_around.patch
> mm-pefer-fault_around_pages-to-fault_around_bytes.patch
> maintainers-add-myself-as-vmalloc-reviewer.patch
> mm-remove-unused-vmf_insert_mixed_prot.patch
> mm-remove-vmf_insert_pfn_xxx_prot-for-huge-page-table-entries.patch
> drm-ttm-remove-comment-referencing-now-removed-vmf_insert_mixed_prot.patch
> fs-proc-kcore-avoid-bounce-buffer-for-ktext-data.patch
> fs-proc-kcore-convert-read_kcore-to-read_kcore_iter.patch
> iov_iter-add-copy_page_to_iter_nofault.patch
> mm-vmalloc-convert-vread-to-vread_iter.patch
> mm-vmalloc-convert-vread-to-vread_iter-v8.patch
> mm-mmap-vma_merge-further-improve-prev-next-vma-naming.patch
> mm-mmap-vma_merge-fold-curr-next-assignment-logic.patch
> mm-mmap-vma_merge-explicitly-assign-res-vma-extend-invariants.patch
> mm-mmap-vma_merge-init-cleanup-be-explicit-about-the-non-mergeable-case.patch
>