All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Wilck <martin.wilck@suse.com>
To: Benjamin Marzinski <bmarzins@redhat.com>
Cc: Christophe Varoqui <christophe.varoqui@opensvc.com>,
	 dm-devel@lists.linux.dev
Subject: Re: [PATCH 00/41] multipath-tools: comply with C library reserved names
Date: Tue, 13 Aug 2024 11:43:09 +0200	[thread overview]
Message-ID: <61f99010436e6fd8f07e1592294de11bde2a28c8.camel@suse.com> (raw)
In-Reply-To: <ZrqHZDgAZ6C7pokR@redhat.com>

On Mon, 2024-08-12 at 18:06 -0400, Benjamin Marzinski wrote:
> On Mon, Aug 12, 2024 at 06:44:54PM +0200, Martin Wilck wrote:
> > On Fri, 2024-08-09 at 14:14 -0400, Benjamin Marzinski wrote:
> > > 
> > > Despite some small misgivings about renaming symbols (which quite
> > > possibly aren't being used by anyone) in our user-facing
> > > libraries,
> > > libpmathpersist and libmpathcmd, for the set:
> > 
> > As the functions are only renamed, we could use (weak) aliases here
> > to
> > still provide the old function names if some caller requires them.
> > Would you prefer that?
> > 
> > We'd still be violating the glibc standards somewhat, but the
> > probability for an actual symbol name clash is very low, so I guess
> > it
> > would be acceptable to do this in the name of backward
> > compatibility.
> 
> While it's not that big of a deal, I do think we should prioritize
> backwards compatibility over this level of standards compliance.

Ok. Sending patches right now. I've given them quick tests with  qemu-
pr-helper and some trivial test program that use the "old" ABI, but I'd
be grateful if you could double-check.

Martin


      reply	other threads:[~2024-08-13  9:43 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-08 15:25 [PATCH 00/41] multipath-tools: comply with C library reserved names Martin Wilck
2024-08-08 15:25 ` [PATCH 01/41] multipath-tools tests: make alias.test succeed with increased verbosity Martin Wilck
2024-08-08 15:25 ` [PATCH 02/41] libmpathutil: avoid extra memory allocation in print_strbuf() Martin Wilck
2024-08-08 15:25 ` [PATCH 03/41] libmpathutil: rename strlcat and strlcpy Martin Wilck
2024-08-08 15:25 ` [PATCH 04/41] libmpathutil.version: remove strlcpy and strlcat, and LIBMULTIPATH section Martin Wilck
2024-08-08 15:25 ` [PATCH 05/41] libmpathutil: fix __attribute typo in log_pthread.c Martin Wilck
2024-08-08 15:25 ` [PATCH 06/41] libmultipath: devmapper.c: rename __DR_UNUSED__, and fix __attribute typo Martin Wilck
2024-08-08 15:25 ` [PATCH 07/41] multipathd: " Martin Wilck
2024-08-08 15:25 ` [PATCH 08/41] multipath-tools: use common convention for "header file included" macros Martin Wilck
2024-08-08 15:25 ` [PATCH 09/41] libmpathcmd: rename __mpath_connect() to mpath_connect__() Martin Wilck
2024-08-08 15:25 ` [PATCH 10/41] libmpathutil: rename _MAX_CMD_LEN Martin Wilck
2024-08-08 15:25 ` [PATCH 11/41] libmpathutil: rename __append_strbuf_str() and __get_strbuf_buf() Martin Wilck
2024-08-08 15:25 ` [PATCH 12/41] libmpathutil: rename _log_bitfield_overflow() Martin Wilck
2024-08-08 15:25 ` [PATCH 13/41] libmpathutil: rename _install_keyword() to install_keyword__() Martin Wilck
2024-08-08 15:25 ` [PATCH 14/41] libmultipath: rename _NVME_LIB_C Martin Wilck
2024-08-08 15:25 ` [PATCH 15/41] libmultipath: checkers.h: fix __CHECKER_FIRST_MSG in comment Martin Wilck
2024-08-08 15:25 ` [PATCH 16/41] libmultipath: rename identifiers with leading underscores in devmapper.h Martin Wilck
2024-08-08 15:25 ` [PATCH 17/41] libmultipath: rename identifiers with leading underscores in discovery.h Martin Wilck
2024-08-08 15:25 ` [PATCH 18/41] libmultipath: rename __snprint_config() Martin Wilck
2024-08-08 15:25 ` [PATCH 19/41] libmultipath: rename __unlock() Martin Wilck
2024-08-08 15:25 ` [PATCH 20/41] libmultipath: rename __sysfs_attr_get_value() Martin Wilck
2024-08-08 15:26 ` [PATCH 21/41] libmultipath: rename __snprint_foreign_topology() Martin Wilck
2024-08-08 15:26 ` [PATCH 22/41] libmultipath: rename macros with double underscores in propsel.c Martin Wilck
2024-08-08 15:26 ` [PATCH 23/41] libmultipath: rename enum values with double underscores in structs.h Martin Wilck
2024-08-08 15:26 ` [PATCH 24/41] libmultipath: rename macros starting with underscore Martin Wilck
2024-08-08 15:26 ` [PATCH 25/41] libmultipath: rename __internal_config variable Martin Wilck
2024-08-08 15:26 ` [PATCH 26/41] libmultipath: rename _dm_flush_map() to dm_flush_map__() Martin Wilck
2024-08-08 15:26 ` [PATCH 27/41] libmultipath: foreign: rename _check() to check__() Martin Wilck
2024-08-08 15:26 ` [PATCH 28/41] libmultipath: rename _cleanup_foreign() Martin Wilck
2024-08-08 15:26 ` [PATCH 29/41] libmultipath: rename _init_config() Martin Wilck
2024-08-08 15:26 ` [PATCH 30/41] libmultipath: rename symbols starting with underscore in print.h Martin Wilck
2024-08-08 15:26 ` [PATCH 31/41] libmultipath: remove dead code in pgpolicies.h Martin Wilck
2024-08-08 15:26 ` [PATCH 32/41] libmultipath: remove struct and union names in cciss.h Martin Wilck
2024-08-08 15:26 ` [PATCH 33/41] libmpathpersist: rename functions with double leading underscore Martin Wilck
2024-08-08 15:26 ` [PATCH 34/41] libdmmp: rename macros starting with _DMMP Martin Wilck
2024-08-08 15:26 ` [PATCH 35/41] libdmmp: rename enum values and variables starting with _DMMP_ Martin Wilck
2024-08-08 15:26 ` [PATCH 36/41] libdmmp: rename non-static functions starting with underscore Martin Wilck
2024-08-08 15:26 ` [PATCH 37/41] multipathd: rename symbols with double leading underscore Martin Wilck
2024-08-08 15:26 ` [PATCH 38/41] multipath-tools tests: rename functions with double underscores Martin Wilck
2024-08-08 15:26 ` [PATCH 39/41] kpartx: rename macros with leading underscores Martin Wilck
2024-08-08 15:26 ` [PATCH 40/41] libmpathutil: rename struct _vector to vector_s Martin Wilck
2024-08-08 15:26 ` [PATCH 41/41] libmultipath: don't define __user Martin Wilck
2024-08-09 18:14 ` [PATCH 00/41] multipath-tools: comply with C library reserved names Benjamin Marzinski
2024-08-12 16:44   ` Martin Wilck
2024-08-12 22:06     ` Benjamin Marzinski
2024-08-13  9:43       ` Martin Wilck [this message]

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=61f99010436e6fd8f07e1592294de11bde2a28c8.camel@suse.com \
    --to=martin.wilck@suse.com \
    --cc=bmarzins@redhat.com \
    --cc=christophe.varoqui@opensvc.com \
    --cc=dm-devel@lists.linux.dev \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.