From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurence Oberman Subject: Re: [PATCH] st: implement sysfs based tape statistics v2 Date: Thu, 5 Feb 2015 12:40:20 -0500 (EST) Message-ID: <995891679.6002631.1423158020737.JavaMail.zimbra@redhat.com> References: <5B997489-2D5C-4F85-BAB8-317E62F694E5@kolumbus.fi> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mx3-phx2.redhat.com ([209.132.183.24]:48027 "EHLO mx3-phx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750884AbbBERkk convert rfc822-to-8bit (ORCPT ); Thu, 5 Feb 2015 12:40:40 -0500 In-Reply-To: <5B997489-2D5C-4F85-BAB8-317E62F694E5@kolumbus.fi> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: Kai =?utf-8?Q?M=C3=A4kisara_=28Kolumbus=29?= Cc: Laurence Oberman , Shane M Seymour , linux-scsi@vger.kernel.org, "James E.J. Bottomley (JBottomley@parallels.com)" , jeffm@suse.com ----- Original Message ----- =46rom: "Kai M=C3=A4kisara (Kolumbus)" To: "Laurence Oberman" Cc: "Shane M Seymour" , loberman@redhat.com, linu= x-scsi@vger.kernel.org, "James E.J. Bottomley (JBottomley@parallels.com= )" , jeffm@suse.com Sent: Thursday, February 5, 2015 12:03:29 PM Subject: Re: [PATCH] st: implement sysfs based tape statistics v2 > On 2.2.2015, at 17.16, Laurence Oberman wrote: >=20 > I pulled this this morning and will be testing. The prior version was > stable for me on the upstream and RHEL 6.5 kernel without exhaustive > testing. > We also just received more requests to get this into RHEL from HP / > Red Hat customers. >=20 > Kai, what are your thoughts. I realize this is a large amount of > additional code. I am not keen to create a driver just for stats as w= e > would have to keep the rest of the st driver changes always in sync. >=20 I still think that the tape statistics should be exported like the stat= istics of =E2=80=9Creal=E2=80=9D block devices, i.e., one sysfs file ex= porting on a single line the statistics that temporally belong together= =2E James rejected this approach. I am leaving the decision about this = code to him. I will neither ack nor nak this code. Thanks, Kai Hello Kai, I missed the earlier conversations with James, I will go search for the= m. Do you mean add them so they are similar to the /proc/diskstats cat /proc/diskstats =2E. 8 0 sda 2258346 152801 291907067 5263795 388817 1518048 150138= 33 4542062 0 4794931 9803495 8 1 sda1 717 102 26154 1179 8 2 80 76 0 1172 1254 8 2 sda2 328 31 2872 1554 0 0 0 0 0 1554 1554 8 3 sda3 2195205 151617 290898283 5203627 355053 1518046 15009= 528 4370598 0 4594137 9571937 8 4 sda4 61921 1050 978350 57218 18 0 4225 34 0 56384 57185 11 0 sr0 0 0 0 0 0 0 0 0 0 0 0 =2E. Laurence Oberman Red Hat Global Support Service SEG Team -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html