Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: jRRV+35hz7OyM7UzpAbrXaSzIdsn014AAlwguvVkEdMYdypBLqLiUURcRh9B1IlM9/JBBRUY9M
 HNb6BwaYo3tviC1cxA138W0xqpkG9jNAGUDkxx4ZL1QppJ8vvXGNadyzN91NBsj5vmAngwRnKg
 EFMfxbXmV2X3Wdr+nuoQ1esaMXKW8dCmAIBG2pxlMwiGze/rsBQMWQ/7xhDstHO48/Tnsv4Q9l
 qtpUXwF8iaEv6GU1B/5MPb0iYuJBrOqICGu12kuEiUs3e7dRJuZMwYbRIMN9esdbNBIe8fxSel
 43M=
From:   Arthur Simchaev <Arthur.Simchaev@wdc.com>
To:     Arthur Simchaev <Arthur.Simchaev@wdc.com>,
        "James E . J . Bottomley" <jejb@linux.vnet.ibm.com>,
        "Martin K . Petersen" <martin.petersen@oracle.com>,
        "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
CC:     "alim.akhtar@samsung.com" <alim.akhtar@samsung.com>,
        Bean Huo <beanhuo@micron.com>
Subject: RE: [PATCH v2] scsi: ufs: sysfs: Print string descriptors as raw data
Thread-Topic: [PATCH v2] scsi: ufs: sysfs: Print string descriptors as raw
 data
Thread-Index: AQHXA8HGKUtY27DSkEKVXm/ukaBroqqGpqXw
Date:   Tue, 16 Mar 2021 11:27:16 +0000
Message-ID: <BY5PR04MB6327DEDB1CB9EEBC8365E848ED6B9@BY5PR04MB6327.namprd04.prod.outlook.com>
References: <1613410846-16883-1-git-send-email-Arthur.Simchaev@wdc.com>
In-Reply-To: <1613410846-16883-1-git-send-email-Arthur.Simchaev@wdc.com>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: bc08c85b-21aa-4c01-7c4d-08d8e86e7488
X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Mar 2021 11:27:16.9982
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: YFPrpL4+qHNB/7YFVfYx5DSw0fV806RFJ+jSXywRqd4xtz07feUbhkQBpd6og/ZiFKHz7VXWuS69de4ncllDnA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR04MB6817
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Martin

Could you please consider to take this patch?

Regards
Arthur

> -----Original Message-----
> From: Arthur Simchaev <Arthur.Simchaev@wdc.com>
> Sent: Monday, February 15, 2021 7:41 PM
> To: James E . J . Bottomley <jejb@linux.vnet.ibm.com>; Martin K . Peterse=
n
> <martin.petersen@oracle.com>; linux-scsi@vger.kernel.org; linux-
> kernel@vger.kernel.org
> Cc: alim.akhtar@samsung.com; Bean Huo <beanhuo@micron.com>; Arthur
> Simchaev <Arthur.Simchaev@wdc.com>
> Subject: [PATCH v2] scsi: ufs: sysfs: Print string descriptors as raw dat=
a
>=20
> Currently the string descriptors sysfs entries are printing in ascii
> format. According to Jedec UFS spec the string descriptors data is
> Unicode and need-not be ascii convertible. Therefore in case the device
> string descriptor contains non ascii convertible characters, it will
> produce a wrong output. In order to fix this issue, the new
> string descriptors entries will added to sysfs directory.
> Those entries will show the string descriptors in raw format
>=20
> Signed-off-by: Arthur Simchaev <Arthur.Simchaev@wdc.com>
> ---
>  Documentation/ABI/testing/sysfs-driver-ufs | 44
> ++++++++++++++++++++++++++++++
>  drivers/scsi/ufs/ufs-sysfs.c               | 34 +++++++++++++++++------
>  2 files changed, 70 insertions(+), 8 deletions(-)
>=20
> diff --git a/Documentation/ABI/testing/sysfs-driver-ufs
> b/Documentation/ABI/testing/sysfs-driver-ufs
> index d1bc23c..f6d6a46 100644
> --- a/Documentation/ABI/testing/sysfs-driver-ufs
> +++ b/Documentation/ABI/testing/sysfs-driver-ufs
> @@ -561,6 +561,50 @@ Description:	This file contains a product revision
> string. The full
>=20
>  		The file is read only.
>=20
> +What:
> 	/sys/bus/platform/drivers/ufshcd/*/string_descriptors/manufacture
> r_name_raw
> +Date:		February 2021
> +Contact:	Arthur Simchaev <arthur.simchaev@wdc.com>
> +Description:	This file contains a device manufactureer name string
> +		as raw data. The full information about the descriptor
> +		could be found at UFS specifications 2.1.
> +
> +		The file is read only.
> +
> +What:
> 	/sys/bus/platform/drivers/ufshcd/*/string_descriptors/product_na
> me_raw
> +Date:		February 2021
> +Contact:	Arthur Simchaev <arthur.simchaev@wdc.com>
> +Description:	This file contains a product name string as raw data.
> +		The full information about the descriptor could be found at
> +		UFS specifications 2.1.
> +
> +		The file is read only.
> +
> +What:
> 	/sys/bus/platform/drivers/ufshcd/*/string_descriptors/oem_id_raw
> +Date:		February 2021
> +Contact:	Arthur Simchaev <arthur.simchaev@wdc.com>
> +Description:	This file contains a OEM ID string as raw data.
> +		The full information about the descriptor could be found at
> +		UFS specifications 2.1.
> +
> +		The file is read only.
> +
> +What:
> 	/sys/bus/platform/drivers/ufshcd/*/string_descriptors/serial_numb
> er_raw
> +Date:		February 2021
> +Contact:	Arthur Simchaev <arthur.simchaev@wdc.com>
> +Description:	This file contains a device serial number string
> +		as raw data. The full information about the descriptor could
> be
> +		found at UFS specifications 2.1.
> +
> +		The file is read only.
> +
> +What:
> 	/sys/bus/platform/drivers/ufshcd/*/string_descriptors/product_revi
> sion_raw
> +Date:		February 2021
> +Contact:	Arthur Simchaev <arthur.simchaev@wdc.com>
> +Description:	This file contains a product revision string as raw data.
> +		The full information about the descriptor could be found at
> +		UFS specifications 2.1.
> +
> +		The file is read only.
>=20
>  What:
> 	/sys/class/scsi_device/*/device/unit_descriptor/boot_lun_id
>  Date:		February 2018
> diff --git a/drivers/scsi/ufs/ufs-sysfs.c b/drivers/scsi/ufs/ufs-sysfs.c
> index acc54f5..f1407ff 100644
> --- a/drivers/scsi/ufs/ufs-sysfs.c
> +++ b/drivers/scsi/ufs/ufs-sysfs.c
> @@ -658,7 +658,7 @@ static const struct attribute_group
> ufs_sysfs_power_descriptor_group =3D {
>  	.attrs =3D ufs_sysfs_power_descriptor,
>  };
>=20
> -#define UFS_STRING_DESCRIPTOR(_name, _pname)
> 	\
> +#define UFS_STRING_DESCRIPTOR(_name, _pname, _is_ascii)
> 	\
>  static ssize_t _name##_show(struct device *dev,
> 	\
>  	struct device_attribute *attr, char *buf)			\
>  {									\
> @@ -690,10 +690,18 @@ static ssize_t _name##_show(struct device *dev,
> 				\
>  	kfree(desc_buf);						\
>  	desc_buf =3D NULL;						\
>  	ret =3D ufshcd_read_string_desc(hba, index, &desc_buf,		\
> -				      SD_ASCII_STD);			\
> +				      _is_ascii);			\
>  	if (ret < 0)							\
>  		goto out;						\
> -	ret =3D sysfs_emit(buf, "%s\n", desc_buf);			\
> +	if (_is_ascii) {						\
> +		ret =3D sysfs_emit(buf, "%s\n", desc_buf);		\
> +	} else {							\
> +		int i;							\
> +									\
> +		for (i =3D 0; i < desc_buf[0]; i++)			\
> +			hex_byte_pack(buf + i * 2, desc_buf[i]);	\
> +		ret =3D sysfs_emit(buf, "%s\n", buf);			\
> +	}			\
>  out:									\
>  	pm_runtime_put_sync(hba->dev);
> 	\
>  	kfree(desc_buf);						\
> @@ -702,11 +710,16 @@ out:
> 			\
>  }									\
>  static DEVICE_ATTR_RO(_name)
>=20
> -UFS_STRING_DESCRIPTOR(manufacturer_name, _MANF_NAME);
> -UFS_STRING_DESCRIPTOR(product_name, _PRDCT_NAME);
> -UFS_STRING_DESCRIPTOR(oem_id, _OEM_ID);
> -UFS_STRING_DESCRIPTOR(serial_number, _SN);
> -UFS_STRING_DESCRIPTOR(product_revision, _PRDCT_REV);
> +UFS_STRING_DESCRIPTOR(manufacturer_name, _MANF_NAME, 1);
> +UFS_STRING_DESCRIPTOR(product_name, _PRDCT_NAME, 1);
> +UFS_STRING_DESCRIPTOR(oem_id, _OEM_ID, 1);
> +UFS_STRING_DESCRIPTOR(serial_number, _SN, 1);
> +UFS_STRING_DESCRIPTOR(product_revision, _PRDCT_REV, 1);
> +UFS_STRING_DESCRIPTOR(manufacturer_name_raw, _MANF_NAME, 0);
> +UFS_STRING_DESCRIPTOR(product_name_raw, _PRDCT_NAME, 0);
> +UFS_STRING_DESCRIPTOR(oem_id_raw, _OEM_ID, 0);
> +UFS_STRING_DESCRIPTOR(serial_number_raw, _SN, 0);
> +UFS_STRING_DESCRIPTOR(product_revision_raw, _PRDCT_REV, 0);
>=20
>  static struct attribute *ufs_sysfs_string_descriptors[] =3D {
>  	&dev_attr_manufacturer_name.attr,
> @@ -714,6 +727,11 @@ static struct attribute
> *ufs_sysfs_string_descriptors[] =3D {
>  	&dev_attr_oem_id.attr,
>  	&dev_attr_serial_number.attr,
>  	&dev_attr_product_revision.attr,
> +	&dev_attr_manufacturer_name_raw.attr,
> +	&dev_attr_product_name_raw.attr,
> +	&dev_attr_oem_id_raw.attr,
> +	&dev_attr_serial_number_raw.attr,
> +	&dev_attr_product_revision_raw.attr,
>  	NULL,
>  };
>=20
> --
> 2.7.4