On Thu, 2006-04-20 at 10:12 +0900, MAEDA Naoaki wrote: > Hi Chandra, > > [EMAIL PROTECTED] wrote: > > Hi All, > > > > f-series CKRM went thru a major overhaul, mainly to reduce the codesize > > and make it easy to understand. > > > > As usual, I uploaded the patches to the project website in two formats. > > > > External visible changes are: > > For user: > > guarantee, limit, total_guarantee and max_limit are replaced by > > min_shares - same usage as guarantee > > max_shares - same usage as limit > > child_shares_divisor - replaces both total_guarantee and max_limit > > > > config file is removed, instead we are now using module parameters > > changeable through /sys/module/ > > > > attribute file is removed. > > Since this is a major overhaul, I propose one more specification change. > > I think min_shares and max_shares shouldn't be exposed in the shares > file of the root class, because changing these two parameters in the > root class is no effect.
on the same token child_shares_divisor will have no effect on the leaf classes. We say both of these in the documentation. > > # I was confused a lot by the specification when I started to use CKRM. > > Setting these two parameters in the root class to CKRM_SHARE_UNSUPPORTED > is alternative, but eliminating them is more straightforward. I added UNSUPPORTED only for taking care of controller not supporting the parameter. > > What do you think? > > Thanks, > MAEDA Naoaki > -- ---------------------------------------------------------------------- Chandra Seetharaman | Be careful what you choose.... - [EMAIL PROTECTED] | .......you may get it. ---------------------------------------------------------------------- ------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ ckrm-tech mailing list https://lists.sourceforge.net/lists/listinfo/ckrm-tech