Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#1314 closed enhancement (fixed)

Add support for JMS56x RAID

Reported by: elbandi Owned by: Christian Franke
Priority: minor Milestone: Release 7.2
Component: all Version: 7.1
Keywords: jmicron jms56x Cc:

Description

I have this device: https://www.speeddragon.com/index.php?controller=Default&action=ProductInfo&Id=590

Actually, it'a a clone from this: https://www.sybausa.com/index.php?route=product/product&path=64_148&product_id=850

It has a JMicron JMS562 chip. If i modify some parameters in dev_jmb39x_raid, it works.

SCRAMLED_CMD: 0x197b0562
CRC: 0xde10952b

Attachments (3)

logicalvolume.txt (1.3 KB ) - added by elbandi 4 years ago.
result from query the raid volume
debug.txt (118.7 KB ) - added by elbandi 4 years ago.
query device with modified code
0001-Change-dev_jmb39x_raid-to-JMS56x.patch (1.1 KB ) - added by elbandi 4 years ago.

Download all attachments as: .zip

Change History (12)

by elbandi, 4 years ago

Attachment: logicalvolume.txt added

result from query the raid volume

by elbandi, 4 years ago

Attachment: debug.txt added

query device with modified code

comment:1 by Christian Franke, 4 years ago

Milestone: undecided

... If i modify some parameters in dev_jmb39x_raid, it works.

Please provide these modifications as a patch file.

Related: tickets #705, #1283, #1290.

comment:2 by elbandi, 4 years ago

I just changed the two parameters, but thats breaks the jmb39x support, so not good for long time. I think this jmicron device support need a refactor for a common support: rename the dev_jmb39x to something dev_jmicron_raid.cpp, handle better the different chips and the chips variants. like:
-d jmicron,chiptype[jmb39x|jms56x|xxx],variantid

it this idea is good, i can make this changes.

Version 0, edited 4 years ago by elbandi (next)

comment:3 by Christian Franke, 4 years ago

Of course your changes would break existing support. So please provide you changes as a patch file to document it cleanly. Of course, the patch will not be applied.

I leave any refactoring for later as more enhancements (#1290) are expected.

comment:4 by Christian Franke, 4 years ago

Milestone: undecidedRelease 7.2
Summary: Add support for JMS56x seriesAdd support for JMS56x RAID

Thanks. I will add another (possibly preliminary) option to select this variant.

Any refactoring and unifying of option syntax will done later, possibly close to next release.

comment:5 by Christian Franke, 4 years ago

Owner: set to Christian Franke
Status: newaccepted

comment:6 by Christian Franke, 4 years ago

Resolution: fixed
Status: acceptedclosed

comment:7 by Christian Franke, 4 years ago

Please test smartctl -d jms56x,N ... with a build from current SVN trunk.

Source tarballs and various binaries are available at https://builds.smartmontools.org/.

comment:8 by Christian Franke, 4 years ago

Any test results so far?

comment:9 by elbandi, 4 years ago

it works for me.

Note: See TracTickets for help on using tickets.