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
next prev parent 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