Re: [Xen-devel] [PATCH] xen/sched_rt: Move repl_timer into struct rt_private

[Dario Faggioli <dfaggioli@xxxxxxxx>]

On Thu, 2018-01-11 at 17:00 +0000, Andrew Cooper wrote:
> struct timer is only 48 bytes and repl_timer has a 1-to-1
> correspondance with
> struct rt_private, so having it referenced by pointer is wasteful.
> 
> This avoids one memory allocation in rt_init(), and the resulting
> diffstat is:
> 
>   add/remove: 0/0 grow/shrink: 0/7 up/down: 0/-156 (-156)
>   function                                     old     new   delta
>   rt_switch_sched                              134     133      -1
>   rt_context_saved                             278     271      -7
>   rt_vcpu_remove                               253     245      -8
>   rt_vcpu_sleep                                234     218     -16
>   repl_timer_handler                           761     744     -17
>   rt_deinit                                     44      20     -24
>   rt_init                                      219     136     -83
> 
> As an extra bit of cleanup noticed while making this change, there is
> no need
> to call cpumask_clear() on an zeroed memory allocation.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>
Acked-by: Dario Faggioli <dfaggioli@xxxxxxxx>

Regards,
Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Software Engineer @ SUSE https://www.suse.com/

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel