[PATCH] gpu/cuda: missing slash in libcuda.so path

[PATCH] gpu/cuda: missing slash in libcuda.so path

[eagostini]

From: Elena Agostini <eagostini@nvidia.com>

Signed-off-by: Elena Agostini <eagostini@nvidia.com>
---
 drivers/gpu/cuda/cuda.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/cuda/cuda.c b/drivers/gpu/cuda/cuda.c
index 2f8c4684ce..8505d39d64 100644
--- a/drivers/gpu/cuda/cuda.c
+++ b/drivers/gpu/cuda/cuda.c
@@ -169,7 +169,7 @@ cuda_loader(void)
 	if (getenv("CUDA_PATH_L") == NULL)
 		snprintf(cuda_path, 1024, "%s", "libcuda.so");
 	else
-		snprintf(cuda_path, 1024, "%s%s", getenv("CUDA_PATH_L"), "libcuda.so");
+		snprintf(cuda_path, 1024, "%s/%s", getenv("CUDA_PATH_L"), "libcuda.so");
 
 	cudalib = dlopen(cuda_path, RTLD_LAZY);
 	if (cudalib == NULL) {
-- 
2.17.1

Re: [PATCH] gpu/cuda: missing slash in libcuda.so path

[Thomas Monjalon]

01/03/2022 20:42, eagostini@nvidia.com:
> From: Elena Agostini <eagostini@nvidia.com>
> 
> Signed-off-by: Elena Agostini <eagostini@nvidia.com>

Applied with this title and commit log:

    gpu/cuda: fix dependency loading path
    
    A slash was missing in libcuda.so path for dlopen.

Re: [PATCH] gpu/cuda: missing slash in libcuda.so path

[Kevin Traynor]

Hi Elena,

On 07/03/2022 21:47, Thomas Monjalon wrote:
> 01/03/2022 20:42, eagostini@nvidia.com:
>> From: Elena Agostini <eagostini@nvidia.com>
>>
>> Signed-off-by: Elena Agostini <eagostini@nvidia.com>
> 
> Applied with this title and commit log:
> 
>      gpu/cuda: fix dependency loading path
>      
>      A slash was missing in libcuda.so path for dlopen.
> 
> 
> 

This seems like a good candidate for backport to LTS, so I added it to 
the queue for 21.11.1.

Please let me know if you do *not* want it backported 21.11.1.

thanks,
Kevin.

Re: [PATCH] gpu/cuda: missing slash in libcuda.so path

[Elena Agostini]

[-- Attachment #1: Type: text/plain, Size: 933 bytes --]

Yes please

Thanks
EA

From: Kevin Traynor <ktraynor@redhat.com>
Date: Tuesday, 8 March 2022 at 15:24
To: Elena Agostini <eagostini@nvidia.com>
Cc: dev@dpdk.org <dev@dpdk.org>, NBU-Contact-Thomas Monjalon (EXTERNAL) <thomas@monjalon.net>
Subject: Re: [PATCH] gpu/cuda: missing slash in libcuda.so path
External email: Use caution opening links or attachments


Hi Elena,

On 07/03/2022 21:47, Thomas Monjalon wrote:
> 01/03/2022 20:42, eagostini@nvidia.com:
>> From: Elena Agostini <eagostini@nvidia.com>
>>
>> Signed-off-by: Elena Agostini <eagostini@nvidia.com>
>
> Applied with this title and commit log:
>
>      gpu/cuda: fix dependency loading path
>
>      A slash was missing in libcuda.so path for dlopen.
>
>
>

This seems like a good candidate for backport to LTS, so I added it to
the queue for 21.11.1.

Please let me know if you do *not* want it backported 21.11.1.

thanks,
Kevin.

[-- Attachment #2: Type: text/html, Size: 3536 bytes --]

Re: [PATCH] gpu/cuda: missing slash in libcuda.so path

[Thomas Monjalon]

08/03/2022 15:24, Elena Agostini:
> Yes please
> 
> Thanks
> EA
> 
> From: Kevin Traynor <ktraynor@redhat.com>
> 
> Hi Elena,
> 
> On 07/03/2022 21:47, Thomas Monjalon wrote:
> > 01/03/2022 20:42, eagostini@nvidia.com:
> >> From: Elena Agostini <eagostini@nvidia.com>
> >>
> >> Signed-off-by: Elena Agostini <eagostini@nvidia.com>
> >
> > Applied with this title and commit log:
> >
> >      gpu/cuda: fix dependency loading path
> >
> >      A slash was missing in libcuda.so path for dlopen.
> >
> >
> >
> 
> This seems like a good candidate for backport to LTS, so I added it to
> the queue for 21.11.1.
> 
> Please let me know if you do *not* want it backported 21.11.1.
> 
> thanks,
> Kevin.


I forgot the Fixes: line and Cc: stable@dpdk.org

Elena, please remind to add such lines next time,
it makes the process more automatic.