From nobody Wed Nov 12 07:03:43 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1568648608; cv=none; d=zoho.com; s=zohoarc; b=fTQhGh7LEez7ylFVOZrN2WzIAyDZtI0nVydl7W7xAU0zyZtghNjyIbjy8kcRaa4K85OhIiEztRGVp+rhQ1qi1glhvLPXofknrzIUh+OiU4JyDOMYWTHZeokRe49ng3x8FwnAeMoNX0YfpvhTg8FZKXQO9AKDbkW9dk99T28VS7Y= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1568648608; h=Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=15ASw7Sda32hAHbQApt0nrRp6KAHHCRUIPuWH1EYvBg=; b=HhEz/7DH6yVBXIO+ZeoGDzOcPc4AHG++J3U1vPRWXAh5TAeREh2uOXiMRzaLiv30gW89hzMtDxRIIs10obcYP3d7qwM6Q0rpE69OKv8NjJ8uOzM8CrfPn9w3r1dndsKGmUTru4YHuXWQ4BZ6K8EZO+pTLz/mAqWU0uUzuEFEez0= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1568648608359580.5089980188878; Mon, 16 Sep 2019 08:43:28 -0700 (PDT) Received: from localhost ([::1]:36074 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1i9t9y-0001Hz-Bc for importer@patchew.org; Mon, 16 Sep 2019 11:43:26 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:53715) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1i9sCy-0006WQ-8R for qemu-devel@nongnu.org; Mon, 16 Sep 2019 10:42:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1i9sCv-0007UB-Oq for qemu-devel@nongnu.org; Mon, 16 Sep 2019 10:42:28 -0400 Received: from mail-wr1-x444.google.com ([2a00:1450:4864:20::444]:45714) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1i9sCu-0007TP-VM for qemu-devel@nongnu.org; Mon, 16 Sep 2019 10:42:25 -0400 Received: by mail-wr1-x444.google.com with SMTP id r5so5213255wrm.12 for ; Mon, 16 Sep 2019 07:42:24 -0700 (PDT) Received: from 640k.lan ([93.56.166.5]) by smtp.gmail.com with ESMTPSA id i14sm17024298wra.78.2019.09.16.07.42.21 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 16 Sep 2019 07:42:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=15ASw7Sda32hAHbQApt0nrRp6KAHHCRUIPuWH1EYvBg=; b=OMOU6mtrfGuxY/0J7L+RmnKRgwzrrpCXLBu1CTTFUIFVF1eb0j3pD0cuxjpAhVyWJ1 w/y/BRDuqD8UvJYB6FuQypcnZbma6V8Jwmzx5b4e9oKrF1UQPDUhwUqzQ7X87iH9L3m+ QrzGSeVx5RuSIB1Q9qgehlSIZOeYJjC4DIZBuToDrX/Z03y0LW1Supn7p9wEV+Z3N7kf HWVbHwZihD5yZNvY+xradXxJkL1vePQSa3JXR75X3P7ytm6bHW5tVDjbiA2LzJsg2yRE am5Cby3z04DOgYRm3++Q6ecv0CgsoIz1lmqN0MhMbHPRBOVxrAmHW7qjrnkmh6mRiBHa kY5Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references; bh=15ASw7Sda32hAHbQApt0nrRp6KAHHCRUIPuWH1EYvBg=; b=DrJYvGzUauly16a2roQu5RYqnzSuUU+AJCr6wjDISCXneEJCrIOC5I5bEvkU3snZP8 hChMN8HYW0YdbhJ+w9LBWHhyE5oR4yAM6E1ifbz9hdfHcOdMS0BVggMKMjlLeMeMv8Od Z+BFTAmLvw8nzuA0MWmRUzOx9pQ1cdqyWqKBSG8qRiVEdbcPnFEXdt4ffuTIQrl2tIes n6Coi80iW29reVnhPu18SzfMVga57crSU5bDIM2jPs59xmOurGzVUB0jOtq0XcR8z1nk 9p6oOIG9bAtuag8B2+QdH7d64QCIlkbCoj/S2LaCDeEjcCwJVmQa7AdV68IiW5V0skq1 lemA== X-Gm-Message-State: APjAAAUBrr8anK6wBLwnoiROMt0kGPDXNpXJcn7wtjmFLS+Km9TZ35rz EIwC/oB4iAmn0eYJl4EEVs1j2LMY X-Google-Smtp-Source: APXvYqw2adYeU9ECLXlFSIAwn/FLPlPa0xwiPidSvx5xC1vhMCYPmb/IE3hRwcpqudxOTAg1I24pQQ== X-Received: by 2002:a5d:548d:: with SMTP id h13mr93168wrv.215.1568644943237; Mon, 16 Sep 2019 07:42:23 -0700 (PDT) From: Paolo Bonzini To: qemu-devel@nongnu.org Date: Mon, 16 Sep 2019 16:41:51 +0200 Message-Id: <1568644929-9124-12-git-send-email-pbonzini@redhat.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1568644929-9124-1-git-send-email-pbonzini@redhat.com> References: <1568644929-9124-1-git-send-email-pbonzini@redhat.com> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2a00:1450:4864:20::444 Subject: [Qemu-devel] [PULL 11/29] memory: fetch pmem size in get_file_size() X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Stefan Hajnoczi Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Stefan Hajnoczi Neither stat(2) nor lseek(2) report the size of Linux devdax pmem character device nodes. Commit 314aec4a6e06844937f1677f6cba21981005f389 ("hostmem-file: reject invalid pmem file sizes") added code to hostmem-file.c to fetch the size from sysfs and compare against the user-provided size=3DNUM parameter: if (backend->size > size) { error_setg(errp, "size property %" PRIu64 " is larger than " "pmem file \"%s\" size %" PRIu64, backend->size, fb->mem_path, size); return; } It turns out that exec.c:qemu_ram_alloc_from_fd() already has an equivalent size check but it skips devdax pmem character devices because lseek(2) returns 0: if (file_size > 0 && file_size < size) { error_setg(errp, "backing store %s size 0x%" PRIx64 " does not match 'size' option 0x" RAM_ADDR_FMT, mem_path, file_size, size); return NULL; } This patch moves the devdax pmem file size code into get_file_size() so that we check the memory size in a single place: qemu_ram_alloc_from_fd(). This simplifies the code and makes it more general. This also fixes the problem that hostmem-file only checks the devdax pmem file size when the pmem=3Don parameter is given. An unchecked size=3DNUM parameter can lead to SIGBUS in QEMU so we must always fetch the file size for Linux devdax pmem character device nodes. Signed-off-by: Stefan Hajnoczi Message-Id: <20190830093056.12572-1-stefanha@redhat.com> Reviewed-by: Eduardo Habkost Signed-off-by: Paolo Bonzini --- backends/hostmem-file.c | 22 -------------------- exec.c | 34 ++++++++++++++++++++++++++++++- include/qemu/osdep.h | 13 ------------ util/oslib-posix.c | 54 ---------------------------------------------= ---- util/oslib-win32.c | 6 ------ 5 files changed, 33 insertions(+), 96 deletions(-) diff --git a/backends/hostmem-file.c b/backends/hostmem-file.c index ecc15e3..be64020 100644 --- a/backends/hostmem-file.c +++ b/backends/hostmem-file.c @@ -58,28 +58,6 @@ file_backend_memory_alloc(HostMemoryBackend *backend, Er= ror **errp) return; } =20 - /* - * Verify pmem file size since starting a guest with an incorrect size - * leads to confusing failures inside the guest. - */ - if (fb->is_pmem) { - Error *local_err =3D NULL; - uint64_t size; - - size =3D qemu_get_pmem_size(fb->mem_path, &local_err); - if (local_err) { - error_propagate(errp, local_err); - return; - } - - if (size && backend->size > size) { - error_setg(errp, "size property %" PRIu64 " is larger than " - "pmem file \"%s\" size %" PRIu64, backend->size, - fb->mem_path, size); - return; - } - } - backend->force_prealloc =3D mem_prealloc; name =3D host_memory_backend_get_name(backend); memory_region_init_ram_from_file(&backend->mr, OBJECT(backend), diff --git a/exec.c b/exec.c index b9511be..8b99897 100644 --- a/exec.c +++ b/exec.c @@ -1791,7 +1791,39 @@ long qemu_maxrampagesize(void) #ifdef CONFIG_POSIX static int64_t get_file_size(int fd) { - int64_t size =3D lseek(fd, 0, SEEK_END); + int64_t size; +#if defined(__linux__) + struct stat st; + + if (fstat(fd, &st) < 0) { + return -errno; + } + + /* Special handling for devdax character devices */ + if (S_ISCHR(st.st_mode)) { + g_autofree char *subsystem_path =3D NULL; + g_autofree char *subsystem =3D NULL; + + subsystem_path =3D g_strdup_printf("/sys/dev/char/%d:%d/subsystem", + major(st.st_rdev), minor(st.st_rd= ev)); + subsystem =3D g_file_read_link(subsystem_path, NULL); + + if (subsystem && g_str_has_suffix(subsystem, "/dax")) { + g_autofree char *size_path =3D NULL; + g_autofree char *size_str =3D NULL; + + size_path =3D g_strdup_printf("/sys/dev/char/%d:%d/size", + major(st.st_rdev), minor(st.st_rdev)); + + if (g_file_get_contents(size_path, &size_str, NULL, NULL)) { + return g_ascii_strtoll(size_str, NULL, 0); + } + } + } +#endif /* defined(__linux__) */ + + /* st.st_size may be zero for special files yet lseek(2) works */ + size =3D lseek(fd, 0, SEEK_END); if (size < 0) { return -errno; } diff --git a/include/qemu/osdep.h b/include/qemu/osdep.h index af2b91f..c7d242f 100644 --- a/include/qemu/osdep.h +++ b/include/qemu/osdep.h @@ -571,19 +571,6 @@ void os_mem_prealloc(int fd, char *area, size_t sz, in= t smp_cpus, Error **errp); =20 /** - * qemu_get_pmem_size: - * @filename: path to a pmem file - * @errp: pointer to a NULL-initialized error object - * - * Determine the size of a persistent memory file. Besides supporting fil= es on - * DAX file systems, this function also supports Linux devdax character - * devices. - * - * Returns: the size or 0 on failure - */ -uint64_t qemu_get_pmem_size(const char *filename, Error **errp); - -/** * qemu_get_pid_name: * @pid: pid of a process * diff --git a/util/oslib-posix.c b/util/oslib-posix.c index 5fda67d..f869338 100644 --- a/util/oslib-posix.c +++ b/util/oslib-posix.c @@ -514,60 +514,6 @@ void os_mem_prealloc(int fd, char *area, size_t memory= , int smp_cpus, } } =20 -uint64_t qemu_get_pmem_size(const char *filename, Error **errp) -{ - struct stat st; - - if (stat(filename, &st) < 0) { - error_setg(errp, "unable to stat pmem file \"%s\"", filename); - return 0; - } - -#if defined(__linux__) - /* Special handling for devdax character devices */ - if (S_ISCHR(st.st_mode)) { - char *subsystem_path =3D NULL; - char *subsystem =3D NULL; - char *size_path =3D NULL; - char *size_str =3D NULL; - uint64_t ret =3D 0; - - subsystem_path =3D g_strdup_printf("/sys/dev/char/%d:%d/subsystem", - major(st.st_rdev), minor(st.st_rd= ev)); - subsystem =3D g_file_read_link(subsystem_path, NULL); - if (!subsystem) { - error_setg(errp, "unable to read subsystem for pmem file \"%s\= "", - filename); - goto devdax_err; - } - - if (!g_str_has_suffix(subsystem, "/dax")) { - error_setg(errp, "pmem file \"%s\" is not a dax device", filen= ame); - goto devdax_err; - } - - size_path =3D g_strdup_printf("/sys/dev/char/%d:%d/size", - major(st.st_rdev), minor(st.st_rdev)); - if (!g_file_get_contents(size_path, &size_str, NULL, NULL)) { - error_setg(errp, "unable to read size for pmem file \"%s\"", - size_path); - goto devdax_err; - } - - ret =3D g_ascii_strtoull(size_str, NULL, 0); - -devdax_err: - g_free(size_str); - g_free(size_path); - g_free(subsystem); - g_free(subsystem_path); - return ret; - } -#endif /* defined(__linux__) */ - - return st.st_size; -} - char *qemu_get_pid_name(pid_t pid) { char *name =3D NULL; diff --git a/util/oslib-win32.c b/util/oslib-win32.c index 9583fb4..c62cd43 100644 --- a/util/oslib-win32.c +++ b/util/oslib-win32.c @@ -562,12 +562,6 @@ void os_mem_prealloc(int fd, char *area, size_t memory= , int smp_cpus, } } =20 -uint64_t qemu_get_pmem_size(const char *filename, Error **errp) -{ - error_setg(errp, "pmem support not available"); - return 0; -} - char *qemu_get_pid_name(pid_t pid) { /* XXX Implement me */ --=20 1.8.3.1