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

Reply via email to