On Wed, 2019-09-25 at 15:13 +0200, Jürgen Groß wrote: > On 24.09.19 14:16, Jan Beulich wrote: > > > > While at the first glance "sd" is used twice here, this 2nd use is > > actually unnecessary - "cpu" hasn't changed from the beginning of > > the function afaics, and hence "ops" could be used here. > > "sd" is now "sr" everywhere. > > And yes, using ops here seems okay. > > > Preferably with the suggested adjustments (but final judgement is > > with the scheduler maintainers anyway) > > Reviewed-by: Jan Beulich <jbeul...@suse.com> > Reviewed-by: Dario Faggioli <dfaggi...@suse.com>
(With sd->sr and with using ops.) Regards -- Dario Faggioli, Ph.D http://about.me/dario.faggioli Virtualization Software Engineer SUSE Labs, SUSE https://www.suse.com/ ------------------------------------------------------------------- <<This happens because _I_ choose it to happen!>> (Raistlin Majere)
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel