Thread (19 messages) 19 messages, 6 authors, 2016-10-04

Re: [PATCH v1] dpdk-devbind.py: Virtio interface issue.

flat view

From: Stephen Hemminger <stephen@networkplumber.org>
Date: 2016-08-26 15:54:48

On Wed, 24 Aug 2016 22:25:46 -0400
souvikdey33 [off-list ref] wrote:
quoted hunk ↗ jump to hunk
+    #The path for virtio devices are different. Get the correct path.
+	virtio = "/sys/bus/pci/devices/%s/" % dev_id
+    cmd = " ls %s | grep 'virt' " %virtio
+    virtio = commands.getoutput(cmd)
+    virtio_sys_path = "/sys/bus/pci/devices/%s/%s/net/" % (dev_id,virtio)
     if exists(sys_path):
         device["Interface"] = ",".join(os.listdir(sys_path))
There should be a way to do this in python without going out to shell.
This would be safer and more secure.

The code already uses os.listdir() (which is the python library version
of ls) in later section. Why not use that here to check for virtio bus.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help