From: Douglas Gilbert <dgilbert@interlog.com>
To: "Akers, Jason B" <jason.b.akers@intel.com>,
Phillip Susi <psusi@ubuntu.com>,
"linux-ide@vger.kernel.org" <linux-ide@vger.kernel.org>
Cc: "Karkra, Kapil" <kapil.karkra@intel.com>,
"Williams, Dan J" <dan.j.williams@intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH] sg3_utils: Added hybrid information log utility
Date: Tue, 25 Nov 2014 15:27:44 -0500 [thread overview]
Message-ID: <5474E640.7020206@interlog.com> (raw)
In-Reply-To: <372C7AE269BA5E4D92B1082F530ED4B88A13F692@ORSMSX110.amr.corp.intel.com>
On 14-11-25 02:13 PM, Akers, Jason B wrote:
> Hi Phillip,
> It turns out that this patch was based on an old github repository that doesn't appear to be updated. Doug Gilbert reached out after the initial RFC and directed us to his page (http://sg.danny.cz/sg/) where he has updated sg3_utils code.
>
> As we continue to work through the feedback from our SSHD kernel patches, I'm going to follow up with Doug again on sg3_utils.
Hi,
As discussed earlier in this thread adding a new utility
for each log page like sg_sat_get_hybrid_log would lead to
hundreds of utilities. There are now these groups of
storage related pages (with user space utilities to decode
them shown in parentheses):
- SCSI VPD pages (sg_vpd (sdparm and legacy sg_inq))
- SCSI mode pages (sdparm and sg_modes)
- SCSI log pages (sg_logs)
- ATA GP log pages (from 1.40 sg_sat_read_gplog)
- ATA SMART log pages (smartctl)
At the start of this thread the proposed patch was very
close to the release of sg3_utils version 1.40 and Hannes
Reinecke contributed a more general approach:
sg_sat_read_gplog which I did incorporate. Currently it
only decodes pages to hex.
Please look at that utility (i.e. sg_sat_read_gplog) as the
basis for adding ATA GP log page decoding such as the SATA
hybrid information page. Please send patches against its source.
Doug Gilbert
>> -----Original Message-----
>> From: Phillip Susi [mailto:psusi@ubuntu.com]
>> Sent: Monday, November 24, 2014 4:34 PM
>> To: Akers, Jason B; linux-ide@vger.kernel.org
>> Cc: Karkra, Kapil; Williams, Dan J; linux-kernel@vger.kernel.org;
>> dgilbert@interlog.com
>> Subject: Re: [RFC PATCH] sg3_utils: Added hybrid information log utility
>>
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA512
>>
>> On 10/29/2014 03:57 PM, Jason B. Akers wrote:
>>> From: Kapil Karkra <kapil.karkra@intel.com>
>>>
>>> A solid state hybrid drive (SSHD) is a hard drive with a small
>>> embedded NAND. Host drivers can manage this NAND using the
>> commands
>>> defined by SATA 3.2 standard. One SATA command that provides
>>> visibility into the SSHD's cache behavior is get hybrid log command.
>>> This augmentation allows users to issue the command as
>>> follows:
>>>
>>> sg_sat_get_hybrid_log /dev/sdc assuming /dev/sdc is an SSHD.
>>>
>>> patch applies to the following sourcebase from where to clone:
>>> https://github.com/hreinecke/sg3_utils.git
>>
>> I cloned that repo and the patch does not apply cleanly. git am -3 says the
>> referenced blobs are not found and it has conflicts in src/Makefile.am. I
>> manually fixed that up and then it fails to link due to unresolved externals.
>>
>>
>>
>> -----BEGIN PGP SIGNATURE-----
>> Version: GnuPG v1
>>
>> iQEcBAEBCgAGBQJUc85UAAoJENRVrw2cjl5RR3YIAJOF0fcTkF5pM0OIA87hK8k
>> 8
>> ujwflBTy66ZbPzXMD97cTGYQ6gPp3UCeT/ZcElBfrGGAuD5u/ETXMJY7lqAZgYt
>> K
>> bELPv6oLhz3ya2IO442sUlRYO4dVfwhsnak+2rLiQkXCy4WKSJ1EYiMIP/VHj8ne
>> 5e8VYkuc0KamKjvT4v6EwJ7tM0blfJbYgmFGruIxOSgCAo94Zp9thAcctb9PbRex
>> HnPTcnw5cI1sCoLf5fJafhc9w/1H45WjJTnGF7zs4hq/GGsYCwryvI0ovwAzU9ST
>> R3jRvE+70oiyQDsj9EBak63hRYIAd8QzQ3TkCjaM5E1XHSU7vTnxvxAjKb4ZyF4=
>> =a8Vh
>> -----END PGP SIGNATURE-----
next prev parent reply other threads:[~2014-11-25 20:28 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-29 19:57 [RFC PATCH] sg3_utils Hybrid Information Log utility Jason B. Akers
2014-10-29 19:57 ` [RFC PATCH] sg3_utils: Added hybrid information log utility Jason B. Akers
2014-11-25 0:33 ` Phillip Susi
2014-11-25 19:13 ` Akers, Jason B
2014-11-25 20:27 ` Douglas Gilbert [this message]
2014-12-16 0:46 ` Phillip Susi
2015-03-24 2:27 ` Phillip Susi
2015-03-24 19:07 ` Douglas Gilbert
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5474E640.7020206@interlog.com \
--to=dgilbert@interlog.com \
--cc=dan.j.williams@intel.com \
--cc=jason.b.akers@intel.com \
--cc=kapil.karkra@intel.com \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=psusi@ubuntu.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).