DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] eal/windows: add interrupt functions stub
@ 2020-12-11  0:26 Pallavi Kadam
  2020-12-11 19:53 ` Narcisa Ana Maria Vasile
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Pallavi Kadam @ 2020-12-11  0:26 UTC (permalink / raw)
  To: dev, thomas
  Cc: ranjit.menon, dmitry.kozliuk, Narcisa.Vasile, talshn, pallavi.kadam

Add some missing interrupt implementations on Windows.
Also add respestive functions to export file.

Signed-off-by: Tal Shnaiderman <talshn@nvidia.com>
Signed-off-by: Pallavi Kadam <pallavi.kadam@intel.com>
Reviewed-by: Ranjit Menon <ranjit.menon@intel.com>
---
 lib/librte_eal/rte_eal_exports.def      | 11 ++++
 lib/librte_eal/windows/eal_interrupts.c | 76 +++++++++++++++++++++++++
 2 files changed, 87 insertions(+)

diff --git a/lib/librte_eal/rte_eal_exports.def b/lib/librte_eal/rte_eal_exports.def
index 6a6be1cfa..98a38ed32 100644
--- a/lib/librte_eal/rte_eal_exports.def
+++ b/lib/librte_eal/rte_eal_exports.def
@@ -63,6 +63,17 @@ EXPORTS
 	rte_get_tsc_hz
 	rte_hexdump
 	rte_hypervisor_get
+	rte_intr_ack
+	rte_intr_allow_others
+	rte_intr_callback_register
+	rte_intr_callback_unregister
+	rte_intr_cap_multiple
+	rte_intr_disable
+	rte_intr_dp_is_en
+	rte_intr_efd_disable
+	rte_intr_efd_enable
+	rte_intr_enable
+	rte_intr_free_epoll_fd
 	rte_intr_rx_ctl
 	rte_lcore_count
 	rte_lcore_has_role
diff --git a/lib/librte_eal/windows/eal_interrupts.c b/lib/librte_eal/windows/eal_interrupts.c
index 6c64a48f3..feaaf3afe 100644
--- a/lib/librte_eal/windows/eal_interrupts.c
+++ b/lib/librte_eal/windows/eal_interrupts.c
@@ -105,3 +105,79 @@ eal_intr_thread_schedule(void (*func)(void *arg), void *arg)
 
 	return 0;
 }
+
+int
+rte_intr_callback_register(
+	__rte_unused const struct rte_intr_handle *intr_handle,
+	__rte_unused rte_intr_callback_fn cb, __rte_unused void *cb_arg)
+{
+	return -ENOTSUP;
+}
+
+int
+rte_intr_callback_unregister(
+	__rte_unused const struct rte_intr_handle *intr_handle,
+	__rte_unused rte_intr_callback_fn cb_fn, __rte_unused void *cb_arg)
+{
+	return 0;
+}
+
+int
+rte_intr_enable(__rte_unused const struct rte_intr_handle *intr_handle)
+{
+	return -ENOTSUP;
+}
+
+int
+rte_intr_ack(__rte_unused const struct rte_intr_handle *intr_handle)
+{
+	return -ENOTSUP;
+}
+
+int
+rte_intr_disable(__rte_unused const struct rte_intr_handle *intr_handle)
+{
+	return -ENOTSUP;
+}
+
+int
+rte_intr_efd_enable(struct rte_intr_handle *intr_handle, uint32_t nb_efd)
+{
+	RTE_SET_USED(intr_handle);
+	RTE_SET_USED(nb_efd);
+
+	return 0;
+}
+
+void
+rte_intr_efd_disable(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+}
+
+int
+rte_intr_dp_is_en(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+	return 0;
+}
+
+int
+rte_intr_allow_others(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+	return 1;
+}
+
+int
+rte_intr_cap_multiple(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+	return 0;
+}
+
+void
+rte_intr_free_epoll_fd(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+}
-- 
2.18.0.windows.1


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH] eal/windows: add interrupt functions stub
  2020-12-11  0:26 [dpdk-dev] [PATCH] eal/windows: add interrupt functions stub Pallavi Kadam
@ 2020-12-11 19:53 ` Narcisa Ana Maria Vasile
  2020-12-11 21:07 ` Dmitry Kozlyuk
  2020-12-15  0:26 ` [dpdk-dev] [PATCH v2] " Pallavi Kadam
  2 siblings, 0 replies; 7+ messages in thread
From: Narcisa Ana Maria Vasile @ 2020-12-11 19:53 UTC (permalink / raw)
  To: Pallavi Kadam
  Cc: dev, thomas, ranjit.menon, dmitry.kozliuk, Narcisa.Vasile, talshn

On Thu, Dec 10, 2020 at 04:26:07PM -0800, Pallavi Kadam wrote:
> Add some missing interrupt implementations on Windows.
> Also add respestive functions to export file.
> 
> Signed-off-by: Tal Shnaiderman <talshn@nvidia.com>
> Signed-off-by: Pallavi Kadam <pallavi.kadam@intel.com>
> Reviewed-by: Ranjit Menon <ranjit.menon@intel.com>
> ---
>  lib/librte_eal/rte_eal_exports.def      | 11 ++++
>  lib/librte_eal/windows/eal_interrupts.c | 76 +++++++++++++++++++++++++
>  2 files changed, 87 insertions(+)
> 
Acked-by: Narcisa Vasile <navasile@linux.microsoft.com>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH] eal/windows: add interrupt functions stub
  2020-12-11  0:26 [dpdk-dev] [PATCH] eal/windows: add interrupt functions stub Pallavi Kadam
  2020-12-11 19:53 ` Narcisa Ana Maria Vasile
@ 2020-12-11 21:07 ` Dmitry Kozlyuk
  2020-12-15  0:38   ` Kadam, Pallavi
  2020-12-15  0:26 ` [dpdk-dev] [PATCH v2] " Pallavi Kadam
  2 siblings, 1 reply; 7+ messages in thread
From: Dmitry Kozlyuk @ 2020-12-11 21:07 UTC (permalink / raw)
  To: Pallavi Kadam; +Cc: dev, thomas, ranjit.menon, Narcisa.Vasile, talshn

On Thu, 10 Dec 2020 16:26:07 -0800, Pallavi Kadam wrote:
> Add some missing interrupt implementations on Windows.
> Also add respestive functions to export file.
> 
> Signed-off-by: Tal Shnaiderman <talshn@nvidia.com>
> Signed-off-by: Pallavi Kadam <pallavi.kadam@intel.com>
> Reviewed-by: Ranjit Menon <ranjit.menon@intel.com>
> ---
>  lib/librte_eal/rte_eal_exports.def      | 11 ++++
>  lib/librte_eal/windows/eal_interrupts.c | 76 +++++++++++++++++++++++++
>  2 files changed, 87 insertions(+)
> 
> diff --git a/lib/librte_eal/rte_eal_exports.def b/lib/librte_eal/rte_eal_exports.def
> index 6a6be1cfa..98a38ed32 100644
> --- a/lib/librte_eal/rte_eal_exports.def
> +++ b/lib/librte_eal/rte_eal_exports.def
> @@ -63,6 +63,17 @@ EXPORTS
>  	rte_get_tsc_hz
>  	rte_hexdump
>  	rte_hypervisor_get
> +	rte_intr_ack
> +	rte_intr_allow_others
> +	rte_intr_callback_register
> +	rte_intr_callback_unregister
> +	rte_intr_cap_multiple
> +	rte_intr_disable
> +	rte_intr_dp_is_en
> +	rte_intr_efd_disable
> +	rte_intr_efd_enable
> +	rte_intr_enable
> +	rte_intr_free_epoll_fd

Might as well add all remaining interrupt APIs:

	rte_intr_callback_unregister_pending
	rte_intr_tls_epfd

>  	rte_intr_rx_ctl
>  	rte_lcore_count
>  	rte_lcore_has_role

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [dpdk-dev] [PATCH v2] eal/windows: add interrupt functions stub
  2020-12-11  0:26 [dpdk-dev] [PATCH] eal/windows: add interrupt functions stub Pallavi Kadam
  2020-12-11 19:53 ` Narcisa Ana Maria Vasile
  2020-12-11 21:07 ` Dmitry Kozlyuk
@ 2020-12-15  0:26 ` Pallavi Kadam
  2020-12-15 22:39   ` Dmitry Kozlyuk
  2 siblings, 1 reply; 7+ messages in thread
From: Pallavi Kadam @ 2020-12-15  0:26 UTC (permalink / raw)
  To: dev, thomas
  Cc: ranjit.menon, dmitry.kozliuk, Narcisa.Vasile, talshn, pallavi.kadam

Add some missing interrupt implementations on Windows.
Also add respective functions to export file.

Signed-off-by: Tal Shnaiderman <talshn@nvidia.com>
Signed-off-by: Pallavi Kadam <pallavi.kadam@intel.com>
Reviewed-by: Ranjit Menon <ranjit.menon@intel.com>
Acked-by: Narcisa Vasile <navasile@linux.microsoft.com>

v2 changes:
	Add remaining interrupt APIs (Dmitry Kozlyuk)
---
 lib/librte_eal/rte_eal_exports.def      |  16 +++
 lib/librte_eal/windows/eal_interrupts.c | 126 ++++++++++++++++++++++++
 2 files changed, 142 insertions(+)

diff --git a/lib/librte_eal/rte_eal_exports.def b/lib/librte_eal/rte_eal_exports.def
index 6a6be1cfa..dd8398f88 100644
--- a/lib/librte_eal/rte_eal_exports.def
+++ b/lib/librte_eal/rte_eal_exports.def
@@ -56,6 +56,9 @@ EXPORTS
 	rte_eal_tailq_register
 	rte_eal_using_phys_addrs
 	rte_eal_wait_lcore
+	rte_epoll_ctl
+	rte_epoll_wait
+	rte_epoll_wait_interruptible
 	rte_exit
 	rte_free
 	rte_get_main_lcore
@@ -63,7 +66,20 @@ EXPORTS
 	rte_get_tsc_hz
 	rte_hexdump
 	rte_hypervisor_get
+	rte_intr_ack
+	rte_intr_allow_others
+	rte_intr_callback_register
+	rte_intr_callback_unregister
+	rte_intr_callback_unregister_pending
+	rte_intr_cap_multiple
+	rte_intr_disable
+	rte_intr_dp_is_en
+	rte_intr_efd_disable
+	rte_intr_efd_enable
+	rte_intr_enable
+	rte_intr_free_epoll_fd
 	rte_intr_rx_ctl
+	rte_intr_tls_epfd
 	rte_lcore_count
 	rte_lcore_has_role
 	rte_lcore_index
diff --git a/lib/librte_eal/windows/eal_interrupts.c b/lib/librte_eal/windows/eal_interrupts.c
index 6c64a48f3..67928b46d 100644
--- a/lib/librte_eal/windows/eal_interrupts.c
+++ b/lib/librte_eal/windows/eal_interrupts.c
@@ -105,3 +105,129 @@ eal_intr_thread_schedule(void (*func)(void *arg), void *arg)
 
 	return 0;
 }
+
+int
+rte_intr_callback_register(
+	__rte_unused const struct rte_intr_handle *intr_handle,
+	__rte_unused rte_intr_callback_fn cb, __rte_unused void *cb_arg)
+{
+	return -ENOTSUP;
+}
+
+int
+rte_intr_callback_unregister_pending(
+	__rte_unused const struct rte_intr_handle *intr_handle,
+	__rte_unused rte_intr_callback_fn cb_fn, __rte_unused void *cb_arg,
+	__rte_unused rte_intr_unregister_callback_fn ucb_fn)
+{
+	return -ENOTSUP;
+}
+
+int
+rte_intr_callback_unregister(
+	__rte_unused const struct rte_intr_handle *intr_handle,
+	__rte_unused rte_intr_callback_fn cb_fn, __rte_unused void *cb_arg)
+{
+	return 0;
+}
+
+int
+rte_intr_enable(__rte_unused const struct rte_intr_handle *intr_handle)
+{
+	return -ENOTSUP;
+}
+
+int
+rte_intr_ack(__rte_unused const struct rte_intr_handle *intr_handle)
+{
+	return -ENOTSUP;
+}
+
+int
+rte_intr_disable(__rte_unused const struct rte_intr_handle *intr_handle)
+{
+	return -ENOTSUP;
+}
+
+int
+rte_intr_efd_enable(struct rte_intr_handle *intr_handle, uint32_t nb_efd)
+{
+	RTE_SET_USED(intr_handle);
+	RTE_SET_USED(nb_efd);
+
+	return 0;
+}
+
+void
+rte_intr_efd_disable(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+}
+
+int
+rte_intr_dp_is_en(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+	return 0;
+}
+
+int
+rte_intr_allow_others(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+	return 1;
+}
+
+int
+rte_intr_cap_multiple(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+	return 0;
+}
+
+int
+rte_epoll_wait(int epfd, struct rte_epoll_event *events,
+		int maxevents, int timeout)
+{
+	RTE_SET_USED(epfd);
+	RTE_SET_USED(events);
+	RTE_SET_USED(maxevents);
+	RTE_SET_USED(timeout);
+
+	return -ENOTSUP;
+}
+
+int
+rte_epoll_wait_interruptible(int epfd, struct rte_epoll_event *events,
+			     int maxevents, int timeout)
+{
+	RTE_SET_USED(epfd);
+	RTE_SET_USED(events);
+	RTE_SET_USED(maxevents);
+	RTE_SET_USED(timeout);
+
+	return -ENOTSUP;
+}
+
+int
+rte_epoll_ctl(int epfd, int op, int fd, struct rte_epoll_event *event)
+{
+	RTE_SET_USED(epfd);
+	RTE_SET_USED(op);
+	RTE_SET_USED(fd);
+	RTE_SET_USED(event);
+
+	return -ENOTSUP;
+}
+
+int
+rte_intr_tls_epfd(void)
+{
+	return -ENOTSUP;
+}
+
+void
+rte_intr_free_epoll_fd(struct rte_intr_handle *intr_handle)
+{
+	RTE_SET_USED(intr_handle);
+}
-- 
2.18.0.windows.1


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH] eal/windows: add interrupt functions stub
  2020-12-11 21:07 ` Dmitry Kozlyuk
@ 2020-12-15  0:38   ` Kadam, Pallavi
  0 siblings, 0 replies; 7+ messages in thread
From: Kadam, Pallavi @ 2020-12-15  0:38 UTC (permalink / raw)
  To: Dmitry Kozlyuk; +Cc: dev, thomas, ranjit.menon, Narcisa.Vasile, talshn

Hi Dmitry,

On 12/11/2020 1:07 PM, Dmitry Kozlyuk wrote:
> On Thu, 10 Dec 2020 16:26:07 -0800, Pallavi Kadam wrote:
>> Add some missing interrupt implementations on Windows.
>> Also add respestive functions to export file.
>>
>> Signed-off-by: Tal Shnaiderman <talshn@nvidia.com>
>> Signed-off-by: Pallavi Kadam <pallavi.kadam@intel.com>
>> Reviewed-by: Ranjit Menon <ranjit.menon@intel.com>
>> ---
>>   lib/librte_eal/rte_eal_exports.def      | 11 ++++
>>   lib/librte_eal/windows/eal_interrupts.c | 76 +++++++++++++++++++++++++
>>   2 files changed, 87 insertions(+)
>>
>> diff --git a/lib/librte_eal/rte_eal_exports.def b/lib/librte_eal/rte_eal_exports.def
>> index 6a6be1cfa..98a38ed32 100644
>> --- a/lib/librte_eal/rte_eal_exports.def
>> +++ b/lib/librte_eal/rte_eal_exports.def
>> @@ -63,6 +63,17 @@ EXPORTS
>>   	rte_get_tsc_hz
>>   	rte_hexdump
>>   	rte_hypervisor_get
>> +	rte_intr_ack
>> +	rte_intr_allow_others
>> +	rte_intr_callback_register
>> +	rte_intr_callback_unregister
>> +	rte_intr_cap_multiple
>> +	rte_intr_disable
>> +	rte_intr_dp_is_en
>> +	rte_intr_efd_disable
>> +	rte_intr_efd_enable
>> +	rte_intr_enable
>> +	rte_intr_free_epoll_fd
> Might as well add all remaining interrupt APIs:
>
> 	rte_intr_callback_unregister_pending
> 	rte_intr_tls_epfd

Sent out v2 with additional interrupt APIs.

Please let me know if anything is missing.

Thanks,

>
>>   	rte_intr_rx_ctl
>>   	rte_lcore_count
>>   	rte_lcore_has_role

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v2] eal/windows: add interrupt functions stub
  2020-12-15  0:26 ` [dpdk-dev] [PATCH v2] " Pallavi Kadam
@ 2020-12-15 22:39   ` Dmitry Kozlyuk
  2021-01-05 21:39     ` Thomas Monjalon
  0 siblings, 1 reply; 7+ messages in thread
From: Dmitry Kozlyuk @ 2020-12-15 22:39 UTC (permalink / raw)
  To: Pallavi Kadam; +Cc: dev, thomas, ranjit.menon, Narcisa.Vasile, talshn

On Mon, 14 Dec 2020 16:26:22 -0800, Pallavi Kadam wrote:
> Add some missing interrupt implementations on Windows.
> Also add respective functions to export file.
> 
> Signed-off-by: Tal Shnaiderman <talshn@nvidia.com>
> Signed-off-by: Pallavi Kadam <pallavi.kadam@intel.com>
> Reviewed-by: Ranjit Menon <ranjit.menon@intel.com>
> Acked-by: Narcisa Vasile <navasile@linux.microsoft.com>

Acked-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v2] eal/windows: add interrupt functions stub
  2020-12-15 22:39   ` Dmitry Kozlyuk
@ 2021-01-05 21:39     ` Thomas Monjalon
  0 siblings, 0 replies; 7+ messages in thread
From: Thomas Monjalon @ 2021-01-05 21:39 UTC (permalink / raw)
  To: Pallavi Kadam; +Cc: dev, ranjit.menon, Narcisa.Vasile, talshn, Dmitry Kozlyuk

15/12/2020 23:39, Dmitry Kozlyuk:
> On Mon, 14 Dec 2020 16:26:22 -0800, Pallavi Kadam wrote:
> > Add some missing interrupt implementations on Windows.
> > Also add respective functions to export file.
> > 
> > Signed-off-by: Tal Shnaiderman <talshn@nvidia.com>
> > Signed-off-by: Pallavi Kadam <pallavi.kadam@intel.com>
> > Reviewed-by: Ranjit Menon <ranjit.menon@intel.com>
> > Acked-by: Narcisa Vasile <navasile@linux.microsoft.com>
> 
> Acked-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>

Applied, thanks

Note: rte_eal_exports.def is sorted as version.map to ease comparison.



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2021-01-05 21:39 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-11  0:26 [dpdk-dev] [PATCH] eal/windows: add interrupt functions stub Pallavi Kadam
2020-12-11 19:53 ` Narcisa Ana Maria Vasile
2020-12-11 21:07 ` Dmitry Kozlyuk
2020-12-15  0:38   ` Kadam, Pallavi
2020-12-15  0:26 ` [dpdk-dev] [PATCH v2] " Pallavi Kadam
2020-12-15 22:39   ` Dmitry Kozlyuk
2021-01-05 21:39     ` Thomas Monjalon

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git