[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4 4/5] docs/misra: Remove earlycpio.c from the Eclair exclusion list.


  • To: Alejandro Vallejo <alejandro.garciavallejo@xxxxxxx>, Andrew Cooper3 <andrew.cooper3@xxxxxxxxxx>, Jbeulich <jbeulich@xxxxxxxx>
  • From: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
  • Date: Tue, 20 Jan 2026 15:20:23 +0100
  • Arc-authentication-results: i=1; bugseng.com; arc=none smtp.remote-ip=162.55.131.47
  • Arc-message-signature: i=1; d=bugseng.com; s=openarc; a=rsa-sha256; c=relaxed/relaxed; t=1768918823; h=MIME-Version:Date:From:To:Cc:Subject:In-Reply-To:References: Message-ID:X-Sender:Organization:Content-Type: Content-Transfer-Encoding; bh=qt+S6uyW/K0pgpTSAzdgtw0W1RwId9+iIyB8zzdbrfI=; b=E6OPlXr1amIxx3qVVkbBP08WaN5hFEODpnAOBbzMqC1Vu+dUEEANwfcbA6tIdmr6BtCv 7l8xgL2EOZC9HTOFwTd8zMOkSrt8rzYmHBz1HUOhmBsI0NwkRGIX0ibA6EfBgSaK0uDmC jikTzshw6vsRp4kiYowUmTXuBy0i9SjTBZfmUNK6wH1gdSubi0TmyfelYrXTBfdsBBM5v 2qxZJlNSh/dW6X5ZYE044VIbIEI7iKgArDZ6PogQ1rJ+p7EigfYycQQ3beYhTkdYohAZk l1v1fKDQEeOStY9bwaA1QanOeOQe0E9YmBIZwsbhzOqbrAIio2d9cagYdF9WLiGWdaJF/ 51pWLosHtsENxoDv1js03+lwmBrlL/PbNWTWWvWB3Fb5gy1S2v5YJ5vHn1aeLfctmeJW5 GiDPWHr3DTf+JyQK3dsG2hP9WT5vcp/Zkavw3tnWv0jE6AKkfBYyjXphoH50tV+/fy/F9 9Xz6QSS3FQru0mwOLCaOLhFMIOMXFGlgnC0TaeMr6XCJSdNZKEYxX0cuGfaGKXTbbr4BM 3V6CoLrhs2CtuREpfJ8pZqjiqKY5U0m7guZK83GsgGj5h8OD28y3lBRuOtLGJPCSECIKb BV6vEpQXCnH2KmekJTRGdztjW4YzyEKdTC0iNg1trK25fGH5aXp6Xl6s6jYh0JE=
  • Arc-seal: i=1; d=bugseng.com; s=openarc; a=rsa-sha256; cv=none; t=1768918823; b=CB55ofH04DEjlVEofm4qEM0P2XnqAo7KmX7qq2xqP7FFRkqY19mKui3Ys2iTElMAUi3t L6wlJwmcnZ7fn3tjlA/bQvjqmKBG13i7dBZExzS0/DdxqYy3nEZ2hrt+CIfY9iOuijQ10 BjRGUWXJ2+zrZ/pVuuK4o+k1FAASlP7GZJsftAVudYRZhhiPvt3LQfGBrrKwuLyUnT8yw lsVXiTbMumns4cWzIoZLtzbDx3RBFdOuFBZuvOgnNRi/qP8WSmhOPz1oiJNGJaI0okedb j/zs5qI90nYdD7hb3TXtBraX/uIwklSeF5YLLLUcEm2OId7KnNKTlZLlkjq6fvsnjxa7M NDjTZC6x4ge/pt/bcXfO6oYxzY73brpIaECUUPEMyn1teJp4J/D0yFFyLJBRM6c1dHGdP u/ADPqpG8dkQOBNPzLFH0lqt0kuV3qvXRkr7Ee+O+4sHSq4g2ecLqsl9L8snN/ixxt9Nw Vx70a8wp/T/vHswSrJedENUgqSJ6UVlgcx5c5dB88lqKdefl7ZOylBBD23bM0+xSmbGQf sbB6bZAl84aMA+/I+c7JZnu9g3VYzffk09r5fV37uLEOcCtvbhrt+vSi/F9xOyG7Q+SD+ MAxDxozY815K1sQ2bU1ZH1ZPxB3BkQAPAG9dCmR2vnXwAAaFlbmMnbJX95bXnkE=
  • Authentication-results: bugseng.com; arc=none smtp.remote-ip=162.55.131.47
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Julien Grall <julien@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Xen-devel <xen-devel-bounces@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Tue, 20 Jan 2026 14:20:32 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 2026-01-20 13:09, Alejandro Vallejo wrote:
On Tue Jan 20, 2026 at 12:51 PM CET, Alejandro Vallejo wrote:
On Tue Jan 20, 2026 at 12:41 PM CET, Nicola Vetrini wrote:
On 2026-01-20 12:27, Alejandro Vallejo wrote:
On Tue Jan 20, 2026 at 12:21 PM CET, Nicola Vetrini wrote:
On 2026-01-20 10:38, Alejandro Vallejo wrote:
It's clean.

Signed-off-by: Alejandro Vallejo <alejandro.garciavallejo@xxxxxxx>
---
 docs/misra/exclude-list.json | 4 ----
 1 file changed, 4 deletions(-)


Hi. Do you have a link to a pipeline?

In the cover letter. I only run it on allcode.


I see. I can spot these additional violations from earlycpio.c. It does not result in a failure, but only because x86_64-allcode has also other
non-clean guidelines and is thus allowed to fail. Ideally in some
copious free time I'd send a patch to create a subset of clean
guidelines for the *-allcode analysis that is failing, so that the
"allow_fail: true" can be removed.

https://eclair-analysis-logs.xenproject.org/fs/space/verdesse0/XEN.ecdf/xen-project/people/agvallejo/xen/ECLAIR_normal/ucode-disable_v4/X86_64/12771570090/PROJECT.ecd;/by_main_file/xen/lib/earlycpio.c.html

The web interface doesn't allow to search?! Sigh... thanks for the pointer.

It's your usual mess of miscasting, enum-as-int, etc.

Would you rather keep the exclusion and deal with it later or let it pile up?
I just don't have the time to go into it myself.


Well, including more stuff in the scan doesn't hurt and it's only a handful of reports that could be fixed, but the maintainers will have the final say. This file is not really inside my area as a reviewer, but if it helps:

Reviewed-by: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>

--
Nicola Vetrini, B.Sc.
Software Engineer
BUGSENG (https://bugseng.com)
LinkedIn: https://www.linkedin.com/in/nicola-vetrini-a42471253



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.