Re: [libvirt PATCH] hypervisor: Move interface mgmt methods to hypervisor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Oct 15, 2023 at 03:03:40PM -0400, Laine Stump wrote:
> On 10/12/23 3:37 PM, Praveen K Paladugu wrote:
> > Move guest interface management methods from qemu to hypervisor. These
> > methods will be shared by networking support in ch driver.
> > 
> > Signed-off-by: Praveen K Paladugu <prapal@xxxxxxxxxxxxxxxxxxx>
> > ---
> >   po/POTFILES                       |   1 +
> >   src/hypervisor/domain_interface.c | 279 ++++++++++++++++++++++++++++++
> >   src/hypervisor/domain_interface.h |  38 ++++
> >   src/hypervisor/meson.build        |   1 +
> >   src/libvirt_private.syms          |   6 +
> >   src/qemu/qemu_command.c           |   7 +-
> >   src/qemu/qemu_hotplug.c           |   3 +-
> >   src/qemu/qemu_interface.c         | 246 +-------------------------
> >   src/qemu/qemu_interface.h         |   6 -
> >   src/qemu/qemu_process.c           |   3 +-
> >   10 files changed, 341 insertions(+), 249 deletions(-)
> >   create mode 100644 src/hypervisor/domain_interface.c
> >   create mode 100644 src/hypervisor/domain_interface.h
> > 
> > diff --git a/po/POTFILES b/po/POTFILES
> > index 3a51aea5cb..023c041f61 100644
> > --- a/po/POTFILES
> > +++ b/po/POTFILES
> > @@ -92,6 +92,7 @@ src/hyperv/hyperv_util.c
> >   src/hyperv/hyperv_wmi.c
> >   src/hypervisor/domain_cgroup.c
> >   src/hypervisor/domain_driver.c
> > +src/hypervisor/domain_interface.c
> >   src/hypervisor/virhostdev.c
> >   src/interface/interface_backend_netcf.c
> >   src/interface/interface_backend_udev.c
> > diff --git a/src/hypervisor/domain_interface.c b/src/hypervisor/domain_interface.c
> > new file mode 100644
> > index 0000000000..b01b6ee767
> > --- /dev/null
> > +++ b/src/hypervisor/domain_interface.c
> > @@ -0,0 +1,279 @@
> > +/*
> > + * Copyright Microsoft Corp. 2023
> 
> I haven't had time to look through the rest of this yet (although it seems
> pretty straightforward, and I think it might have been me who suggested it
> in the first place :-)), I did want to bring up this one topic now:
> 
> 
> Best practices for the copyright notice in a new file have changed over the
> years (for example, we no longer list the "Author" of new files and even
> removed Author from existing files, since a more accurate and complete
> version of that info is all available from git), and I haven't paid close
> attention, but since this entire file is comprised of functions that were
> just moved from an existing file and renamed (rather than actual new code),
> I don't think it's appropriate for it to erase all trace of the original
> copyright holder and simply assign the copyright entirely to Microsoft.

If the original file that the code was copied from had any
Copyright lines, those must be preserved in the new file.
If the original file had no copyright lines, you're not
required to do any archeology to figure out copyrighth
holders, just leave the new file similarly devoid.

Simply moving code is also not a copyrightable enhancement,
so adding your own copyright on moved code is inappropriate.

Once you add new functionality to the newfile though, you
are free to add your own copyright statement at that point,
if desired.


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux