[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 7/8] symbols: drop symbols-dummy.c
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
- Date: Mon, 8 Dec 2025 15:19:53 +0000
- Accept-language: en-GB, en-US
- Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 4.158.2.129) smtp.rcpttodomain=suse.com smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=arm.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
- Arc-message-signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=uZglb5GKpkqPRjhl3e5G+9xIPgNinmbvgen4Y5C6SjU=; b=smeNDDsK4zWTy9ctOa/SUKQS2rEVma1s5DDy9ZDzC9dGimpXUQP7ulwxcd/p11vhSbVTVWblIOe1ZPLzvSJjWM8xxXelnuG7YO85MENv2hXs6BouyLuPMnoyn/d1o757U9TU49G3Pe7oQpbUaUula8WxUAIOXLROkWKNWTeDneApprurZp8wy6+jEDt8USqPTX0aJfSjtMO5g6ogVsxpZFxQDLFj1fg1cEG8uW7sW4/PRkED0tiRuIDkwbsQVKfrS1CILP+Nod7As2seQujzeCH5oLxlV+bt036ZxrgQh46JkAv3bBsThL6LMdRaUll8SU692+RrMeBagKPt4KpDEQ==
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=uZglb5GKpkqPRjhl3e5G+9xIPgNinmbvgen4Y5C6SjU=; b=xC1l41p00Y94BSloLG8NDIcSlges9KhxFQ9YKx7vler/Uh7Rsp9YCbHzlHwP3UTv6kLcMR6KY5FipHVcCvCnErmBDWvmX9XEJ8YhGAJPS1HAK51YQ5XKTpDsnHYPEx9t8/n5NdFet+oIjrv31zE7AKvgElkynEHgZJH7NJ7WtUiczSBrK7kjslCNOBtT8RRlwUY0fwtlW6jsrEk1a44EsyDLEUkkwpIoRLptYct+QCHPVi4B5V6URhfZl4bxwH8Rkug5kPSm4UuWn7Px8jbZy1bRFQrDGLhkUeqk0tavrUA/lmMXyirznGNpnKjhE6p8aLgMDHB7J1MpLc6AEoqnsw==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=pass; b=cWlRSUB3N9d5LZThH7ze9GM7oBx8OrlPlfP3NPnPdNFbFy5R5TkBOuiRCDkneaY7peNpTrGQ/cLPXucfnGkqE7S4o42S9geykBN9X8x2D9Fvun4sgjsvrh20nd/FwXA3eRV5lU+JZhlEy8ZR6DcumiW5IlGQpHK/bL93Fvh3zeA+rYLURajSxwZYQuVPmDAMAV4Rcr1ZGJKM8iEg0TFNY1kcPE09zgDtCTcqcPhi87NBPWuu0pI18GMYnRrdxxGbxOXg9VVQM4LqD9DL9KmFSBr8luejSZMdeuDdEopbDvagAC96p8MRLLas4Ahj3Fl93Y04aJFq4znGgt05q/onaA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ZPYiAEJBIMjYyMYAzoLXAPMGQU+BqO5VcLscRuaUyUDQGHl5jkHVW7hN8K5OgFQfyYb5mGByO/TE/0I9bELpRQXinwdNVwSBRW4eLajmnmj98RjJR6XdzoP08thHHNbLBbuogDBRNrj3aNTvKL540Mi2os6SUx0rNW3b6tFvvmmbQvxkww7mN6FsSCC6pRd8Rosm7okYPOrg84ZiFD+ccWeWVjyeBa90+1xXD7hLfDdLx716RU8hHM/O8JwagpWJiXWnMHxuDSasjc02Qx8q7ZiV4qf2OBwK2bwOlN1VGEY7/u05eAS5bRq5oSiVbCNvl3stfeuKeSgUjpLPpMe5IQ==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Mon, 08 Dec 2025 15:21:08 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Nodisclaimer: true
- Thread-index: AQHcaFHEZgCWYRtzmkm7gvYg23aA3rUX05+AgAABswCAAAYbgA==
- Thread-topic: [PATCH 7/8] symbols: drop symbols-dummy.c
Hi Jan,
>>>>
>>>
>>> Now this is (to me at least) absurd: I'm removing a file, just to find the
>>> pipeline
>>> fails because cppcheck doesn't like docs/misra/exclude-list.json containing
>>> a
>>> reference to a non-existing file.
>>>
>>> I'll amend the commit with
>>>
>>> --- a/docs/misra/exclude-list.json
>>> +++ b/docs/misra/exclude-list.json
>>> @@ -170,10 +170,6 @@
>>> "comment": "Imported from Linux, ignore for now"
>>> },
>>> {
>>> - "rel_path": "common/symbols-dummy.c",
>>> - "comment": "The resulting code is not included in the final
>>> Xen binary, ignore for now"
>>> - },
>>> - {
>>> "rel_path": "crypto/*",
>>> "comment": "Origin is external and documented in
>>> crypto/README.source"
>>> },
>>>
>>> but I think such tidying should be optional.
>>
>> Can you share the error?
>
> + xen/scripts/xen-analysis.py --run-cppcheck --cppcheck-misra -- -j16
> ERROR: Issue with reading file
> /builds/xen-project/hardware/xen-staging/docs/misra/exclude-list.json:
> Malformed path: common/symbols-dummy.c refers to
> /builds/xen-project/hardware/xen-staging/xen/common/symbols-dummy.c that does
> not exists
>
Oh ok I see, seems to me a good feedback from the tool to keep everything
consistent.
Cheers,
Luca
|