All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yong Zhang <yong.zhang0@gmail.com>
To: john stultz <johnstul@us.ibm.com>
Cc: linux-kernel@vger.kernel.org, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH 1/3] clocksource: Also tweak ->maxadj in clocksource_calc_mult_shift()
Date: Fri, 2 Dec 2011 09:35:38 +0800	[thread overview]
Message-ID: <20111202013538.GA15687@zhy> (raw)
In-Reply-To: <1322783907.21423.153.camel@work-vm>

On Thu, Dec 01, 2011 at 03:58:27PM -0800, john stultz wrote:
> On Thu, 2011-12-01 at 15:20 +0800, Yong Zhang wrote:
> > There is still user of clocksource_calc_mult_shift(), and it's
> > complement to commit d65670a7 [clocksource: Avoid selecting mult
> > values that might overflow when adjusted]
> 
> I'd probably drop this patch, as its just extra churn.

Hmm, so you drop option#1,

> 
> Again, I've queued 2/3 to resolve the issue and queued a simplified 3/3
> to just remove clocksource_calc_mult_shift. 

and take option#2 instead (and 3/3 will become more simple).
Both work to me ;-)

Thank you John.

-Yong

  reply	other threads:[~2011-12-02  1:35 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-01  7:20 [PATCH 0/3] Patches focus on clocksource_calc_mult_shift() Yong Zhang
2011-12-01  7:20 ` [PATCH 1/3] clocksource: Also tweak ->maxadj in clocksource_calc_mult_shift() Yong Zhang
2011-12-01 23:58   ` john stultz
2011-12-02  1:35     ` Yong Zhang [this message]
2011-12-01  7:20 ` [PATCH 2/3] clocksource: dbx500: convert to clocksource_register_hz() Yong Zhang
2011-12-01 13:03   ` Linus Walleij
2011-12-01 23:55   ` John Stultz
2011-12-02  8:29     ` Linus Walleij
2011-12-02 19:28       ` John Stultz
2011-12-01  7:20 ` [PATCH 3/3] clocksource: get rid of clocksource_calc_mult_shift() Yong Zhang

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=20111202013538.GA15687@zhy \
    --to=yong.zhang0@gmail.com \
    --cc=johnstul@us.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tglx@linutronix.de \
    /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.