Re: [PATCH v1] tools/hv: reduce resouce usage in hv_kvp_daemon

2024-12-08 Thread Olaf Hering
Sun, 8 Dec 2024 23:52:54 + Wei Liu : > There's one conflict caused by an earlier patch by Vitaly. I've resolved > the conflict and applied the patch to hyperv-fixes. Please check the > final result. Commit b8ea8cd0fbd358feee3e9172c5ef8afd671e0d11 looks good, thanks. Olaf pgp7RZdDSAY7I.pgp

[PATCH v1] tools/hv: reduce resouce usage in hv_kvp_daemon

2024-12-02 Thread Olaf Hering
;exec'. Signed-off-by: Olaf Hering --- tools/hv/hv_kvp_daemon.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/hv/hv_kvp_daemon.c b/tools/hv/hv_kvp_daemon.c index ae57bf69ad4a..91b50cadfc52 100644 --- a/tools/hv/hv_kvp_daemon.c +++ b/tools/hv/hv_kvp_daemon.c

[PATCH v1] tools/hv: add a .gitignore file

2024-12-02 Thread Olaf Hering
Remove generated files from 'git status' output after 'make -C tools/hv'. Signed-off-by: Olaf Hering --- tools/hv/.gitignore | 3 +++ 1 file changed, 3 insertions(+) create mode 100644 tools/hv/.gitignore diff --git a/tools/hv/.gitignore b/tools/hv/.gitignore new fi

[PATCH v1] tools/hv: reduce resouce usage in hv_get_dns_info helper

2024-12-02 Thread Olaf Hering
Remove the usage of cat. Replace the shell process with awk with 'exec'. Also use a generic shell because no bash specific features will be used. Signed-off-by: Olaf Hering --- tools/hv/hv_get_dns_info.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v1] tools/hv: update route parsing in kvp daemon

2024-12-02 Thread Olaf Hering
xthop via fe80::26fc:4e00:3b:74 dev eth0 weight 1 Adjust the route parsing to use a single line for each line of output. Also use a single shell invocation to retrieve both IPv4 and IPv6 information. The actual IP addresses are expected after the "via" keyword. Signed-off-by

[PATCH v2] tools/hv: terminate fcopy daemon if read from uio fails

2024-11-05 Thread Olaf Hering
tem call, and continue in this case. Signed-off-by: Olaf Hering --- v2: update commit message A more complete fix is to handle this properly in the kernel, by making the file descriptor unavailable for further operations. tools/hv/hv_fcopy_uio_daemon.c | 4 +++- 1 file changed, 3 insertions(+

Re: [PATCH v1] tools/hv: terminate fcopy daemon if read from uio fails

2024-10-28 Thread Olaf Hering
Mon, 28 Oct 2024 02:06:20 -0700 Saurabh Singh Sengar : > Thanks for the patch. Changes look good to me, I will suggest to improve this > log message incase the error type is EIO by suggesting that users verify if > 'Guest Services' is enabled. No error happens if the state of "Guest services" re

[PATCH v1] tools/hv: terminate fcopy daemon if read from uio fails

2024-10-25 Thread Olaf Hering
Terminate endless loop in reading fails, to avoid flooding syslog. This happens if the "Guest services" integration service is disabled at runtime in the VM settings. Also handle an interrupted system call, and continue in this case. Signed-off-by: Olaf Hering --- A more complete

[PATCH v1] tools: hv: change permissions of NetworkManager configuration file

2024-10-16 Thread Olaf Hering
Align permissions of the resulting .nmconnection file, instead of the input file from hv_kvp_daemon. To avoid the tiny time frame where the output file is world-readable, use umask instead of chmod. Fixes: 42999c90 ("Support for keyfile based connection profile") Signed-off-by: O

[PATCH v2] hyperv: reduce size of ms_hyperv_info

2023-09-22 Thread Olaf Hering
Use the hole prior shared_gpa_boundary to store the result of get_vtl. This reduces the size by 8 bytes. Signed-off-by: Olaf Hering --- v2: move vtl up, as suggested by Dexuan Cui include/asm-generic/mshyperv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/asm

Re: [PATCH v1] hyperv: reduce size of ms_hyperv_info

2023-09-19 Thread Olaf Hering
Tue, 19 Sep 2023 06:18:53 + Dexuan Cui : > How about moving the 'vtl' field to an even earlier place: I have not tried it, but I think this would just move the hole up. The end result will likely be the same, pahole -E vmlinux will show it. Olaf pgpDmiOmWAMyl.pgp Description: Digitale Sig

[PATCH v1] hyperv: reduce size of ms_hyperv_info

2023-09-18 Thread Olaf Hering
Use the hole prior shared_gpa_boundary to store the result of get_vtl. This reduces the size by 8 bytes. Signed-off-by: Olaf Hering --- include/asm-generic/mshyperv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/asm-generic/mshyperv.h b/include/asm-generic