All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rodolfo Giometti <giometti@enneenne.com>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: linux-kernel@vger.kernel.org,
	Andrew Morton <akpm@linux-foundation.org>,
	David Woodhouse <dwmw2@infradead.org>,
	Dave Jones <davej@redhat.com>, Sam Ravnborg <sam@ravnborg.org>,
	Greg KH <greg@kroah.com>, Randy Dunlap <randy.dunlap@oracle.com>,
	Kay Sievers <kay.sievers@vrfy.org>,
	Alan Cox <alan@lxorguk.ukuu.org.uk>, Ingo Molnar <mingo@elte.hu>,
	Michael Kerrisk <mtk.manpages@gmail.com>,
	Christoph Hellwig <hch@infradead.org>
Subject: Re: [PATCH 09/11] pps: low level IRQ timestamps recording.
Date: Thu, 3 Dec 2009 13:01:30 +0100	[thread overview]
Message-ID: <20091203120130.GD12943@gundam.enneenne.com> (raw)
In-Reply-To: <4B170204.4080604@zytor.com>

On Wed, Dec 02, 2009 at 04:10:44PM -0800, H. Peter Anvin wrote:
> On 12/02/2009 09:18 AM, Rodolfo Giometti wrote:
> > Add low level IRQ timestamps recording for x86 (32 and 64 bits)
> > platforms and enable UART clients in order to use it.
> > 
> > This improves PPS precision. :)
> 
> It also invokes getnstimeofday on every single interrupt, including ones
> which have absolutely nothing to do with the PPS, and are potentially
> high volume.  getnstimeofday can be a fairly expensive operation,
> especially if the best available clock is away from the CPU, or it
> requires extensive arithmetic in order to normalize the clock value...

I see, that's why I propose my patches to you: in order to have some
feedbacks for better solutions. :)

Can I add a IRQF_TIMESTAMP in order to ask to the system to record irq
timestamps for particular irq lines? The problem is that I cannot set
that flag at irq_register() time but I have to do it later... it can
be a problem?

Ciao,

Rodolfo

-- 

GNU/Linux Solutions                  e-mail: giometti@enneenne.com
Linux Device Driver                          giometti@linux.it
Embedded Systems                     phone:  +39 349 2432127
UNIX programming                     skype:  rodolfo.giometti
Freelance ICT Italia - Consulente ICT Italia - www.consulenti-ict.it

  reply	other threads:[~2009-12-03 12:01 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-02 17:18 LinuxPPS new functionalities Rodolfo Giometti
2009-12-02 17:18 ` [PATCH 01/11] pps: userland header file for PPS API Rodolfo Giometti
2009-12-02 17:18   ` [PATCH 02/11] pps: documentation programs and examples Rodolfo Giometti
2009-12-02 17:18     ` [PATCH 03/11] pps: LinuxPPS clients support Rodolfo Giometti
2009-12-02 17:18       ` [PATCH 04/11] ldisc: new dcd_change() method for line disciplines Rodolfo Giometti
2009-12-02 17:18         ` [PATCH 05/11] ldisc n_tty: export all N_TTY ldisc methods Rodolfo Giometti
2009-12-02 17:18           ` [PATCH 06/11] pps: serial clients support Rodolfo Giometti
2009-12-02 17:18             ` [PATCH 07/11] serial 8250: enable PPS support Rodolfo Giometti
2009-12-02 17:18               ` [PATCH 08/11] pps: parallel port clients support Rodolfo Giometti
2009-12-02 17:18                 ` [PATCH 09/11] pps: low level IRQ timestamps recording Rodolfo Giometti
2009-12-02 17:18                   ` [PATCH 10/11] serial amba-pl010: enable PPS support Rodolfo Giometti
2009-12-02 17:18                     ` [PATCH 11/11] arm: add low level IRQ timestamps recording for arm platforms Rodolfo Giometti
2009-12-03  0:10                   ` [PATCH 09/11] pps: low level IRQ timestamps recording H. Peter Anvin
2009-12-03 12:01                     ` Rodolfo Giometti [this message]
2009-12-05  6:46                       ` H. Peter Anvin
2009-12-02 18:59             ` [PATCH 06/11] pps: serial clients support Alan Cox
2009-12-03 10:53               ` Rodolfo Giometti
2009-12-03 11:46                 ` Alan Cox
2009-12-03 12:05                   ` Rodolfo Giometti
2009-12-03 12:24                     ` Alan Cox
2009-12-02 18:18           ` [PATCH 05/11] ldisc n_tty: export all N_TTY ldisc methods Greg KH

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=20091203120130.GD12943@gundam.enneenne.com \
    --to=giometti@enneenne.com \
    --cc=akpm@linux-foundation.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=davej@redhat.com \
    --cc=dwmw2@infradead.org \
    --cc=greg@kroah.com \
    --cc=hch@infradead.org \
    --cc=hpa@zytor.com \
    --cc=kay.sievers@vrfy.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=mtk.manpages@gmail.com \
    --cc=randy.dunlap@oracle.com \
    --cc=sam@ravnborg.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.