From nobody Mon Feb 9 19:30:45 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass(p=quarantine dis=none) header.from=suse.com ARC-Seal: i=1; a=rsa-sha256; t=1699632561; cv=none; d=zohomail.com; s=zohoarc; b=kITeE95jKSYvuhe82mJjxO7YAuDT4UioUyY6ZNo9yt4O6ql1XDYtAbjqDUO15bmdK/tYCYeOxcgWQhGvHIuVJu1TI8VntfKjJX3ObVicOfNDV1JHZjf/dsbPqaBxtWpIhK8sRTqUmNUosaH6UqUU57BS7XfdIY4rvAPzKwB8DUA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1699632561; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=DXMUzNYrgUE7yKY/59/v2Vn6B+SqNKU5mV5KPcdYjpU=; b=bV5+i878FyxbpZ6vYwPCS+zs9e5Ve19jjC4WveAScuE2CEXa4bNzKaPP+itzWztX6UTRLdrL9cMpiswi1eUoML9RausjhT1lft/HCU/tU71xUWDbwXiSeI47oDvcc5iNf4KyEHmcEb80yqoDrDmM2eeiGzVL4w5kzEGWJ6BK7DU= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1699632561071402.8410570154367; Fri, 10 Nov 2023 08:09:21 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.630476.983360 (Exim 4.92) (envelope-from ) id 1r1U3t-0006WT-D3; Fri, 10 Nov 2023 16:08:49 +0000 Received: by outflank-mailman (output) from mailman id 630476.983360; Fri, 10 Nov 2023 16:08:49 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1r1U3t-0006WG-9e; Fri, 10 Nov 2023 16:08:49 +0000 Received: by outflank-mailman (input) for mailman id 630476; Fri, 10 Nov 2023 16:08:48 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1r1U3s-0004cS-JM for xen-devel@lists.xenproject.org; Fri, 10 Nov 2023 16:08:48 +0000 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id 6d4ee1bb-7fe3-11ee-9b0e-b553b5be7939; Fri, 10 Nov 2023 17:08:46 +0100 (CET) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 3D5432187C; Fri, 10 Nov 2023 16:08:46 +0000 (UTC) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 14AF013398; Fri, 10 Nov 2023 16:08:46 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id i96uA45VTmWmCAAAMHmgww (envelope-from ); Fri, 10 Nov 2023 16:08:46 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 6d4ee1bb-7fe3-11ee-9b0e-b553b5be7939 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1699632526; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=DXMUzNYrgUE7yKY/59/v2Vn6B+SqNKU5mV5KPcdYjpU=; b=fqE2QvSzbCJMwaxIhGm0OkBdBxXN6SUXC82ZvxfZaKcwnfbjpOoe0eg8jO/xT6vYkW0EM7 JWOBKwBKgtcBNXHsxu2PtauLhlPTTzHHhRe0//VV7MpMtNpqle12Ib6mqIoaO5lSEueOuQ RQFfe9ClnyuKyvPk47PzZQEPb5vkEXQ= From: Juergen Gross To: xen-devel@lists.xenproject.org Cc: Juergen Gross , Wei Liu , Anthony PERARD Subject: [PATCH v2 07/29] tools/xenlogd: add 9pfs attach request support Date: Fri, 10 Nov 2023 17:07:42 +0100 Message-Id: <20231110160804.29021-8-jgross@suse.com> X-Mailer: git-send-email 2.35.3 In-Reply-To: <20231110160804.29021-1-jgross@suse.com> References: <20231110160804.29021-1-jgross@suse.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @suse.com) X-ZM-MESSAGEID: 1699632562010100005 Content-Type: text/plain; charset="utf-8" Add the attach request of the 9pfs protocol. This introduces the "fid" scheme of the 9pfs protocol. As this will be needed later, use a dedicated memory allocation function in alloc_fid() and prepare a fid reference count. For filling the qid data take the approach from the qemu 9pfs backend implementation. Signed-off-by: Juergen Gross Reviewed-by: Jason Andryuk --- V2: - make fill_qid() parameter stbuf const (Jason Andryuk) - free fids after disconnecting guest (Jason Andryuk) --- tools/xen-9pfsd/io.c | 155 ++++++++++++++++++++++++++++++++++++ tools/xen-9pfsd/xen-9pfsd.c | 6 ++ tools/xen-9pfsd/xen-9pfsd.h | 14 ++++ 3 files changed, 175 insertions(+) diff --git a/tools/xen-9pfsd/io.c b/tools/xen-9pfsd/io.c index 18f81319b4..03e45d5f8b 100644 --- a/tools/xen-9pfsd/io.c +++ b/tools/xen-9pfsd/io.c @@ -16,6 +16,8 @@ #include #include #include +#include +#include #include /* For cpu barriers. */ #include =20 @@ -23,6 +25,7 @@ =20 /* P9 protocol commands (response is either cmd+1 or P9_CMD_ERROR). */ #define P9_CMD_VERSION 100 +#define P9_CMD_ATTACH 104 #define P9_CMD_ERROR 107 =20 #define P9_MIN_MSIZE 2048 @@ -443,6 +446,118 @@ static int fill_data(struct ring *ring, const char *f= mt, ...) return pars; } =20 +static struct p9_fid *find_fid(device *device, unsigned int fid) +{ + struct p9_fid *fidp; + + XEN_TAILQ_FOREACH(fidp, &device->fids, list) + { + if ( fidp->fid =3D=3D fid ) + return fidp; + } + + return NULL; +} + +static struct p9_fid *alloc_fid_mem(device *device, unsigned int fid, + const char *path) +{ + struct p9_fid *fidp; + size_t pathlen; + + pathlen =3D strlen(device->host_path) + strlen(path) + 1; + fidp =3D calloc(sizeof(*fidp) + pathlen, 1); + if ( !fidp ) + return NULL; + + fidp->fid =3D fid; + snprintf(fidp->path, pathlen, "%s%s", device->host_path, path); + + return fidp; +} + +static struct p9_fid *alloc_fid(device *device, unsigned int fid, + const char *path) +{ + struct p9_fid *fidp =3D NULL; + + pthread_mutex_lock(&device->fid_mutex); + + if ( find_fid(device, fid) ) + { + errno =3D EBADFD; + goto out; + } + + if ( device->n_fids >=3D device->max_open_files ) + { + errno =3D EMFILE; + goto out; + } + + fidp =3D alloc_fid_mem(device, fid, path); + if ( !fidp ) + goto out; + + fidp->ref =3D 1; + XEN_TAILQ_INSERT_HEAD(&device->fids, fidp, list); + device->n_fids++; + + out: + pthread_mutex_unlock(&device->fid_mutex); + + return fidp; +} + +static void free_fid(device *device, struct p9_fid *fidp) +{ + if ( !fidp ) + return; + + pthread_mutex_lock(&device->fid_mutex); + + fidp->ref--; + if ( !fidp->ref ) + { + device->n_fids--; + XEN_TAILQ_REMOVE(&device->fids, fidp, list); + free(fidp); + } + + pthread_mutex_unlock(&device->fid_mutex); +} + +void free_fids(device *device) +{ + struct p9_fid *fidp; + + while ( (fidp =3D XEN_TAILQ_FIRST(&device->fids)) !=3D NULL ) + { + XEN_TAILQ_REMOVE(&device->fids, fidp, list); + free(fidp); + } +} + +static int fill_qid(const char *path, struct p9_qid *qid, + const struct stat *stbuf) +{ + struct stat st; + + if ( !stbuf ) + { + if ( stat(path, &st) ) + return errno; + + stbuf =3D &st; + } + + qid->type =3D S_ISDIR(stbuf->st_mode) ? QID_TYPE_DIR : 0; + qid->version =3D stbuf->st_mtime ^ (stbuf->st_size << 8); + qid->path =3D stbuf->st_ino; + + return 0; +} + static void p9_error(struct ring *ring, uint16_t tag, uint32_t err) { unsigned int erroff; @@ -484,6 +599,42 @@ static void p9_version(struct ring *ring, struct p9_he= ader *hdr) fill_buffer(ring, hdr->cmd + 1, hdr->tag, "US", &ring->max_size, versi= on); } =20 +static void p9_attach(struct ring *ring, struct p9_header *hdr) +{ + device *device =3D ring->device; + uint32_t fid; + uint32_t dummy_u32; + unsigned int dummy_uint; + struct p9_qid qid; + int ret; + + ret =3D fill_data(ring, "UUSSU", &fid, &dummy_u32, &dummy_uint, &dummy= _uint, + &dummy_u32); + if ( ret !=3D 5 ) + { + p9_error(ring, hdr->tag, errno); + return; + } + + device->root_fid =3D alloc_fid(device, fid, ""); + if ( !device->root_fid ) + { + p9_error(ring, hdr->tag, errno); + return; + } + + ret =3D fill_qid(device->host_path, &qid, NULL); + if ( ret ) + { + free_fid(device, device->root_fid); + device->root_fid =3D NULL; + p9_error(ring, hdr->tag, ret); + return; + } + + fill_buffer(ring, hdr->cmd + 1, hdr->tag, "Q", &qid); +} + void *io_thread(void *arg) { struct ring *ring =3D arg; @@ -543,6 +694,10 @@ void *io_thread(void *arg) p9_version(ring, &hdr); break; =20 + case P9_CMD_ATTACH: + p9_attach(ring, &hdr); + break; + default: syslog(LOG_DEBUG, "%u.%u sent unhandled command %u\n", ring->device->domid, ring->device->devid, hdr.cmd); diff --git a/tools/xen-9pfsd/xen-9pfsd.c b/tools/xen-9pfsd/xen-9pfsd.c index cc5734402d..534c0cefa5 100644 --- a/tools/xen-9pfsd/xen-9pfsd.c +++ b/tools/xen-9pfsd/xen-9pfsd.c @@ -278,6 +278,9 @@ static device *new_device(unsigned int domid, unsigned = int devid) return NULL; } =20 + pthread_mutex_init(&device->fid_mutex, NULL); + XEN_TAILQ_INIT(&device->fids); + val =3D read_backend_node(device, "security_model"); if ( !val || strcmp(val, "none") ) { @@ -370,6 +373,8 @@ static void disconnect_guest(device *device) } =20 device->num_rings =3D 0; + + free_fids(device); } =20 static void close_device(device *device, enum xenbus_state state) @@ -454,6 +459,7 @@ static void remove_device(device *device) XEN_TAILQ_REMOVE(&devs, device, list); =20 disconnect_guest(device); + pthread_mutex_destroy(&device->fid_mutex); free_device(device); } =20 diff --git a/tools/xen-9pfsd/xen-9pfsd.h b/tools/xen-9pfsd/xen-9pfsd.h index 5ff87ca5bd..e511acdb76 100644 --- a/tools/xen-9pfsd/xen-9pfsd.h +++ b/tools/xen-9pfsd/xen-9pfsd.h @@ -21,6 +21,13 @@ struct p9_header { uint16_t tag; } __attribute__((packed)); =20 +struct p9_fid { + XEN_TAILQ_ENTRY(struct p9_fid) list; + unsigned int fid; + unsigned int ref; + char path[]; +}; + typedef struct device device; =20 struct ring { @@ -70,10 +77,17 @@ struct device { enum xenbus_state frontend_state; unsigned int num_rings; struct ring *ring[MAX_RINGS]; + + /* File system handling. */ + pthread_mutex_t fid_mutex; + XEN_TAILQ_HEAD(fidhead, struct p9_fid) fids; + struct p9_fid *root_fid; + unsigned int n_fids; }; =20 extern xenevtchn_handle *xe; =20 void *io_thread(void *arg); +void free_fids(device *device); =20 #endif /* XEN_9PFSD_H */ --=20 2.35.3