Re: [PATCH] virtiofs: rename member to 'openfiles' for clarity

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

 



On 7/23/24 14:47, Adam Julis wrote:
> New element 'openfiles' had confusing name. Since the patch with
> this new element wasn't propagate yet, old name ('rlimit_nofile')
> was changed.
> 
> ...
> <binary>
>   <openfiles max='122333'/>
> </binary>
> ...
> 
> Signed-off-by: Adam Julis <ajulis@xxxxxxxxxx>
> ---
>  docs/formatdomain.rst                              |  6 +++---
>  src/conf/domain_conf.c                             | 14 +++++++-------
>  src/conf/domain_conf.h                             |  2 +-
>  src/conf/schemas/domaincommon.rng                  |  4 ++--
>  src/qemu/qemu_virtiofs.c                           |  4 ++--
>  ... vhost-user-fs-fd-openfiles.x86_64-latest.args} |  0
>  .../vhost-user-fs-fd-openfiles.x86_64-latest.xml   |  1 +
>  ...d-rlimit.xml => vhost-user-fs-fd-openfiles.xml} |  2 +-
>  .../vhost-user-fs-fd-rlimit.x86_64-latest.xml      |  1 -
>  tests/qemuxmlconftest.c                            |  2 +-
>  10 files changed, 18 insertions(+), 18 deletions(-)
>  rename tests/qemuxmlconfdata/{vhost-user-fs-fd-rlimit.x86_64-latest.args => vhost-user-fs-fd-openfiles.x86_64-latest.args} (100%)
>  create mode 120000 tests/qemuxmlconfdata/vhost-user-fs-fd-openfiles.x86_64-latest.xml
>  rename tests/qemuxmlconfdata/{vhost-user-fs-fd-rlimit.xml => vhost-user-fs-fd-openfiles.xml} (97%)
>  delete mode 120000 tests/qemuxmlconfdata/vhost-user-fs-fd-rlimit.x86_64-latest.xml
> 

Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Michal



[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