Opened 9 days ago

Last modified 8 days ago

#1877 new enhancement

Regex adjustment for Micron 5300 MAX (MTFDDAK1T9TDT)

Reported by: Robin Owned by:
Priority: minor Milestone: unscheduled
Component: drivedb Version: 7.3
Keywords: ssd Cc: Robin

Description (last modified by Robin)

The regex for Micron 5300 MAX needs to be adjusted.

It is currently:
"Micron_5300(HC)?_MTFDDA[KV](240|480|960|1T9|3T8|7T6)TD[ST]"
and it needs to be:
"(Micron_5300)?(_|HC_)?MTFDDA[KV](240|480|960|1T9|3T8|7T6)TD[ST]"

Attachments (1)

smartctl-micron-5300-max.txt​ (12.9 KB ) - added by Robin 9 days ago.

Download all attachments as: .zip

Change History (7)

by Robin, 9 days ago

Attachment: smartctl-micron-5300-max.txt​ added

comment:1 by Christian Franke, 9 days ago

Keywords: ssd added
Milestone: Release 7.5 → unscheduled
Type: defect → enhancement

This would no longer match Micron_5300_... without HC.

Fix:
"(Micron_5300(HC)?_)?MTFDDA[KV](240|480|960|1T9|3T8|7T6)TD[ST]"

PS: Please don't set a milestone in future submissions.

comment:2 by Christian Franke, 9 days ago

Summary: Regex adjustment for Micron 5300 MAX → Regex adjustment for Micron 5300 MAX (MTFDDAK1T9TDT)

comment:3 by Robin, 9 days ago

Description: modified (diff)

comment:4 by Robin, 9 days ago

I updated the original ticket with a more inclusive regex. You can see that it matches here: ​https://regex101.com/r/vOvjwq/1

comment:5 by Christian Franke, 8 days ago

This variant also matches _ and HC_ prefix but should not:
​https://regex101.com/r/9LoZb8/1
Therefore I prefer:
​https://regex101.com/r/HHFTGX/1

comment:6 by Robin, 8 days ago

I'd be happy to go with your preference!

Note: See TracTickets for help on using tickets.