Sorry, not excepted, those are expected!

Patch name:             librte_vhost: Fix compilation issue
Brief description:      to fix compilation error on ubuntu14.04 for vhost lib 
and example
Test Flag:                      Tested-by
Tester name:            jingguox.fu at intel.com
Commit ID:                      1ab07743b21b785a71fa334641ab58e779532600
Result summary:         total 3 cases, 3 passed, 0 failed

Test Case 1:            
Name:                           compile vhost on fc20 with gcc 4.8.3 x86_64     
Environment:            OS: Fedora20 3.11.10-301.fc20.x86_64
                                GCC: gcc 4.8.3
                                CPU: Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz
                                NIC: Intel Corporation 82599ES 10-Gigabit 
SFI/SFP+ Network Connection [8086:10fb]
expected:                       compilation success
Test result:            PASSED

Test Case 2:            
Name:                           compile vhost on fc20 with icc 14.0.2 x86_64    
Environment:            OS: Fedora20 3.11.10-301.fc20.x86_64
                                ICC: icc 14.0.2
                                CPU: Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz
                                NIC: Intel Corporation 82599ES 10-Gigabit 
SFI/SFP+ Network Connection [8086:10fb]
expected:                       compilation success
Test result:            PASSED

Test Case 3:            
Name:                           compile vhost on UB14.04 with gcc 4.8.2 x86_64  
Environment:            OS: Ubuntu14.04 3.13.0-24-generic x86_64
                                GCC: gcc 4.8.2
                                CPU: Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz
                                NIC: Intel Corporation 82599ES 10-Gigabit 
SFI/SFP+ Network Connection [8086:10fb]
expected:                       compilation success
Test result:            PASSED

-----Original Message-----
From: Fu, JingguoX 
Sent: Wednesday, October 29, 2014 16:17
To: Ouyang, Changchun; dev at dpdk.org
Cc: Cao, Waterman
Subject: RE: [dpdk-dev] [PATCH] librte_vhost: Fix compilation issue

Patch name:             librte_vhost: Fix compilation issue
Brief description:      to fix compilation error on ubuntu14.04 for vhost lib 
and example
Test Flag:                      Tested-by
Tester name:            jingguox.fu at intel.com
Commit ID:                      1ab07743b21b785a71fa334641ab58e779532600
Result summary:         total 3 cases, 3 passed, 0 failed

Test Case 1:            
Name:                           compile vhost on fc20 with gcc 4.8.3 x86_64     
Environment:            OS: Fedora20 3.11.10-301.fc20.x86_64
                                GCC: gcc 4.8.3
                                CPU: Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz
                                NIC: Intel Corporation 82599ES 10-Gigabit 
SFI/SFP+ Network Connection [8086:10fb]
Excepted:                       compilation success
Test result:            PASSED

Test Case 2:            
Name:                           compile vhost on fc20 with icc 14.0.2 x86_64    
Environment:            OS: Fedora20 3.11.10-301.fc20.x86_64
                                ICC: icc 14.0.2
                                CPU: Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz
                                NIC: Intel Corporation 82599ES 10-Gigabit 
SFI/SFP+ Network Connection [8086:10fb]
Excepted:                       compilation success
Test result:            PASSED

Test Case 3:            
Name:                           compile vhost on UB14.04 with gcc 4.8.2 x86_64  
Environment:            OS: Ubuntu14.04 3.13.0-24-generic x86_64
                                GCC: gcc 4.8.2
                                CPU: Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz
                                NIC: Intel Corporation 82599ES 10-Gigabit 
SFI/SFP+ Network Connection [8086:10fb]
Excepted:                       compilation success
Test result:            PASSED




-----Original Message-----
From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Ouyang Changchun
Sent: Wednesday, October 29, 2014 14:40
To: dev at dpdk.org
Subject: [dpdk-dev] [PATCH] librte_vhost: Fix compilation issue

It fixes this compilation complain: "error: ignoring return value of 
'realpath', 
declared with attribute warn_unused_result [-Werror=unused-result]"

Signed-off-by: Changchun Ouyang <changchun.ouyang at intel.com>
---
 lib/librte_vhost/virtio-net.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/lib/librte_vhost/virtio-net.c b/lib/librte_vhost/virtio-net.c
index 27ba175..8015dd8 100644
--- a/lib/librte_vhost/virtio-net.c
+++ b/lib/librte_vhost/virtio-net.c
@@ -133,6 +133,7 @@ host_memory_map(struct virtio_net *dev, struct 
virtio_memory *mem,
        char mapfile[PATH_MAX];
        char procdir[PATH_MAX];
        char resolved_path[PATH_MAX];
+       char *path = NULL;
        FILE            *fmap;
        void            *map;
        uint8_t         found = 0;
@@ -235,9 +236,11 @@ host_memory_map(struct virtio_net *dev, struct 
virtio_memory *mem,
        while (NULL != (dptr = readdir(dp))) {
                snprintf(memfile, PATH_MAX, "/proc/%u/fd/%s",
                                pid, dptr->d_name);
-           realpath(memfile, resolved_path);
-               if (resolved_path == NULL) {
-                       RTE_LOG(ERR, VHOST_CONFIG, "(%"PRIu64") Failed to 
resolve fd directory\n", dev->device_fh);
+               path = realpath(memfile, resolved_path);
+               if (path == NULL) {
+                       RTE_LOG(ERR, VHOST_CONFIG,
+                               "(%"PRIu64") Failed to resolve fd directory\n",
+                               dev->device_fh);
                        closedir(dp);
                        return -1;
                }
-- 
1.8.4.2

Reply via email to