Re: [PATCH] RTC: Add mmap method to rtc character driver

On Tue, Jul 25, 2006 at 07:55:39PM +0200, Arjan van de Ven wrote:
@@ -265,6 +269,7 @@ irqreturn_t rtc_interrupt(int irq, void

kill_fasync (&rtc_async_queue, SIGIO, POLL_IN);

+ *count_ptr = (*count_ptr)++;

Hi,

it's a cute idea, however 3 questions:
1) you probably want to add a few memory barriers around this, right?
Actually, I was curious about that. I had initially planned on using xchg to
update the counter on the mmaped page, but then it occured to me that an
unsigned long would always be an atomic update (or so I thought). Is there a
case in which userspace will read a munged value the way the code is now?

2) why use the rtc and not the regular timer interrupt?

Honestly, because it seemed to be a quick way forward. The real time clock
driver was there, and its infrastructure lent itself rather easily toward adding
this functionality. If you can elaborate on a better suggestion, I'll happily
take a crack at it.

(and
3) this will negate the power gain you get for tickless kernels, since
now they need to start ticking again ;( )

That is true, but only in the case where someone opens up /dev/rtc, and if they
open that driver and send it a UIE or PIE ioctl, it will start ticking
regardless of this patch (or that is at least my impression).

Regards
Neil

Greetings,
Arjan van de Ven

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

--
/***************************************************
*Neil Horman
*Software Engineer
*gpg keyid: 1024D / 0x92A74FA1 - http://pgp.mit.edu
***************************************************/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Relevant Pages

  • Re: Net: e1000 driver: TX Hang message
    ... It receives packets but keeps on giving "Tx hang" ... > the same piece of code is in the 2.6.13.2 kernel's e1000 driver. ... > More majordomo info at http://vger.kernel.org/majordomo-info.html ... > Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • Re: MODULE_LICENSE usage
    ... GPL, so that others are free to distribute and modify the driver. ... After asking the FSF for advice and working through their FAQ, ... More majordomo info at http://vger.kernel.org/majordomo-info.html ...
    (Linux-Kernel)
  • Re: [RFC PATCH] move drm to pci_request_irq
    ... Actually, this is the exit path, I felt like it could be safer if it was ... should probably be done in each driver. ... More majordomo info at http://vger.kernel.org/majordomo-info.html ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • Re: [ALSA] 2.6.21 loading of module snd_ymfpci takes 2 minutes
    ... Santiago Garcia Mantinan wrote: ... driver I decided to replace the ymfpci_main.c from 2.6.21 with the one from ... More majordomo info at http://vger.kernel.org/majordomo-info.html ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • Re: [PATCH] RTC: Add mmap method to rtc character driver
    ... open that driver and send it a UIE or PIE ioctl, it will start ticking ... regardless of this patch. ... More majordomo info at http://vger.kernel.org/majordomo-info.html ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)