All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Weinberger <richard@nod.at>
To: Thomas Meyer <thomas@m3y3r.de>
Cc: user-mode-linux-devel <user-mode-linux-devel@lists.sourceforge.net>
Subject: Re: [uml-devel] [PATCH v6] um: Add a high resolution timer subsystem
Date: Sun, 10 May 2015 22:51:51 +0200	[thread overview]
Message-ID: <554FC4E7.6070701@nod.at> (raw)
In-Reply-To: <3985DC9F-E8A3-431E-8A7F-B867DB0B99AF@m3y3r.de>

Am 10.05.2015 um 16:34 schrieb Thomas Meyer:
> 
>> Am 10.05.2015 um 14:35 schrieb Richard Weinberger <richard.weinberger@gmail.com>:
>>
>>> On Sun, May 10, 2015 at 1:14 AM, Thomas Meyer <thomas@m3y3r.de> wrote:
>>> Hi,
>>>
>>> Changes:
>>> - also create posix timer in stub_clone_handler()
>>> - incorporated antons remarks
>>
>> Hm, this patch does a *lot* more than the changelog says.
> 
> Hi, yes PATCH was probably the wrong keyword in the subject line. It should have been RFC.
> I just wanted to have feedback of the current state of this patch/work.

Ahh, ok!

I was a bit confused as it contains beside the new features also new comments to unrelated
code, changes coding style, etc...


> I'm currently working on cleaning up the patch and switch from SIGUSR2 to SIGNALRM, which seems to be the natural thing for posix timers.
> I will send this next patch as something that should be includable into the kernel, i.e. With correct description and signed off line and so on.
> 
> But feel free to have a look at v6 and give feedback.

I will. :-)

Thanks,
//richard

------------------------------------------------------------------------------
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


  parent reply	other threads:[~2015-05-10 20:52 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 [this message]
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
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=554FC4E7.6070701@nod.at \
    --to=richard@nod.at \
    --cc=thomas@m3y3r.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.