DPDK patches and discussions
 help / color / mirror / Atom feed
From: Dariusz Sosnowski <dsosnowski@nvidia.com>
To: Stephen Hemminger <stephen@networkplumber.org>,
	Yang Ming <mosesyyoung@gmail.com>
Cc: <dev@dpdk.org>
Subject: Re: [PATCH v3 1/2] net/mlx5: improve socket file path handling
Date: Tue, 18 Nov 2025 15:45:59 +0100	[thread overview]
Message-ID: <20251118144559.hpcgungifxd2hv3l@ds-vm-debian.local> (raw)
In-Reply-To: <20251117095842.496757e5@phoenix>

On Mon, Nov 17, 2025 at 09:58:42AM -0800, Stephen Hemminger wrote:
> On Fri, 29 Aug 2025 22:49:52 +0800
> Yang Ming <mosesyyoung@gmail.com> wrote:
> 
> > -#define MLX5_SOCKET_PATH "/var/tmp/dpdk_net_mlx5_%d"
> > +#define MLX5_SOCKET_DPATH "/var/tmp"
> > +#define MLX5_SOCKET_FNAME "dpdk_net_mlx5"
> >  #define MLX5_ALL_PORT_IDS 0xffff
> >  
> NAK - should follow current Linux defined policy for services.
> 
> 
>     http://0pointer.net/blog/projects/tmp.html
>     https://en.wikipedia.org/wiki/Filesystem_Hierarchy_Standard
>     https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html*

We have discussed the series internally and we agree that the proper
solution would require dropping unix socket in /var/tmp and replacing it
with either DPDK runtime directory or another path in /var/run.
Since such a change would break external tooling,
we are unable to do that in time for 25.11.
Regarding the issue reported i.e., not being able to create unix socket in
/var/tmp when this directory is not writeable.
It is not critical, because it does not prevent probing.
Unix socket creation failure is logged and PMD moves on.
The series will be dropped from next-net-mlx tree and
we'll work on fixing that in next release cycle.

@Yang: We are sorry for the inconvenience caused by dropping
of this patch. Since the changes required are also depending
on tooling external to DPDK, we would like to propose that our DPDK team
will handle this fix in the next release cycle.
Of course, such a fix would include your attribution.
Thank you for raising this issue and sorry again for the inconvience.

Best regards,
Dariusz Sosnowski

  reply	other threads:[~2025-11-18 14:46 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-13  9:24 [PATCH 1/2] net/mlx5: improve socket file path Yang Ming
2024-12-13  9:24 ` [PATCH 2/2] net/mlx5: improve log " Yang Ming
2025-03-04  6:23   ` Bing Zhao
2025-03-05  3:20     ` Yang Ming
2025-03-10 14:59     ` Stephen Hemminger
2025-03-12  2:32       ` [External] " Yang Ming
2025-03-17 16:05         ` Bing Zhao
2025-03-21  9:48           ` Ming 1. Yang (NSB)
2024-12-13 17:12 ` [PATCH 1/2] net/mlx5: improve socket " Stephen Hemminger
2024-12-13 17:16   ` Bruce Richardson
2025-01-03  2:51     ` Yang Ming
2025-03-12  2:55       ` Yang Ming
2025-03-14 11:48         ` Dariusz Sosnowski
2025-03-21 13:27 ` [PATCH v2 1/2] net/mlx5: enhance " Yang Ming
2025-03-21 13:27   ` [PATCH v2 2/2] net/mlx5: enhance log " Yang Ming
2025-04-10  7:53     ` Yang Ming
2025-05-12 10:12       ` Moses Young
2025-06-04  6:54         ` Maayan Kashani
2025-06-26 13:06           ` Dariusz Sosnowski
2025-07-20  8:57             ` Moses Young
2025-06-26 12:48   ` [PATCH v2 1/2] net/mlx5: enhance socket " Dariusz Sosnowski
2025-08-29 14:49   ` [PATCH v3 1/2] net/mlx5: improve socket file path handling Yang Ming
2025-08-29 14:49     ` [PATCH v3 2/2] net/mlx5: improve debug dump " Yang Ming
2025-11-07 16:48       ` Dariusz Sosnowski
2025-11-17 17:56       ` Stephen Hemminger
2025-08-29 19:57     ` [PATCH v3 1/2] net/mlx5: improve socket " Stephen Hemminger
2025-08-31  0:55       ` Moses Young
2025-08-31  3:21         ` Stephen Hemminger
2025-09-01  1:40           ` Moses Young
2025-11-17 13:43     ` Raslan Darawsheh
2025-11-17 17:58     ` Stephen Hemminger
2025-11-18 14:45       ` Dariusz Sosnowski [this message]
2025-11-08  1:16   ` [PATCH v2 1/2] net/mlx5: enhance socket file path Stephen Hemminger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20251118144559.hpcgungifxd2hv3l@ds-vm-debian.local \
    --to=dsosnowski@nvidia.com \
    --cc=dev@dpdk.org \
    --cc=mosesyyoung@gmail.com \
    --cc=stephen@networkplumber.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).