[Lustre-devel] moving obd_fail_check to libcfs

di wang di.wang at sun.com
Mon Feb 23 14:50:57 PST 2009


Hello,
Eric Barton wrote:
> Although I could agree that there should be levels of abstraction
> above libcfs, it is, de facto, the place we put _all_ generic code
> - not just stateless porting primitives, but everything that can be
> used everywhere.
>
> I don't actually think of obd_fail_check as inexorably bound with
> /proc.  But since that's the current implementation, it's probably
> my oversight not to have shared that sense of direction.
>
> Nic, is the patch totally /proc - centric?  Wangdi is doing the work
> to remove /proc-ness and make our tuneables, configurables and monitoring
> more portable.  He needs to be involved...
>
>   
Yes, after we have our own /proc stuff in lustre, which might be land to 
HEAD in 2 or 3 weeks. All the new proc stuff is implemented in libcfs 
layer, then we will move as much as obd proc stuff(obd lprocfs layer) to 
libcfs layer, then they(include obd_fail_check) can be shared with LNET.

The only difference for those sysctl parameters is that you may not use 
/etc/sysctl.conf to control them anymore, and lctl set_param is the only 
interface here.

Actually, you can also move this now. but it means you need move those 
obd proc api to libcfs layer,  which  might  not  be  small  amount  of  
work.

Thanks
WangDi
>     Cheers,
>               Eric
>
>   




More information about the lustre-devel mailing list