Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-12 Thread Greg Kroah-Hartman
On Fri, Apr 12, 2013 at 08:32:03AM -0700, Jacob Pan wrote: > If so, the reason being I don't know if there is a standard way > to a transaction via sysfs where I can commit multiple inputs > atomically. That is by design. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-12 Thread Jacob Pan
On Fri, 5 Apr 2013 14:35:18 -0700 Greg Kroah-Hartman wrote: > On Fri, Apr 05, 2013 at 02:26:35PM -0700, Joe Perches wrote: > > On Fri, 2013-04-05 at 14:02 -0700, Jacob Pan wrote: > > > +static ssize_t store_event_control(struct device *dev, > > > + struct device_attribute

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-12 Thread Jacob Pan
On Fri, 5 Apr 2013 14:35:18 -0700 Greg Kroah-Hartman gre...@linuxfoundation.org wrote: On Fri, Apr 05, 2013 at 02:26:35PM -0700, Joe Perches wrote: On Fri, 2013-04-05 at 14:02 -0700, Jacob Pan wrote: +static ssize_t store_event_control(struct device *dev, + struct

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-12 Thread Greg Kroah-Hartman
On Fri, Apr 12, 2013 at 08:32:03AM -0700, Jacob Pan wrote: If so, the reason being I don't know if there is a standard way to a transaction via sysfs where I can commit multiple inputs atomically. That is by design. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Jacob Pan
On Fri, 05 Apr 2013 14:26:35 -0700 Joe Perches wrote: > > +/* in the order of enum rapl_primitives */ > > +static struct rapl_primitive_info rpi[] = { > > const? I do need to override one entry for a special case. The hardware uses a different bit location for the same lock functionality.

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Greg Kroah-Hartman
On Fri, Apr 05, 2013 at 02:26:35PM -0700, Joe Perches wrote: > On Fri, 2013-04-05 at 14:02 -0700, Jacob Pan wrote: > > +static ssize_t store_event_control(struct device *dev, > > + struct device_attribute *attr, > > + const char *buf, > > +

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Joe Perches
On Fri, 2013-04-05 at 14:02 -0700, Jacob Pan wrote: > RAPL(Running Average Power Limit) interface provides platform software > with the ability to monitor, control, and get notifications on SOC > power consumptions. Since its first appearance on Sandy Bridge, more > features have being added to

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Greg Kroah-Hartman
On Fri, Apr 05, 2013 at 02:02:04PM -0700, Jacob Pan wrote: > RAPL(Running Average Power Limit) interface provides platform software > with the ability to monitor, control, and get notifications on SOC > power consumptions. Since its first appearance on Sandy Bridge, more > features have being

[PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Jacob Pan
RAPL(Running Average Power Limit) interface provides platform software with the ability to monitor, control, and get notifications on SOC power consumptions. Since its first appearance on Sandy Bridge, more features have being added to extend its usage. In RAPL, platforms are divided into domains

[PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Jacob Pan
RAPL(Running Average Power Limit) interface provides platform software with the ability to monitor, control, and get notifications on SOC power consumptions. Since its first appearance on Sandy Bridge, more features have being added to extend its usage. In RAPL, platforms are divided into domains

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Greg Kroah-Hartman
On Fri, Apr 05, 2013 at 02:02:04PM -0700, Jacob Pan wrote: RAPL(Running Average Power Limit) interface provides platform software with the ability to monitor, control, and get notifications on SOC power consumptions. Since its first appearance on Sandy Bridge, more features have being added to

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Joe Perches
On Fri, 2013-04-05 at 14:02 -0700, Jacob Pan wrote: RAPL(Running Average Power Limit) interface provides platform software with the ability to monitor, control, and get notifications on SOC power consumptions. Since its first appearance on Sandy Bridge, more features have being added to extend

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Greg Kroah-Hartman
On Fri, Apr 05, 2013 at 02:26:35PM -0700, Joe Perches wrote: On Fri, 2013-04-05 at 14:02 -0700, Jacob Pan wrote: +static ssize_t store_event_control(struct device *dev, + struct device_attribute *attr, + const char *buf, +

Re: [PATCH v2] Introduce Intel RAPL cooling device driver

2013-04-05 Thread Jacob Pan
On Fri, 05 Apr 2013 14:26:35 -0700 Joe Perches j...@perches.com wrote: +/* in the order of enum rapl_primitives */ +static struct rapl_primitive_info rpi[] = { const? I do need to override one entry for a special case. The hardware uses a different bit location for the same lock