From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <pve-devel-bounces@lists.proxmox.com> Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id 253A61FF162 for <inbox@lore.proxmox.com>; Sat, 22 Mar 2025 16:31:15 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id B3A3F3C5B4; Sat, 22 Mar 2025 16:31:04 +0100 (CET) To: pve-devel@lists.proxmox.com Date: Sun, 23 Mar 2025 00:17:19 +0900 In-Reply-To: <20250322152004.1646886-1-jing@jing.rocks> References: <20250322152004.1646886-1-jing@jing.rocks> MIME-Version: 1.0 Message-ID: <mailman.92.1742657463.359.pve-devel@lists.proxmox.com> List-Id: Proxmox VE development discussion <pve-devel.lists.proxmox.com> List-Post: <mailto:pve-devel@lists.proxmox.com> From: Jing Luo via pve-devel <pve-devel@lists.proxmox.com> Precedence: list Cc: Jing Luo <jing@jing.rocks> X-Mailman-Version: 2.1.29 X-BeenThere: pve-devel@lists.proxmox.com List-Subscribe: <https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel>, <mailto:pve-devel-request@lists.proxmox.com?subject=subscribe> List-Unsubscribe: <https://lists.proxmox.com/cgi-bin/mailman/options/pve-devel>, <mailto:pve-devel-request@lists.proxmox.com?subject=unsubscribe> List-Archive: <http://lists.proxmox.com/pipermail/pve-devel/> Reply-To: Proxmox VE development discussion <pve-devel@lists.proxmox.com> List-Help: <mailto:pve-devel-request@lists.proxmox.com?subject=help> Subject: [pve-devel] [PATCH qemu-server] tree-wide: change /var/run to /run and /var/lock to /run/lock Content-Type: multipart/mixed; boundary="===============1628971884275870679==" Errors-To: pve-devel-bounces@lists.proxmox.com Sender: "pve-devel" <pve-devel-bounces@lists.proxmox.com> --===============1628971884275870679== Content-Type: message/rfc822 Content-Disposition: inline Return-Path: <jing@jing.rocks> X-Original-To: pve-devel@lists.proxmox.com Delivered-To: pve-devel@lists.proxmox.com Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id 0C4E4C944B for <pve-devel@lists.proxmox.com>; Sat, 22 Mar 2025 16:31:03 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 1EA2A3C362 for <pve-devel@lists.proxmox.com>; Sat, 22 Mar 2025 16:30:32 +0100 (CET) Received: from mail-gw3.jing.rocks (mail-gw3.jing.rocks [219.117.250.209]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS for <pve-devel@lists.proxmox.com>; Sat, 22 Mar 2025 16:30:27 +0100 (CET) Received: from mail-gw3.jing.rocks (localhost [127.0.0.1]) by mail-gw3.jing.rocks (Proxmox) with ESMTP id B8462164F3 for <pve-devel@lists.proxmox.com>; Sun, 23 Mar 2025 00:20:23 +0900 (JST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=jing.rocks; h=cc :cc:content-transfer-encoding:date:from:from:in-reply-to :message-id:mime-version:references:reply-to:subject:subject:to :to; s=mail-gw; bh=+BDxtf32J9gPT2O0w9gELa06zP4v6lrQo4eK0K38aYo=; b= bBViTMQqWJ+UYp6wCqibDG+SXza6r0VUTPeHzRS3LTGdZgh1s9fDn/8z+FrWu34m d4HlhKeyKgSlh9LLQoNzbhyz5f4r3aKubhVDAlLw4IEjTv45JNcG6jolBQT5QcSJ qsQV312IPZ2yyyvmkxJXPNvjvB7+paY3LJSZwGdygzyVjMCivPZTIkab2aGKRjLf VbU01U7dlqiM0pgaHtiNYnNwLDqhFtWV8reUieIJKqiyNrkov0HwTRWbR0zvmUqf L8NOT1ZDbkBPaAbEIRseXdemD0DVDItz4ce3TgmJJT/agsolgf2bLm8ot1gfbWVm EBJTB+DXsKdBNbTokH3DLw== Received: from mail.jing.rocks (mail.jing.rocks [192.168.0.222]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail-gw3.jing.rocks (Proxmox) with ESMTPS id CB3F7165F1 for <pve-devel@lists.proxmox.com>; Sun, 23 Mar 2025 00:20:22 +0900 (JST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=jing.rocks; s=default; t=1742656822; bh=24oaaOEwvU9O/2J5/wVBjmIo3DFvQVCzXk01kWUCGe8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=dgPu5rN61etEQzTU6YTRVkg02qar2Kdg8+OLKUwh2ImtKd++0ztT1ywM0vLKGnitF yiePPaHFfcqjir74Ia8Vt5ybyBktwLqNIT3O91kf5O/584RUUuC0+UiDiYGit5PWhv Vtlhq1/DRzEyokt5EQGhK3oarH/ZyAMKOK9CzeKDsvNxtS1r8/nA3KgN2Qa1XgDq+H UtzvJs6B2lXCkHgQnOPIWbogD30AfoFZRxIKhlTiED+dRcZvyBr3ZUFfoEcOdQmr2L GW5mtuZCUhHrC4tYYF0zBn/nfecmAIuqUV7omI6XhoqCNcugU7uHD1wHJRX8XEHJHr 8iXsedD8B9WSw== Received: from X570AM.jing.rocks (X570AM.lan [IPv6:240b:10:f00:1b00::7e82]) (Authenticated sender: jing@jing.rocks) by mail.jing.rocks (Postfix) with ESMTPSA id C5B5F37F46; Sun, 23 Mar 2025 00:20:22 +0900 (JST) From: Jing Luo <jing@jing.rocks> To: pve-devel@lists.proxmox.com Cc: Jing Luo <jing@jing.rocks> Subject: [PATCH qemu-server] tree-wide: change /var/run to /run and /var/lock to /run/lock Date: Sun, 23 Mar 2025 00:17:19 +0900 Message-ID: <20250322152004.1646886-12-jing@jing.rocks> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250322152004.1646886-1-jing@jing.rocks> References: <20250322152004.1646886-1-jing@jing.rocks> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 0 AWL 0.000 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DKIM_SIGNED 0.1 Message has a DKIM or DK signature, not necessarily valid DKIM_VALID -0.1 Message has at least one valid DKIM or DK signature DKIM_VALID_AU -0.1 Message has a valid DKIM or DK signature from author's domain DKIM_VALID_EF -0.1 Message has a valid DKIM or DK signature from envelope-from domain DMARC_PASS -0.1 DMARC pass policy KAM_INFOUSMEBIZ 0.75 Prevalent use of .info|.us|.me|.me.uk|.biz|xyz|id|rocks|life domains in spam/malware KAM_OTHER_BAD_TLD 0.75 Other untrustworthy TLDs SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record "/var/run" and "/var/lock" are deprecated. This is to comply with Debian Policy 9.1.4 "/run and /run/lock". (https://www.debian.org/doc/debian-policy/ch-opersys.html#run-and-run-lock) Signed-off-by: Jing Luo <jing@jing.rocks> --- PVE/CLI/qm.pm | 2 +- PVE/QemuConfig.pm | 2 +- PVE/QemuServer.pm | 14 +++++++------- PVE/QemuServer/Helpers.pm | 2 +- PVE/QemuServer/Memory.pm | 2 +- qmeventd/qmeventd.service | 4 ++-- 6 files changed, 13 insertions(+), 13 deletions(-) diff --git a/PVE/CLI/qm.pm b/PVE/CLI/qm.pm index 4214a7ca..5e1321c6 100755 --- a/PVE/CLI/qm.pm +++ b/PVE/CLI/qm.pm @@ -703,7 +703,7 @@ __PACKAGE__->register_method ({ die "VM $vmid not running\n" if !PVE::QemuServer::check_running($vmid); - my $socket = "/var/run/qemu-server/${vmid}.$iface"; + my $socket = "/run/qemu-server/${vmid}.$iface"; my $cmd = "socat UNIX-CONNECT:$socket STDIO,raw,echo=0$escape"; diff --git a/PVE/QemuConfig.pm b/PVE/QemuConfig.pm index ffdf9f03..b414db99 100644 --- a/PVE/QemuConfig.pm +++ b/PVE/QemuConfig.pm @@ -24,7 +24,7 @@ my $nodename = PVE::INotify::nodename(); mkdir "/etc/pve/nodes/$nodename"; mkdir "/etc/pve/nodes/$nodename/qemu-server"; -my $lock_dir = "/var/lock/qemu-server"; +my $lock_dir = "/run/lock/qemu-server"; mkdir $lock_dir; sub assert_config_exists_on_node { diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 5bb86f7a..268140ce 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -3204,8 +3204,8 @@ sub audio_devs { sub get_tpm_paths { my ($vmid) = @_; return { - socket => "/var/run/qemu-server/$vmid.swtpm", - pid => "/var/run/qemu-server/$vmid.swtpm.pid", + socket => "/run/qemu-server/$vmid.swtpm", + pid => "/run/qemu-server/$vmid.swtpm.pid", }; } @@ -3461,7 +3461,7 @@ sub query_supported_cpu_flags { $qemu_cmd, '-machine', $default_machine, '-display', 'none', - '-chardev', "socket,id=qmp,path=/var/run/qemu-server/$fakevmid.qmp,server=on,wait=off", + '-chardev', "socket,id=qmp,path=/run/qemu-server/$fakevmid.qmp,server=on,wait=off", '-mon', 'chardev=qmp,mode=control', '-pidfile', $pidfile, '-S', '-daemonize' @@ -3710,7 +3710,7 @@ sub config_to_command { push @$cmd, '-mon', "chardev=qmp,mode=control"; if (min_version($machine_version, 2, 12)) { - push @$cmd, '-chardev', "socket,id=qmp-event,path=/var/run/qmeventd.sock,reconnect=5"; + push @$cmd, '-chardev', "socket,id=qmp-event,path=/run/qmeventd.sock,reconnect=5"; push @$cmd, '-mon', "chardev=qmp-event,mode=control"; } @@ -3812,7 +3812,7 @@ sub config_to_command { for (my $i = 0; $i < $MAX_SERIAL_PORTS; $i++) { my $path = $conf->{"serial$i"} or next; if ($path eq 'socket') { - my $socket = "/var/run/qemu-server/${vmid}.serial$i"; + my $socket = "/run/qemu-server/${vmid}.serial$i"; push @$devices, '-chardev', "socket,id=serial$i,path=$socket,server=on,wait=off"; # On aarch64, serial0 is the UART device. QEMU only allows # connecting UART devices via the '-serial' command line, as @@ -6330,7 +6330,7 @@ sub vm_stop_cleanup { } foreach my $ext (qw(mon qmp pid vnc qga)) { - unlink "/var/run/qemu-server/${vmid}.$ext"; + unlink "/run/qemu-server/${vmid}.$ext"; } if ($conf->{ivshmem}) { @@ -8789,7 +8789,7 @@ sub register_qmeventd_handle { my ($vmid) = @_; my $fh; - my $peer = "/var/run/qmeventd.sock"; + my $peer = "/run/qmeventd.sock"; my $count = 0; for (;;) { diff --git a/PVE/QemuServer/Helpers.pm b/PVE/QemuServer/Helpers.pm index 72a46a0a..a27fcfdd 100644 --- a/PVE/QemuServer/Helpers.pm +++ b/PVE/QemuServer/Helpers.pm @@ -21,7 +21,7 @@ my $nodename = PVE::INotify::nodename(); # Paths and directories -our $var_run_tmpdir = "/var/run/qemu-server"; +our $var_run_tmpdir = "/run/qemu-server"; mkdir $var_run_tmpdir; sub qmp_socket { diff --git a/PVE/QemuServer/Memory.pm b/PVE/QemuServer/Memory.pm index e5024cd2..d87b3a06 100644 --- a/PVE/QemuServer/Memory.pm +++ b/PVE/QemuServer/Memory.pm @@ -726,7 +726,7 @@ sub hugepages_update_locked { my $timeout = 60; #could be long if a lot of hugepages need to be allocated - my $lock_filename = "/var/lock/hugepages.lck"; + my $lock_filename = "/run/lock/hugepages.lck"; my $res = lock_file($lock_filename, $timeout, $code, @param); die $@ if $@; diff --git a/qmeventd/qmeventd.service b/qmeventd/qmeventd.service index 1e2465be..d692d1a0 100644 --- a/qmeventd/qmeventd.service +++ b/qmeventd/qmeventd.service @@ -1,11 +1,11 @@ [Unit] Description=PVE Qemu Event Daemon -RequiresMountsFor=/var/run +RequiresMountsFor=/run Before=pve-ha-lrm.service Before=pve-guests.service [Service] -ExecStart=/usr/sbin/qmeventd /var/run/qmeventd.sock +ExecStart=/usr/sbin/qmeventd /run/qmeventd.sock Type=forking [Install] -- 2.49.0 --===============1628971884275870679== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel --===============1628971884275870679==--