Re: [PATCH] [RFCv2] notify userspace about time changes

From: Greg KH
Date: Thu Aug 19 2010 - 00:16:48 EST


On Wed, Aug 18, 2010 at 04:53:03PM -0700, Andrew Morton wrote:
> Is sysfs the right interface for this thing? Bear in mind that
> CONFIG_SYSFS does exist.
>
> > + fd = open("/sys/kernel/time_notify", O_WRONLY);
> > + fdprintf(fd, "%d 1 0 1 1", efd);
>
> why not
>
> sys_time_notify(efd, 1, 0, 1, 1);

Yeah, that would be much better than a sysfs file, this is abusing the
sysfs interface quite a lot.

thanks,

greg k-h
--
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/