[088/166] mm: use fallthrough;

Message ID	20200407030839.2vw2YSSgQ%akpm@linux-foundation.org (mailing list archive)
State	New, archived
Headers	show Return-Path: <SRS0=tmHZ=5X=kvack.org=owner-linux-mm@kernel.org> DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E0B31217D8 Date: Mon, 06 Apr 2020 20:08:39 -0700 From: Andrew Morton <akpm@linux-foundation.org> To: akpm@linux-foundation.org, gustavo@embeddedor.com, joe@perches.com, linux-mm@kvack.org, mm-commits@vger.kernel.org, torvalds@linux-foundation.org Subject: [patch 088/166] mm: use fallthrough; Message-ID: <20200407030839.2vw2YSSgQ%akpm@linux-foundation.org> In-Reply-To: <20200406200254.a69ebd9e08c4074e41ddebaf@linux-foundation.org> User-Agent: s-nail v14.8.16 Sender: owner-linux-mm@kvack.org Precedence: bulk
Series	[001/166] mm, memcg: bypass high reclaim iteration for cgroup hierarchy root \| expand [001/166] mm, memcg: bypass high reclaim iteration for cgroup hierarchy root [002/166] mm: don't prepare anon_vma if vma has VM_WIPEONFORK [003/166] Revert "mm/rmap.c: reuse mergeable anon_vma as parent when fork" [004/166] mm: set vm_next and vm_prev to NULL in vm_area_dup() [005/166] mm/vma: add missing VMA flag readable name for VM_SYNC [006/166] mm/vma: make vma_is_accessible() available for general use [007/166] mm/vma: replace all remaining open encodings with is_vm_hugetlb_page() [008/166] mm/vma: replace all remaining open encodings with vma_is_anonymous() [009/166] mm/vma: append unlikely() while testing VMA access permissions [010/166] mm/vmalloc: fix a typo in comment [011/166] mm: make it clear that gfp reclaim modifiers are valid only for sleepable allocations [012/166] mm/migrate.c: no need to check for i > start in do_pages_move() [013/166] mm/migrate.c: wrap do_move_pages_to_node() and store_status() [014/166] mm/migrate.c: check pagelist in move_pages_and_store_status() [015/166] mm/migrate.c: unify "not queued for migration" handling in do_pages_move() [016/166] mm/migrate.c: migrate PG_readahead flag [017/166] mm, shmem: add vmstat for hugepage fallback [018/166] mm, thp: track fallbacks due to failed memcg charges separately [019/166] include/linux/pagemap.h: optimise find_subpage for !THP [020/166] mm: remove CONFIG_TRANSPARENT_HUGE_PAGECACHE [021/166] mm/ksm.c: update get_user_pages() argument in comment [022/166] mm: code cleanup for MADV_FREE [023/166] mm: adjust shuffle code to allow for future coalescing [024/166] mm: use zone and order instead of free area in free_list manipulators [025/166] mm: add function __putback_isolated_page [026/166] mm: introduce Reported pages [027/166] virtio-balloon: pull page poisoning config out of free page hinting [028/166] virtio-balloon: add support for providing free page reports to host [029/166] mm/page_reporting: rotate reported pages to the tail of the list [030/166] mm/page_reporting: add budget limit on how many pages can be reported per pass [031/166] mm/page_reporting: add free page reporting documentation [032/166] virtio-balloon: switch back to OOM handler for VIRTIO_BALLOON_F_DEFLATE_ON_OOM [033/166] userfaultfd: wp: add helper for writeprotect check [034/166] userfaultfd: wp: hook userfault handler to write protection fault [035/166] userfaultfd: wp: add WP pagetable tracking to x86 [036/166] userfaultfd: wp: userfaultfd_pte/huge_pmd_wp() helpers [037/166] userfaultfd: wp: add UFFDIO_COPY_MODE_WP [038/166] mm: merge parameters for change_protection() [039/166] userfaultfd: wp: apply _PAGE_UFFD_WP bit [040/166] userfaultfd: wp: drop _PAGE_UFFD_WP properly when fork [041/166] userfaultfd: wp: add pmd_swp_uffd_wp() helpers [042/166] userfaultfd: wp: support swap and page migration [043/166] khugepaged: skip collapse if uffd-wp detected [044/166] userfaultfd: wp: support write protection for userfault vma range [045/166] userfaultfd: wp: add the writeprotect API to userfaultfd ioctl [046/166] userfaultfd: wp: enabled write protection in userfaultfd API [047/166] userfaultfd: wp: don't wake up when doing write protect [048/166] userfaultfd: wp: UFFDIO_REGISTER_MODE_WP documentation update [049/166] userfaultfd: wp: declare _UFFDIO_WRITEPROTECT conditionally [050/166] userfaultfd: selftests: refactor statistics [051/166] userfaultfd: selftests: add write-protect test [052/166] drivers/base/memory.c: drop section_count [053/166] drivers/base/memory.c: drop pages_correctly_probed() [054/166] mm/page_ext.c: drop pfn_present() check when onlining [055/166] mm/memory_hotplug.c: only respect mem= parameter during boot stage [056/166] mm/memory_hotplug.c: simplify calculation of number of pages in __remove_pages() [057/166] mm/memory_hotplug.c: cleanup __add_pages() [058/166] mm/sparse.c: introduce new function fill_subsection_map() [059/166] mm/sparse.c: introduce a new function clear_subsection_map() [060/166] mm/sparse.c: only use subsection map in VMEMMAP case [061/166] mm/sparse.c: add note about only VMEMMAP supporting sub-section hotplug [062/166] mm/sparse.c: move subsection_map related functions together [063/166] drivers/base/memory: rename MMOP_ONLINE_KEEP to MMOP_ONLINE [064/166] drivers/base/memory: map MMOP_OFFLINE to 0 [065/166] drivers/base/memory: store mapping between MMOP_ and string in an array [066/166] powernv/memtrace: always online added memory blocks [067/166] hv_balloon: don't check for memhp_auto_online manually [068/166] mm/memory_hotplug: unexport memhp_auto_online [069/166] mm/memory_hotplug: convert memhp_auto_online to store an online_type [070/166] mm/memory_hotplug: allow to specify a default online_type [071/166] mm/memory_hotplug.c: use __pfn_to_section() instead of open-coding [072/166] mm/shmem.c: distribute switch variables for initialization [073/166] mm/shmem.c: clean code by removing unnecessary assignment [074/166] mm: huge tmpfs: try to split_huge_page() when punching hole [075/166] mm: prevent a warning when casting void* -> enum [076/166] mm/zswap: allow setting default status, compressor and allocator in Kconfig [077/166] mm/compaction: add missing annotation for compact_lock_irqsave [078/166] mm/hugetlb: add missing annotation for gather_surplus_pages() [079/166] mm/mempolicy: add missing annotation for queue_pages_pmd() [080/166] mm/slub: add missing annotation for get_map() [081/166] mm/slub: add missing annotation for put_map() [082/166] mm/zsmalloc: add missing annotation for migrate_read_lock() [083/166] mm/zsmalloc: add missing annotation for migrate_read_unlock() [084/166] mm/zsmalloc: add missing annotation for pin_tag() [085/166] mm/zsmalloc: add missing annotation for unpin_tag() [086/166] mm: fix ambiguous comments for better code readability [087/166] mm/mm_init.c: clean code. Use BUILD_BUG_ON when comparing compile time constant [088/166] mm: use fallthrough; [089/166] include/linux/swapops.h: correct guards for non_swap_entry() [090/166] include/linux/memremap.h: remove stale comments [091/166] mm/dmapool.c: micro-optimisation remove unnecessary branch [092/166] mm: remove dummy struct bootmem_data/bootmem_data_t [093/166] fs/proc/inode.c: annotate close_pdeo() for sparse [094/166] proc: faster open/read/close with "permanent" files [095/166] proc: speed up /proc//statm [096/166] proc: inline vma_stop into m_stop [097/166] proc: remove m_cache_vma [098/166] proc: use ppos instead of m->version [099/166] seq_file: remove m->version [100/166] proc: inline m_next_vma into m_next [101/166] asm-generic: fix unistd_32.h generation format [102/166] kernel/extable.c: use address-of operator on section symbols [103/166] sparc,x86: vdso: remove meaningless undefining CONFIG_OPTIMIZE_INLINING [104/166] compiler: remove CONFIG_OPTIMIZE_INLINING entirely [105/166] compiler.h: fix error in BUILD_BUG_ON() reporting [106/166] MAINTAINERS: list the section entries in the preferred order [107/166] bitops: always inline sign extension helpers [108/166] lib/test_lockup: test module to generate lockups [109/166] lib/test_lockup.c: fix spelling mistake "iteraions" -> "iterations" [110/166] lib/test_lockup.c: add parameters for locking generic vfs locks [111/166] lib/bch.c: replace zero-length array with flexible-array member [112/166] lib/ts_bm.c: replace zero-length array with flexible-array member [113/166] lib/ts_fsm.c: replace zero-length array with flexible-array member [114/166] lib/ts_kmp.c: replace zero-length array with flexible-array member [115/166] lib/scatterlist: fix sg_copy_buffer() kerneldoc [116/166] lib: test_stackinit.c: XFAIL switch variable init tests [117/166] lib/stackdepot.c: check depot_index before accessing the stack slab [118/166] lib/stackdepot.c: build with -fno-builtin [119/166] kasan: stackdepot: move filter_irq_stacks() to stackdepot.c [120/166] percpu_counter: fix a data race at vm_committed_as [121/166] lib/test_bitmap.c: make use of EXP2_IN_BITS [122/166] lib/rbtree: fix coding style of assignments [123/166] lib/test_kmod.c: remove a NULL test [124/166] linux/bits.h: add compile time sanity check of GENMASK inputs [125/166] lib/list: prevent compiler reloads inside 'safe' list iteration [126/166] lib/dynamic_debug.c: use address-of operator on section symbols [127/166] checkpatch: remove email address comment from email address comparisons [128/166] checkpatch: check SPDX tags in YAML files [129/166] checkpatch: support "base-commit:" format [130/166] checkpatch: prefer fallthrough; over fallthrough comments [131/166] checkpatch: fix minor typo and mixed space+tab in indentation [132/166] checkpatch: fix multiple const types [133/166] checkpatch: add command-line option for TAB size [134/166] checkpatch: improve Gerrit Change-Id: test [135/166] checkpatch: check proper licensing of Devicetree bindings [136/166] checkpatch: avoid warning about uninitialized_var() [137/166] kselftest: introduce new epoll test case [138/166] fs/epoll: make nesting accounting safe for -rt kernel [139/166] fs/binfmt_elf.c: delete "loc" variable [140/166] fs/binfmt_elf.c: allocate less for static executable [141/166] fs/binfmt_elf.c: don't free interpreter's ELF pheaders on common path [142/166] samples/hw_breakpoint: drop HW_BREAKPOINT_R when reporting writes [143/166] samples/hw_breakpoint: drop use of kallsyms_lookup_name() [144/166] kallsyms: unexport kallsyms_lookup_name() and kallsyms_on_each_symbol() [145/166] reiserfs: clean up several indentation issues [146/166] kernel/kmod.c: fix a typo "assuems" -> "assumes" [147/166] gcov: gcc_4_7: replace zero-length array with flexible-array member [148/166] gcov: gcc_3_4: replace zero-length array with flexible-array member [149/166] kernel/gcov/fs.c: replace zero-length array with flexible-array member [150/166] init/Kconfig: clean up ANON_INODES and old IO schedulers options [151/166] kcov: cleanup debug messages [152/166] kcov: fix potential use-after-free in kcov_remote_start [153/166] kcov: move t->kcov assignments into kcov_start/stop [154/166] kcov: move t->kcov_sequence assignment [155/166] kcov: use t->kcov_mode as enabled indicator [156/166] kcov: collect coverage from interrupts [157/166] usb: core: kcov: collect coverage from usb complete callback [158/166] ubsan: add trap instrumentation option [159/166] ubsan: split "bounds" checker from other options [160/166] drivers/misc/lkdtm/bugs.c: add arithmetic overflow and array bounds checks [161/166] ubsan: check panic_on_warn [162/166] kasan: unset panic_on_warn before calling panic() [163/166] ubsan: include bug type in report header [164/166] lib/Kconfig.debug: fix a typo "capabilitiy" -> "capability" [165/166] ipc/mqueue.c: fix a brace coding style issue [166/166] ipc/shm.c: make compat_ksys_shmctl() static

Message ID

20200407030839.2vw2YSSgQ%akpm@linux-foundation.org (mailing list archive)

State

New, archived

Headers

DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E0B31217D8
Date: Mon, 06 Apr 2020 20:08:39 -0700
From: Andrew Morton <akpm@linux-foundation.org>
To: akpm@linux-foundation.org, gustavo@embeddedor.com, joe@perches.com,
 linux-mm@kvack.org, mm-commits@vger.kernel.org,
 torvalds@linux-foundation.org
Subject: [patch 088/166] mm: use fallthrough;
Message-ID: <20200407030839.2vw2YSSgQ%akpm@linux-foundation.org>
In-Reply-To: <20200406200254.a69ebd9e08c4074e41ddebaf@linux-foundation.org>
User-Agent: s-nail v14.8.16
Sender: owner-linux-mm@kvack.org
Precedence: bulk

Series

[001/166] mm, memcg: bypass high reclaim iteration for cgroup hierarchy root | expand

Commit Message

Andrew Morton April 7, 2020, 3:08 a.m. UTC

From: Joe Perches <joe@perches.com>
Subject: mm: use fallthrough;

Convert the various /* fallthrough */ comments to the pseudo-keyword
fallthrough;

Done via script:
https://lore.kernel.org/lkml/b56602fcf79f849e733e7b521bb0e17895d390fa.1582230379.git.joe@perches.com/

Link: http://lkml.kernel.org/r/f62fea5d10eb0ccfc05d87c242a620c261219b66.camel@perches.com
Signed-off-by: Joe Perches <joe@perches.com>
Reviewed-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 mm/gup.c            |    2 +-
 mm/hugetlb_cgroup.c |    6 +++---
 mm/ksm.c            |    3 +--
 mm/list_lru.c       |    2 +-
 mm/memcontrol.c     |    2 +-
 mm/mempolicy.c      |    3 ---
 mm/mmap.c           |    5 ++---
 mm/shmem.c          |    2 +-
 mm/zsmalloc.c       |    2 +-
 9 files changed, 11 insertions(+), 16 deletions(-)

--- a/mm/gup.c~mm-use-fallthrough
+++ a/mm/gup.c
@@ -1102,7 +1102,7 @@  retry:
 				goto retry;
 			case -EBUSY:
 				ret = 0;
-				/* FALLTHRU */
+				fallthrough;
 			case -EFAULT:
 			case -ENOMEM:
 			case -EHWPOISON:
--- a/mm/hugetlb_cgroup.c~mm-use-fallthrough
+++ a/mm/hugetlb_cgroup.c
@@ -467,14 +467,14 @@  static int hugetlb_cgroup_read_u64_max(s
 	switch (MEMFILE_ATTR(cft->private)) {
 	case RES_RSVD_USAGE:
 		counter = &h_cg->rsvd_hugepage[idx];
-		/* Fall through. */
+		fallthrough;
 	case RES_USAGE:
 		val = (u64)page_counter_read(counter);
 		seq_printf(seq, "%llu\n", val * PAGE_SIZE);
 		break;
 	case RES_RSVD_LIMIT:
 		counter = &h_cg->rsvd_hugepage[idx];
-		/* Fall through. */
+		fallthrough;
 	case RES_LIMIT:
 		val = (u64)counter->max;
 		if (val == limit)
@@ -514,7 +514,7 @@  static ssize_t hugetlb_cgroup_write(stru
 	switch (MEMFILE_ATTR(of_cft(of)->private)) {
 	case RES_RSVD_LIMIT:
 		rsvd = true;
-		/* Fall through. */
+		fallthrough;
 	case RES_LIMIT:
 		mutex_lock(&hugetlb_limit_mutex);
 		ret = page_counter_set_max(
--- a/mm/ksm.c~mm-use-fallthrough
+++ a/mm/ksm.c
@@ -2813,8 +2813,7 @@  static int ksm_memory_callback(struct no
 		 */
 		ksm_check_stable_tree(mn->start_pfn,
 				      mn->start_pfn + mn->nr_pages);
-		/* fallthrough */
-
+		fallthrough;
 	case MEM_CANCEL_OFFLINE:
 		mutex_lock(&ksm_thread_mutex);
 		ksm_run &= ~KSM_RUN_OFFLINE;
--- a/mm/list_lru.c~mm-use-fallthrough
+++ a/mm/list_lru.c
@@ -223,7 +223,7 @@  restart:
 		switch (ret) {
 		case LRU_REMOVED_RETRY:
 			assert_spin_locked(&nlru->lock);
-			/* fall through */
+			fallthrough;
 		case LRU_REMOVED:
 			isolated++;
 			nlru->nr_items--;
--- a/mm/memcontrol.c~mm-use-fallthrough
+++ a/mm/memcontrol.c
@@ -5813,7 +5813,7 @@  retry:
 		switch (get_mctgt_type(vma, addr, ptent, &target)) {
 		case MC_TARGET_DEVICE:
 			device = true;
-			/* fall through */
+			fallthrough;
 		case MC_TARGET_PAGE:
 			page = target.page;
 			/*
--- a/mm/mempolicy.c~mm-use-fallthrough
+++ a/mm/mempolicy.c
@@ -881,7 +881,6 @@  static void get_policy_nodemask(struct m
 
 	switch (p->mode) {
 	case MPOL_BIND:
-		/* Fall through */
 	case MPOL_INTERLEAVE:
 		*nodes = p->v.nodes;
 		break;
@@ -2066,7 +2065,6 @@  bool init_nodemask_of_mempolicy(nodemask
 		break;
 
 	case MPOL_BIND:
-		/* Fall through */
 	case MPOL_INTERLEAVE:
 		*mask =  mempolicy->v.nodes;
 		break;
@@ -2333,7 +2331,6 @@  bool __mpol_equal(struct mempolicy *a, s
 
 	switch (a->mode) {
 	case MPOL_BIND:
-		/* Fall through */
 	case MPOL_INTERLEAVE:
 		return !!nodes_equal(a->v.nodes, b->v.nodes);
 	case MPOL_PREFERRED:
--- a/mm/mmap.c~mm-use-fallthrough
+++ a/mm/mmap.c
@@ -1460,7 +1460,7 @@  unsigned long do_mmap(struct file *file,
 			 * with MAP_SHARED to preserve backward compatibility.
 			 */
 			flags &= LEGACY_MAP_MASK;
-			/* fall through */
+			fallthrough;
 		case MAP_SHARED_VALIDATE:
 			if (flags & ~flags_mask)
 				return -EOPNOTSUPP;
@@ -1487,8 +1487,7 @@  unsigned long do_mmap(struct file *file,
 			vm_flags |= VM_SHARED | VM_MAYSHARE;
 			if (!(file->f_mode & FMODE_WRITE))
 				vm_flags &= ~(VM_MAYWRITE | VM_SHARED);
-
-			/* fall through */
+			fallthrough;
 		case MAP_PRIVATE:
 			if (!(file->f_mode & FMODE_READ))
 				return -EACCES;
--- a/mm/shmem.c~mm-use-fallthrough
+++ a/mm/shmem.c
@@ -3996,7 +3996,7 @@  bool shmem_huge_enabled(struct vm_area_s
 			if (i_size >= HPAGE_PMD_SIZE &&
 					i_size >> PAGE_SHIFT >= off)
 				return true;
-			/* fall through */
+			fallthrough;
 		case SHMEM_HUGE_ADVISE:
 			/* TODO: implement fadvise() hints */
 			return (vma->vm_flags & VM_HUGEPAGE);
--- a/mm/zsmalloc.c~mm-use-fallthrough
+++ a/mm/zsmalloc.c
@@ -424,7 +424,7 @@  static void *zs_zpool_map(void *pool, un
 	case ZPOOL_MM_WO:
 		zs_mm = ZS_MM_WO;
 		break;
-	case ZPOOL_MM_RW: /* fall through */
+	case ZPOOL_MM_RW:
 	default:
 		zs_mm = ZS_MM_RW;
 		break;

[088/166] mm: use fallthrough;

Commit Message

Patch