[Qemu-devel] 答复: Re: [PATCH v2] qga: replace GetIfEntry

[lu.zhipeng]

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: address@hidden 
www.zte.com.cn










原始邮件




发件人：芦志朋10108272
收件人： <address@hidden>;
抄送人： <address@hidden>;
日 期 ：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: address@hidden 
www.zte.com.cn

















发件人： <address@hidden>;
收件人：芦志朋10108272;
抄送人： <address@hidden>;芦志朋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 <address@hidden>
> ---
>  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
>