From: Bruce Richardson <bruce.richardson@intel.com>
To: dev@dpdk.org
Cc: patrick.fu@intel.com, Kevin Laatz <kevin.laatz@intel.com>,
stable@dpdk.org, Sunil Pai G <sunil.pai.g@intel.com>
Subject: Re: [dpdk-dev] [PATCH v3 02/25] raw/ioat: fix missing close function
Date: Fri, 25 Sep 2020 12:12:33 +0100 [thread overview]
Message-ID: <20200925111233.GA943@bricha3-MOBL.ger.corp.intel.com> (raw)
In-Reply-To: <20200925110910.284098-3-bruce.richardson@intel.com>
On Fri, Sep 25, 2020 at 12:08:47PM +0100, Bruce Richardson wrote:
> From: Kevin Laatz <kevin.laatz@intel.com>
>
> When rte_rawdev_pmd_release() is called, rte_rawdev_close() looks for a
> dev_close function for the device causing a segmentation fault when no
> close() function is implemented for a driver.
>
> This patch resolves the issue by adding a stub function ioat_dev_close().
>
> Fixes: f687e842e328 ("raw/ioat: introduce IOAT driver")
> Cc: stable@dpdk.org
>
> Reported-by: Sunil Pai G <sunil.pai.g@intel.com>
> Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
> ---
Forgot to add my own reviewed-by to this:
Reviewed-by: Bruce Richardson <bruce.richardson@intel.com>
next prev parent reply other threads:[~2020-09-25 11:14 UTC|newest]
Thread overview: 157+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-21 9:51 [dpdk-dev] [PATCH 20.11 00/20] raw/ioat: enhancements and new hardware support Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 01/20] raw/ioat: add a flag to control copying handle parameters Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 02/20] raw/ioat: support multiple devices being tested Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 03/20] app/test: change rawdev autotest to run selftest on all devs Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 04/20] app/test: remove ioat-specific autotest Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 05/20] raw/ioat: split header for readability Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 06/20] raw/ioat: make the HW register spec private Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 07/20] usertools/dpdk-devbind.py: add support for DSA HW Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 08/20] raw/ioat: add skeleton for vfio/uio based DSA device Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 09/20] raw/ioat: add vdev probe for DSA/idxd devices Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 10/20] raw/ioat: create rawdev instances on idxd PCI probe Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 11/20] raw/ioat: create rawdev instances for idxd vdevs Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 12/20] raw/ioat: add datapath data structures for idxd devices Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 13/20] raw/ioat: add configure function " Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 14/20] raw/ioat: add start and stop functions " Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 15/20] raw/ioat: add data path support " Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 16/20] raw/ioat: add info function " Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 17/20] raw/ioat: create separate statistics structure Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 18/20] raw/ioat: move xstats functions to common file Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 19/20] raw/ioat: add xstats tracking for idxd devices Bruce Richardson
2020-07-21 9:51 ` [dpdk-dev] [PATCH 20.11 20/20] raw/ioat: clean up use of common test function Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 00/18] raw/ioat: enhancements and new hardware support Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 01/18] raw/ioat: add a flag to control copying handle parameters Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 02/18] raw/ioat: split header for readability Bruce Richardson
2020-08-25 15:27 ` Laatz, Kevin
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 03/18] raw/ioat: make the HW register spec private Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 04/18] usertools/dpdk-devbind.py: add support for DSA HW Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 05/18] raw/ioat: add skeleton for VFIO/UIO based DSA device Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 06/18] raw/ioat: add vdev probe for DSA/idxd devices Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 07/18] raw/ioat: include example configuration script Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 08/18] raw/ioat: create rawdev instances on idxd PCI probe Bruce Richardson
2020-08-25 15:27 ` Laatz, Kevin
2020-08-26 15:45 ` Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 09/18] raw/ioat: create rawdev instances for idxd vdevs Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 10/18] raw/ioat: add datapath data structures for idxd devices Bruce Richardson
2020-08-25 15:27 ` Laatz, Kevin
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 11/18] raw/ioat: add configure function " Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 12/18] raw/ioat: add start and stop functions " Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 13/18] raw/ioat: add data path " Bruce Richardson
2020-08-25 15:27 ` Laatz, Kevin
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 14/18] raw/ioat: add info function " Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 15/18] raw/ioat: create separate statistics structure Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 16/18] raw/ioat: move xstats functions to common file Bruce Richardson
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 17/18] raw/ioat: add xstats tracking for idxd devices Bruce Richardson
2020-08-24 9:56 ` Laatz, Kevin
2020-08-21 16:29 ` [dpdk-dev] [PATCH v2 18/18] raw/ioat: clean up use of common test function Bruce Richardson
2020-08-21 16:39 ` [dpdk-dev] [PATCH v2 00/18] raw/ioat: enhancements and new hardware support Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 00/25] " Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 01/25] doc/api: add ioat driver to index Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 02/25] raw/ioat: fix missing close function Bruce Richardson
2020-09-25 11:12 ` Bruce Richardson [this message]
2020-09-25 11:12 ` Pai G, Sunil
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 03/25] raw/ioat: enable use from C++ code Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 04/25] raw/ioat: include extra info in error messages Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 05/25] raw/ioat: add a flag to control copying handle parameters Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 06/25] raw/ioat: split header for readability Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 07/25] raw/ioat: rename functions to be operation-agnostic Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 08/25] raw/ioat: add separate API for fence call Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 09/25] raw/ioat: make the HW register spec private Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 10/25] usertools/dpdk-devbind.py: add support for DSA HW Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 11/25] raw/ioat: add skeleton for VFIO/UIO based DSA device Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 12/25] raw/ioat: add vdev probe for DSA/idxd devices Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 13/25] raw/ioat: include example configuration script Bruce Richardson
2020-09-25 11:08 ` [dpdk-dev] [PATCH v3 14/25] raw/ioat: create rawdev instances on idxd PCI probe Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 15/25] raw/ioat: create rawdev instances for idxd vdevs Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 16/25] raw/ioat: add datapath data structures for idxd devices Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 17/25] raw/ioat: add configure function " Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 18/25] raw/ioat: add start and stop functions " Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 19/25] raw/ioat: add data path " Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 20/25] raw/ioat: add info function " Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 21/25] raw/ioat: create separate statistics structure Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 22/25] raw/ioat: move xstats functions to common file Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 23/25] raw/ioat: add xstats tracking for idxd devices Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 24/25] raw/ioat: clean up use of common test function Bruce Richardson
2020-09-25 11:09 ` [dpdk-dev] [PATCH v3 25/25] raw/ioat: add fill operation Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 00/25] raw/ioat: enhancements and new hardware support Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 01/25] doc/api: add ioat driver to index Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 02/25] raw/ioat: fix missing close function Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 03/25] raw/ioat: enable use from C++ code Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 04/25] raw/ioat: include extra info in error messages Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 05/25] raw/ioat: add a flag to control copying handle parameters Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 06/25] raw/ioat: split header for readability Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 07/25] raw/ioat: rename functions to be operation-agnostic Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 08/25] raw/ioat: add separate API for fence call Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 09/25] raw/ioat: make the HW register spec private Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 10/25] usertools/dpdk-devbind.py: add support for DSA HW Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 11/25] raw/ioat: add skeleton for VFIO/UIO based DSA device Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 12/25] raw/ioat: add vdev probe for DSA/idxd devices Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 13/25] raw/ioat: include example configuration script Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 14/25] raw/ioat: create rawdev instances on idxd PCI probe Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 15/25] raw/ioat: create rawdev instances for idxd vdevs Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 16/25] raw/ioat: add datapath data structures for idxd devices Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 17/25] raw/ioat: add configure function " Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 18/25] raw/ioat: add start and stop functions " Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 19/25] raw/ioat: add data path " Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 20/25] raw/ioat: add info function " Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 21/25] raw/ioat: create separate statistics structure Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 22/25] raw/ioat: move xstats functions to common file Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 23/25] raw/ioat: add xstats tracking for idxd devices Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 24/25] raw/ioat: clean up use of common test function Bruce Richardson
2020-09-28 16:42 ` [dpdk-dev] [PATCH v4 25/25] raw/ioat: add fill operation Bruce Richardson
2020-10-02 14:07 ` [dpdk-dev] [PATCH v4 00/25] raw/ioat: enhancements and new hardware support Nicolau, Radu
2020-10-06 21:10 ` Thomas Monjalon
2020-10-07 9:46 ` Bruce Richardson
2020-10-07 16:29 ` [dpdk-dev] [PATCH v5 " Bruce Richardson
2020-10-07 16:29 ` [dpdk-dev] [PATCH v5 01/25] doc/api: add ioat driver to index Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 02/25] raw/ioat: fix missing close function Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 03/25] raw/ioat: enable use from C++ code Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 04/25] raw/ioat: include extra info in error messages Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 05/25] raw/ioat: add a flag to control copying handle parameters Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 06/25] raw/ioat: split header for readability Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 07/25] raw/ioat: rename functions to be operation-agnostic Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 08/25] raw/ioat: add separate API for fence call Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 09/25] raw/ioat: make the HW register spec private Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 10/25] usertools/dpdk-devbind.py: add support for DSA HW Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 11/25] raw/ioat: add skeleton for VFIO/UIO based DSA device Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 12/25] raw/ioat: add vdev probe for DSA/idxd devices Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 13/25] raw/ioat: include example configuration script Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 14/25] raw/ioat: create rawdev instances on idxd PCI probe Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 15/25] raw/ioat: create rawdev instances for idxd vdevs Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 16/25] raw/ioat: add datapath data structures for idxd devices Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 17/25] raw/ioat: add configure function " Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 18/25] raw/ioat: add start and stop functions " Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 19/25] raw/ioat: add data path " Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 20/25] raw/ioat: add info function " Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 21/25] raw/ioat: create separate statistics structure Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 22/25] raw/ioat: move xstats functions to common file Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 23/25] raw/ioat: add xstats tracking for idxd devices Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 24/25] raw/ioat: clean up use of common test function Bruce Richardson
2020-10-07 16:30 ` [dpdk-dev] [PATCH v5 25/25] raw/ioat: add fill operation Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 00/25] raw/ioat: enhancements and new hardware support Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 01/25] doc/api: add ioat driver to index Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 02/25] raw/ioat: fix missing close function Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 03/25] raw/ioat: enable use from C++ code Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 04/25] raw/ioat: include extra info in error messages Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 05/25] raw/ioat: add a flag to control copying handle parameters Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 06/25] raw/ioat: split header for readability Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 07/25] raw/ioat: rename functions to be operation-agnostic Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 08/25] raw/ioat: add separate API for fence call Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 09/25] raw/ioat: make the HW register spec private Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 10/25] usertools/dpdk-devbind.py: add support for DSA HW Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 11/25] raw/ioat: add skeleton for VFIO/UIO based DSA device Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 12/25] raw/ioat: add vdev probe for DSA/idxd devices Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 13/25] raw/ioat: include example configuration script Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 14/25] raw/ioat: create rawdev instances on idxd PCI probe Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 15/25] raw/ioat: create rawdev instances for idxd vdevs Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 16/25] raw/ioat: add datapath data structures for idxd devices Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 17/25] raw/ioat: add configure function " Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 18/25] raw/ioat: add start and stop functions " Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 19/25] raw/ioat: add data path " Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 20/25] raw/ioat: add info function " Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 21/25] raw/ioat: create separate statistics structure Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 22/25] raw/ioat: move xstats functions to common file Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 23/25] raw/ioat: add xstats tracking for idxd devices Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 24/25] raw/ioat: clean up use of common test function Bruce Richardson
2020-10-08 9:51 ` [dpdk-dev] [PATCH v6 25/25] raw/ioat: add fill operation Bruce Richardson
2020-10-08 12:32 ` [dpdk-dev] [PATCH v6 00/25] raw/ioat: enhancements and new hardware support Thomas Monjalon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200925111233.GA943@bricha3-MOBL.ger.corp.intel.com \
--to=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
--cc=kevin.laatz@intel.com \
--cc=patrick.fu@intel.com \
--cc=stable@dpdk.org \
--cc=sunil.pai.g@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).