Re: [uml-devel] [PATCH v6] um: Add a high resolution timer subsystem

[Richard Weinberger <richard@nod.at>]

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

The timer stuff looks generally okay but please submit again as a clean patch series
with one logical change per patch such that I can review it in depth. :-)

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