+ rbtree-avoid-generating-code-twice-for-the-cached-versions-checkpatch-fixes.patch added to -mm tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The patch titled
     Subject: rbtree-avoid-generating-code-twice-for-the-cached-versions-checkpatch-fixes
has been added to the -mm tree.  Its filename is
     rbtree-avoid-generating-code-twice-for-the-cached-versions-checkpatch-fixes.patch

This patch should soon appear at
    http://ozlabs.org/~akpm/mmots/broken-out/rbtree-avoid-generating-code-twice-for-the-cached-versions-checkpatch-fixes.patch
and later at
    http://ozlabs.org/~akpm/mmotm/broken-out/rbtree-avoid-generating-code-twice-for-the-cached-versions-checkpatch-fixes.patch

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 and is updated
there every 3-4 working days

------------------------------------------------------
From: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
Subject: rbtree-avoid-generating-code-twice-for-the-cached-versions-checkpatch-fixes

WARNING: 'implemntation' may be misspelled - perhaps 'implementation'?
#8: 
rb_leftmost was wired deeper into the implemntation, but there were some

ERROR: Macros with complex values should be enclosed in parentheses
#180: FILE: include/linux/rbtree.h:130:
+#define RB_ROOT_CACHED (struct rb_root_cached) { {NULL, }, NULL }

ERROR: Macros with complex values should be enclosed in parentheses
#183: FILE: include/linux/rbtree.h:133:
+#define rb_first_cached(root) (root)->rb_leftmost

ERROR: code indent should use tabs where possible
#197: FILE: include/linux/rbtree.h:147:
+        if (root->rb_leftmost == node)$

WARNING: please, no spaces at the start of a line
#197: FILE: include/linux/rbtree.h:147:
+        if (root->rb_leftmost == node)$

ERROR: code indent should use tabs where possible
#198: FILE: include/linux/rbtree.h:148:
+                root->rb_leftmost = rb_next(node);$

WARNING: please, no spaces at the start of a line
#198: FILE: include/linux/rbtree.h:148:
+                root->rb_leftmost = rb_next(node);$

total: 4 errors, 3 warnings, 246 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

NOTE: Whitespace errors detected.
      You may wish to use scripts/cleanpatch or scripts/cleanfile

./patches/rbtree-avoid-generating-code-twice-for-the-cached-versions.patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.

Please run checkpatch prior to sending patches

Cc: Michel Lespinasse <walken@xxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---

 include/linux/rbtree.h |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

--- a/include/linux/rbtree.h~rbtree-avoid-generating-code-twice-for-the-cached-versions-checkpatch-fixes
+++ a/include/linux/rbtree.h
@@ -144,8 +144,8 @@ static inline void rb_insert_color_cache
 static inline void rb_erase_cached(struct rb_node *node,
 				   struct rb_root_cached *root)
 {
-        if (root->rb_leftmost == node)
-                root->rb_leftmost = rb_next(node);
+	if (root->rb_leftmost == node)
+		root->rb_leftmost = rb_next(node);
 	rb_erase(node, &root->rb_root);
 }
 
_

Patches currently in -mm which might be from akpm@xxxxxxxxxxxxxxxxxxxx are

mm-z3foldc-lock-z3fold-page-before-__setpagemovable-fix.patch
scripts-spellingtxt-drop-sepc-from-the-misspelling-list-fix.patch
ocfs2-add-locking-filter-debugfs-file-fix.patch
ocfs2-clear-zero-in-unaligned-direct-io-checkpatch-fixes.patch
mm.patch
include-linux-pfn_th-remove-pfn_t_to_virt.patch
mm-swap-use-rbtree-for-swap_extent-fix.patch
mm-gup-speed-up-check_and_migrate_cma_pages-on-huge-page-fix.patch
mm-section-numbers-use-the-type-unsigned-long-fix.patch
mm-sparsemem-cleanup-section-number-data-types-fix.patch
mm-vmscanc-add-checks-for-incorrect-handling-of-current-reclaim_state.patch
proc-use-down_read_killable-mmap_sem-for-proc-pid-map_files-fix.patch
mm-oom_killer-add-task-uid-to-info-message-on-an-oom-kill-fix.patch
mm-thp-make-transhuge_vma_suitable-available-for-anonymous-thp-fix.patch
vmcore-add-a-kernel-parameter-novmcoredd-fix.patch
vmcore-add-a-kernel-parameter-novmcoredd-fix-fix.patch
rbtree-avoid-generating-code-twice-for-the-cached-versions-checkpatch-fixes.patch
coda-add-hinting-support-for-partial-file-caching-fix.patch
selftests-ptrace-add-a-test-case-for-ptrace_get_syscall_info-checkpatch-fixes.patch
resource-fix-locking-in-find_next_iomem_res-fix.patch
linux-next-rejects.patch
proc-sysctl-add-shared-variables-for-range-check-fix-2-fix.patch
proc-sysctl-add-shared-variables-for-range-check-fix-4.patch
drivers-tty-serial-sh-scic-suppress-warning.patch
kernel-forkc-export-kernel_thread-to-modules.patch




[Index of Archives]     [Kernel Archive]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux