raw/ioat: fix missing device name in idxd bus scan
Checks
Commit Message
The device name is not being initialized during the idxd bus scan which
will cause segmentation faults when an appliation tries to access this
information.
This patch adds the required initialization of the device name so that it
can be read without issues.
Fixes: b7aaf417f936 ("raw/ioat: add bus driver for device scanning automatically")
Reported-by: Sunil Pai G <sunil.pai.g@intel.com>
Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
---
drivers/raw/ioat/idxd_bus.c | 1 +
1 file changed, 1 insertion(+)
Comments
On Thu, May 27, 2021 at 02:36:09PM +0100, Kevin Laatz wrote:
> The device name is not being initialized during the idxd bus scan which
> will cause segmentation faults when an appliation tries to access this
> information.
>
> This patch adds the required initialization of the device name so that it
> can be read without issues.
>
> Fixes: b7aaf417f936 ("raw/ioat: add bus driver for device scanning automatically")
>
> Reported-by: Sunil Pai G <sunil.pai.g@intel.com>
> Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
> ---
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> -----Original Message-----
> From: Richardson, Bruce <bruce.richardson@intel.com>
> Sent: Thursday, May 27, 2021 7:58 PM
> To: Laatz, Kevin <kevin.laatz@intel.com>
> Cc: dev@dpdk.org; stable@dpdk.org; Pai G, Sunil <sunil.pai.g@intel.com>
> Subject: Re: [PATCH] raw/ioat: fix missing device name in idxd bus scan
>
> On Thu, May 27, 2021 at 02:36:09PM +0100, Kevin Laatz wrote:
> > The device name is not being initialized during the idxd bus scan
> > which will cause segmentation faults when an appliation tries to
> > access this information.
> >
> > This patch adds the required initialization of the device name so that
> > it can be read without issues.
> >
> > Fixes: b7aaf417f936 ("raw/ioat: add bus driver for device scanning
> > automatically")
> >
> > Reported-by: Sunil Pai G <sunil.pai.g@intel.com>
> > Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
> > ---
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>
Tested-by: Sunil Pai G <sunil.pai.g@intel.com>
> > > The device name is not being initialized during the idxd bus scan
> > > which will cause segmentation faults when an appliation tries to
> > > access this information.
> > >
> > > This patch adds the required initialization of the device name so that
> > > it can be read without issues.
> > >
> > > Fixes: b7aaf417f936 ("raw/ioat: add bus driver for device scanning
> > > automatically")
> > >
> > > Reported-by: Sunil Pai G <sunil.pai.g@intel.com>
> > > Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
> > > ---
> > Acked-by: Bruce Richardson <bruce.richardson@intel.com>
>
> Tested-by: Sunil Pai G <sunil.pai.g@intel.com>
Applied, thanks
@@ -313,6 +313,7 @@ dsa_scan(void)
read_device_int(dev, "numa_node", &numa_node);
dev->device.numa_node = numa_node;
+ dev->device.name = dev->wq_name;
}
closedir(dev_dir);