[ovs-dev] [PATCH] netdev_dpdk: pci_dev pointer check.
Ben Pfaff
blp at ovn.org
Tue Dec 22 00:09:52 UTC 2015
It looks obviously correct to me but I guess that Daniele or Pravin
should apply it.
On Mon, Dec 21, 2015 at 10:04:55AM +0000, Weglicki, MichalX wrote:
> Hello,
>
> Is it possible to merge this patch? It is quite simple, and very important for integration with DPDK 2.2.
>
> Br,
> Michal.
>
> -----Original Message-----
> From: Weglicki, MichalX
> Sent: Thursday, December 3, 2015 7:30 AM
> To: dev at openvswitch.org
> Cc: Weglicki, MichalX <michalx.weglicki at intel.com>
> Subject: [PATCH] netdev_dpdk: pci_dev pointer check.
>
> This change prevents netdev_dpdk from accessing pointer
> which is not valid.
>
> Signed-off-by: Michal Weglicki <michalx.weglicki at intel.com>
> ---
> lib/netdev-dpdk.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
> index 4658416..2831a1b 100644
> --- a/lib/netdev-dpdk.c
> +++ b/lib/netdev-dpdk.c
> @@ -1690,8 +1690,12 @@ netdev_dpdk_get_status(const struct netdev *netdev_, struct smap *args)
> smap_add_format(args, "max_vfs", "%u", dev_info.max_vfs);
> smap_add_format(args, "max_vmdq_pools", "%u", dev_info.max_vmdq_pools);
>
> - smap_add_format(args, "pci-vendor_id", "0x%u", dev_info.pci_dev->id.vendor_id);
> - smap_add_format(args, "pci-device_id", "0x%x", dev_info.pci_dev->id.device_id);
> + if (dev_info.pci_dev) {
> + smap_add_format(args, "pci-vendor_id", "0x%u",
> + dev_info.pci_dev->id.vendor_id);
> + smap_add_format(args, "pci-device_id", "0x%x",
> + dev_info.pci_dev->id.device_id);
> + }
>
> return 0;
> }
> --
> 1.9.3
>
> --------------------------------------------------------------
> Intel Research and Development Ireland Limited
> Registered in Ireland
> Registered Office: Collinstown Industrial Park, Leixlip, County Kildare
> Registered Number: 308263
>
>
> This e-mail and any attachments may contain confidential material for the sole
> use of the intended recipient(s). Any review or distribution by others is
> strictly prohibited. If you are not the intended recipient, please contact the
> sender and delete all copies.
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
More information about the dev
mailing list