[PATCH] provider: only call discover for the provider implementations

[PATCH] provider: only call discover for the provider implementations

[Kris Van Hees]

Since the discover() functionality is provided by the implementation
of a provider type rather than being specific to a provider, we only
need to loop over dt_providers[].  The discover hooks themselves should
take care of any newly discovered probes.

Suggested-by: Eugene Loh <eugene.loh@oracle.com>
Signed-off-by: Kris Van Hees <kris.van.hees@oracle.com>
---
 libdtrace/dt_provider.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/libdtrace/dt_provider.c b/libdtrace/dt_provider.c
index e073e1f2..818390d2 100644
--- a/libdtrace/dt_provider.c
+++ b/libdtrace/dt_provider.c
@@ -186,12 +186,13 @@ int
 dt_provider_discover(dtrace_hdl_t *dtp)
 {
 	int		prid = dtp->dt_probe_id;
-	dt_htab_next_t	*it = NULL;
-	dt_provider_t	*pvp;
+	int		i;
 
 	/* Discover new probes. */
-	while ((pvp = dt_htab_next(dtp->dt_provs, &it)) != NULL) {
-		if (pvp->impl->discover && pvp->impl->discover(dtp) < 0)
+	for (i = 0; dt_providers[i]; i++) {
+		const dt_provimpl_t	*pvops = dt_providers[i];
+
+		if (pvops->discover && pvops->discover(dtp) < 0)
 			return -1;        /* errno is already set */
 	}
 
-- 
2.52.0

Re: [PATCH] provider: only call discover for the provider implementations

[Elena Zannoni]

Reviewed-by: Elena Zannoni <elena.zannoni@oracle.com>

On 8/28/26 12:48 PM, Kris Van Hees wrote:

> Since the discover() functionality is provided by the implementation
> of a provider type rather than being specific to a provider, we only
> need to loop over dt_providers[].  The discover hooks themselves should
> take care of any newly discovered probes.
> 
> Suggested-by: Eugene Loh <eugene.loh@oracle.com>
> Signed-off-by: Kris Van Hees <kris.van.hees@oracle.com>
> ---
>  libdtrace/dt_provider.c | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/libdtrace/dt_provider.c b/libdtrace/dt_provider.c
> index e073e1f2..818390d2 100644
> --- a/libdtrace/dt_provider.c
> +++ b/libdtrace/dt_provider.c
> @@ -186,12 +186,13 @@ int
>  dt_provider_discover(dtrace_hdl_t *dtp)
>  {
>  	int		prid = dtp->dt_probe_id;
> -	dt_htab_next_t	*it = NULL;
> -	dt_provider_t	*pvp;
> +	int		i;
>  
>  	/* Discover new probes. */
> -	while ((pvp = dt_htab_next(dtp->dt_provs, &it)) != NULL) {
> -		if (pvp->impl->discover && pvp->impl->discover(dtp) < 0)
> +	for (i = 0; dt_providers[i]; i++) {
> +		const dt_provimpl_t	*pvops = dt_providers[i];
> +
> +		if (pvops->discover && pvops->discover(dtp) < 0)
>  			return -1;        /* errno is already set */
>  	}
>  
> -- 
> 2.52.0
> 
>