[libvirt] [PATCH] BSD: implement virProcess{Get,Set}Affinity

Eric Blake eblake at redhat.com
Mon Jan 27 17:09:18 UTC 2014


On 01/25/2014 09:11 AM, Roman Bogorodskiy wrote:
> Implement virProcess{Get,Set}Affinity() using cpuset_getaffinity()
> and cpuset_setaffinity() calls. Quick search showed that they are
> only available on FreeBSD, so placed it inside existing #ifdef
> blocks for FreeBSD instead of adding configure checks.

That's not the ideal approach.  Feature checking has the advantage that
if some other platform adds the feature in the future, then we
automatically start using it, even if FreeBSD is the only platform that
offers the feature right now.  However, the existing code really was
tied to just a FreeBSD check, so I could live with a followup that adds
the configure check in order to do a feature-based rather than
platform-based conditional compilation.

ACK and pushed.

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140127/7f6e9393/attachment-0001.sig>


More information about the libvir-list mailing list