lsdh-fei

Results 6 comments of lsdh-fei

@billhollings this bug also exists in the latest MoltenVK, it crash as i described in https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/8473 ,please check it, thx

@wwbmmm @MJY-HUST 看了下提交记录,不知道是不是 #2645 引入的,辛苦看下哈

@chenBright 不是,就是rpc处理的bthread中使用的

@MJY-HUST 没有单独使用bthread-local,就是task_runner中的,我门最近把brpc从1.8升级到了1.10,然后就发现新版有内存泄漏,LogStream占用了大量内存,看了下brpc源代码,发现log stream的buf是存在keytable中,然后就怀疑是keytable的问题,然后看vars监控,发现keytable的数量一直在不断增长,所以才怀疑是新版的改动导致的

现在观察到的现象是经常会出现 LogStream** get_or_new_tls_stream_array() => bthread_getspecific 无法获取LogStream,然后new了LogStream之后通过 bthread_setspecific 设置到tls_bls.keytable,但是同时在这里new一个KeyTable。所以新增的LogStream和KeyTable的数量是相同的。 ![image](https://github.com/user-attachments/assets/995833d5-c734-435e-8588-1e38f5e4dac4) ![image](https://github.com/user-attachments/assets/88d5960b-bec6-4d10-9ead-2b2f8c321ce0) 应该就是new log stream时创建的keytable泄漏了

这里有个疑问,为什么不直接把bthread_keytable_pool_t改成thread-local的呢?@MJY-HUST