On Wed, Jun 9, 2010 at 10:16 AM, Frank Steiner <[email protected]> wrote: > Hi, > > Dhaval Giani wrote: >> Hi, >> >> Annoucing v0.36.2.rc > > just tried this version and it fails to parse my formerly working > cgconfig.conf. Indeed, any 0.36* version fails, while 0.34 and 0.35 > work fine. But from the manpage I can't figure out what has changed. > > My cgconfig.conf: > mount { > cpuset = /dev/cgroup/cpuset; > memory = /dev/cgroup/memory; > } > > group applications { > cpuset { > cpuset.cpus = 0-3; > cpuset.mems = 0-1; > } > > memory { > memory.limit_in_bytes=7.5G;
OK, this is the line that is failing. Earlier we were not checking the return value of writes (till v0.35), and we added fixes to ensure we actually wrote to the file successfully. AFAIK, the only valid values are k and M, not G. Balbir can confirm that. So yes, this is an invalid configuration file, we do parse it, but we fail while creating the groups. I could add a printf to stderr there to give us more useful information on failure, but i am not sure if that is a good idea to do from the library. thanks for the report! Dhaval ------------------------------------------------------------------------------ ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo _______________________________________________ Libcg-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/libcg-devel
