The patch titled itimers-fix-itimer-many-thread-hang-checkpatch-fixes has been added to the -mm tree. Its filename is itimers-fix-itimer-many-thread-hang-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/SubmitChecklist when testing your code *** See http://userweb.kernel.org/~akpm/added-to-mm.txt to find out what to do about this The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ ------------------------------------------------------ Subject: itimers-fix-itimer-many-thread-hang-checkpatch-fixes From: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> WARNING: line over 80 characters #400: FILE: include/linux/sched.h:2096: +extern void thread_group_cputime_smp(struct task_struct *, struct task_cputime *); ERROR: code indent should use tabs where possible #1079: FILE: kernel/posix-cpu-timers.c:97: + cputime_lt(current->signal->it_prof_expires, cputime)) {$ WARNING: braces {} are not necessary for single statement blocks #1577: FILE: kernel/posix-cpu-timers.c:1375: + if (unlikely(!sig) || !fastpath_timer_check(tsk, sig)) { + return; + } total: 1 errors, 2 warnings, 1612 lines checked ./patches/itimers-fix-itimer-many-thread-hang.patch has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Please run checkpatch prior to sending patches Cc: Alexey Dobriyan <adobriyan@xxxxxxxxx> Cc: Frank Mayhar <fmayhar@xxxxxxxxxx> Cc: Ingo Molnar <mingo@xxxxxxx> Cc: Oleg Nesterov <oleg@xxxxxxxxxx> Cc: Roland McGrath <roland@xxxxxxxxxx> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- include/linux/sched.h | 3 ++- kernel/posix-cpu-timers.c | 10 ++++------ 2 files changed, 6 insertions(+), 7 deletions(-) diff -puN include/linux/sched.h~itimers-fix-itimer-many-thread-hang-checkpatch-fixes include/linux/sched.h --- a/include/linux/sched.h~itimers-fix-itimer-many-thread-hang-checkpatch-fixes +++ a/include/linux/sched.h @@ -2093,7 +2093,8 @@ static inline int spin_needbreak(spinloc #ifdef CONFIG_SMP extern int thread_group_cputime_alloc_smp(struct task_struct *); -extern void thread_group_cputime_smp(struct task_struct *, struct task_cputime *); +extern void thread_group_cputime_smp(struct task_struct *tsk, + struct task_cputime *times); static inline void thread_group_cputime_init(struct signal_struct *sig) { diff -puN kernel/posix-cpu-timers.c~itimers-fix-itimer-many-thread-hang-checkpatch-fixes kernel/posix-cpu-timers.c --- a/kernel/posix-cpu-timers.c~itimers-fix-itimer-many-thread-hang-checkpatch-fixes +++ a/kernel/posix-cpu-timers.c @@ -58,9 +58,8 @@ int thread_group_cputime_alloc_smp(struc * Walk the list of CPUs to sum the per-CPU time fields in the thread group * time structure. */ -void thread_group_cputime_smp( - struct task_struct *tsk, - struct task_cputime *times) +void thread_group_cputime_smp(struct task_struct *tsk, + struct task_cputime *times) { struct signal_struct *sig; int i; @@ -94,7 +93,7 @@ void update_rlimit_cpu(unsigned long rli cputime = secs_to_cputime(rlim_new); if (cputime_eq(current->signal->it_prof_expires, cputime_zero) || - cputime_lt(current->signal->it_prof_expires, cputime)) { + cputime_lt(current->signal->it_prof_expires, cputime)) { spin_lock_irq(¤t->sighand->siglock); set_process_cpu_timer(current, CPUCLOCK_PROF, &cputime, NULL); spin_unlock_irq(¤t->sighand->siglock); @@ -1372,9 +1371,8 @@ void run_posix_cpu_timers(struct task_st * tsk->signal is non-NULL; this probably can't happen but cover the * possibility anyway. */ - if (unlikely(!sig) || !fastpath_timer_check(tsk, sig)) { + if (unlikely(!sig) || !fastpath_timer_check(tsk, sig)) return; - } sighand = lock_task_sighand(tsk, &flags); if (likely(sighand)) { /* _ Patches currently in -mm which might be from akpm@xxxxxxxxxxxxxxxxxxxx are origin.patch include-linux-ioporth-add-missing-macro-argument-for-devm_release_-family-checkpatch-fixes.patch rescan_partitions-make-device-capacity-errors-non-fatal.patch linux-next.patch linux-next-fixups.patch next-remove-localversion.patch linux-next-git-rejects.patch security-selinux-include-netlabelh-fix-two-build-errors.patch drivers-mfd-ucb1400_corec-needs-gpio.patch drivers-mfd-ucb1400_corec-further-unbork.patch kbuild-ftrace-dont-assume-that-scripts-recordmcountpl-is-executable.patch toshiba_acpi-add-support-for-bluetooth-toggling-through-rfkill-v7-fix.patch toshiba_acpi-add-support-for-bluetooth-toggling-through-rfkill-v7-fix-fix.patch revert-acpi-ec-do-transaction-from-interrupt-context.patch acpi-catch-calls-of-acpi_driver_data-on-pointer-of-wrong-type-fix.patch acpi-catch-calls-of-acpi_driver_data-on-pointer-of-wrong-type-fix-fix.patch zero-based-percpu-use-vmlinux_symbol-in-include-asm-generic-vmlinuxldsh.patch x86-make-poll_idle-behave-more-like-the-other-idle-methods-checkpatch-fixes.patch arch-x86-kernel-early_printkc-remove-unused-enable_debug_console.patch fs-sysfs-dirc-remove-unused-__sysfs_get_dentry.patch fs-dlm-configc-choose-better-identifiers.patch tick-schedc-suppress-needless-timer-reprogramming.patch itimers-fix-itimer-many-thread-hang-checkpatch-fixes.patch drivers-input-touchscreen-ucb1400_tsc-needs-gpio.patch git-jg-misc.patch led-driver-for-leds-on-pcengines-alix2-and-alix3-boards.patch libata-blackfin-pata-driver-add-proper-pm-operation-into-atapi-driver-fix.patch sundance-set-carrier-status-on-link-change-events.patch cassini-use-request_firmware.patch e1000e-avoid-duplicated-output-of-device-name-in-kernel-warning-checkpatch-fixes.patch e1000e-avoid-duplicated-output-of-device-name-in-kernel-warning-fix.patch backlight-driver-for-tabletkiosk-sahara-touchit-213-tablet-pc.patch backlight-driver-for-tabletkiosk-sahara-touchit-213-tablet-pc-update-2-checkpatch-fixes.patch fix-cred-wrap-current-cred-and-a-few-other-accessors-on-mips.patch pci-tidy-pme-support-messages-checkpatch-fixes.patch arch-s390-kernel-ptracec-fix-build.patch sched-clarify-ifdef-tangle.patch debug_locks_warn_on-disable-if-oops_in_progress.patch scsi-gdthc-use-unaligned-access-helpers.patch drivers-scsi-device_handler-scsi_dh_emcc-suppress-warning.patch drivers-rtc-kconfig-dont-build-rtc-cmoso-on-sparc32.patch drivers-usb-musb-disable-it-on-superh.patch git-xtensa.patch modules-remove-stop_machine-during-module-load-checkpatch-fixes.patch scsi-dpt_i2o-is-bust-on-ia64.patch drivers-media-video-cafe_ccicc-needs-mmh.patch e1000e-prevent-corruption-of-eeprom-nvm.patch drivers-net-mlx4-allocc-needs-mmh.patch net-forcedeth-call-restore-mac-addr-in-nv_shutdown-path-v2.patch mm-page_allocc-free_area_init_nodes-fix-inappropriate-use-of-enum.patch mm-cleanup-to-make-remove_memory-arch-neutral-fix.patch vmscan-give-referenced-active-and-unmapped-pages-a-second-trip-around-the-lru.patch vm-dont-run-touch_buffer-during-buffercache-lookups.patch vmscan-move-isolate_lru_page-to-vmscanc.patch define-page_file_cache-function.patch vmscan-split-lru-lists-into-anon-file-sets.patch unevictable-lru-infrastructure.patch unevictable-lru-page-statistics.patch mlock-mlocked-pages-are-unevictable.patch mlock-mlocked-pages-are-unevictable-fix.patch mmap-handle-mlocked-pages-during-map-remap-unmap.patch mmap-handle-mlocked-pages-during-map-remap-unmap-mlock-resubmit-locked_vm-adjustment-as-separate-patch-fix.patch vmscan-unevictable-lru-scan-sysctl.patch mlock-make-mlock-error-return-posixly-correct-fix.patch mm-rewrite-vmap-layer-fix.patch mm-rewrite-vmap-layer-fix-fix.patch mm-rewrite-vmap-layer-fix-fix-fix.patch h8300-generic_bug-support-checkpatch-fixes.patch pm-rework-disabling-of-user-mode-helpers-during-suspend-hibernation-cleanup.patch container-freezer-add-tif_freeze-flag-to-all-architectures-fix.patch container-freezer-implement-freezer-cgroup-subsystem-checkpatch-fixes.patch container-freezer-implement-freezer-cgroup-subsystem-uninline-thaw_process-fix.patch separate-atomic_t-declaration-from-asm-atomich-into-asm-atomic_defh-fix.patch separate-atomic_t-declaration-from-asm-atomich-into-asm-atomic_defh-fix-fix.patch fix-a-race-condtion-of-oops_in_progress-fix.patch vsprintf-use-new-vsprintf-symbolic-function-pointer-format-cleanup.patch make-taint-bit-reliable-v3-fix.patch kernel-sysc-improve-code-generation.patch compat-generic-compat-get-settimeofday-checkpatch-fixes.patch hwmon-pc87360-separate-alarm-files-add-in-min-max-alarms-cleanup.patch hwmon-pc87360-separate-alarm-files-add-temp-min-max-crit-fault-alarms-cleanup.patch hwmon-pc87360-separate-alarm-files-add-therm-min-max-crit-alarms-cleanup.patch allow-recursion-in-binfmt_script-and-binfmt_misc-fix.patch autofs4-track-uid-and-gid-of-last-mount-requester-fix.patch autofs4-add-miscellaneous-device-for-ioctls-fix.patch autofs4-add-miscellaneous-device-for-ioctls-fix-2.patch autofs4-add-miscellaneous-device-for-ioctls-fix-fix-3.patch rtc-add-device-driver-for-dallas-ds3234-spi-rtc-chip-fix.patch fb-push-down-the-bkl-in-the-ioctl-handler-checkpatch-fixes.patch viafb-viafbmodes-viafbtxt-fix-fix.patch viafb-accelc-accelh-checkpatch-fixes.patch viafb-dvic-dvih-globalc-and-globalh-checkpatch-fixes.patch viafb-hwc-hwh-checkpatch-fixes.patch viafb-viafbdevc-viafbdevh-checkpatch-fixes.patch fbdev-kconfig-update-fix.patch eink_apollofb-new-driver-for-apollo-eink-controller.patch cgroups-fix-probable-race-with-put_css_set-and-find_css_set-fix.patch memrlimit-add-memrlimit-controller-accounting-and-control-mm_owner-fix-checkpatch-fixes.patch genirq-record-trigger-type-fix.patch message-queues-increase-range-limits-checkpatch-fixes.patch tty-fix-echo-tab-erase-and-locking-fix.patch applicomc-fix-apparently-broken-code-in-do_ac_read.patch drivers-char-tpm-tpmc-fix-error-patch-memory-leak.patch w1-be-able-to-manually-add-and-remove-slaves-fix.patch gru-driver-minor-updates-fix.patch gcov-architecture-specific-compile-flag-adjustments-powerpc-moved-stuff.patch drivers-rtc-rtc-bq4802c-dont-use-bin_2_bcd-and-bcd_2_bin.patch x86-sysfs-kill-owner-field-from-attribute-fix.patch nilfs2-continuous-snapshotting-file-system-fix.patch nilfs2-continuous-snapshotting-file-system-fix-fix-2.patch reiser4.patch reiser4-tree_lock-fixes.patch reiser4-tree_lock-fixes-fix.patch reiser4-semaphore-fix.patch slb-drop-kmem-cache-argument-from-constructor-reiser4.patch reiser4-suid.patch reiser4-track-upstream-changes.patch page-owner-tracking-leak-detector.patch nr_blockdev_pages-in_interrupt-warning.patch slab-leaks3-default-y.patch put_bh-debug.patch shrink_slab-handle-bad-shrinkers.patch getblk-handle-2tb-devices.patch getblk-handle-2tb-devices-fix.patch undeprecate-pci_find_device.patch notify_change-callers-must-hold-i_mutex.patch profile-likely-unlikely-macros.patch drivers-net-bonding-bond_sysfsc-suppress-uninitialized-var-warning.patch w1-build-fix.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html