Re: [REGRESSION][BISECTED] BU40N Blu-Ray drive broken since 7627a0edef54

[Damien Le Moal <dlemoal@kernel.org>]

On 2025/01/10 18:42, Christian Heusel wrote:
> On 25/01/10 06:24PM, Damien Le Moal wrote:
>> On 2025/01/10 18:04, Christian Heusel wrote:
>>> Hello everyone,
>>>
>>> we have recently received a report by a user in the Arch Linux Forums
>>> user that their Blue Ray player, a HL-DT-ST BD-RE BU40N stopped working
>>> somewhere since kernel version v6.8.0. We have then bisected the issue
>>> together with them within the mainline kernel sources to the following
>>> commit that changes the default power policy:
>>>
>>>     7627a0edef54 ("ata: ahci: Drop low power policy board type")
>>>
>>> The user reports that adding "ahci.mobile_lpm_policy=1" as a kernel
>>> parameter fixes the issue for them.
>>>
>>> Additionally fellow forum user @loqs came up with a quirk patch, that
>>> sadly was reported to not work:
>>>
>>> diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c
>>> index c085dd8..ef01ccd 100644
>>> --- a/drivers/ata/libata-core.c
>>> +++ b/drivers/ata/libata-core.c
>>> @@ -4118,6 +4118,9 @@ static const struct ata_dev_quirks_entry __ata_dev_quirks[] = {
>>>  	{ "SAMSUNG MZ7TD256HAFV-000L9", NULL,       ATA_QUIRK_NOLPM },
>>>  	{ "SAMSUNG MZ7TE512HMHP-000L1", "EXT06L0Q", ATA_QUIRK_NOLPM },
>>>  
>>> +	/* Hitachi-LG Data Storage models with LPM issues */
>>> +	{ "HL-DT-ST BD-RE BU40N", NULL, ATA_QUIRK_NOLPM },
>>
>> Looks like "BU40N" may be the FW rev ? If we can get a dmesg ourput of the
>> failing case, we should be able to sort out this. Likely it is a bad string
>> reference here.
> 
> I have attached the dmesg outputs, here is the relevant output:
> 
> [   81.230713] ata4: SATA link up 1.5 Gbps (SStatus 113 SControl 300)
> [   81.235708] ata4.00: ATAPI: HL-DT-ST BD-RE BU40N, 1.03, max UDMA/133
> [   81.240789] ata4.00: configured for UDMA/133
> [   81.248585] scsi 3:0:0:0: CD-ROM            HL-DT-ST BD-RE BU40N      1.03 PQ: 0 ANSI: 5
> [   81.302830] scsi 3:0:0:0: Attached scsi generic sg2 type 5
> [   81.382131] sr 3:0:0:0: [sr0] scsi3-mmc drive: 24x/24x writer dvd-ram cd/rw xa/form2 cdda tray

Is this with the latest kernel 6.13-rc6 ?

> 
> Aswell as the output for "hdparm -i /dev/sr0":
> 
> /dev/sr0:
> 
>  Model=HL-DT-ST BD-RE BU40N, FwRev=1.03, SerialNo=

Odd... The device model string is correct in the patch, so it should work.
May be something is not being done correctly for ATAPI. Let me check.

>  Config={ Fixed Removable DTR<=5Mbs DTR>10Mbs nonMagnetic }
>  RawCHS=0/0/0, TrkSize=0, SectSize=0, ECCbytes=0
>  BuffType=unknown, BuffSize=unknown, MaxMultSect=0
>  CurCHS=0/0/0, CurSects=0, LBA=yes, LBAsects=0
>  IORDY=on/off, tPIO={min:120,w/IORDY:120}, tDMA={min:120,rec:120}
>  PIO modes:  pio0 pio1 pio2 pio3 pio4 
>  DMA modes:  mdma0 mdma1 mdma2 
>  UDMA modes: udma0 udma1 udma2 udma3 udma4 *udma5 udma6 
>  AdvancedPM=no
>  Drive conforms to: unknown:  ATA/ATAPI-3,4,5,6,7
> 
>  * signifies the current active mode
> 
> Cheers,
> Chris


-- 
Damien Le Moal
Western Digital Research