i think the code is better if (OSver.dwMajorVersion >= 6) { MIB_IF_ROW2 aMib_ifrow; typedef NETIOAPI_API (WINAPI *getifentry2_t)(PMIB_IF_ROW2 Row); memset(&aMib_ifrow, 0, sizeof(aMib_ifrow)); aMib_ifrow.InterfaceIndex = nicId; HMODULE module = GetModuleHandle("iphlpapi"); PVOID fun = GetProcAddress(module, "GetIfEntry2"); if (fun == NULL) { error_setg(errp, QERR_QGA_COMMAND_FAILED, "Failed to get address of GetIfEntry2"); return NULL; } getifentry2_t getifentry2_ex = (getifentry2_t)fun; if (NO_ERROR == getifentry2_ex(&aMib_ifrow)){ }
为了让您的VPlat虚拟机故障和docker故障得到高效的处理,请上报故障到: $VPlat技术支持。 芦志朋 luzhipeng IT开发工程师 IT Development Engineer 操作系统产品部/中心研究院/系统产品 OS Product Dept./Central R&D Institute/System Product 四川省成都市天府大道中段800号 E: lu.zhip...@zte.com.cn www.zte.com.cn 原始邮件 发件人:芦志朋10108272 收件人: <mdr...@linux.vnet.ibm.com>; 抄送人: <qemu-devel@nongnu.org>; 日 期 :2017年11月08日 10:54 主 题 :答复: Re: [PATCH v2] qga: replace GetIfEntry >I have the modified patch staged here: > > https://github.com/mdroth/qemu/commit/de597a8b27722ce4f9cc660f930f7dccc712712d>Let > me know if it looks okay and I'll send a pull request for rc1 the code right, but running QGA on the Windows XP, error: can not find the link to GetIfEntry2 in iphlpapi.DLL. 为了让您的VPlat虚拟机故障和docker故障得到高效的处理,请上报故障到: $VPlat技术支持。 芦志朋 luzhipeng IT开发工程师 IT Development Engineer 操作系统产品部/中心研究院/系统产品 OS Product Dept./Central R&D Institute/System Product 四川省成都市天府大道中段800号 E: lu.zhip...@zte.com.cn www.zte.com.cn 发件人: <mdr...@linux.vnet.ibm.com>; 收件人:芦志朋10108272; 抄送人: <qemu-devel@nongnu.org>;芦志朋10108272; 日 期 :2017年11月08日 01:07 主 题 :Re: [PATCH v2] qga: replace GetIfEntry Quoting ZhiPeng Lu (2017-11-03 09:54:20) > The data obtained by GetIfEntry is 32 bits, and it may overflow. Thus using > GetIfEntry2 instead of GetIfEntry. > > Signed-off-by: ZhiPeng Lu <lu.zhip...@zte.com.cn> > --- > qga/commands-win32.c | 31 ++++++++++++++++++------------- > 1 file changed, 18 insertions(+), 13 deletions(-) > > diff --git a/qga/commands-win32.c b/qga/commands-win32.c > index 0322188..d096dc2 100644 > --- a/qga/commands-win32.c > +++ b/qga/commands-win32.c > @@ -1173,20 +1173,25 @@ static int guest_get_network_stats(const char *name, > GuestNetworkInterfaceStat *stats) > { > DWORD if_index = 0; > - MIB_IFROW a_mid_ifrow; > - memset(&a_mid_ifrow, 0, sizeof(a_mid_ifrow)); > + OSVERSIONINFO OSver; > if_index = get_interface_index(name); > - a_mid_ifrow.dwIndex = if_index; > - if (NO_ERROR == GetIfEntry(&a_mid_ifrow)) { > - stats->rx_bytes = a_mid_ifrow.dwInOctets; > - stats->rx_packets = a_mid_ifrow.dwInUcastPkts; > - stats->rx_errs = a_mid_ifrow.dwInErrors; > - stats->rx_dropped = a_mid_ifrow.dwInDiscards; > - stats->tx_bytes = a_mid_ifrow.dwOutOctets; > - stats->tx_packets = a_mid_ifrow.dwOutUcastPkts; > - stats->tx_errs = a_mid_ifrow.dwOutErrors; > - stats->tx_dropped = a_mid_ifrow.dwOutDiscards; > - return 0; > + OSver.dwOSVersionInfoSize = sizeof(OSVERSIONINFO); > + GetVersionEx(&OSver); > + if (OSver.dwMajorVersion >= 6) { > + MIB_IF_ROW2 a_mid_ifrow; > + memset(&a_mid_ifrow, 0, sizeof(a_mid_ifrow)); > + a_mid_ifrow.dwIndex = if_index; > + if (NO_ERROR == GetIfEntry2(&a_mid_ifrow)) { > + stats->rx_bytes = a_mid_ifrow.dwInOctets; > + stats->rx_packets = a_mid_ifrow.dwInUcastPkts; > + stats->rx_errs = a_mid_ifrow.dwInErrors; > + stats->rx_dropped = a_mid_ifrow.dwInDiscards; > + stats->tx_bytes = a_mid_ifrow.dwOutOctets; > + stats->tx_packets = a_mid_ifrow.dwOutUcastPkts; > + stats->tx_errs = a_mid_ifrow.dwOutErrors; > + stats->tx_dropped = a_mid_ifrow.dwOutDiscards; > + return 0; This fails to build in my fc20 mingw64 environment: /home/mdroth/w/qemu4.git/qga/commands-win32.c:1184:20: error: 'MIB_IF_ROW2' has no member named 'dwIndex' a_mid_ifrow.dwIndex = if_index; ^ /home/mdroth/w/qemu4.git/qga/commands-win32.c:1186:42: error: 'MIB_IF_ROW2' has no member named 'dwInOctets' stats->rx_bytes = a_mid_ifrow.dwInOctets; ^ /home/mdroth/w/qemu4.git/qga/commands-win32.c:1187:44: error: 'MIB_IF_ROW2' has no member named 'dwInUcastPkts' stats->rx_packets = a_mid_ifrow.dwInUcastPkts; etc ... The patch seems to assume MIB_IFROW and MIB_IF_ROW2 have the same field names for the stats, but according to this they're slightly different: https://msdn.microsoft.com/en-us/library/windows/desktop/aa814491(v=vs.85).aspx This change on top of your patch seems to work: diff --git a/qga/commands-win32.c b/qga/commands-win32.c index e17fc3c284..58e470877f 100644 --- a/qga/commands-win32.c +++ b/qga/commands-win32.c @@ -1181,16 +1181,16 @@ static int guest_get_network_stats(const char *name, if (os_ver.dwMajorVersion >= 6) { MIB_IF_ROW2 a_mid_ifrow; memset(&a_mid_ifrow, 0, sizeof(a_mid_ifrow)); - a_mid_ifrow.dwIndex = if_index; + a_mid_ifrow.InterfaceIndex = if_index; if (NO_ERROR == GetIfEntry2(&a_mid_ifrow)) { - stats->rx_bytes = a_mid_ifrow.dwInOctets; - stats->rx_packets = a_mid_ifrow.dwInUcastPkts; - stats->rx_errs = a_mid_ifrow.dwInErrors; - stats->rx_dropped = a_mid_ifrow.dwInDiscards; - stats->tx_bytes = a_mid_ifrow.dwOutOctets; - stats->tx_packets = a_mid_ifrow.dwOutUcastPkts; - stats->tx_errs = a_mid_ifrow.dwOutErrors; - stats->tx_dropped = a_mid_ifrow.dwOutDiscards; + stats->rx_bytes = a_mid_ifrow.InOctets; + stats->rx_packets = a_mid_ifrow.InUcastPkts; + stats->rx_errs = a_mid_ifrow.InErrors; + stats->rx_dropped = a_mid_ifrow.InDiscards; + stats->tx_bytes = a_mid_ifrow.OutOctets; + stats->tx_packets = a_mid_ifrow.OutUcastPkts; + stats->tx_errs = a_mid_ifrow.OutErrors; + stats->tx_dropped = a_mid_ifrow.OutDiscards; return 0; } } I have the modified patch staged here: https://github.com/mdroth/qemu/commit/de597a8b27722ce4f9cc660f930f7dccc712712d Let me know if it looks okay and I'll send a pull request for rc1 > + } > } > return -1; > } > -- > 1.8.3.1 >