linux-um archives
 help / color / mirror / Atom feed
From: Thomas Meyer <thomas@m3y3r.de>
To: Anton Ivanov <anton.ivanov@kot-begemot.co.uk>
Cc: user-mode-linux-devel@lists.sourceforge.net
Subject: Re: [uml-devel] [PATCH v6] um: Add a high resolution timer subsystem
Date: Mon, 11 May 2015 23:22:55 +0200	[thread overview]
Message-ID: <36e3b03a-755f-4860-a8fb-8734f7c7d672@email.android.com> (raw)
In-Reply-To: <5551061E.9080006@kot-begemot.co.uk>

Am 11.05.2015 9:42 nachm. schrieb Anton Ivanov <anton.ivanov@kot-begemot.co.uk>:
>
> On 11/05/15 18:41, Thomas Meyer wrote: 
> > Am 11.05.2015 7:31 nachm. schrieb Anton Ivanov <anton.ivanov@kot-begemot.co.uk>: 
> >> On 11/05/15 18:20, Anton Ivanov wrote: 
> >>> On 11/05/15 18:00, Thomas Meyer wrote: 
> >>>> Hi, 
> >>>> 
> >>>> maybe there is a bug in how the timers are created for all user space processes. 
> >>>> In the latest patch I use os__timer_remain for the initial interval. 
> >>>> The idea was to launch all timers at the same time. But I now think this can never work using relative times, especially when os__timer_remain returns 0, then the new timer is never launched. 
> >>> That is not a bad idea, just requires a more complex check in setting up 
> >>> interval so it is reset to default when a zero value or value > HZ is 
> >>> passed. 
> >>> 
> >>> 
> >>>> That may explain the hangs you see. 
> >>>>      You could try to replace os__timer_remain with the current HZ value in nanoseconds. 
> >>> I am going to adjust time.c instead as above. 
> >> Both approaches fail - the check (that is expected actually, I did not 
> >> think properly here - it does not cover the stub in kernel/skas) and the 
> >> setting of data to the correct initial values. 
> >> 
> >> Also, if it was just not starting timers it would have been possible to 
> >> beat the process with USR2 on the head until its morale improves. Well, 
> >> "while true; do kill -USR2 PID ; done" does not change the behavior, it 
> >> still hangs and is still possible to terminate the errant process by 
> >> sending SIGSEGV to the UML thread which is shown as 100% CPU and in R state. 
> > Yes, sending USR2 should do the same. Correct. 
> > 
> >> Problem is somewhere else. I am surprised it worked correctly in the 
> >> original one as there for 100% flat CPU usage it should have converged 
> >> to a behavior which is similar to what we have now. 
> > Mhh. Strange. Any hints how I can reproduce this? 
> > 
> > I need to see the same behaviour, i.e. the hang on my machine to begin to understand what is going on here. 
>
> Start off with a Debian image of let's say wheezy and hit it with a 
> dist-upgrade to the next release. This produces enough IO+CPU at the 
> same time to trigger it. I get it 100% reproducible every time it tries 
> to unpack the first couple of packages. 

Okay, I'll try that!

>
> I tried to trigger backtraces but could not get anything informative. 
> The old trick of hitting the kernel with SIGILL which worked very nicely 
> in older kernel versions no longer works :( 

You may can trigger an backtrace via uml_console.

>
> > 
> > Thanks for testing Anton. 
>
> I wish I had more time at the moment to debug it and fix it :( We are 
> nearly there and it is worth it. 

Yes, same here.
But we will finish this!

>
> A. 
>
> > 
> > 
> >> A. 
> >> 
> >>> A. 
> >>> 
> >>>> Kind regards 
> >>>> Thomad 
> >>>> 
> >>>> A 
> >>> [snip] 
> >>> 
> >>> A. 
> >>> 
> >>> ------------------------------------------------------------------------------ 
> >>> One dashboard for servers and applications across Physical-Virtual-Cloud 
> >>> Widest out-of-the-box monitoring support with 50+ applications 
> >>> Performance metrics, stats and reports that give you Actionable Insights 
> >>> Deep dive visibility with transaction tracing using APM Insight. 
> >>> http://ad.doubleclick.net/ddm/clk/290420510;117567292;y 
> >>> _______________________________________________ 
> >>> User-mode-linux-devel mailing list 
> >>> User-mode-linux-devel@lists.sourceforge.net 
> >>> https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel 
> >>> 
> >> 
> >> ------------------------------------------------------------------------------ 
> >> One dashboard for servers and applications across Physical-Virtual-Cloud 
> >> Widest out-of-the-box monitoring support with 50+ applications 
> >> Performance metrics, stats and reports that give you Actionable Insights 
> >> Deep dive visibility with transaction tracing using APM Insight. 
> >> http://ad.doubleclick.net/ddm/clk/290420510;117567292;y 
> >> _______________________________________________ 
> >> User-mode-linux-devel mailing list 
> >> User-mode-linux-devel@lists.sourceforge.net 
> >> https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel 
>
>
> ------------------------------------------------------------------------------ 
> One dashboard for servers and applications across Physical-Virtual-Cloud 
> Widest out-of-the-box monitoring support with 50+ applications 
> Performance metrics, stats and reports that give you Actionable Insights 
> Deep dive visibility with transaction tracing using APM Insight. 
> http://ad.doubleclick.net/ddm/clk/290420510;117567292;y 
> _______________________________________________ 
> User-mode-linux-devel mailing list 
> User-mode-linux-devel@lists.sourceforge.net 
> https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel 
------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

  reply	other threads:[~2015-05-11 21:23 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-09 23:14 [uml-devel] [PATCH v6] um: Add a high resolution timer subsystem Thomas Meyer
2015-05-10 12:35 ` Richard Weinberger
2015-05-10 13:32   ` Anton Ivanov
2015-05-10 14:34   ` Thomas Meyer
2015-05-10 18:25     ` Anton Ivanov
2015-05-10 20:51     ` Richard Weinberger
2015-05-11 12:52     ` Anton Ivanov
2015-05-11 15:05       ` Anton Ivanov
2015-05-11 15:43         ` Anton Ivanov
2015-05-11 17:00           ` Thomas Meyer
2015-05-11 17:20             ` Anton Ivanov
2015-05-11 17:31               ` Anton Ivanov
2015-05-11 17:41                 ` Thomas Meyer
2015-05-11 19:42                   ` Anton Ivanov
2015-05-11 21:22                     ` Thomas Meyer [this message]
2015-05-26 10:56           ` stian
2015-05-11 22:34     ` Richard Weinberger

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=36e3b03a-755f-4860-a8fb-8734f7c7d672@email.android.com \
    --to=thomas@m3y3r.de \
    --cc=anton.ivanov@kot-begemot.co.uk \
    --cc=user-mode-linux-devel@lists.sourceforge.net \
    /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