Opened 8 years ago
Last modified 7 years ago
#832 new patch
Enable / disable the certain device features by specifying feature value and sector count value.
Reported by: | JonghwanChoi | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | undecided |
Component: | smartctl | Version: | 6.5 |
Keywords: | Cc: |
Description
There are many device features. To support these features, we have to add a lot of code.
This patch set enable / disable the certain device features by specifying feature value and sector count value.
example) ./smartctl -s features,0x64,0x2 /dev/sdb -> DSN control
Do not turn dangerous features on.Use with care.
This patch is quite useful for the FW developer.
Attachments (1)
Change History (4)
by , 8 years ago
Attachment: | 0001-Enable-Disable-the-certain-device-features-by-specif.patch added |
---|
comment:1 by , 8 years ago
Milestone: | → undecided |
---|
comment:2 by , 8 years ago
Component: | all → smartctl |
---|
comment:3 by , 7 years ago
I think smartctl should not be a tool doing this. Some of my arguments:
- User is not expecting that smartctl/smartd can do any hard and we are trying to avoid it as much as we can (e.g. by blacklisting some "known to be bad" commands on some controllers)
- On some OS/controllers there is only limited interface (e.g. on Darwin, some raid-s) which simply not allowing to send anything but predefined commands, so it will not work anyway.
May be its could be some other binary in the same source.
Note:
See TracTickets
for help on using tickets.
Of course this useful for FW developers - but possibly dangerous (disconnect or brick device, damage data) for our typical user audience.
I'm not yet sure whether we want such send arbitrary ATA commands functionality in smartctl. Please discuss on smartmontools-devel mailing list.
Review of patch is left for later.