diff mbox series

net/ring: support secondary process

Message ID 20200930110241.1882308-1-ferruh.yigit@intel.com (mailing list archive)
State Changes Requested
Delegated to: Ferruh Yigit
Headers show
Series net/ring: support secondary process | expand

Checks

Context Check Description
ci/travis-robot success Travis build: passed
ci/iol-testing success Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Ferruh Yigit Sept. 30, 2020, 11:02 a.m. UTC
Also add missing .ini feature list documentation and document some
existing features including new "Multiprocess aware" feature.

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
Cc: Kevin Laatz <kevin.laatz@intel.com>
---
 doc/guides/nics/features/ring.ini | 10 ++++++++++
 drivers/net/ring/rte_eth_ring.c   | 17 +++++++++++++++++
 2 files changed, 27 insertions(+)
 create mode 100644 doc/guides/nics/features/ring.ini

Comments

Ferruh Yigit Feb. 17, 2021, 5:49 p.m. UTC | #1
On 9/30/2020 12:02 PM, Ferruh Yigit wrote:
> Also add missing .ini feature list documentation and document some
> existing features including new "Multiprocess aware" feature.
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
> Cc: Kevin Laatz <kevin.laatz@intel.com>

Hi Bruce,

Reminder of this patch waiting for review.

Thanks,
ferruh
Bruce Richardson March 1, 2021, 1:38 p.m. UTC | #2
On Wed, Sep 30, 2020 at 12:02:40PM +0100, Ferruh Yigit wrote:
> Also add missing .ini feature list documentation and document some
> existing features including new "Multiprocess aware" feature.
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
> Cc: Kevin Laatz <kevin.laatz@intel.com>
> ---
>  doc/guides/nics/features/ring.ini | 10 ++++++++++
>  drivers/net/ring/rte_eth_ring.c   | 17 +++++++++++++++++
>  2 files changed, 27 insertions(+)
>  create mode 100644 doc/guides/nics/features/ring.ini
> 
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
Ferruh Yigit March 4, 2021, 5:14 p.m. UTC | #3
On 3/1/2021 1:38 PM, Bruce Richardson wrote:
> On Wed, Sep 30, 2020 at 12:02:40PM +0100, Ferruh Yigit wrote:
>> Also add missing .ini feature list documentation and document some
>> existing features including new "Multiprocess aware" feature.
>>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>> ---
>> Cc: Kevin Laatz <kevin.laatz@intel.com>
>> ---
>>   doc/guides/nics/features/ring.ini | 10 ++++++++++
>>   drivers/net/ring/rte_eth_ring.c   | 17 +++++++++++++++++
>>   2 files changed, 27 insertions(+)
>>   create mode 100644 doc/guides/nics/features/ring.ini
>>
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> 

Applied to dpdk-next-net/main, thanks.
Thomas Monjalon March 7, 2021, 9:02 p.m. UTC | #4
04/03/2021 18:14, Ferruh Yigit:
> On 3/1/2021 1:38 PM, Bruce Richardson wrote:
> > On Wed, Sep 30, 2020 at 12:02:40PM +0100, Ferruh Yigit wrote:
> >> Also add missing .ini feature list documentation and document some
> >> existing features including new "Multiprocess aware" feature.
> >>
> >> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> >> ---
> >> Cc: Kevin Laatz <kevin.laatz@intel.com>
> >> ---
> >>   doc/guides/nics/features/ring.ini | 10 ++++++++++
> >>   drivers/net/ring/rte_eth_ring.c   | 17 +++++++++++++++++
> >>   2 files changed, 27 insertions(+)
> >>   create mode 100644 doc/guides/nics/features/ring.ini
> >>
> > Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> > 
> 
> Applied to dpdk-next-net/main, thanks.

Sorry, just seeing now.
I think we should not re-introduce the ring PMD in the features table.
It is not significant enough to occupy a column in this table I think.

This is the commit where I removed it from the table:
https://git.dpdk.org/dpdk/commit/?id=7b103c4ccd
Ferruh Yigit March 8, 2021, 11:30 a.m. UTC | #5
On 3/7/2021 9:02 PM, Thomas Monjalon wrote:
> 04/03/2021 18:14, Ferruh Yigit:
>> On 3/1/2021 1:38 PM, Bruce Richardson wrote:
>>> On Wed, Sep 30, 2020 at 12:02:40PM +0100, Ferruh Yigit wrote:
>>>> Also add missing .ini feature list documentation and document some
>>>> existing features including new "Multiprocess aware" feature.
>>>>
>>>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>>>> ---
>>>> Cc: Kevin Laatz <kevin.laatz@intel.com>
>>>> ---
>>>>    doc/guides/nics/features/ring.ini | 10 ++++++++++
>>>>    drivers/net/ring/rte_eth_ring.c   | 17 +++++++++++++++++
>>>>    2 files changed, 27 insertions(+)
>>>>    create mode 100644 doc/guides/nics/features/ring.ini
>>>>
>>> Acked-by: Bruce Richardson <bruce.richardson@intel.com>
>>>
>>
>> Applied to dpdk-next-net/main, thanks.
> 
> Sorry, just seeing now.
> I think we should not re-introduce the ring PMD in the features table.
> It is not significant enough to occupy a column in this table I think.
> 
> This is the commit where I removed it from the table:
> https://git.dpdk.org/dpdk/commit/?id=7b103c4ccd
> 
> 
That is OK, I can send a new version without the .ini file.
diff mbox series

Patch

diff --git a/doc/guides/nics/features/ring.ini b/doc/guides/nics/features/ring.ini
new file mode 100644
index 0000000000..fbfa6555d7
--- /dev/null
+++ b/doc/guides/nics/features/ring.ini
@@ -0,0 +1,10 @@ 
+;
+; Supported features of the 'ring' network poll mode driver.
+;
+; Refer to default.ini for the full list of available PMD features.
+;
+[Features]
+Basic stats          = Y
+Multiprocess aware   = Y
+ARMv8                = Y
+x86-64               = Y
diff --git a/drivers/net/ring/rte_eth_ring.c b/drivers/net/ring/rte_eth_ring.c
index 12046f5a00..f33951ee2e 100644
--- a/drivers/net/ring/rte_eth_ring.c
+++ b/drivers/net/ring/rte_eth_ring.c
@@ -595,6 +595,23 @@  rte_pmd_ring_probe(struct rte_vdev_device *dev)
 
 	PMD_LOG(INFO, "Initializing pmd_ring for %s", name);
 
+	if (rte_eal_process_type() == RTE_PROC_SECONDARY) {
+		eth_dev = rte_eth_dev_attach_secondary(name);
+		if (!eth_dev) {
+			PMD_LOG(ERR, "Failed to probe %s", name);
+			return -1;
+		}
+		eth_dev->dev_ops = &ops;
+		eth_dev->device = &dev->device;
+
+		eth_dev->rx_pkt_burst = eth_ring_rx;
+		eth_dev->tx_pkt_burst = eth_ring_tx;
+
+		rte_eth_dev_probing_finish(eth_dev);
+
+		return 0;
+	}
+
 	if (params == NULL || params[0] == '\0') {
 		ret = eth_dev_ring_create(name, dev, rte_socket_id(), DEV_CREATE,
 				&eth_dev);