On 02/12/2015 10:01 AM, Pavel Hrdina wrote:
> Commit b6a2828e introduced new functions to set process scheduler. There
> is a small typo in ELSE path for systems where scheduler is not
> available.
> 
> Also some of the definitions were introduced later in kernel. For
> example RHEL-5 is running on kernel 2.6.18, but SCHED_IDLE was introduces
> in 2.6.23 [1] and SCHED_BATCH in 2.6.16 [1]. We should not count only on
> existence of function sched_setscheduler(), we must also check for
> existence of used macros as they might not be defined.
> 
> [1] see 'man 7 sched'
> 
> Signed-off-by: Pavel Hrdina <phrd...@redhat.com>
> ---
>  src/util/virprocess.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)

ACK

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to