On 07/23/2014 07:49 AM, Martin Kletzander wrote: > On Tue, Jul 01, 2014 at 02:00:57PM -0400, Matthew Rosato wrote: >> Defer MAC registration until net devices are actually going >> to be used by the guest. This patch does so by setting the >> devices online just before starting guest CPUs. >> > > Does this have some upside/downside? Are you trying to fix some > problem? It would be nice to describe it in the commit message, so I > know what to focus on or why it's needed. Depending on the answer > there might be a way how to unit-test it. > >> +++ b/src/qemu/qemu_interface.c >> @@ -0,0 +1,65 @@ >> +/* >> + * qemu_interface.c: QEMU interface management >> + * >> + * Copyright (C) 2014 Red Hat, Inc. >> + * Copyright IBM Corp. 2014 >> + * > > I don't understand this double copyright here, copy-paste mistake? If this file is copied from a pre-existing file with double copyright, and substantially borrows from that content rather than being fresh material, then keeping double copyright is ideal. If this is something you mostly wrote yourself, and then just copied in boilerplate, then it's simpler to just put one copyright line for yourself (in this case, IBM); then later, if other contributors make contributions, a second line can be added then (I tend to add Red Hat copyright whenever I touch files, since I'm doing my work on Red Hat time). I don't see it as a show-stopper either way, because these days, git log is more reliable for telling who contributed what portions of a file. -- 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@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list