All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: Mark Lord <liml@rtr.ca>
Cc: IDE/ATA development list <linux-ide@vger.kernel.org>,
	Tejun Heo <htejun@gmail.com>, Alan Cox <alan@lxorguk.ukuu.org.uk>
Subject: Re: Per ATA host sysfs attributes?
Date: Tue, 10 Mar 2009 19:07:42 -0400	[thread overview]
Message-ID: <49B6F2BE.3000509@pobox.com> (raw)
In-Reply-To: <49B6C99B.9030507@rtr.ca>

Mark Lord wrote:
> Guys,
> 
> I have added IRQ coalescing to sata_mv, but need a way to activate/tune it.
> There are two simple parameters:  an IRQ count threshold, and a time 
> threshold.
> 
> Whenever either the count or the time threshold is exceeded,
> the chip will generate an interrupt (assuming at least one completed 
> event).
> 
> These two parameters are per-chip (actually, per group of four ports 
> within the chip).
> How can I export these for setting/query withing sysfs ?
> 
> It is all rather easy to create per-SCSI-host attrs in sysfs,
> but I don't see any obvious way to do it per-chip.
> Suggestions?

The absence of a kobject associated with ata_host is largely the root 
cause of this problem you've noted.  Long term, we want to move in a 
direction where we have a "storage_host" container object, containing 
one or more port-style objects that can be ATA or SCSI.  In that 
scenario (and what we want to move towards, over the years), the sysfs 
attachment points should be fairly obvious.

Lacking a better suggestion, I suppose the answer is exporting per-chip 
variables multiple times, once per scsi-host.  Just get the locking 
right, since multiple sysfs attributes access the same underlying 
per-chip variable.


> For now, I'm doing it with a module parameter,
> but that's really rather stone-age in the sysfs era.

Indeed!  :)

	Jeff




      reply	other threads:[~2009-03-10 23:07 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-10 20:12 Per ATA host sysfs attributes? Mark Lord
2009-03-10 23:07 ` Jeff Garzik [this message]

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=49B6F2BE.3000509@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=htejun@gmail.com \
    --cc=liml@rtr.ca \
    --cc=linux-ide@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.