On 9/1/07, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote: > On Fri, 31 Aug 2007 00:22:46 +0530 > Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx> wrote: > > > +/* > > + * Strategy routines for formating read/write data > > + */ > > +int mem_container_read_strategy(unsigned long long val, char *buf) > > +{ > > + return sprintf(buf, "%llu Bytes\n", val); > > +} > > It's a bit cheesy to be printing the units like this. It's better to just > print the raw number. > > If you really want to remind the user what units that number is in (not a > bad idea) then it can be encoded in the filename, like > /proc/sys/vm/min_free_kbytes, /proc/sys/vm/dirty_expire_centisecs, etc. > Sounds good, I'll change the file to memory.limit_in_bytes and memory.usage_in_bytes. > > > +int mem_container_write_strategy(char *buf, unsigned long long *tmp) > > +{ > > + *tmp = memparse(buf, &buf); > > + if (*buf != '\0') > > + return -EINVAL; > > + > > + printk("tmp is %llu\n", *tmp); > > don't think we want that. > Yes, I'll redo the patch and resend. > > + /* > > + * Round up the value to the closest page size > > + */ > > + *tmp = ((*tmp + PAGE_SIZE - 1) >> PAGE_SHIFT) << PAGE_SHIFT; > > + return 0; > > +} Thanks, Balbir _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers