From nobody Fri Oct 18 05:17:53 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=citrix.com ARC-Seal: i=1; a=rsa-sha256; t=1720046210; cv=none; d=zohomail.com; s=zohoarc; b=hy/xsljBv1+A6raOjau9/19dp8fxEM5J7fw8yHWkofhGZlH8BBvFugP1otoB63ucpPj0HQe4hwfmv8s1kW4nKi7CZ1HWAaLkU9nga6YRetkiHrrb0AcCk3P7uBOpYBR0glSKk8u64fjb4kh85hZvOUuu6qnuZK5RlTjSoFIbRIY= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1720046210; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=qartpj94QO9IIjwcNKztD+etg1loQQXr0SUDN4EHJJo=; b=Emuq8G7jkOwDVu6bN5jW3ujRr3qUnvd0urg5QMQdyhcagLe66zEVIfmZjMOUxPUq7oxp9VEahpKwZpTqGOIwvN4Bb35HUbmBOg87kbBi32GLG7EPcg8Q6hCj1QJtnt+36JXDXeESRNxj/8uGHqDxH6k3wyvSQPN8G/mCj2hNOXI= 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 1720046210377484.9829454485581; Wed, 3 Jul 2024 15:36:50 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.753337.1161570 (Exim 4.92) (envelope-from ) id 1sP8aL-00011e-7t; Wed, 03 Jul 2024 22:36:21 +0000 Received: by outflank-mailman (output) from mailman id 753337.1161570; Wed, 03 Jul 2024 22:36:21 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1sP8aL-00011X-4V; Wed, 03 Jul 2024 22:36:21 +0000 Received: by outflank-mailman (input) for mailman id 753337; Wed, 03 Jul 2024 22:36:20 +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 1sP8aK-00011R-Cu for xen-devel@lists.xenproject.org; Wed, 03 Jul 2024 22:36:20 +0000 Received: from mail-ej1-x635.google.com (mail-ej1-x635.google.com [2a00:1450:4864:20::635]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id a9b403a3-398c-11ef-8776-851b0ebba9a2; Thu, 04 Jul 2024 00:36:17 +0200 (CEST) Received: by mail-ej1-x635.google.com with SMTP id a640c23a62f3a-a727d9dd367so743866b.3 for ; Wed, 03 Jul 2024 15:36:17 -0700 (PDT) Received: from andrewcoop.eng.citrite.net ([185.25.67.249]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-a72aaf6336csm544875166b.70.2024.07.03.15.36.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 03 Jul 2024 15:36:14 -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: a9b403a3-398c-11ef-8776-851b0ebba9a2 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=citrix.com; s=google; t=1720046176; x=1720650976; darn=lists.xenproject.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=qartpj94QO9IIjwcNKztD+etg1loQQXr0SUDN4EHJJo=; b=sMi7d7TMyGg8Ml2jqL8aEGOLzFmiIMAvy0zBXfsw3qVG0epmOFnkKB/77gcZovnMss wksxJTeQsyHBgWAY2+u4IImQ1KRVPKrKkEaME6RDED9sMh/TyTN9drZ/RkshzwTUrLID Kbutj2KqBfSaAeEXxg/Lpn/qtzguiuECCk3gI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1720046176; x=1720650976; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=qartpj94QO9IIjwcNKztD+etg1loQQXr0SUDN4EHJJo=; b=adevRxh2zl5bEJ2UqRs5+xrA3BlvF6leTYQ0AiZ41bDXfzlu9dq1cum71vAa+3nx9R a8UjQw9jHaPVtXJx5e+ZVgUugprPxumrdlImcvlxZ3v7YwoCQpc0rPFpURrajLISJRD0 PDNMp7yGtCqmMupFBCrvm8UKupifGC5v4rK/GB4EJ7PDc/QBRpR9fRjOB4GlLE09W+tg qqXYIHW3OR9vW3HtgL4A4Fo0Lxb5nXn29E2kK2ai5yEm6xJ+2zEgvKTvUQsilW2noC/9 uC3HfobSRVuhl+c1motvmYhI+m0wXKqT2h+2BF8/wTbx0c2Fco7bOuDwgs4d2pXQzAf2 vbqw== X-Gm-Message-State: AOJu0YxdalsDGME5eUG2HK5k9R5/QrSTEjRlgLYSaq4LkvGrDXn3Xo3u CuNkp3gklgjseldpMDWaJKuQUCUEiNHFT+MLkWujS4yz2rWih7utI47OWP3YKs5jM7GbjkkjJyj j X-Google-Smtp-Source: AGHT+IExX5Nv+LsC1H8bO2T30R1oXX3JKM+1DztHjEm/4lrMkXRq0laGW2wrpRu5eTpZS/ReS8DqOA== X-Received: by 2002:a17:907:9407:b0:a6f:bf0f:4209 with SMTP id a640c23a62f3a-a7514475b49mr1121407666b.42.1720046175886; Wed, 03 Jul 2024 15:36:15 -0700 (PDT) From: Andrew Cooper To: Xen-devel Cc: Andrew Cooper , Anthony PERARD , Juergen Gross , Oleksii Kurochko Subject: [PATCH for-4.19] tools/xs-clients: Fix `make clean` rule Date: Wed, 3 Jul 2024 23:36:11 +0100 Message-Id: <20240703223611.1754927-1-andrew.cooper3@citrix.com> X-Mailer: git-send-email 2.39.2 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @citrix.com) X-ZM-MESSAGEID: 1720046211049100001 Content-Type: text/plain; charset="utf-8" Prior to the split, "the clients" used tools/xenstored/Makefile.common whose clean rule includes *.o whereas after the split, the removal of *.o was lost by virtule of not including Makefile.common any more. This is the bug behind the following build error: make[2]: Entering directory '/local/xen.git/tools/xs-clients' gcc xenstore_client.o (snip) /usr/bin/ld: xenstore_client.o: relocation R_X86_64_32S against `.rodata'= can not be used when making a PIE object; recompile with -fPIE /usr/bin/ld: failed to set dynamic section sizes: bad value collect2: error: ld returned 1 exit status make[2]: *** [Makefile:35: xenstore] Error 1 which was caused by `make clean` not properly cleaning the tree as I was swapping between various build containers. Switch to a plain single-colon clean rule. Fixes: 5c293058b130 ("tools/xenstore: move xenstored sources into dedicated= directory") Signed-off-by: Andrew Cooper Reviewed-by: Juergen Gross --- CC: Anthony PERARD CC: Juergen Gross CC: Oleksii Kurochko For 4.19. It's a minor build system corner case, but is also very safe too. Ultimately it needs backporting to 4.18 too. --- tools/xs-clients/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/xs-clients/Makefile b/tools/xs-clients/Makefile index 4e78d365e0df..5415c441920a 100644 --- a/tools/xs-clients/Makefile +++ b/tools/xs-clients/Makefile @@ -38,8 +38,8 @@ xenstore-control: xenstore_control.o $(CC) $(LDFLAGS) $^ $(LDLIBS) -o $@ $(APPEND_LDFLAGS) =20 .PHONY: clean -clean:: - $(RM) $(TARGETS) $(DEPS_RM) +clean: + $(RM) *.o $(TARGETS) $(DEPS_RM) =20 .PHONY: distclean distclean: clean --=20 2.39.2