From nobody Sun Feb 8 12:39:05 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org ARC-Seal: i=1; a=rsa-sha256; t=1655180533; cv=none; d=zohomail.com; s=zohoarc; b=DQ/iclGS8L7/57E166ucnyjXHRD3fUf9G0IdslbP0+Pq/CopvYXQy4pHqfiNUSa+jsxJ5Mj5mUEcRofmBjtYYO3OMpV53pYxmn41b9GyHua9KA468BwYLcgnSZUOeQ/BOdmQ54fZywvTDP59eDsUCSGijTm9OCkCO8MWvmBFyIs= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1655180533; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=Az+dT+4hpKTWFpzpvd6dSLcOyh/SnPbqA6UEA7oKevo=; b=OM1goYskazOwzoOcrQ3db5ukD/DZg9knKEMHcGyHgjoW5eExiXGN/Xrzn5fW17zyxnYosBTHdMenB1+yyyp4iprK4SEvX8fvnT2+6DtscI8hkTdtfXF0o+K3c3RaBrfpEj9c7T4/KbqqkAPXqbiwpdlnR+pEy0fHFo2mqBU3qLQ= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1655180533276567.7776916308202; Mon, 13 Jun 2022 21:22:13 -0700 (PDT) Received: from localhost ([::1]:36106 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1o0y4C-0006Kv-79 for importer@patchew.org; Tue, 14 Jun 2022 00:22:12 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:48942) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1o0y25-0003Qo-L3 for qemu-devel@nongnu.org; Tue, 14 Jun 2022 00:20:01 -0400 Received: from mail-io1-xd30.google.com ([2607:f8b0:4864:20::d30]:33579) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1o0y23-0001dG-QH for qemu-devel@nongnu.org; Tue, 14 Jun 2022 00:20:01 -0400 Received: by mail-io1-xd30.google.com with SMTP id p69so8279706iod.0 for ; Mon, 13 Jun 2022 21:19:59 -0700 (PDT) Received: from dune.bsdimp.com (50-253-99-174-static.hfc.comcastbusiness.net. [50.253.99.174]) by smtp.gmail.com with ESMTPSA id a3-20020a5ec303000000b0066579afd3cbsm4803742iok.50.2022.06.13.21.19.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Jun 2022 21:19:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=Az+dT+4hpKTWFpzpvd6dSLcOyh/SnPbqA6UEA7oKevo=; b=1UP2Z2L0Yc75otrxndcwNVCwqvYN+3lXK9wBW3HfeEBRorFevvwgbl+RnEfuL8Am1z pQSfPQb+26cmwBEtPZAfkl7/ja5B9wYFzg1Xcj+XXVtOL+eOJOtXGcimrhZ3eHY6Sife TIAMD8jqcgDZdZS5r+FX+m97C5YPz1DJrU1ju8m4pJ3SKUptEyD0jXXURzvwJxRMPmRK DCcRa7gjPBhf1v2N15Z1t6UuV0M9OXXeqev1L41zpR3knUWyLeXF00121R/L7HjsFUYT z9oHhJ1iqskZexWZsvpC3YCi1u5qWQVal4GBcdGFNQH0uSdtUDipVm5JEDD2cnnBXUXh TsPA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=Az+dT+4hpKTWFpzpvd6dSLcOyh/SnPbqA6UEA7oKevo=; b=jfQFR3ci/wmBVatBxieKhA/I5mKc4bX4tYfuQHVZtuRQFG2JUwmCeHoyIym/HvgT6S 4X2AxcrB8YqLXlXQ6qmJF/5q3Csqg1MraVT+D02TCOBs0IDuLh+LMFAdGbdQxJdo+6ix TxabPZ/liSHnuqd0TNox7v8zOGVPDFymzdsUiXFTQUcEuH90l9uo3lXd+X4cSanWaX6G g2sCqEZ2IjKjRo4XpTyHqcaD/P8U1dquBhVaOqqsFBm5FEHUnUmRTMgWLASYftDUzCmj WEvkbvlouMmHocXgSVOsU/HBGvEaKSOrVaipg396nkxgxFmdTcCKfeTw4jVow2Gx3iED FfKA== X-Gm-Message-State: AOAM530vbA2hh06vbKBoFDmyhlJVG8krLM2laMSfe9+o6RmAhbNQyEaF Qub2DHw38IbXn9coKZrUiUPqnTslgPIBSg== X-Google-Smtp-Source: ABdhPJygxe1xB8dobK5Ak8hvlXA/GWNpFj0/fHB3cmOGGcdS7esrWOVrWLlonlGWR45wtFkGts1yWA== X-Received: by 2002:a02:2305:0:b0:331:a026:b650 with SMTP id u5-20020a022305000000b00331a026b650mr1764890jau.314.1655180397910; Mon, 13 Jun 2022 21:19:57 -0700 (PDT) From: Warner Losh To: qemu-devel@nongnu.org Cc: Kyle Evans , arrowd@freebsd.org, Richard Henderson , jrtc27@FreeBSD.org, Warner Losh , def@FreeBSD.org, Stacey Son , Jung-uk Kim , Kyle Evans Subject: [PATCH v2 01/11] bsd-user: Implement open, openat and close Date: Mon, 13 Jun 2022 22:20:00 -0600 Message-Id: <20220614042010.40327-2-imp@bsdimp.com> X-Mailer: git-send-email 2.33.1 In-Reply-To: <20220614042010.40327-1-imp@bsdimp.com> References: <20220614042010.40327-1-imp@bsdimp.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.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; Received-SPF: none client-ip=2607:f8b0:4864:20::d30; envelope-from=imp@bsdimp.com; helo=mail-io1-xd30.google.com X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_NONE=0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @bsdimp-com.20210112.gappssmtp.com) X-ZM-MESSAGEID: 1655180536195100002 Content-Type: text/plain; charset="utf-8" Add the open, openat and close system calls. We need to lock paths, so implmenent that as well. Signed-off-by: Stacey Son Signed-off-by: Jung-uk Kim Signed-off-by: Kyle Evans Signed-off-by: Warner Losh Reviewed-by: Richard Henderson --- bsd-user/bsd-file.h | 49 +++++++++++++++++++++++++++++++++++ bsd-user/freebsd/os-syscall.c | 16 ++++++++++++ bsd-user/syscall_defs.h | 4 +++ 3 files changed, 69 insertions(+) diff --git a/bsd-user/bsd-file.h b/bsd-user/bsd-file.h index e9e2c85eb67..2bd312f8e18 100644 --- a/bsd-user/bsd-file.h +++ b/bsd-user/bsd-file.h @@ -22,11 +22,25 @@ =20 #include "qemu/path.h" =20 +#define LOCK_PATH(p, arg) \ +do { \ + (p) =3D lock_user_string(arg); \ + if ((p) =3D=3D NULL) { \ + return -TARGET_EFAULT; \ + } \ +} while (0) + +#define UNLOCK_PATH(p, arg) unlock_user(p, arg, 0) + + extern struct iovec *lock_iovec(int type, abi_ulong target_addr, int count, int copy); extern void unlock_iovec(struct iovec *vec, abi_ulong target_addr, int cou= nt, int copy); =20 +int safe_open(const char *path, int flags, mode_t mode); +int safe_openat(int fd, const char *path, int flags, mode_t mode); + ssize_t safe_read(int fd, void *buf, size_t nbytes); ssize_t safe_pread(int fd, void *buf, size_t nbytes, off_t offset); ssize_t safe_readv(int fd, const struct iovec *iov, int iovcnt); @@ -190,4 +204,39 @@ static abi_long do_bsd_pwritev(void *cpu_env, abi_long= arg1, return ret; } =20 +/* open(2) */ +static abi_long do_bsd_open(abi_long arg1, abi_long arg2, abi_long arg3) +{ + abi_long ret; + void *p; + + LOCK_PATH(p, arg1); + ret =3D get_errno(safe_open(path(p), target_to_host_bitmask(arg2, + fcntl_flags_tbl), arg3)); + UNLOCK_PATH(p, arg1); + + return ret; +} + +/* openat(2) */ +static abi_long do_bsd_openat(abi_long arg1, abi_long arg2, + abi_long arg3, abi_long arg4) +{ + abi_long ret; + void *p; + + LOCK_PATH(p, arg2); + ret =3D get_errno(safe_openat(arg1, path(p), + target_to_host_bitmask(arg3, fcntl_flags_tbl), arg4)); + UNLOCK_PATH(p, arg2); + + return ret; +} + +/* close(2) */ +static inline abi_long do_bsd_close(abi_long arg1) +{ + return get_errno(close(arg1)); +} + #endif /* BSD_FILE_H */ diff --git a/bsd-user/freebsd/os-syscall.c b/bsd-user/freebsd/os-syscall.c index 71aa0d38e03..a824785fee8 100644 --- a/bsd-user/freebsd/os-syscall.c +++ b/bsd-user/freebsd/os-syscall.c @@ -44,6 +44,10 @@ #include "bsd-proc.h" =20 /* I/O */ +safe_syscall3(int, open, const char *, path, int, flags, mode_t, mode); +safe_syscall4(int, openat, int, fd, const char *, path, int, flags, mode_t, + mode); + safe_syscall3(ssize_t, read, int, fd, void *, buf, size_t, nbytes); safe_syscall4(ssize_t, pread, int, fd, void *, buf, size_t, nbytes, off_t, offset); @@ -257,6 +261,18 @@ static abi_long freebsd_syscall(void *cpu_env, int num= , abi_long arg1, ret =3D do_bsd_pwritev(cpu_env, arg1, arg2, arg3, arg4, arg5, arg6= ); break; =20 + case TARGET_FREEBSD_NR_open: /* open(2) */ + ret =3D do_bsd_open(arg1, arg2, arg3); + break; + + case TARGET_FREEBSD_NR_openat: /* openat(2) */ + ret =3D do_bsd_openat(arg1, arg2, arg3, arg4); + break; + + case TARGET_FREEBSD_NR_close: /* close(2) */ + ret =3D do_bsd_close(arg1); + break; + default: qemu_log_mask(LOG_UNIMP, "Unsupported syscall: %d\n", num); ret =3D -TARGET_ENOSYS; diff --git a/bsd-user/syscall_defs.h b/bsd-user/syscall_defs.h index f5797b28e39..b6d113d24a7 100644 --- a/bsd-user/syscall_defs.h +++ b/bsd-user/syscall_defs.h @@ -226,4 +226,8 @@ type safe_##name(type1 arg1, type2 arg2, type3 arg3, ty= pe4 arg4, \ return safe_syscall(SYS_##name, arg1, arg2, arg3, arg4, arg5, arg6); \ } =20 +/* So far all target and host bitmasks are the same */ +#define target_to_host_bitmask(x, tbl) (x) +#define host_to_target_bitmask(x, tbl) (x) + #endif /* SYSCALL_DEFS_H */ --=20 2.33.1