From nobody Sun Nov 24 14:38:00 2024 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=reject dis=none) header.from=cloud.com ARC-Seal: i=1; a=rsa-sha256; t=1729182079; cv=none; d=zohomail.com; s=zohoarc; b=biy1UPIp9Xm6JNi0Uwpe3vcKVk9cVVFYHwPZtV5wqf6hQyio6OqXPpkeRHrjv4Qty5tGgoHitXRw4dwnwnHKaQVJKZmvZifkM0VvrAoP5vrdlqRI0y1RoNjrZ4EmYQRkc5RIyJNsiJ0DufW/HAGmpVqkQPXfFNU4Eonb7c2hsMM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1729182079; 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=kgnhx5agu8oUcqb9qagvcSva4nEzimHG4s1giyUXOgs=; b=f4QtiGR4W0B1R9hRMayt5saMhLz+sm/UcKze7o5A9KCnDV24Jkjunq5uOWEchm8ZRQtuAKG4d2fQBYCg7itDTQpVuip1uTY2/jbNLqJI3GSbLhcKiZaSsr+rHdRsEIybmOnYH3DlYfsvVNtopA1Oh59t7ZSxZ6dcl3E+oSahI3E= 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=reject dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1729182079717160.59627572565114; Thu, 17 Oct 2024 09:21:19 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.820763.1234373 (Exim 4.92) (envelope-from ) id 1t1TF3-0005Rq-Si; Thu, 17 Oct 2024 16:20:49 +0000 Received: by outflank-mailman (output) from mailman id 820763.1234373; Thu, 17 Oct 2024 16:20: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 1t1TF3-0005Re-Py; Thu, 17 Oct 2024 16:20:49 +0000 Received: by outflank-mailman (input) for mailman id 820763; Thu, 17 Oct 2024 16:20:48 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1t1TF2-0004vC-3B for xen-devel@lists.xenproject.org; Thu, 17 Oct 2024 16:20:48 +0000 Received: from mail-ej1-x630.google.com (mail-ej1-x630.google.com [2a00:1450:4864:20::630]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id c45a0a80-8ca3-11ef-a0be-8be0dac302b0; Thu, 17 Oct 2024 18:20:47 +0200 (CEST) Received: by mail-ej1-x630.google.com with SMTP id a640c23a62f3a-a99fa009adcso58626766b.0 for ; Thu, 17 Oct 2024 09:20:47 -0700 (PDT) Received: from localhost.localdomain ([185.25.67.249]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-5c98d77995esm2922739a12.65.2024.10.17.09.20.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 17 Oct 2024 09:20:46 -0700 (PDT) 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: c45a0a80-8ca3-11ef-a0be-8be0dac302b0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cloud.com; s=cloud; t=1729182047; x=1729786847; darn=lists.xenproject.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=kgnhx5agu8oUcqb9qagvcSva4nEzimHG4s1giyUXOgs=; b=DoxzSI2tFJ1jDiYPjcQ+I6PTOyZz3genmfTBvKgINuTb5soggOqQpzk1kbx6nSpR7d ab33TJelsGc0T5yHxITxS5TcSPlGG9Pkx2hjrGXq1ivFvW0AYw9LElRB8yVsRZXqRzDq 3D1YrpSF5gL1zzwhcjOXCTIWzcetZVQCCY5zI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1729182047; x=1729786847; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=kgnhx5agu8oUcqb9qagvcSva4nEzimHG4s1giyUXOgs=; b=f3qlJ6ruI6H5QFGqbPRkrx8EyhkOnrUGPrUmsAV5DMLMwLW1+KQDet2vIhizGTsDai OH+ja73d9PNICQJpaQFu2w241zF9O41GDYGFfgmkUJdtLht5OolCOxZeV0QLzr29gPcl 5svFi+3i60idHAOOb6jd8YEqVz2k5s7BiTu03W1s9RmK4GTeLx6+YH0bIYpGFw/2I9oY 5+depgMQxuDC2ZVnJOQSfI1QfA6UEb5jVLb3Ky151NW2XEhvmzJlWr3BPx/Pe41UfMLv c3gR6sA/21wiIMkpBnLWjQADTtVbjDeKxeBDCx01KxbtLsjue2SuqT/5XDc2NtURa3/U DGBA== X-Gm-Message-State: AOJu0Yy3aFmnFdrA2duiYYB5jA9JJqCepuXvIL4fbfxQ/22YMGUaGl+6 zXljrM2Q0ZvcmBGVQRXOjUlCdIp8LJlxuRnpdBYN05PYeF2RJFbmHF1Yl6c6sp7plah9ERCPBJH Nfwk= X-Google-Smtp-Source: AGHT+IGi48jW39nfLzkZ0jShufFEA0sV9ND93jydXIr0b+hTtD4pgFjW5DI8ncZwdeIP/YBXSsiSbA== X-Received: by 2002:a05:6402:5c9:b0:5c9:5745:de9a with SMTP id 4fb4d7f45d1cf-5c95ac15776mr22549115a12.9.1729182046684; Thu, 17 Oct 2024 09:20:46 -0700 (PDT) From: Javi Merino To: xen-devel@lists.xenproject.org Cc: Javi Merino , Christian Lindig , David Scott , Anthony PERARD , Andrew Cooper , Andrii Sultanov Subject: [XEN PATCH v1 2/3] ocaml/libs: Fill build failure due to unused variable in ocaml macro Date: Thu, 17 Oct 2024 17:20:20 +0100 Message-ID: X-Mailer: git-send-email 2.46.1 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @cloud.com) X-ZM-MESSAGEID: 1729182080655116600 Content-Type: text/plain; charset="utf-8" On Fedora 40, the build fails with: gcc -m64 -DBUILD_ID -fno-strict-aliasing -std=3Dgnu99 -Wall -Wstrict-pr= ototypes -Wno-unused-but-set-variable -Wno-unused-local-typedefs -Werror = -O2 -fomit-frame-pointer -D__XEN_INTERFACE_VERSION__=3D__XEN_LATEST_INTERFA= CE_VERSION__ -MMD -MP -MF .domain_getinfo_stubs_v1.o.d -D_LARGEFILE_SOURCE = -D_LARGEFILE64_SOURCE -fPIC -I/usr/lib64/ocaml -I /build/tools/ocaml/libs= /xsd_glue/domain_getinfo_plugin_v1/../../../libs -I /build/tools/ocaml/libs= /xsd_glue/domain_getinfo_plugin_v1/../../../libs/xsd_glue -I/build/tools/oc= aml/libs/xsd_glue/domain_getinfo_plugin_v1/../../../../../tools/include -D_= _XEN_TOOLS__ -I/build/tools/ocaml/libs\/xsd_glue/domain_getinfo_plugin_v1/.= ./../../../../tools/include -c -o domain_getinfo_stubs_v1.o domain_getinfo= _stubs_v1.c In file included from domain_getinfo_stubs_v1.c:10: domain_getinfo_stubs_v1.c: In function 'xsd_glue_failwith': /usr/lib64/ocaml/caml/memory.h:275:29: error: unused variable 'caml__fr= ame' [-Werror=3Dunused-variable] 275 | struct caml__roots_block *caml__frame =3D *caml_local_roots_p= tr | ^~~~~~~~~~~ domain_getinfo_stubs_v1.c:48:9: note: in expansion of macro 'CAMLparam0' 48 | CAMLparam0(); | ^~~~~~~~~~ cc1: all warnings being treated as errors The CAMLparam0 macro is defined in /usr/lib64/ocaml/caml/memory.h:255 as: #define CAMLparam0() = \ struct caml__roots_block** caml_local_roots_ptr =3D = \ (DO_CHECK_CAML_STATE ? Caml_check_caml_state() : (void)0, = \ &CAML_LOCAL_ROOTS); = \ struct caml__roots_block *caml__frame =3D *caml_local_roots_ptr We can't modify the macro. Mark the xsd_glue_failwith() function with ignore "-Wunused-variable" to prevent gcc from failing the build due to the unused variable. Fixes: a6576011a4d2 ("ocaml/libs: Implement a dynamically-loaded plugin for= Xenctrl.domain_getinfo") Signed-off-by: Javi Merino Acked-by: Christian Lindig --- While it is not ideal, this is the most effective option I have found that avoids treating warnings as errors. I can add a comment above the pragma explaining why if we decide that this is the way forward. .../domain_getinfo_plugin_v1/domain_getinfo_stubs_v1.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/ocaml/libs/xsd_glue/domain_getinfo_plugin_v1/domain_geti= nfo_stubs_v1.c b/tools/ocaml/libs/xsd_glue/domain_getinfo_plugin_v1/domain_= getinfo_stubs_v1.c index 7be386f4d4c2..df2b3c74c727 100644 --- a/tools/ocaml/libs/xsd_glue/domain_getinfo_plugin_v1/domain_getinfo_stu= bs_v1.c +++ b/tools/ocaml/libs/xsd_glue/domain_getinfo_plugin_v1/domain_getinfo_stu= bs_v1.c @@ -42,6 +42,8 @@ static struct custom_operations xsd_glue_xenctrl_ops =3D { .compare_ext =3D custom_compare_ext_default, /* Can't compare */ }; =20 +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wunused-variable" static void Noreturn xsd_glue_failwith( xc_interface *xch, const char *func, unsigned int line) { @@ -70,6 +72,7 @@ static void Noreturn xsd_glue_failwith( =20 caml_raise_with_arg(*caml_named_value("xsg.error_v1"), msg); } +#pragma GCC diagnostic pop #define xsd_glue_failwith(xch) xsd_glue_failwith(xch, __func__, __LINE__) =20 CAMLprim value stub_xsd_glue_xc_interface_open(value unit) --=20 2.46.1