linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: mark gross <markgross@thegnar.org>
To: MyungJoo Ham <myungjoo.ham@gmail.com>
Cc: markgross@thegnar.org, "Turquette, Mike" <mturquette@ti.com>,
	linux-kernel@vger.kernel.org,
	Linux PM list <linux-pm@vger.kernel.org>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	"Rafael J. Wysocki" <rjw@sisk.pl>, Kevin Hilman <khilman@ti.com>
Subject: Re: [PATCH 1/2] PM / devfreq: add min/max_freq limit requested by users.
Date: Tue, 17 Jan 2012 19:27:56 -0800	[thread overview]
Message-ID: <20120118032756.GC27153@mgross-G62> (raw)
In-Reply-To: <CAJ0PZbTMuJfBL4iZyc=R4TkdfFuh1_XzV-KQr73330PUt6fcNQ@mail.gmail.com>

Either my mail reader is acting up or this email is a bit off topic.  It
seems to be talking about DVFS/ondemand sampling rates when the
discussion has been about anding CPU operating frequency constraint
requests to pm_qos

On Tue, Jan 17, 2012 at 07:32:05PM +0900, MyungJoo Ham wrote:
> On Fri, Jan 13, 2012 at 1:47 PM, mark gross <markgross@thegnar.org> wrote:
> > On Thu, Jan 12, 2012 at 11:08:44AM +0900, MyungJoo Ham wrote:
> >>
> >> In practice, we have been using min/max to test DVFS behaviors and its
> >> side effects. And we are going to use them to 1. restrict power
> >> consumption forcibly by the platform software if it is too hot or the
> >> battery is low, and to 2. guarantee the minimum performance for
> >> specific tasks controlled by the platform software.
> >>
> >> Anyway, the reason 2 could be tackled by pm-qos if we allow more
> >> options in pm-qos with 1. pm qos type to enforce DVFS response time.
> > what would pm_qos do with DVFS response time?  What power management
> > knob would it enable a constraint for?
> >
> > pm_qos doesn't do anything but enable power throttling code to consider
> > a constraint on how far to throttle "something".  pm_qos has no
> > enforcement power.
> 
> 
> - The control knob: polling interval of ondemand-like DVFS mechanisms
> - It's ok to have no enforcement power. The DVFS mechanism only needs
> an interface (PM QoS seems fine for this) from user space / device
> drivers to get the response-time requirement.
> 
> With some events, we need to adjust DVFS polling interval.

This would be a new type of request.  I don't see it as a pm_qos type of
data item but, I can see it may be useful.

My first thought is that this should be exported by cpufreq.

> For now, we do this in our devices for user input events (key input,
> touchscreen input, ...). And some peripheral device drivers want to
> get "guaranteed response time" depending on their operational modes
> from memory and bus at the start of their operations.
> 
> With user input events, user may (doing something heavy) or may not
> (doing something light) want fast reaction from CPU/MEM/GPU/... in
> many occasions, and we cannot determine it until the DVFS polling has
> been done.
> 
> In average, with near 100% threshold, ondemand-like governors will
> take 1.5 x polling interval to response. In a system with 100ms
> polling interval, DVFS mechanism will take usuallly 150ms (and up to
> 200ms) to react and this is significantly noticable to human users.
> With 60Hz display system, this is loss of almost 10 frames.

Do you really want to change the sampling rate of the DVFS governors or
just use a pm_qos request to limit to minimum frequency the governor
goes for and use a timer to modtime to remove the request on every UI
event from the touch screen driver interrupt handler?

> In order to address this, a touch event handler (or any
> thread/callback or anything deals with it) may request QoS with an
> incoming event to reduce polling interval temporarily.
> 
> Although PM-QoS does not have the QoS Type for this kind of metric;
> however, DVFS response time seems to be another QoS metric candidate.

sure its a candidate but I think perhaps an export from cpufreq may be a
good candidate too.  But, I like my timer scheme best.

However; this still needs a pm_qos class for cpu_min_feq to work.

--mark

> 
> 
> >
> >> 2. pm qos type to enforce graphics performance. And adding a duration
> >> option to pm-qos requests will be helpful (sort of a helper function):
> >> i.e., pm_qos_timed_request(struct pm_qos_request *req, int
> >> pm_qos_class, s32 value, unsigned long duration_ms);
> >
> > What would be good units for graphics throughput?
> > Where in the graphics driver would you insert the equivalent of cpufreq?
> > to control the GPU core frequency?
> 
> I've not thought about this much yet. I've just seen the need for QoS
> requirements from GPU people because DVFS mechanism loses a frame or
> frames often during GPU usage without QoS information.
> 
> I'm not so familiar with GPUs, so I can't be sure about the metric for
> graphics throughput. However, could it be "FLOPS", "triangles per
> second", or GPU clock speed?
> 
> We have GPU DVFS drivers in linux/drivers/media/video/..., which
> controls GPU core frequency and measures GPU usage. However, they can
> be implemented with devfreq framework and move into drivers/devfreq/
> later.
> 
> 
> 
> Cheers!
> MyungJoo.
> 
> 
> -- 
> MyungJoo Ham, Ph.D.
> Mobile Software Platform Lab, DMC Business, Samsung Electronics
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2012-01-18  3:28 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-11 10:02 [PATCH 1/2] PM / devfreq: add min/max_freq limit requested by users MyungJoo Ham
2012-01-11 10:02 ` [PATCH 2/2] PM / devfreq: fixed syntax errors MyungJoo Ham
2012-01-12  0:21   ` Turquette, Mike
2012-01-11 22:54 ` [PATCH 1/2] PM / devfreq: add min/max_freq limit requested by users Rafael J. Wysocki
2012-01-12  1:51   ` MyungJoo Ham
2012-01-12  0:20 ` Turquette, Mike
2012-01-12  2:08   ` MyungJoo Ham
2012-01-13  4:47     ` mark gross
2012-01-17 10:32       ` MyungJoo Ham
2012-01-18  3:27         ` mark gross [this message]
2012-01-19  5:05           ` MyungJoo Ham

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=20120118032756.GC27153@mgross-G62 \
    --to=markgross@thegnar.org \
    --cc=khilman@ti.com \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mturquette@ti.com \
    --cc=myungjoo.ham@gmail.com \
    --cc=rjw@sisk.pl \
    /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).