Re: [Cluster-devel] [PATCH RESEND 4/8] dlm: fix creating multiple node structures
Greg KH <[email protected]> Wed, 11 Oct 2023 08:25:15 +0200
| Newsgroups | com.redhat.cluster-devel,dev.linux.lists.gfs2,org.kernel.vger.stable |
|---|---|
| Message-ID | <2023101138-rickety-collector-e9fc@gregkh> |
On Tue, Oct 10, 2023 at 06:04:44PM -0400, Alexander Aring wrote: > This patch will lookup existing nodes instead of always creating them > when dlm_midcomms_addr() is called. The idea is here to create midcomms > nodes when user space getting informed that nodes joins the cluster. This > is the case when dlm_midcomms_addr() is called, however it can be called > multiple times by user space to add several address configurations to one > node e.g. when using SCTP. Those multiple times need to be filtered out > and we doing that by looking up if the node exists before. Due configfs > entry it is safe that this function gets only called once at a time. > > Cc: [email protected] > Fixes: 63e711b08160 ("fs: dlm: create midcomms nodes when configure") > Signed-off-by: Alexander Aring <[email protected]> > --- > fs/dlm/midcomms.c | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) Why does patch 4/8 have a cc: stable, when it depends on patches 1-3 as well? That is going to drive us crazy when it hits Linus's tree, how do we know the dependancies here anymore? thanks, greg k-h