<div class="zcontentRow"> >On 07/17/2017 08:00 AM, John Ferlan wrote: >> > On 07/16/2017 05:04 PM, Laine Stump wrote: >> On 07/11/2017 09:01 PM, ZhiPeng Lu wrote: >>> This patchs allow to set the buffer size for netlink socket in >>> the libvirtd configuration file. The default buffer size remain >>> as before at 128k. >> See my more detailed response to your earlier patch here: >> >> >> https://www.redhat.com/archives/libvir-list/2017-July/msg00566.html >> >> There should be no need to configure the initial libnl buffer size, >> because we enable MSG_PEEK on the libnl sockets (and recent versions of >> libnl have it turned on by default anyway). If that's not permitting the >> buffer to auto-grow as necessary, then there is a different bug somewhere. >> >> If an old version of libnl is the problem, then perhaps a patch which >> just adds a comment in virNetlinkCreateSocket to "summarize" what gets >> discovered w/r/t MSG_PEEK and the "correct" minimum version of libnl so > that the "next" person to come this way will have a chance at >> understanding what needs to be done without going through the submit a >> patch changing the size again! >> > >All that said, having it be configurable could be useful for someone who > >has a system that doesn't have that version, while still working as> >expected for the right version. >I think it may need to be a fairly unusual combination of kernel, >libvirt, and libnl versions, combined with pretty "big" hardware in >order for that to happen. More information from ZhiPeng about the >versions of those packages might allow us to make a better informed >decision. >Workarounds are okay when necessary. But adding a config parameter is >something that would need to be left in forever, leaving more code to >maintain, and all for a bug that shouldn't even be there today, much >less 6 months or a year from now - turning on message peek was supposed >to "eliminate this problem totally and permanently". If it didn't, I'd >like to know why. >ZhipPeng - can you tell us more about your setup? package versions, >hardware, example XML, gdb backtrace at the instant the error message is >logged? ----- Thanks.i will try to update libnl3 to 3.2.29 ,now libnl3 is 3.2.8 in my host <div class="zMailSign"><div><div><div><div> 芦志朋 luzhipeng IT开发工程师 IT Development Engineer 操作系统产品部/中心研究院/系统产品 OS Product Dept./Central R＆D Institute/System Product <table style="color: rgb(0, 0, 0); font-family: 宋体; widows: 1;"><tbody><tr class="firstRow"><td valign="top" width="100"><img id="sign-icon" src="cid:9ae3e214c17d49ed935d87c674ba3ee2" width="130" height="120"></td><td valign="top" width="500"><img id="sign-logo" src="cid:24242e5637af428891c4db731e7765ad" width="115" height="38"> 深圳市南山区科技南路55号中兴通讯研发大楼33楼 33/F, R&D Building, ZTE Corporation Hi-tech Road South, Hi-tech Industrial Park Nanshan District, Shenzhen, P.R.China, 518057 T: +86 755 xxxxxxxx F:+86 755 xxxxxxxx M: +86 xxxxxxxxxxx E: <a href="mailto:lu.zhipeng@zte.com.cn">lu.zhipeng@zte.com.cn</a> <a href="http://www.zte.com.cn/" target="_blank">www.zte.com.cn</a></td></tr></tbody></table></div></div></div></div></div><div><div class="zhistoryRow" style="display:block"><div class="zhistoryDes" style="width: 100%; height: 28px; line-height: 28px; background-color: #E0E5E9; color: #1388FF; text-align: center;" language-data="HistoryOrgTxt">原始邮件</div><div id="zwriteHistoryContainer"><div class="control-group zhistoryPanel"><div class="zhistoryHeader" style="padding: 8px; background-color: #F5F6F8;"><div>发件人： <laine@laine.org>;</div><div>收件人： <libvir-list@redhat.com>;</div><div>抄送人： <jferlan@redhat.com>;芦志朋10108272;</div><div>日期：2017年07月21日 10:01</div><div>主题：Re: [libvirt] [PATCH v2] network: allow to specify buffer size fornetlink socket</div></div> <div>On 07/17/2017 08:00 AM, John Ferlan wrote: > > On 07/16/2017 05:04 PM, Laine Stump wrote: >> On 07/11/2017 09:01 PM, ZhiPeng Lu wrote: >>> This patchs allow to set the buffer size for netlink socket in >>> the libvirtd configuration file. The default buffer size remain >>> as before at 128k. >> See my more detailed response to your earlier patch here: >> >> >> https://www.redhat.com/archives/libvir-list/2017-July/msg00566.html >> >> There should be no need to configure the initial libnl buffer size, >> because we enable MSG_PEEK on the libnl sockets (and recent versions of >> libnl have it turned on by default anyway). If that's not permitting the >> buffer to auto-grow as necessary, then there is a different bug somewhere. >> > If an old version of libnl is the problem, then perhaps a patch which > just adds a comment in virNetlinkCreateSocket to "summarize" what gets > discovered w/r/t MSG_PEEK and the "correct" minimum version of libnl so > that the "next" person to come this way will have a chance at > understanding what needs to be done without going through the submit a > patch changing the size again! > > All that said, having it be configurable could be useful for someone who > has a system that doesn't have that version, while still working as > expected for the right version. I think it may need to be a fairly unusual combination of kernel, libvirt, and libnl versions, combined with pretty "big" hardware in order for that to happen. More information from ZhiPeng about the versions of those packages might allow us to make a better informed decision. Workarounds are okay when necessary. But adding a config parameter is something that would need to be left in forever, leaving more code to maintain, and all for a bug that shouldn't even be there today, much less 6 months or a year from now - turning on message peek was supposed to "eliminate this problem totally and permanently". If it didn't, I'd like to know why. ZhipPeng - can you tell us more about your setup? package versions, hardware, example XML, gdb backtrace at the instant the error message is logged? </div> </div></div></div></div> </div>