On Tue, Mar 15, 2005 at 09:17:52AM -0800, Nishanth Aravamudan wrote:
> Chandra,
> 
> > Index: 2611-rc5-numem/include/linux/ckrm_mem_inline.h
> > ===================================================================
> > --- 2611-rc5-numem.orig/include/linux/ckrm_mem_inline.h
> > +++ 2611-rc5-numem/include/linux/ckrm_mem_inline.h
> > @@ -0,0 +1,67 @@
> > +/* include/linux/ckrm_mem_inline.h : memory control for CKRM
> > + *
> > + * Copyright (C) Jiantao Kong, IBM Corp. 2003
> > + *           (C) Shailabh Nagar, IBM Corp. 2003
> > + *           (C) Chandra Seetharaman, IBM Corp. 2004
> > + *
> > + *
> > + * Memory control functions of the CKRM kernel API
> > + *
> > + * Latest version, more details at http://ckrm.sf.net
> > + *
> > + * This program is free software; you can redistribute it and/or modify
> > + * it under the terms of the GNU General Public License as published by
> > + * the Free Software Foundation; either version 2 of the License, or
> > + * (at your option) any later version.
> > + *
> > + */
> > +
> > +#ifndef _LINUX_CKRM_MEM_INLINE_H_
> > +#define _LINUX_CKRM_MEM_INLINE_H_
> > +
> > +#ifdef CONFIG_CKRM_RES_MEM
> > +
> > +#error "Memory controller for CKRM is not available."
> > +
> > +#else
> 
> Not sure I understand what this comment insinuates. If the config option
> for the CKRM Memory RC is set, we error? Should that perhaps be a
> #ifndef?

Nevermind, should read the complete patchset first :)

Thanks,
Nish


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
ckrm-tech mailing list
https://lists.sourceforge.net/lists/listinfo/ckrm-tech

Reply via email to