public inbox for linux-rt-users@vger.kernel.org
 help / color / mirror / Atom feed
From: Alexander Gerasiov <gq@cs.msu.su>
To: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: linux-rt-users@vger.kernel.org, zlobin_d@redlab-i.ru
Subject: Re: Problem with hrtimer_cancel() does not cancel timer
Date: Thu, 2 Mar 2017 02:48:08 +0300	[thread overview]
Message-ID: <20170302024808.7f5f29d9@brick.gerasiov.net> (raw)
In-Reply-To: <20170301150103.4ttnvz72d4jsgui6@linutronix.de>

Hello Sebastian,

On Wed, 1 Mar 2017 16:01:04 +0100
Sebastian Andrzej Siewior <bigeasy@linutronix.de> wrote:

> On 2017-02-18 01:15:22 [+0300], Alexander Gerasiov wrote:
> > Hello folks,  
> Hi,
> 
> > I met some problem with preempt version of kernel 4.4.
> > 
> > I have hrtimer which runs callback, it forwards time and returns
> > HRTIMER_RESTART. Very common scheme. When I stop the routine I call
> > hrtimer_cancel(). It should wait for callback completion if any and
> > unqueue any future timers.  
> 
> Could you please check if the hunk here fixes the problem you see?

Great! Looks like the problem is solved.

When submitting patch to git, take a note, that not only 4.4 is
affected. I'm quite sure, that 4.1 was affected too when I tested it and
Dmitry comfirmed 4.9 is also broken.

-- 
Best regards,
 Alexander Gerasiov

 Contacts:
 e-mail: gq@cs.msu.su  Homepage: http://gerasiov.net  Skype: gerasiov
 PGP fingerprint: 04B5 9D90 DF7C C2AB CD49  BAEA CA87 E9E8 2AAC 33F1

  reply	other threads:[~2017-03-02  0:05 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-17 22:15 Problem with hrtimer_cancel() does not cancel timer Alexander Gerasiov
2017-02-21 14:15 ` Dmitry
2017-02-28 13:02 ` Alexander Gerasiov
2017-03-01 15:01 ` Sebastian Andrzej Siewior
2017-03-01 23:48   ` Alexander Gerasiov [this message]
2017-03-02  8:19     ` Sebastian Andrzej Siewior

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170302024808.7f5f29d9@brick.gerasiov.net \
    --to=gq@cs.msu.su \
    --cc=bigeasy@linutronix.de \
    --cc=linux-rt-users@vger.kernel.org \
    --cc=zlobin_d@redlab-i.ru \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox