linux-kernel - Re: [PATCH] PCI: eliminate abnormal characters when reads help information of "PCI

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:   Tue, 2 Aug 2022 09:11:52 +0800
From:   Liu Song <liusong@...ux.alibaba.com>
To:     Bjorn Helgaas <helgaas@...nel.org>
Cc:     bhelgaas@...gle.com, linux-pci@...r.kernel.org,
        linux-kernel@...r.kernel.org, Andy Whitcroft <apw@...onical.com>,
        Joe Perches <joe@...ches.com>
Subject: Re: [PATCH] PCI: eliminate abnormal characters when reads help
 information of "PCI_P2PDMA" under menuconfig


在 2022/8/2 05:56, Bjorn Helgaas 写道:
> On Wed, Jul 20, 2022 at 12:23:29PM -0500, Bjorn Helgaas wrote:
>> [+cc Andy, Joe, possible checkpatch question]
>>
>> On Wed, Jul 20, 2022 at 03:22:03PM +0800, Liu Song wrote:
>>> From: Liu Song <liusong@...ux.alibaba.com>
>>>
>>> Read the help information of PCI_P2PDMA through make menuconfig,
>>> "Enables" is partially displayed as garbled characters, so fix it.
>>>
>>> Signed-off-by: Liu Song <liusong@...ux.alibaba.com>
>>> ---
>>>   drivers/pci/Kconfig | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
>>> index 133c732..8102b78 100644
>>> --- a/drivers/pci/Kconfig
>>> +++ b/drivers/pci/Kconfig
>>> @@ -166,7 +166,7 @@ config PCI_P2PDMA
>>>   	depends on ZONE_DEVICE
>>>   	select GENERIC_ALLOCATOR
>>>   	help
>>> -	  Enableѕ drivers to do PCI peer-to-peer transactions to and from
>>> +	  Enables drivers to do PCI peer-to-peer transactions to and from
>> I see this problem ("Enables" renders as "Enable ~U" because the "s"
>> is actually UTF-8 D195, CYRILLIC SMALL LETTER DZE).
>>
>> "file" found the following other Kconfig files that also contain
>> UTF-8:
>>
>>    drivers/pci/Kconfig
>>      D195 CYRILLIC SMALL LETTER DZE, which looks like "s"
>>      "Enables" renders as "Enable ~U"
>>    net/netfilter/ipvs/Kconfig
>>      C2A0 NO-BREAK SPACE
>>      renders fine
>>    drivers/mtd/nand/raw/Kconfig
>>      MTD_NAND_CAFE  C389 LATIN CAPITAL LETTER E WITH ACUTE
>>      "CAFÉ" renders as "CAF ~I"
>>    drivers/mtd/spi-nor/Kconfig
>>      MTD_SPI_NOR_USE_4K_SECTORS
>>      "16 × 4 KiB" renders as "16  ~W 4 KiB"
>>    drivers/net/can/usb/Kconfig
>>    drivers/net/can/peak_canfd/Kconfig
>>    drivers/gpu/drm/panel/Kconfig
>>    drivers/platform/mellanox/Kconfig
>>    kernel/time/Kconfig
>>    crypto/Kconfig
>>    arch/Kconfig
>>
>> Some of these are clearly wrong (Cyrillic letter), some are
>> unnecessary (non-breakable space), some are arguable ("CAFÉ" and "16 ×
>> 4 KiB" -- these take advantage of UTF-8 in useful ways).
>>
>> Not being a charset guru, I dunno if the rendering problem means
>> my terminal is set incorrectly or if they all need to be changed.
>>
>> But I think we should address all of them at the same time.  If we do
>> need to avoid UTF-8 in Kconfig help, maybe checkpatch should look for
>> it.
> Any progress on this problem?  I certainly agree that it should be
> fixed; I would just like to fix all the problems, not just the PCI
> one.
>
> Bjorn


Totally agree, according to your suggestion, I will resubmit with all 
the revisions.


Thanks