Re: [PATCH] libata: Fix wrong HPA horkage

[Niklas Cassel <cassel@kernel.org>]

On Tue, Oct 13, 2026 at 07:21:46AM +0300, Ioannis Barkas wrote:
>  Code wrongly filters disks by their serial numbers

I assume you found this using LLM, so a better commit message is something
like:

    ata: libata-core: Remove BROKEN_HPA quirks that can never match

    The ATA_QUIRK_BROKEN_HPA entries for "WDC WD3200JD-00KLB0" and
    "WDC WD2500JD-00HBB0" use "WD-WCAMR1130137" and "WD-WMAL71490727" as
    model_rev. These look like drive serial numbers. model_rev is matched
    against the firmware revision, which is at most ATA_ID_FW_REV_LEN (8)
    characters long, so these 15 character patterns without wildcards can
    never match. This has been the case since the entries were added in
    commit 16c55b038033 ("libata: implement BROKEN_HPA horkage and apply it
    to affected drives"), which compared the strings using strcmp().

    The firmware revisions of the affected drives are not known. Matching
    on the model only would apply ATA_QUIRK_BROKEN_HPA to drives that have
    never had it, and these entries would then take precedence over the
    "WDC WD3200JD-*" and "WDC WD2500JD-*" entries that match the same
    drives. Remove the two entries instead.

    No functional change.

    Assisted-by: LLM



i.e. I would simply let the patch drop these two entries, such that the
existing wildcard entries are the only ones used.


> 
> Fixes: 16c55b038033 (\libata: implement BROKEN_HPA horkage and apply it to affected drives")
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=8888
> Signed-off-by: Ioannis Barkas jnyb.de@gmail.com
> Signed-off-by: Nick Barkas nickbgrk@gmail.com
> ---
>  drivers/ata/libata-core.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
> index 59e87b987d..b23e1c3e4c 100644
> --- a/drivers/ata/libata-core.c
> +++ b/drivers/ata/libata-core.c
> @@ -4248,8 +4248,8 @@ static const struct ata_dev_quirks_entry __ata_dev_quirks[] = {
>  
>  	/* devices which puke on READ_NATIVE_MAX */
>  	{ "HDS724040KLSA80",	"KFAOA20N",	ATA_QUIRK_BROKEN_HPA },
> -	{ "WDC WD3200JD-00KLB0", "WD-WCAMR1130137", ATA_QUIRK_BROKEN_HPA },
> -	{ "WDC WD2500JD-00HBB0", "WD-WMAL71490727", ATA_QUIRK_BROKEN_HPA },
> +	{ "WDC WD3200JD-00KLB0", "08.05J08", ATA_QUIRK_BROKEN_HPA },
> +	{ "WDC WD2500JD-00HBB0", "08.02D08", ATA_QUIRK_BROKEN_HPA },
>  	{ "MAXTOR 6L080L4",	"A93.0500",	ATA_QUIRK_BROKEN_HPA },
>  
>  	/* this one allows HPA unlocking but fails IOs on the area */
> -- 
> 2.53.0
>