util: Refactor fetching of virtual function list

[PATCH 0/6] util: Refactor fetching of virtual function list

Peter Krempa posted 6 patches 2 years, 8 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1628169131.git.pkrempa@redhat.com

src/conf/node_device_conf.c |  16 +++--
src/libvirt_private.syms    |   2 +
src/network/bridge_driver.c |  23 ++-----
src/util/virnetdev.c        |  65 +++---------------
src/util/virnetdev.h        |   8 +--
src/util/virpci.c           | 127 +++++++++++++++++++++---------------
src/util/virpci.h           |  24 +++++--
7 files changed, 122 insertions(+), 143 deletions(-)

Expand all Fold all

[PATCH 0/6] util: Refactor fetching of virtual function list

Posted by Peter Krempa 2 years, 8 months ago

Note that this applies on top of the 'VIR_APPEND_ELEMENT' refactor I've
sent earlier.

Pipeline:
https://gitlab.com/pipo.sk/libvirt/-/pipelines/348524810

Peter Krempa (6):
  virNetDevGetVirtualFunctions: Remove 'max_vfs' argument
  virPCIGetVirtualFunctions: Simplify cleanup of returned data
  virPCIGetVirtualFunctionIndex: Refactor cleanup
  virPCIGetNetName: Make 'physPortID' argument const
  virPCIGetVirtualFunctions: Fetch also network interface name if needed
  virNetDevGetVirtualFunctions: Directly return
    virPCIVirtualFunctionList

 src/conf/node_device_conf.c |  16 +++--
 src/libvirt_private.syms    |   2 +
 src/network/bridge_driver.c |  23 ++-----
 src/util/virnetdev.c        |  65 +++---------------
 src/util/virnetdev.h        |   8 +--
 src/util/virpci.c           | 127 +++++++++++++++++++++---------------
 src/util/virpci.h           |  24 +++++--
 7 files changed, 122 insertions(+), 143 deletions(-)

-- 
2.31.1

Re: [PATCH 0/6] util: Refactor fetching of virtual function list

Posted by Jano Tomko 2 years, 8 months ago

On a %A in %Y, Peter Krempa wrote:
> Note that this applies on top of the 'VIR_APPEND_ELEMENT' refactor I've
> sent earlier.
> 
> Pipeline:
> https://gitlab.com/pipo.sk/libvirt/-/pipelines/348524810
> 
> Peter Krempa (6):
>   virNetDevGetVirtualFunctions: Remove 'max_vfs' argument
>   virPCIGetVirtualFunctions: Simplify cleanup of returned data
>   virPCIGetVirtualFunctionIndex: Refactor cleanup
>   virPCIGetNetName: Make 'physPortID' argument const
>   virPCIGetVirtualFunctions: Fetch also network interface name if needed
>   virNetDevGetVirtualFunctions: Directly return
>     virPCIVirtualFunctionList
> 
>  src/conf/node_device_conf.c |  16 +++--
>  src/libvirt_private.syms    |   2 +
>  src/network/bridge_driver.c |  23 ++-----
>  src/util/virnetdev.c        |  65 +++---------------
>  src/util/virnetdev.h        |   8 +--
>  src/util/virpci.c           | 127 +++++++++++++++++++++---------------
>  src/util/virpci.h           |  24 +++++--
>  7 files changed, 122 insertions(+), 143 deletions(-)
> 

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano