[dpdk-dev] [PATCH] nfp: restore the unlink operation
Aaron Conole
aconole at redhat.com
Fri May 18 17:57:04 CEST 2018
Luca Boccassi <bluca at debian.org> writes:
> On Mon, 2018-05-14 at 10:46 -0400, Aaron Conole wrote:
>> For the stable versions of DPDK, the NFP driver's NSP user space
>> driver was modified to account for non-root usage. When that
>> happened, commit 515933ad8385 ("nfp: allow for non-root user")
>> inadvertently removed the unlink() call.
>>
>> Fixes: 515933ad8385 ("nfp: allow for non-root user")
>> Cc: Alejandro Lucero <alejandro.lucero at netronome.com>
>> Cc: Luca Boccassi <bluca at debian.org>
>> Cc: Eelco Chaudron <echaudro at redhat.com>
>> Cc: Yuanhan Liu <yliu at fridaylinux.org>
>> Signed-off-by: Aaron Conole <aconole at redhat.com>
>> ---
>> drivers/net/nfp/nfp_nfpu.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/net/nfp/nfp_nfpu.c b/drivers/net/nfp/nfp_nfpu.c
>> index ae2e07220..c003640c7 100644
>> --- a/drivers/net/nfp/nfp_nfpu.c
>> +++ b/drivers/net/nfp/nfp_nfpu.c
>> @@ -121,5 +121,6 @@ nfpu_close(nfpu_desc_t *desc)
>> close(desc->lock);
>>
>> nspu_get_lockfile_path(lockname, sizeof(lockname), desc);
>> + unlink(lockname);
>> return 0;
>> }
>
> Applied and pushed to stable/18.02
Thanks, Luca!
> Next time, please send patches that are only for stable to stable at dpdk.
> org and do not include dev at dpdk.org
Will do.
> Also use --subject-prefix='PATCH <branch>' so that we can understand
> for which branches a patch is
Thanks. I'll keep this in mind.
More information about the dev
mailing list