Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1443680
| Path | csiph.com!eternal-september.org!feeder.eternal-september.org!news.unit0.net!news.panservice.it!bofh.it!news.nic.it!robomod |
|---|---|
| From | Sebastian Andrzej Siewior <bigeasy@linutronix.de> |
| Newsgroups | linux.kernel |
| Subject | Re: [RT PATCH 1/2] timers: wakeup all timer waiters |
| Date | Thu, 14 Jul 2016 20:40:03 +0200 |
| Message-ID | <rUTIf-4Mw-47@gated-at.bofh.it> (permalink) |
| References | <rURn4-3qz-31@gated-at.bofh.it> <rURwK-3tZ-17@gated-at.bofh.it> <rUSsN-45I-9@gated-at.bofh.it> <rUSsN-45I-7@gated-at.bofh.it> |
| User-Agent | Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.1.0 |
| MIME-Version | 1.0 |
| Content-Type | text/plain; charset=windows-1252 |
| Content-Transfer-Encoding | 7bit |
| X-Linutronix-Spam-Score | -1.0 |
| X-Linutronix-Spam-Level | - |
| X-Linutronix-Spam-Status | No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 |
| Sender | robomod@news.nic.it |
| List-ID | <linux-kernel.vger.kernel.org> |
| X-Mailing-List | linux-kernel@vger.kernel.org |
| Approved | robomod@news.nic.it |
| Lines | 21 |
| Organization | linux.* mail to news gateway |
| X-Original-Cc | linux-rt-users@vger.kernel.org, linux-kernel@vger.kernel.org, tglx@linutronix.de |
| X-Original-Date | Thu, 14 Jul 2016 20:37:42 +0200 |
| X-Original-Message-ID | <bf4277f7-0da1-bbf6-4425-cfaeb74afab6@linutronix.de> |
| X-Original-References | <1468512304-15310-1-git-send-email-bigeasy@linutronix.de> <20160714121341.2b0e70f5@gandalf.local.home> <e014d4ad-316f-3afe-fcac-923cd2f9a7e6@linutronix.de> <20160714131945.67cc8ce5@gandalf.local.home> |
| X-Original-Sender | linux-kernel-owner@vger.kernel.org |
| Xref | csiph.com linux.kernel:1443680 |
Show key headers only | View raw
On 07/14/2016 07:19 PM, Steven Rostedt wrote: >> While doing so I noticed that we drop the base lock during timer >> invocations and so it could be possible that we have two invocations >> of del_timer_sync() on a timer on the same "base" (one after the >> other). This is patch #1. >> >> After that I saw that we do the wake up under the base lock but there >> is no reason for it. So here is patch #2. >> >> Patch #1 is something that could happen in theory and I did not run in >> any problem. >> > > OK, so patch 2 was just discovered by reviewing code? both, not just patch 2. > > -- Steve Sebastian
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[RT PATCH 1/2] timers: wakeup all timer waiters Sebastian Andrzej Siewior <bigeasy@linutronix.de> - 2016-07-14 18:10 +0200
Re: [RT PATCH 1/2] timers: wakeup all timer waiters Steven Rostedt <rostedt@goodmis.org> - 2016-07-14 18:20 +0200
Re: [RT PATCH 1/2] timers: wakeup all timer waiters Steven Rostedt <rostedt@goodmis.org> - 2016-07-14 19:20 +0200
Re: [RT PATCH 1/2] timers: wakeup all timer waiters Sebastian Andrzej Siewior <bigeasy@linutronix.de> - 2016-07-14 20:40 +0200
Re: [RT PATCH 1/2] timers: wakeup all timer waiters Sebastian Andrzej Siewior <bigeasy@linutronix.de> - 2016-07-14 19:20 +0200
csiph-web