一个slab.c中的疑问。。。。

lsm2624

在slab.c中的kmem_cache_create（）

down(&cache_chain_sem);设置信号量，以便可以访问缓存链
{
　struct list_head *p;
　　
　list_for_each(p, &cache_chain) {
　 kmem_cache_t *pc = list_entry(p, kmem_cache_t, next);
　　
　 /* The name field is constant - no lock needed. */名称出错，则报错
　 if (!strcmp(pc->name, name))
　　BUG();
　}
}

　 if (!strcmp(pc->name, name))
　　BUG();
这个地方为什么名字都出错了只报错不返回或者goto到前面呢？
还有一个是：
down(&cache_chain_sem);
{
     。。。
}怎么突然来一对花括号？？？

lsm2624

菜鸟学习中，估计问题有点弱智还请不吝赐教。。。

amarant

有些错误用BUG，有些用goto，看错误严重程度而定吧
{}这个加不加都无所谓啦，只是看起来更像一个代码段一点吧

lsm2624

回复 3# amarant


    谢谢！那使用BUG的既然是错误为什么不需要返回或退出？
这种错误是属于什么样的错呢？

tempname2

加花括号产生一个新的block，这样可以就地声明变量。BUG不了解，好像是发生了无可挽回的错误了，应该跟panic差不多吧。这里有一段注释：

/*
* Don't use BUG() or BUG_ON() unless there's really no way out; one
* example might be detecting data structure corruption in the middle
* of an operation that can't be backed out of.  If the (sub)system
* can somehow continue operating, perhaps with reduced functionality,
* it's probably not BUG-worthy.
*
* If you're tempted to BUG(), think again:  is completely giving up
* really the *only* solution?  There are usually better options, where
* users don't need to reboot ASAP and can mostly shut down cleanly.
*/

lsm2624

回复 5# tempname2


    感谢！简单点说就是死机了，系统崩溃。。。:wink: