]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
drop alarmtimer-rate-limit-periodic-intervals.patch from kernels that it is broken...
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 19 Jun 2017 02:04:59 +0000 (10:04 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 19 Jun 2017 02:04:59 +0000 (10:04 +0800)
queue-3.18/alarmtimer-rate-limit-periodic-intervals.patch [deleted file]
queue-3.18/series
queue-4.4/alarmtimer-rate-limit-periodic-intervals.patch [deleted file]
queue-4.4/series
queue-4.9/alarmtimer-rate-limit-periodic-intervals.patch [deleted file]
queue-4.9/series

diff --git a/queue-3.18/alarmtimer-rate-limit-periodic-intervals.patch b/queue-3.18/alarmtimer-rate-limit-periodic-intervals.patch
deleted file mode 100644 (file)
index 2995d28..0000000
+++ /dev/null
@@ -1,60 +0,0 @@
-From ff86bf0c65f14346bf2440534f9ba5ac232c39a0 Mon Sep 17 00:00:00 2001
-From: Thomas Gleixner <tglx@linutronix.de>
-Date: Tue, 30 May 2017 23:15:35 +0200
-Subject: alarmtimer: Rate limit periodic intervals
-
-From: Thomas Gleixner <tglx@linutronix.de>
-
-commit ff86bf0c65f14346bf2440534f9ba5ac232c39a0 upstream.
-
-The alarmtimer code has another source of potentially rearming itself too
-fast. Interval timers with a very samll interval have a similar CPU hog
-effect as the previously fixed overflow issue.
-
-The reason is that alarmtimers do not implement the normal protection
-against this kind of problem which the other posix timer use:
-
-  timer expires -> queue signal -> deliver signal -> rearm timer
-
-This scheme brings the rearming under scheduler control and prevents
-permanently firing timers which hog the CPU.
-
-Bringing this scheme to the alarm timer code is a major overhaul because it
-lacks all the necessary mechanisms completely.
-
-So for a quick fix limit the interval to one jiffie. This is not
-problematic in practice as alarmtimers are usually backed by an RTC for
-suspend which have 1 second resolution. It could be therefor argued that
-the resolution of this clock should be set to 1 second in general, but
-that's outside the scope of this fix.
-
-Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
-Cc: Peter Zijlstra <peterz@infradead.org>
-Cc: Kostya Serebryany <kcc@google.com>
-Cc: syzkaller <syzkaller@googlegroups.com>
-Cc: John Stultz <john.stultz@linaro.org>
-Cc: Dmitry Vyukov <dvyukov@google.com>
-Link: http://lkml.kernel.org/r/20170530211655.896767100@linutronix.de
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
----
- kernel/time/alarmtimer.c |    8 ++++++++
- 1 file changed, 8 insertions(+)
-
---- a/kernel/time/alarmtimer.c
-+++ b/kernel/time/alarmtimer.c
-@@ -614,6 +614,14 @@ static int alarm_timer_set(struct k_itim
-       /* start the timer */
-       timr->it.alarm.interval = timespec_to_ktime(new_setting->it_interval);
-+
-+      /*
-+       * Rate limit to the tick as a hot fix to prevent DOS. Will be
-+       * mopped up later.
-+       */
-+      if (timr->it.alarm.interval < TICK_NSEC)
-+              timr->it.alarm.interval = TICK_NSEC;
-+
-       exp = timespec_to_ktime(new_setting->it_value);
-       /* Convert (if necessary) to absolute time */
-       if (flags != TIMER_ABSTIME) {
index 08b37b547c3c406cfb79bff2a5b35b58068c84bd..366e8af442df0eec803e6e9b82bbd3928dc9b4b1 100644 (file)
@@ -28,4 +28,3 @@ usb-xhci-asmedia-asm1042a-chipset-need-shorts-tx-quirk.patch
 mm-memory-failure.c-use-compound_head-flags-for-huge-pages.patch
 swap-cond_resched-in-swap_cgroup_prepare.patch
 genirq-release-resources-in-__setup_irq-error-path.patch
-alarmtimer-rate-limit-periodic-intervals.patch
diff --git a/queue-4.4/alarmtimer-rate-limit-periodic-intervals.patch b/queue-4.4/alarmtimer-rate-limit-periodic-intervals.patch
deleted file mode 100644 (file)
index 7ab4dda..0000000
+++ /dev/null
@@ -1,60 +0,0 @@
-From ff86bf0c65f14346bf2440534f9ba5ac232c39a0 Mon Sep 17 00:00:00 2001
-From: Thomas Gleixner <tglx@linutronix.de>
-Date: Tue, 30 May 2017 23:15:35 +0200
-Subject: alarmtimer: Rate limit periodic intervals
-
-From: Thomas Gleixner <tglx@linutronix.de>
-
-commit ff86bf0c65f14346bf2440534f9ba5ac232c39a0 upstream.
-
-The alarmtimer code has another source of potentially rearming itself too
-fast. Interval timers with a very samll interval have a similar CPU hog
-effect as the previously fixed overflow issue.
-
-The reason is that alarmtimers do not implement the normal protection
-against this kind of problem which the other posix timer use:
-
-  timer expires -> queue signal -> deliver signal -> rearm timer
-
-This scheme brings the rearming under scheduler control and prevents
-permanently firing timers which hog the CPU.
-
-Bringing this scheme to the alarm timer code is a major overhaul because it
-lacks all the necessary mechanisms completely.
-
-So for a quick fix limit the interval to one jiffie. This is not
-problematic in practice as alarmtimers are usually backed by an RTC for
-suspend which have 1 second resolution. It could be therefor argued that
-the resolution of this clock should be set to 1 second in general, but
-that's outside the scope of this fix.
-
-Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
-Cc: Peter Zijlstra <peterz@infradead.org>
-Cc: Kostya Serebryany <kcc@google.com>
-Cc: syzkaller <syzkaller@googlegroups.com>
-Cc: John Stultz <john.stultz@linaro.org>
-Cc: Dmitry Vyukov <dvyukov@google.com>
-Link: http://lkml.kernel.org/r/20170530211655.896767100@linutronix.de
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
----
- kernel/time/alarmtimer.c |    8 ++++++++
- 1 file changed, 8 insertions(+)
-
---- a/kernel/time/alarmtimer.c
-+++ b/kernel/time/alarmtimer.c
-@@ -611,6 +611,14 @@ static int alarm_timer_set(struct k_itim
-       /* start the timer */
-       timr->it.alarm.interval = timespec_to_ktime(new_setting->it_interval);
-+
-+      /*
-+       * Rate limit to the tick as a hot fix to prevent DOS. Will be
-+       * mopped up later.
-+       */
-+      if (timr->it.alarm.interval < TICK_NSEC)
-+              timr->it.alarm.interval = TICK_NSEC;
-+
-       exp = timespec_to_ktime(new_setting->it_value);
-       /* Convert (if necessary) to absolute time */
-       if (flags != TIMER_ABSTIME) {
index 847d3356d1a6fc5a09cf06520f584d26609823e3..6f909803f27d29cd0bafbcd3c5882f2115eb523f 100644 (file)
@@ -25,5 +25,4 @@ swap-cond_resched-in-swap_cgroup_prepare.patch
 genirq-release-resources-in-__setup_irq-error-path.patch
 alarmtimer-prevent-overflow-of-relative-timers.patch
 usb-dwc3-exynos-fix-axius-clock-error-path-to-do-cleanup.patch
-alarmtimer-rate-limit-periodic-intervals.patch
 mips-fix-bnezc-jialc-return-address-calculation.patch
diff --git a/queue-4.9/alarmtimer-rate-limit-periodic-intervals.patch b/queue-4.9/alarmtimer-rate-limit-periodic-intervals.patch
deleted file mode 100644 (file)
index 0e42165..0000000
+++ /dev/null
@@ -1,60 +0,0 @@
-From ff86bf0c65f14346bf2440534f9ba5ac232c39a0 Mon Sep 17 00:00:00 2001
-From: Thomas Gleixner <tglx@linutronix.de>
-Date: Tue, 30 May 2017 23:15:35 +0200
-Subject: alarmtimer: Rate limit periodic intervals
-
-From: Thomas Gleixner <tglx@linutronix.de>
-
-commit ff86bf0c65f14346bf2440534f9ba5ac232c39a0 upstream.
-
-The alarmtimer code has another source of potentially rearming itself too
-fast. Interval timers with a very samll interval have a similar CPU hog
-effect as the previously fixed overflow issue.
-
-The reason is that alarmtimers do not implement the normal protection
-against this kind of problem which the other posix timer use:
-
-  timer expires -> queue signal -> deliver signal -> rearm timer
-
-This scheme brings the rearming under scheduler control and prevents
-permanently firing timers which hog the CPU.
-
-Bringing this scheme to the alarm timer code is a major overhaul because it
-lacks all the necessary mechanisms completely.
-
-So for a quick fix limit the interval to one jiffie. This is not
-problematic in practice as alarmtimers are usually backed by an RTC for
-suspend which have 1 second resolution. It could be therefor argued that
-the resolution of this clock should be set to 1 second in general, but
-that's outside the scope of this fix.
-
-Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
-Cc: Peter Zijlstra <peterz@infradead.org>
-Cc: Kostya Serebryany <kcc@google.com>
-Cc: syzkaller <syzkaller@googlegroups.com>
-Cc: John Stultz <john.stultz@linaro.org>
-Cc: Dmitry Vyukov <dvyukov@google.com>
-Link: http://lkml.kernel.org/r/20170530211655.896767100@linutronix.de
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
----
- kernel/time/alarmtimer.c |    8 ++++++++
- 1 file changed, 8 insertions(+)
-
---- a/kernel/time/alarmtimer.c
-+++ b/kernel/time/alarmtimer.c
-@@ -624,6 +624,14 @@ static int alarm_timer_set(struct k_itim
-       /* start the timer */
-       timr->it.alarm.interval = timespec_to_ktime(new_setting->it_interval);
-+
-+      /*
-+       * Rate limit to the tick as a hot fix to prevent DOS. Will be
-+       * mopped up later.
-+       */
-+      if (timr->it.alarm.interval < TICK_NSEC)
-+              timr->it.alarm.interval = TICK_NSEC;
-+
-       exp = timespec_to_ktime(new_setting->it_value);
-       /* Convert (if necessary) to absolute time */
-       if (flags != TIMER_ABSTIME) {
index 0c676fd8c7e53ae9030ff88d316c7d1547e6cd95..bc145ee4da0c7919f62555daadea28811982a978 100644 (file)
@@ -52,7 +52,6 @@ genirq-release-resources-in-__setup_irq-error-path.patch
 alarmtimer-prevent-overflow-of-relative-timers.patch
 usb-gadget-composite-fix-function-used-to-free-memory.patch
 usb-dwc3-exynos-fix-axius-clock-error-path-to-do-cleanup.patch
-alarmtimer-rate-limit-periodic-intervals.patch
 virtio_balloon-disable-viommu-support.patch
 mips-fix-bnezc-jialc-return-address-calculation.patch
 mips-.its-targets-depend-on-vmlinux.patch