linux-kernel - Re: [EXT] Re: [PATCH v12 4/5] firmware: imx: add driver for NXP EdgeLock Enclave

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <63fb9cc6-b57c-45bc-8da2-7bcb871d9887@kernel.org>
Date: Wed, 22 Jan 2025 12:23:27 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Pankaj Gupta <pankaj.gupta@....com>, Jonathan Corbet <corbet@....net>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
 Sascha Hauer <s.hauer@...gutronix.de>,
 Pengutronix Kernel Team <kernel@...gutronix.de>,
 Fabio Estevam <festevam@...il.com>
Cc: "linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
 "imx@...ts.linux.dev" <imx@...ts.linux.dev>,
 "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [EXT] Re: [PATCH v12 4/5] firmware: imx: add driver for NXP
 EdgeLock Enclave

On 22/01/2025 12:13, Pankaj Gupta wrote:
> 
> 
> -----Original Message----- From: Krzysztof Kozlowski
> <krzk@...nel.org>> Sent: Monday, January 20, 2025 5:54 PM To: Pankaj
> Gupta <pankaj.gupta@....com>>; Jonathan Corbet <corbet@....net>>; 
> Rob Herring <robh@...nel.org>>; Krzysztof Kozlowski
> <krzk+dt@...nel.org>>; Conor Dooley <conor+dt@...nel.org>>; Shawn
> Guo <shawnguo@...nel.org>>; Sascha Hauer <s.hauer@...gutronix.de>>;
> Pengutronix Kernel Team <kernel@...gutronix.de>>; Fabio Estevam
> <festevam@...il.com>> Cc: linux-doc@...r.kernel.org; linux-
> kernel@...r.kernel.org; devicetree@...r.kernel.org;
> imx@...ts.linux.dev; linux-arm-kernel@...ts.infradead.org Subject:
> [EXT] Re: [PATCH v12 4/5] firmware: imx: add driver for NXP EdgeLock
> Enclave
> 
> Caution: This is an external email. Please take care when clicking
> links or opening attachments. When in doubt, report the message
> using the 'Report this email' button


You got comment on this, more than once.

Rest of the email is poorly formatted, so I am just skimming through it.
Fix your email program to send readable content if you want some answers
for stuff I missed. I expect all my comments fully addressed, not just
some of them.


>>> + +static int se_if_probe(struct platform_device *pdev) { +
>>> const struct se_if_node_info_list *info_list; +     const struct
>>> se_if_node_info *info; +     struct device *dev = &pdev->>dev; 
>>> +     struct se_fw_load_info *load_fw; +     struct se_if_priv
>>> *priv; +     u32 idx; +     int ret; +q +     idx =
>>> GET_IDX_FROM_DEV_NODE_NAME(dev->>of_node);
> 
>> NAK. Node can be called firmware and your entire driver collapes.
> The macro is updated to verify the correct-ness of node-name.

NAK, do you understand the term? I provided the reasons for NAK.

> 
> +               (!memcmp(dev_of_node->full_name, NODE_NAME, 
> strlen(NODE_NAME)) ?\ ((strlen(dev_of_node->full_name) >
> strlen(NODE_NAME)) ?\ GET_ASCII_TO_U8((strlen(dev_of_node-
> >full_name) - strlen(NODE_NAME)),\ dev_of_node-
> >full_name[strlen(NODE_NAME) + 1], \ -
> dev_of_node->full_name[strlen(NODE_NAME) + 2]) : 0) 
> +                               dev_of_node-
> >full_name[strlen(NODE_NAME) + 2]) : 0) : -EINVAL)
> 
>>> +     info_list = device_get_match_data(dev); +     if (idx >>=
>>> info_list->>num_mu) { +             dev_err(dev, 
>>> +                     "Incorrect node name :%s\n", 
>>> +                     dev->>of_node->>full_name);
> 
>> Nope. "firmware" or "secure" are correct node names.
> New check is added to validate the correctness of the node name for
> this driver. Replaced the message of " Incorrect node name..", with
> the help message.

You did not resolve the NAK.
1. You cannot reject correct names.
2. You cannot add undocumented ABI. You could try to document it, but it
will not solve the first problem.


Best regards,
Krzysztof