2019-05-29 00:57:20 +08:00
|
|
|
// SPDX-License-Identifier: GPL-2.0-only
|
2006-11-03 00:19:21 +08:00
|
|
|
/******************************************************************************
|
|
|
|
*******************************************************************************
|
|
|
|
**
|
|
|
|
** Copyright (C) Sistina Software, Inc. 1997-2003 All rights reserved.
|
2009-01-29 02:57:40 +08:00
|
|
|
** Copyright (C) 2004-2009 Red Hat, Inc. All rights reserved.
|
2006-11-03 00:19:21 +08:00
|
|
|
**
|
|
|
|
**
|
|
|
|
*******************************************************************************
|
|
|
|
******************************************************************************/
|
|
|
|
|
|
|
|
/*
|
|
|
|
* lowcomms.c
|
|
|
|
*
|
|
|
|
* This is the "low-level" comms layer.
|
|
|
|
*
|
|
|
|
* It is responsible for sending/receiving messages
|
|
|
|
* from other nodes in the cluster.
|
|
|
|
*
|
|
|
|
* Cluster nodes are referred to by their nodeids. nodeids are
|
|
|
|
* simply 32 bit numbers to the locking module - if they need to
|
2009-01-23 05:26:47 +08:00
|
|
|
* be expanded for the cluster infrastructure then that is its
|
2006-11-03 00:19:21 +08:00
|
|
|
* responsibility. It is this layer's
|
|
|
|
* responsibility to resolve these into IP address or
|
|
|
|
* whatever it needs for inter-node communication.
|
|
|
|
*
|
|
|
|
* The comms level is two kernel threads that deal mainly with
|
|
|
|
* the receiving of messages from other nodes and passing them
|
|
|
|
* up to the mid-level comms layer (which understands the
|
|
|
|
* message format) for execution by the locking core, and
|
|
|
|
* a send thread which does all the setting up of connections
|
|
|
|
* to remote nodes and the sending of data. Threads are not allowed
|
|
|
|
* to send their own data because it may cause them to wait in times
|
|
|
|
* of high load. Also, this way, the sending thread can collect together
|
|
|
|
* messages bound for one node and send them in one block.
|
|
|
|
*
|
2009-01-23 05:26:47 +08:00
|
|
|
* lowcomms will choose to use either TCP or SCTP as its transport layer
|
2007-04-17 22:39:57 +08:00
|
|
|
* depending on the configuration variable 'protocol'. This should be set
|
2009-01-23 05:26:47 +08:00
|
|
|
* to 0 (default) for TCP or 1 for SCTP. It should be configured using a
|
2007-04-17 22:39:57 +08:00
|
|
|
* cluster-wide mechanism as it must be the same on all nodes of the cluster
|
|
|
|
* for the DLM to function.
|
2006-11-03 00:19:21 +08:00
|
|
|
*
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include <asm/ioctls.h>
|
|
|
|
#include <net/sock.h>
|
|
|
|
#include <net/tcp.h>
|
|
|
|
#include <linux/pagemap.h>
|
2007-04-17 22:39:57 +08:00
|
|
|
#include <linux/file.h>
|
2008-05-12 23:04:51 +08:00
|
|
|
#include <linux/mutex.h>
|
2007-04-17 22:39:57 +08:00
|
|
|
#include <linux/sctp.h>
|
include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h
percpu.h is included by sched.h and module.h and thus ends up being
included when building most .c files. percpu.h includes slab.h which
in turn includes gfp.h making everything defined by the two files
universally available and complicating inclusion dependencies.
percpu.h -> slab.h dependency is about to be removed. Prepare for
this change by updating users of gfp and slab facilities include those
headers directly instead of assuming availability. As this conversion
needs to touch large number of source files, the following script is
used as the basis of conversion.
http://userweb.kernel.org/~tj/misc/slabh-sweep.py
The script does the followings.
* Scan files for gfp and slab usages and update includes such that
only the necessary includes are there. ie. if only gfp is used,
gfp.h, if slab is used, slab.h.
* When the script inserts a new include, it looks at the include
blocks and try to put the new include such that its order conforms
to its surrounding. It's put in the include block which contains
core kernel includes, in the same order that the rest are ordered -
alphabetical, Christmas tree, rev-Xmas-tree or at the end if there
doesn't seem to be any matching order.
* If the script can't find a place to put a new include (mostly
because the file doesn't have fitting include block), it prints out
an error message indicating which .h file needs to be added to the
file.
The conversion was done in the following steps.
1. The initial automatic conversion of all .c files updated slightly
over 4000 files, deleting around 700 includes and adding ~480 gfp.h
and ~3000 slab.h inclusions. The script emitted errors for ~400
files.
2. Each error was manually checked. Some didn't need the inclusion,
some needed manual addition while adding it to implementation .h or
embedding .c file was more appropriate for others. This step added
inclusions to around 150 files.
3. The script was run again and the output was compared to the edits
from #2 to make sure no file was left behind.
4. Several build tests were done and a couple of problems were fixed.
e.g. lib/decompress_*.c used malloc/free() wrappers around slab
APIs requiring slab.h to be added manually.
5. The script was run on all .h files but without automatically
editing them as sprinkling gfp.h and slab.h inclusions around .h
files could easily lead to inclusion dependency hell. Most gfp.h
inclusion directives were ignored as stuff from gfp.h was usually
wildly available and often used in preprocessor macros. Each
slab.h inclusion directive was examined and added manually as
necessary.
6. percpu.h was updated not to include slab.h.
7. Build test were done on the following configurations and failures
were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my
distributed build env didn't work with gcov compiles) and a few
more options had to be turned off depending on archs to make things
build (like ipr on powerpc/64 which failed due to missing writeq).
* x86 and x86_64 UP and SMP allmodconfig and a custom test config.
* powerpc and powerpc64 SMP allmodconfig
* sparc and sparc64 SMP allmodconfig
* ia64 SMP allmodconfig
* s390 SMP allmodconfig
* alpha SMP allmodconfig
* um on x86_64 SMP allmodconfig
8. percpu.h modifications were reverted so that it could be applied as
a separate patch and serve as bisection point.
Given the fact that I had only a couple of failures from tests on step
6, I'm fairly confident about the coverage of this conversion patch.
If there is a breakage, it's likely to be something in one of the arch
headers which should be easily discoverable easily on most builds of
the specific arch.
Signed-off-by: Tejun Heo <tj@kernel.org>
Guess-its-ok-by: Christoph Lameter <cl@linux-foundation.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Lee Schermerhorn <Lee.Schermerhorn@hp.com>
2010-03-24 16:04:11 +08:00
|
|
|
#include <linux/slab.h>
|
2012-03-08 13:55:59 +08:00
|
|
|
#include <net/sctp/sctp.h>
|
2009-01-23 05:24:49 +08:00
|
|
|
#include <net/ipv6.h>
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2021-11-03 03:17:16 +08:00
|
|
|
#include <trace/events/dlm.h>
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
#include "dlm_internal.h"
|
|
|
|
#include "lowcomms.h"
|
|
|
|
#include "midcomms.h"
|
2021-12-01 03:47:19 +08:00
|
|
|
#include "memory.h"
|
2006-11-03 00:19:21 +08:00
|
|
|
#include "config.h"
|
|
|
|
|
2007-04-17 22:39:57 +08:00
|
|
|
#define NEEDED_RMEM (4*1024*1024)
|
|
|
|
|
2010-11-13 01:15:20 +08:00
|
|
|
/* Number of messages to send before rescheduling */
|
|
|
|
#define MAX_SEND_MSG_COUNT 25
|
2020-07-27 21:13:38 +08:00
|
|
|
#define DLM_SHUTDOWN_WAIT_TIMEOUT msecs_to_jiffies(10000)
|
2010-11-13 01:15:20 +08:00
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
struct connection {
|
|
|
|
struct socket *sock; /* NULL if not connected */
|
|
|
|
uint32_t nodeid; /* So we know who we are in the list */
|
2007-01-24 19:17:59 +08:00
|
|
|
struct mutex sock_mutex;
|
2007-04-17 22:39:57 +08:00
|
|
|
unsigned long flags;
|
2006-11-03 00:19:21 +08:00
|
|
|
#define CF_READ_PENDING 1
|
2017-09-12 17:01:16 +08:00
|
|
|
#define CF_WRITE_PENDING 2
|
2007-04-17 22:39:57 +08:00
|
|
|
#define CF_INIT_PENDING 4
|
|
|
|
#define CF_IS_OTHERCON 5
|
2009-08-12 05:18:23 +08:00
|
|
|
#define CF_CLOSE 6
|
2010-11-11 13:56:39 +08:00
|
|
|
#define CF_APP_LIMITED 7
|
2017-09-12 16:55:50 +08:00
|
|
|
#define CF_CLOSING 8
|
2020-07-27 21:13:38 +08:00
|
|
|
#define CF_SHUTDOWN 9
|
2020-11-03 09:04:20 +08:00
|
|
|
#define CF_CONNECTED 10
|
2021-05-22 03:08:37 +08:00
|
|
|
#define CF_RECONNECT 11
|
|
|
|
#define CF_DELAY_CONNECT 12
|
2021-05-22 03:08:39 +08:00
|
|
|
#define CF_EOF 13
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
struct list_head writequeue; /* List of outgoing writequeue_entries */
|
2006-11-03 00:19:21 +08:00
|
|
|
spinlock_t writequeue_lock;
|
2021-05-22 03:08:39 +08:00
|
|
|
atomic_t writequeue_cnt;
|
2006-11-03 00:19:21 +08:00
|
|
|
int retries;
|
|
|
|
#define MAX_CONNECT_RETRIES 3
|
2009-01-29 02:57:40 +08:00
|
|
|
struct hlist_node list;
|
2006-11-03 00:19:21 +08:00
|
|
|
struct connection *othercon;
|
2021-05-22 03:08:37 +08:00
|
|
|
struct connection *sendcon;
|
2007-01-15 22:33:34 +08:00
|
|
|
struct work_struct rwork; /* Receive workqueue */
|
|
|
|
struct work_struct swork; /* Send workqueue */
|
2020-07-27 21:13:38 +08:00
|
|
|
wait_queue_head_t shutdown_wait; /* wait for graceful shutdown */
|
2020-09-24 22:31:26 +08:00
|
|
|
unsigned char *rx_buf;
|
|
|
|
int rx_buflen;
|
|
|
|
int rx_leftover;
|
2020-08-28 03:02:49 +08:00
|
|
|
struct rcu_head rcu;
|
2006-11-03 00:19:21 +08:00
|
|
|
};
|
|
|
|
#define sock2con(x) ((struct connection *)(x)->sk_user_data)
|
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
struct listen_connection {
|
|
|
|
struct socket *sock;
|
|
|
|
struct work_struct rwork;
|
|
|
|
};
|
|
|
|
|
2021-03-02 06:05:16 +08:00
|
|
|
#define DLM_WQ_REMAIN_BYTES(e) (PAGE_SIZE - e->end)
|
|
|
|
#define DLM_WQ_LENGTH_BYTES(e) (e->end - e->offset)
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
/* An entry waiting to be sent */
|
|
|
|
struct writequeue_entry {
|
|
|
|
struct list_head list;
|
|
|
|
struct page *page;
|
|
|
|
int offset;
|
|
|
|
int len;
|
|
|
|
int end;
|
|
|
|
int users;
|
2021-05-22 03:08:48 +08:00
|
|
|
bool dirty;
|
2006-11-03 00:19:21 +08:00
|
|
|
struct connection *con;
|
2021-05-22 03:08:42 +08:00
|
|
|
struct list_head msgs;
|
|
|
|
struct kref ref;
|
|
|
|
};
|
|
|
|
|
|
|
|
struct dlm_msg {
|
|
|
|
struct writequeue_entry *entry;
|
2021-05-22 03:08:43 +08:00
|
|
|
struct dlm_msg *orig_msg;
|
|
|
|
bool retransmit;
|
2021-05-22 03:08:42 +08:00
|
|
|
void *ppc;
|
|
|
|
int len;
|
|
|
|
int idx; /* new()/commit() idx exchange */
|
|
|
|
|
|
|
|
struct list_head list;
|
|
|
|
struct kref ref;
|
2006-11-03 00:19:21 +08:00
|
|
|
};
|
|
|
|
|
2012-07-27 01:44:30 +08:00
|
|
|
struct dlm_node_addr {
|
|
|
|
struct list_head list;
|
|
|
|
int nodeid;
|
2021-03-02 06:05:09 +08:00
|
|
|
int mark;
|
2012-07-27 01:44:30 +08:00
|
|
|
int addr_count;
|
2013-06-14 17:56:12 +08:00
|
|
|
int curr_addr_index;
|
2012-07-27 01:44:30 +08:00
|
|
|
struct sockaddr_storage *addr[DLM_MAX_ADDR_COUNT];
|
|
|
|
};
|
|
|
|
|
2021-07-17 04:22:40 +08:00
|
|
|
struct dlm_proto_ops {
|
2021-07-17 04:22:43 +08:00
|
|
|
bool try_new_addr;
|
2021-07-17 04:22:41 +08:00
|
|
|
const char *name;
|
|
|
|
int proto;
|
|
|
|
|
2021-07-17 04:22:43 +08:00
|
|
|
int (*connect)(struct connection *con, struct socket *sock,
|
|
|
|
struct sockaddr *addr, int addr_len);
|
|
|
|
void (*sockopts)(struct socket *sock);
|
|
|
|
int (*bind)(struct socket *sock);
|
2021-07-17 04:22:41 +08:00
|
|
|
int (*listen_validate)(void);
|
|
|
|
void (*listen_sockopts)(struct socket *sock);
|
|
|
|
int (*listen_bind)(struct socket *sock);
|
2021-07-17 04:22:40 +08:00
|
|
|
/* What to do to shutdown */
|
|
|
|
void (*shutdown_action)(struct connection *con);
|
|
|
|
/* What to do to eof check */
|
|
|
|
bool (*eof_condition)(struct connection *con);
|
|
|
|
};
|
|
|
|
|
DLM: Fix saving of NULL callbacks
In a previous patch I noted that accept() often copies the struct
sock (sk) which overwrites the sock callbacks. However, in testing
we discovered that the dlm connection structures (con) are sometimes
deleted and recreated as connections come and go, and since they're
zeroed out by kmem_cache_zalloc, the saved callback pointers are
also initialized to zero. But with today's DLM code, the callbacks
are only saved when a socket is added.
During recovery testing, we discovered a common situation in which
the new con is initialized to zero, then a socket is added after
accept(). In this case, the sock's saved values are all NULL, but
the saved values are wiped out, due to accept(). Therefore, we
don't have a known good copy of the callbacks from which we can
restore.
Since the struct sock callbacks are always good after listen(),
this patch saves the known good values after listen(). These good
values are then used for subsequent restores.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Reviewed-by: Tadashi Miyauchi <miyauchi@toshiba-tops.co.jp>
Signed-off-by: David Teigland <teigland@redhat.com>
2017-09-12 16:55:23 +08:00
|
|
|
static struct listen_sock_callbacks {
|
|
|
|
void (*sk_error_report)(struct sock *);
|
|
|
|
void (*sk_data_ready)(struct sock *);
|
|
|
|
void (*sk_state_change)(struct sock *);
|
|
|
|
void (*sk_write_space)(struct sock *);
|
|
|
|
} listen_sock;
|
|
|
|
|
2012-07-27 01:44:30 +08:00
|
|
|
static LIST_HEAD(dlm_node_addrs);
|
|
|
|
static DEFINE_SPINLOCK(dlm_node_addrs_spin);
|
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
static struct listen_connection listen_con;
|
2007-04-17 22:39:57 +08:00
|
|
|
static struct sockaddr_storage *dlm_local_addr[DLM_MAX_ADDR_COUNT];
|
|
|
|
static int dlm_local_count;
|
2021-03-02 06:05:13 +08:00
|
|
|
int dlm_allow_conn;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2007-01-15 22:33:34 +08:00
|
|
|
/* Work queues */
|
|
|
|
static struct workqueue_struct *recv_workqueue;
|
|
|
|
static struct workqueue_struct *send_workqueue;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2009-01-29 02:57:40 +08:00
|
|
|
static struct hlist_head connection_hash[CONN_HASH_SIZE];
|
2020-08-28 03:02:49 +08:00
|
|
|
static DEFINE_SPINLOCK(connections_lock);
|
|
|
|
DEFINE_STATIC_SRCU(connections_srcu);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2021-07-17 04:22:40 +08:00
|
|
|
static const struct dlm_proto_ops *dlm_proto_ops;
|
|
|
|
|
2007-01-15 22:33:34 +08:00
|
|
|
static void process_recv_sockets(struct work_struct *work);
|
|
|
|
static void process_send_sockets(struct work_struct *work);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2021-12-01 03:47:19 +08:00
|
|
|
static void writequeue_entry_ctor(void *data)
|
|
|
|
{
|
|
|
|
struct writequeue_entry *entry = data;
|
|
|
|
|
|
|
|
INIT_LIST_HEAD(&entry->msgs);
|
|
|
|
}
|
|
|
|
|
|
|
|
struct kmem_cache *dlm_lowcomms_writequeue_cache_create(void)
|
|
|
|
{
|
|
|
|
return kmem_cache_create("dlm_writequeue", sizeof(struct writequeue_entry),
|
|
|
|
0, 0, writequeue_entry_ctor);
|
|
|
|
}
|
|
|
|
|
2021-12-01 03:47:20 +08:00
|
|
|
struct kmem_cache *dlm_lowcomms_msg_cache_create(void)
|
|
|
|
{
|
|
|
|
return kmem_cache_create("dlm_msg", sizeof(struct dlm_msg), 0, 0, NULL);
|
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:39 +08:00
|
|
|
/* need to held writequeue_lock */
|
|
|
|
static struct writequeue_entry *con_next_wq(struct connection *con)
|
|
|
|
{
|
|
|
|
struct writequeue_entry *e;
|
|
|
|
|
|
|
|
if (list_empty(&con->writequeue))
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
e = list_first_entry(&con->writequeue, struct writequeue_entry,
|
|
|
|
list);
|
2021-12-01 03:47:15 +08:00
|
|
|
/* if len is zero nothing is to send, if there are users filling
|
|
|
|
* buffers we wait until the users are done so we can send more.
|
|
|
|
*/
|
|
|
|
if (e->users || e->len == 0)
|
2021-07-17 04:22:39 +08:00
|
|
|
return NULL;
|
|
|
|
|
|
|
|
return e;
|
|
|
|
}
|
|
|
|
|
2021-05-22 03:08:35 +08:00
|
|
|
static struct connection *__find_con(int nodeid, int r)
|
2009-01-29 02:57:40 +08:00
|
|
|
{
|
|
|
|
struct connection *con;
|
|
|
|
|
2020-08-28 03:02:49 +08:00
|
|
|
hlist_for_each_entry_rcu(con, &connection_hash[r], list) {
|
2021-05-22 03:08:35 +08:00
|
|
|
if (con->nodeid == nodeid)
|
2009-01-29 02:57:40 +08:00
|
|
|
return con;
|
|
|
|
}
|
2020-08-28 03:02:49 +08:00
|
|
|
|
2009-01-29 02:57:40 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2021-05-22 03:08:39 +08:00
|
|
|
static bool tcp_eof_condition(struct connection *con)
|
|
|
|
{
|
|
|
|
return atomic_read(&con->writequeue_cnt);
|
|
|
|
}
|
|
|
|
|
2020-11-03 09:04:21 +08:00
|
|
|
static int dlm_con_init(struct connection *con, int nodeid)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2020-09-24 22:31:26 +08:00
|
|
|
con->rx_buflen = dlm_config.ci_buffer_size;
|
|
|
|
con->rx_buf = kmalloc(con->rx_buflen, GFP_NOFS);
|
2020-11-03 09:04:21 +08:00
|
|
|
if (!con->rx_buf)
|
|
|
|
return -ENOMEM;
|
2020-09-24 22:31:26 +08:00
|
|
|
|
2007-04-17 22:39:57 +08:00
|
|
|
con->nodeid = nodeid;
|
|
|
|
mutex_init(&con->sock_mutex);
|
|
|
|
INIT_LIST_HEAD(&con->writequeue);
|
|
|
|
spin_lock_init(&con->writequeue_lock);
|
2021-05-22 03:08:39 +08:00
|
|
|
atomic_set(&con->writequeue_cnt, 0);
|
2007-04-17 22:39:57 +08:00
|
|
|
INIT_WORK(&con->swork, process_send_sockets);
|
|
|
|
INIT_WORK(&con->rwork, process_recv_sockets);
|
2020-07-27 21:13:38 +08:00
|
|
|
init_waitqueue_head(&con->shutdown_wait);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2020-11-03 09:04:21 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* If 'allocation' is zero then we don't attempt to create a new
|
|
|
|
* connection structure for this node.
|
|
|
|
*/
|
|
|
|
static struct connection *nodeid2con(int nodeid, gfp_t alloc)
|
|
|
|
{
|
|
|
|
struct connection *con, *tmp;
|
|
|
|
int r, ret;
|
|
|
|
|
2021-05-22 03:08:35 +08:00
|
|
|
r = nodeid_hash(nodeid);
|
|
|
|
con = __find_con(nodeid, r);
|
2020-11-03 09:04:21 +08:00
|
|
|
if (con || !alloc)
|
|
|
|
return con;
|
|
|
|
|
|
|
|
con = kzalloc(sizeof(*con), alloc);
|
|
|
|
if (!con)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
ret = dlm_con_init(con, nodeid);
|
|
|
|
if (ret) {
|
|
|
|
kfree(con);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2020-08-28 03:02:49 +08:00
|
|
|
spin_lock(&connections_lock);
|
2020-10-01 06:37:29 +08:00
|
|
|
/* Because multiple workqueues/threads calls this function it can
|
|
|
|
* race on multiple cpu's. Instead of locking hot path __find_con()
|
|
|
|
* we just check in rare cases of recently added nodes again
|
|
|
|
* under protection of connections_lock. If this is the case we
|
|
|
|
* abort our connection creation and return the existing connection.
|
|
|
|
*/
|
2021-05-22 03:08:35 +08:00
|
|
|
tmp = __find_con(nodeid, r);
|
2020-10-01 06:37:29 +08:00
|
|
|
if (tmp) {
|
|
|
|
spin_unlock(&connections_lock);
|
|
|
|
kfree(con->rx_buf);
|
|
|
|
kfree(con);
|
|
|
|
return tmp;
|
|
|
|
}
|
|
|
|
|
2020-08-28 03:02:49 +08:00
|
|
|
hlist_add_head_rcu(&con->list, &connection_hash[r]);
|
|
|
|
spin_unlock(&connections_lock);
|
|
|
|
|
2007-04-17 22:39:57 +08:00
|
|
|
return con;
|
|
|
|
}
|
|
|
|
|
2009-01-29 02:57:40 +08:00
|
|
|
/* Loop round all connections */
|
|
|
|
static void foreach_conn(void (*conn_func)(struct connection *c))
|
|
|
|
{
|
2021-05-22 03:08:35 +08:00
|
|
|
int i;
|
2009-01-29 02:57:40 +08:00
|
|
|
struct connection *con;
|
|
|
|
|
|
|
|
for (i = 0; i < CONN_HASH_SIZE; i++) {
|
2020-08-28 03:02:49 +08:00
|
|
|
hlist_for_each_entry_rcu(con, &connection_hash[i], list)
|
2009-01-29 02:57:40 +08:00
|
|
|
conn_func(con);
|
|
|
|
}
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2012-07-27 01:44:30 +08:00
|
|
|
static struct dlm_node_addr *find_node_addr(int nodeid)
|
|
|
|
{
|
|
|
|
struct dlm_node_addr *na;
|
|
|
|
|
|
|
|
list_for_each_entry(na, &dlm_node_addrs, list) {
|
|
|
|
if (na->nodeid == nodeid)
|
|
|
|
return na;
|
|
|
|
}
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2020-11-03 09:04:27 +08:00
|
|
|
static int addr_compare(const struct sockaddr_storage *x,
|
|
|
|
const struct sockaddr_storage *y)
|
2007-04-17 22:39:57 +08:00
|
|
|
{
|
2012-07-27 01:44:30 +08:00
|
|
|
switch (x->ss_family) {
|
|
|
|
case AF_INET: {
|
|
|
|
struct sockaddr_in *sinx = (struct sockaddr_in *)x;
|
|
|
|
struct sockaddr_in *siny = (struct sockaddr_in *)y;
|
|
|
|
if (sinx->sin_addr.s_addr != siny->sin_addr.s_addr)
|
|
|
|
return 0;
|
|
|
|
if (sinx->sin_port != siny->sin_port)
|
|
|
|
return 0;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
case AF_INET6: {
|
|
|
|
struct sockaddr_in6 *sinx = (struct sockaddr_in6 *)x;
|
|
|
|
struct sockaddr_in6 *siny = (struct sockaddr_in6 *)y;
|
|
|
|
if (!ipv6_addr_equal(&sinx->sin6_addr, &siny->sin6_addr))
|
|
|
|
return 0;
|
|
|
|
if (sinx->sin6_port != siny->sin6_port)
|
|
|
|
return 0;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
default:
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int nodeid_to_addr(int nodeid, struct sockaddr_storage *sas_out,
|
2021-03-02 06:05:09 +08:00
|
|
|
struct sockaddr *sa_out, bool try_new_addr,
|
|
|
|
unsigned int *mark)
|
2012-07-27 01:44:30 +08:00
|
|
|
{
|
|
|
|
struct sockaddr_storage sas;
|
|
|
|
struct dlm_node_addr *na;
|
2007-04-17 22:39:57 +08:00
|
|
|
|
|
|
|
if (!dlm_local_count)
|
|
|
|
return -1;
|
|
|
|
|
2012-07-27 01:44:30 +08:00
|
|
|
spin_lock(&dlm_node_addrs_spin);
|
|
|
|
na = find_node_addr(nodeid);
|
2013-06-14 17:56:12 +08:00
|
|
|
if (na && na->addr_count) {
|
2015-08-12 06:22:23 +08:00
|
|
|
memcpy(&sas, na->addr[na->curr_addr_index],
|
|
|
|
sizeof(struct sockaddr_storage));
|
|
|
|
|
2013-06-14 17:56:12 +08:00
|
|
|
if (try_new_addr) {
|
|
|
|
na->curr_addr_index++;
|
|
|
|
if (na->curr_addr_index == na->addr_count)
|
|
|
|
na->curr_addr_index = 0;
|
|
|
|
}
|
|
|
|
}
|
2012-07-27 01:44:30 +08:00
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
|
|
|
|
if (!na)
|
|
|
|
return -EEXIST;
|
|
|
|
|
|
|
|
if (!na->addr_count)
|
|
|
|
return -ENOENT;
|
|
|
|
|
2021-03-02 06:05:09 +08:00
|
|
|
*mark = na->mark;
|
|
|
|
|
2012-07-27 01:44:30 +08:00
|
|
|
if (sas_out)
|
|
|
|
memcpy(sas_out, &sas, sizeof(struct sockaddr_storage));
|
|
|
|
|
|
|
|
if (!sa_out)
|
|
|
|
return 0;
|
2007-04-17 22:39:57 +08:00
|
|
|
|
|
|
|
if (dlm_local_addr[0]->ss_family == AF_INET) {
|
2012-07-27 01:44:30 +08:00
|
|
|
struct sockaddr_in *in4 = (struct sockaddr_in *) &sas;
|
|
|
|
struct sockaddr_in *ret4 = (struct sockaddr_in *) sa_out;
|
2007-04-17 22:39:57 +08:00
|
|
|
ret4->sin_addr.s_addr = in4->sin_addr.s_addr;
|
|
|
|
} else {
|
2012-07-27 01:44:30 +08:00
|
|
|
struct sockaddr_in6 *in6 = (struct sockaddr_in6 *) &sas;
|
|
|
|
struct sockaddr_in6 *ret6 = (struct sockaddr_in6 *) sa_out;
|
2011-11-21 11:39:03 +08:00
|
|
|
ret6->sin6_addr = in6->sin6_addr;
|
2007-04-17 22:39:57 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-03-02 06:05:09 +08:00
|
|
|
static int addr_to_nodeid(struct sockaddr_storage *addr, int *nodeid,
|
|
|
|
unsigned int *mark)
|
2012-07-27 01:44:30 +08:00
|
|
|
{
|
|
|
|
struct dlm_node_addr *na;
|
|
|
|
int rv = -EEXIST;
|
2013-06-14 17:56:12 +08:00
|
|
|
int addr_i;
|
2012-07-27 01:44:30 +08:00
|
|
|
|
|
|
|
spin_lock(&dlm_node_addrs_spin);
|
|
|
|
list_for_each_entry(na, &dlm_node_addrs, list) {
|
|
|
|
if (!na->addr_count)
|
|
|
|
continue;
|
|
|
|
|
2013-06-14 17:56:12 +08:00
|
|
|
for (addr_i = 0; addr_i < na->addr_count; addr_i++) {
|
|
|
|
if (addr_compare(na->addr[addr_i], addr)) {
|
|
|
|
*nodeid = na->nodeid;
|
2021-03-02 06:05:09 +08:00
|
|
|
*mark = na->mark;
|
2013-06-14 17:56:12 +08:00
|
|
|
rv = 0;
|
|
|
|
goto unlock;
|
|
|
|
}
|
|
|
|
}
|
2012-07-27 01:44:30 +08:00
|
|
|
}
|
2013-06-14 17:56:12 +08:00
|
|
|
unlock:
|
2012-07-27 01:44:30 +08:00
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
return rv;
|
|
|
|
}
|
|
|
|
|
2020-11-03 09:04:28 +08:00
|
|
|
/* caller need to held dlm_node_addrs_spin lock */
|
|
|
|
static bool dlm_lowcomms_na_has_addr(const struct dlm_node_addr *na,
|
|
|
|
const struct sockaddr_storage *addr)
|
|
|
|
{
|
|
|
|
int i;
|
|
|
|
|
|
|
|
for (i = 0; i < na->addr_count; i++) {
|
|
|
|
if (addr_compare(na->addr[i], addr))
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2012-07-27 01:44:30 +08:00
|
|
|
int dlm_lowcomms_addr(int nodeid, struct sockaddr_storage *addr, int len)
|
|
|
|
{
|
|
|
|
struct sockaddr_storage *new_addr;
|
|
|
|
struct dlm_node_addr *new_node, *na;
|
2020-11-03 09:04:28 +08:00
|
|
|
bool ret;
|
2012-07-27 01:44:30 +08:00
|
|
|
|
|
|
|
new_node = kzalloc(sizeof(struct dlm_node_addr), GFP_NOFS);
|
|
|
|
if (!new_node)
|
|
|
|
return -ENOMEM;
|
|
|
|
|
|
|
|
new_addr = kzalloc(sizeof(struct sockaddr_storage), GFP_NOFS);
|
|
|
|
if (!new_addr) {
|
|
|
|
kfree(new_node);
|
|
|
|
return -ENOMEM;
|
|
|
|
}
|
|
|
|
|
|
|
|
memcpy(new_addr, addr, len);
|
|
|
|
|
|
|
|
spin_lock(&dlm_node_addrs_spin);
|
|
|
|
na = find_node_addr(nodeid);
|
|
|
|
if (!na) {
|
|
|
|
new_node->nodeid = nodeid;
|
|
|
|
new_node->addr[0] = new_addr;
|
|
|
|
new_node->addr_count = 1;
|
2021-03-02 06:05:09 +08:00
|
|
|
new_node->mark = dlm_config.ci_mark;
|
2012-07-27 01:44:30 +08:00
|
|
|
list_add(&new_node->list, &dlm_node_addrs);
|
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2020-11-03 09:04:28 +08:00
|
|
|
ret = dlm_lowcomms_na_has_addr(na, addr);
|
|
|
|
if (ret) {
|
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
kfree(new_addr);
|
|
|
|
kfree(new_node);
|
|
|
|
return -EEXIST;
|
|
|
|
}
|
|
|
|
|
2012-07-27 01:44:30 +08:00
|
|
|
if (na->addr_count >= DLM_MAX_ADDR_COUNT) {
|
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
kfree(new_addr);
|
|
|
|
kfree(new_node);
|
|
|
|
return -ENOSPC;
|
|
|
|
}
|
|
|
|
|
|
|
|
na->addr[na->addr_count++] = new_addr;
|
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
kfree(new_node);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
/* Data available on socket or listen socket received a connect */
|
2014-04-12 04:15:36 +08:00
|
|
|
static void lowcomms_data_ready(struct sock *sk)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2017-09-12 17:01:55 +08:00
|
|
|
struct connection *con;
|
|
|
|
|
|
|
|
con = sock2con(sk);
|
2007-06-01 23:07:26 +08:00
|
|
|
if (con && !test_and_set_bit(CF_READ_PENDING, &con->flags))
|
2007-01-15 22:33:34 +08:00
|
|
|
queue_work(recv_workqueue, &con->rwork);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
static void lowcomms_listen_data_ready(struct sock *sk)
|
|
|
|
{
|
2021-06-02 21:45:18 +08:00
|
|
|
if (!dlm_allow_conn)
|
|
|
|
return;
|
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
queue_work(recv_workqueue, &listen_con.rwork);
|
|
|
|
}
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
static void lowcomms_write_space(struct sock *sk)
|
|
|
|
{
|
2017-09-12 17:01:55 +08:00
|
|
|
struct connection *con;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2017-09-12 17:01:55 +08:00
|
|
|
con = sock2con(sk);
|
2010-11-11 13:56:39 +08:00
|
|
|
if (!con)
|
2021-11-15 21:57:06 +08:00
|
|
|
return;
|
2010-11-11 13:56:39 +08:00
|
|
|
|
2020-11-03 09:04:20 +08:00
|
|
|
if (!test_and_set_bit(CF_CONNECTED, &con->flags)) {
|
|
|
|
log_print("successful connected to node %d", con->nodeid);
|
|
|
|
queue_work(send_workqueue, &con->swork);
|
2021-11-15 21:57:06 +08:00
|
|
|
return;
|
2020-11-03 09:04:20 +08:00
|
|
|
}
|
|
|
|
|
2010-11-11 13:56:39 +08:00
|
|
|
clear_bit(SOCK_NOSPACE, &con->sock->flags);
|
|
|
|
|
|
|
|
if (test_and_clear_bit(CF_APP_LIMITED, &con->flags)) {
|
|
|
|
con->sock->sk->sk_write_pending--;
|
2015-11-30 12:03:10 +08:00
|
|
|
clear_bit(SOCKWQ_ASYNC_NOSPACE, &con->sock->flags);
|
2010-11-11 13:56:39 +08:00
|
|
|
}
|
|
|
|
|
2017-09-12 16:55:14 +08:00
|
|
|
queue_work(send_workqueue, &con->swork);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static inline void lowcomms_connect_sock(struct connection *con)
|
|
|
|
{
|
2009-08-12 05:18:23 +08:00
|
|
|
if (test_bit(CF_CLOSE, &con->flags))
|
|
|
|
return;
|
2017-09-12 16:55:04 +08:00
|
|
|
queue_work(send_workqueue, &con->swork);
|
|
|
|
cond_resched();
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static void lowcomms_state_change(struct sock *sk)
|
|
|
|
{
|
2015-08-12 06:22:23 +08:00
|
|
|
/* SCTP layer is not calling sk_data_ready when the connection
|
|
|
|
* is done, so we catch the signal through here. Also, it
|
|
|
|
* doesn't switch socket state when entering shutdown, so we
|
|
|
|
* skip the write in that case.
|
|
|
|
*/
|
|
|
|
if (sk->sk_shutdown) {
|
|
|
|
if (sk->sk_shutdown == RCV_SHUTDOWN)
|
|
|
|
lowcomms_data_ready(sk);
|
|
|
|
} else if (sk->sk_state == TCP_ESTABLISHED) {
|
2006-11-03 00:19:21 +08:00
|
|
|
lowcomms_write_space(sk);
|
2015-08-12 06:22:23 +08:00
|
|
|
}
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2009-05-07 23:54:16 +08:00
|
|
|
int dlm_lowcomms_connect_node(int nodeid)
|
|
|
|
{
|
|
|
|
struct connection *con;
|
2021-05-22 03:08:35 +08:00
|
|
|
int idx;
|
2009-05-07 23:54:16 +08:00
|
|
|
|
|
|
|
if (nodeid == dlm_our_nodeid())
|
|
|
|
return 0;
|
|
|
|
|
2021-05-22 03:08:35 +08:00
|
|
|
idx = srcu_read_lock(&connections_srcu);
|
2009-05-07 23:54:16 +08:00
|
|
|
con = nodeid2con(nodeid, GFP_NOFS);
|
2021-05-22 03:08:35 +08:00
|
|
|
if (!con) {
|
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
2009-05-07 23:54:16 +08:00
|
|
|
return -ENOMEM;
|
2021-05-22 03:08:35 +08:00
|
|
|
}
|
|
|
|
|
2009-05-07 23:54:16 +08:00
|
|
|
lowcomms_connect_sock(con);
|
2021-05-22 03:08:35 +08:00
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
|
|
|
|
2009-05-07 23:54:16 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-03-02 06:05:09 +08:00
|
|
|
int dlm_lowcomms_nodes_set_mark(int nodeid, unsigned int mark)
|
|
|
|
{
|
|
|
|
struct dlm_node_addr *na;
|
|
|
|
|
|
|
|
spin_lock(&dlm_node_addrs_spin);
|
|
|
|
na = find_node_addr(nodeid);
|
|
|
|
if (!na) {
|
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
return -ENOENT;
|
|
|
|
}
|
|
|
|
|
|
|
|
na->mark = mark;
|
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2015-08-27 22:34:47 +08:00
|
|
|
static void lowcomms_error_report(struct sock *sk)
|
|
|
|
{
|
2016-02-06 03:39:02 +08:00
|
|
|
struct connection *con;
|
|
|
|
void (*orig_report)(struct sock *) = NULL;
|
2021-11-15 21:57:05 +08:00
|
|
|
struct inet_sock *inet;
|
2015-08-27 22:34:47 +08:00
|
|
|
|
2016-02-06 03:39:02 +08:00
|
|
|
con = sock2con(sk);
|
|
|
|
if (con == NULL)
|
|
|
|
goto out;
|
|
|
|
|
DLM: Fix saving of NULL callbacks
In a previous patch I noted that accept() often copies the struct
sock (sk) which overwrites the sock callbacks. However, in testing
we discovered that the dlm connection structures (con) are sometimes
deleted and recreated as connections come and go, and since they're
zeroed out by kmem_cache_zalloc, the saved callback pointers are
also initialized to zero. But with today's DLM code, the callbacks
are only saved when a socket is added.
During recovery testing, we discovered a common situation in which
the new con is initialized to zero, then a socket is added after
accept(). In this case, the sock's saved values are all NULL, but
the saved values are wiped out, due to accept(). Therefore, we
don't have a known good copy of the callbacks from which we can
restore.
Since the struct sock callbacks are always good after listen(),
this patch saves the known good values after listen(). These good
values are then used for subsequent restores.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Reviewed-by: Tadashi Miyauchi <miyauchi@toshiba-tops.co.jp>
Signed-off-by: David Teigland <teigland@redhat.com>
2017-09-12 16:55:23 +08:00
|
|
|
orig_report = listen_sock.sk_error_report;
|
2015-08-27 22:34:47 +08:00
|
|
|
|
2021-11-15 21:57:05 +08:00
|
|
|
inet = inet_sk(sk);
|
|
|
|
switch (sk->sk_family) {
|
|
|
|
case AF_INET:
|
2015-08-27 22:34:47 +08:00
|
|
|
printk_ratelimited(KERN_ERR "dlm: node %d: socket error "
|
2021-11-15 21:57:05 +08:00
|
|
|
"sending to node %d at %pI4, dport %d, "
|
2015-08-27 22:34:47 +08:00
|
|
|
"sk_err=%d/%d\n", dlm_our_nodeid(),
|
2021-11-15 21:57:05 +08:00
|
|
|
con->nodeid, &inet->inet_daddr,
|
|
|
|
ntohs(inet->inet_dport), sk->sk_err,
|
2015-08-27 22:34:47 +08:00
|
|
|
sk->sk_err_soft);
|
2021-11-15 21:57:05 +08:00
|
|
|
break;
|
2021-11-17 22:20:43 +08:00
|
|
|
#if IS_ENABLED(CONFIG_IPV6)
|
2021-11-15 21:57:05 +08:00
|
|
|
case AF_INET6:
|
2015-08-27 22:34:47 +08:00
|
|
|
printk_ratelimited(KERN_ERR "dlm: node %d: socket error "
|
2021-11-15 21:57:05 +08:00
|
|
|
"sending to node %d at %pI6c, "
|
|
|
|
"dport %d, sk_err=%d/%d\n", dlm_our_nodeid(),
|
|
|
|
con->nodeid, &sk->sk_v6_daddr,
|
|
|
|
ntohs(inet->inet_dport), sk->sk_err,
|
2015-08-27 22:34:47 +08:00
|
|
|
sk->sk_err_soft);
|
2021-11-15 21:57:05 +08:00
|
|
|
break;
|
2021-11-17 22:20:43 +08:00
|
|
|
#endif
|
2021-11-15 21:57:05 +08:00
|
|
|
default:
|
|
|
|
printk_ratelimited(KERN_ERR "dlm: node %d: socket error "
|
|
|
|
"invalid socket family %d set, "
|
|
|
|
"sk_err=%d/%d\n", dlm_our_nodeid(),
|
|
|
|
sk->sk_family, sk->sk_err, sk->sk_err_soft);
|
|
|
|
goto out;
|
2015-08-27 22:34:47 +08:00
|
|
|
}
|
2021-05-22 03:08:37 +08:00
|
|
|
|
|
|
|
/* below sendcon only handling */
|
|
|
|
if (test_bit(CF_IS_OTHERCON, &con->flags))
|
|
|
|
con = con->sendcon;
|
|
|
|
|
|
|
|
switch (sk->sk_err) {
|
|
|
|
case ECONNREFUSED:
|
|
|
|
set_bit(CF_DELAY_CONNECT, &con->flags);
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!test_and_set_bit(CF_RECONNECT, &con->flags))
|
|
|
|
queue_work(send_workqueue, &con->swork);
|
|
|
|
|
2016-02-06 03:39:02 +08:00
|
|
|
out:
|
|
|
|
if (orig_report)
|
|
|
|
orig_report(sk);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Note: sk_callback_lock must be locked before calling this function. */
|
DLM: Fix saving of NULL callbacks
In a previous patch I noted that accept() often copies the struct
sock (sk) which overwrites the sock callbacks. However, in testing
we discovered that the dlm connection structures (con) are sometimes
deleted and recreated as connections come and go, and since they're
zeroed out by kmem_cache_zalloc, the saved callback pointers are
also initialized to zero. But with today's DLM code, the callbacks
are only saved when a socket is added.
During recovery testing, we discovered a common situation in which
the new con is initialized to zero, then a socket is added after
accept(). In this case, the sock's saved values are all NULL, but
the saved values are wiped out, due to accept(). Therefore, we
don't have a known good copy of the callbacks from which we can
restore.
Since the struct sock callbacks are always good after listen(),
this patch saves the known good values after listen(). These good
values are then used for subsequent restores.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Reviewed-by: Tadashi Miyauchi <miyauchi@toshiba-tops.co.jp>
Signed-off-by: David Teigland <teigland@redhat.com>
2017-09-12 16:55:23 +08:00
|
|
|
static void save_listen_callbacks(struct socket *sock)
|
2016-02-06 03:39:02 +08:00
|
|
|
{
|
DLM: Fix saving of NULL callbacks
In a previous patch I noted that accept() often copies the struct
sock (sk) which overwrites the sock callbacks. However, in testing
we discovered that the dlm connection structures (con) are sometimes
deleted and recreated as connections come and go, and since they're
zeroed out by kmem_cache_zalloc, the saved callback pointers are
also initialized to zero. But with today's DLM code, the callbacks
are only saved when a socket is added.
During recovery testing, we discovered a common situation in which
the new con is initialized to zero, then a socket is added after
accept(). In this case, the sock's saved values are all NULL, but
the saved values are wiped out, due to accept(). Therefore, we
don't have a known good copy of the callbacks from which we can
restore.
Since the struct sock callbacks are always good after listen(),
this patch saves the known good values after listen(). These good
values are then used for subsequent restores.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Reviewed-by: Tadashi Miyauchi <miyauchi@toshiba-tops.co.jp>
Signed-off-by: David Teigland <teigland@redhat.com>
2017-09-12 16:55:23 +08:00
|
|
|
struct sock *sk = sock->sk;
|
|
|
|
|
|
|
|
listen_sock.sk_data_ready = sk->sk_data_ready;
|
|
|
|
listen_sock.sk_state_change = sk->sk_state_change;
|
|
|
|
listen_sock.sk_write_space = sk->sk_write_space;
|
|
|
|
listen_sock.sk_error_report = sk->sk_error_report;
|
2016-02-06 03:39:02 +08:00
|
|
|
}
|
|
|
|
|
DLM: Fix saving of NULL callbacks
In a previous patch I noted that accept() often copies the struct
sock (sk) which overwrites the sock callbacks. However, in testing
we discovered that the dlm connection structures (con) are sometimes
deleted and recreated as connections come and go, and since they're
zeroed out by kmem_cache_zalloc, the saved callback pointers are
also initialized to zero. But with today's DLM code, the callbacks
are only saved when a socket is added.
During recovery testing, we discovered a common situation in which
the new con is initialized to zero, then a socket is added after
accept(). In this case, the sock's saved values are all NULL, but
the saved values are wiped out, due to accept(). Therefore, we
don't have a known good copy of the callbacks from which we can
restore.
Since the struct sock callbacks are always good after listen(),
this patch saves the known good values after listen(). These good
values are then used for subsequent restores.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Reviewed-by: Tadashi Miyauchi <miyauchi@toshiba-tops.co.jp>
Signed-off-by: David Teigland <teigland@redhat.com>
2017-09-12 16:55:23 +08:00
|
|
|
static void restore_callbacks(struct socket *sock)
|
2016-02-06 03:39:02 +08:00
|
|
|
{
|
DLM: Fix saving of NULL callbacks
In a previous patch I noted that accept() often copies the struct
sock (sk) which overwrites the sock callbacks. However, in testing
we discovered that the dlm connection structures (con) are sometimes
deleted and recreated as connections come and go, and since they're
zeroed out by kmem_cache_zalloc, the saved callback pointers are
also initialized to zero. But with today's DLM code, the callbacks
are only saved when a socket is added.
During recovery testing, we discovered a common situation in which
the new con is initialized to zero, then a socket is added after
accept(). In this case, the sock's saved values are all NULL, but
the saved values are wiped out, due to accept(). Therefore, we
don't have a known good copy of the callbacks from which we can
restore.
Since the struct sock callbacks are always good after listen(),
this patch saves the known good values after listen(). These good
values are then used for subsequent restores.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Reviewed-by: Tadashi Miyauchi <miyauchi@toshiba-tops.co.jp>
Signed-off-by: David Teigland <teigland@redhat.com>
2017-09-12 16:55:23 +08:00
|
|
|
struct sock *sk = sock->sk;
|
|
|
|
|
2021-11-15 21:57:06 +08:00
|
|
|
lock_sock(sk);
|
2016-02-06 03:39:02 +08:00
|
|
|
sk->sk_user_data = NULL;
|
DLM: Fix saving of NULL callbacks
In a previous patch I noted that accept() often copies the struct
sock (sk) which overwrites the sock callbacks. However, in testing
we discovered that the dlm connection structures (con) are sometimes
deleted and recreated as connections come and go, and since they're
zeroed out by kmem_cache_zalloc, the saved callback pointers are
also initialized to zero. But with today's DLM code, the callbacks
are only saved when a socket is added.
During recovery testing, we discovered a common situation in which
the new con is initialized to zero, then a socket is added after
accept(). In this case, the sock's saved values are all NULL, but
the saved values are wiped out, due to accept(). Therefore, we
don't have a known good copy of the callbacks from which we can
restore.
Since the struct sock callbacks are always good after listen(),
this patch saves the known good values after listen(). These good
values are then used for subsequent restores.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Reviewed-by: Tadashi Miyauchi <miyauchi@toshiba-tops.co.jp>
Signed-off-by: David Teigland <teigland@redhat.com>
2017-09-12 16:55:23 +08:00
|
|
|
sk->sk_data_ready = listen_sock.sk_data_ready;
|
|
|
|
sk->sk_state_change = listen_sock.sk_state_change;
|
|
|
|
sk->sk_write_space = listen_sock.sk_write_space;
|
|
|
|
sk->sk_error_report = listen_sock.sk_error_report;
|
2021-11-15 21:57:06 +08:00
|
|
|
release_sock(sk);
|
2015-08-27 22:34:47 +08:00
|
|
|
}
|
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
static void add_listen_sock(struct socket *sock, struct listen_connection *con)
|
|
|
|
{
|
|
|
|
struct sock *sk = sock->sk;
|
|
|
|
|
2021-11-15 21:57:06 +08:00
|
|
|
lock_sock(sk);
|
2020-11-03 09:04:25 +08:00
|
|
|
save_listen_callbacks(sock);
|
|
|
|
con->sock = sock;
|
|
|
|
|
|
|
|
sk->sk_user_data = con;
|
|
|
|
sk->sk_allocation = GFP_NOFS;
|
|
|
|
/* Install a data_ready callback */
|
|
|
|
sk->sk_data_ready = lowcomms_listen_data_ready;
|
2021-11-15 21:57:06 +08:00
|
|
|
release_sock(sk);
|
2020-11-03 09:04:25 +08:00
|
|
|
}
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
/* Make a socket active */
|
2017-09-12 16:55:32 +08:00
|
|
|
static void add_sock(struct socket *sock, struct connection *con)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2016-02-06 03:39:02 +08:00
|
|
|
struct sock *sk = sock->sk;
|
|
|
|
|
2021-11-15 21:57:06 +08:00
|
|
|
lock_sock(sk);
|
2006-11-03 00:19:21 +08:00
|
|
|
con->sock = sock;
|
|
|
|
|
2016-02-06 03:39:02 +08:00
|
|
|
sk->sk_user_data = con;
|
2006-11-03 00:19:21 +08:00
|
|
|
/* Install a data_ready callback */
|
2016-02-06 03:39:02 +08:00
|
|
|
sk->sk_data_ready = lowcomms_data_ready;
|
|
|
|
sk->sk_write_space = lowcomms_write_space;
|
|
|
|
sk->sk_state_change = lowcomms_state_change;
|
|
|
|
sk->sk_allocation = GFP_NOFS;
|
|
|
|
sk->sk_error_report = lowcomms_error_report;
|
2021-11-15 21:57:06 +08:00
|
|
|
release_sock(sk);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2007-04-17 22:39:57 +08:00
|
|
|
/* Add the port number to an IPv6 or 4 sockaddr and return the address
|
2006-11-03 00:19:21 +08:00
|
|
|
length */
|
|
|
|
static void make_sockaddr(struct sockaddr_storage *saddr, uint16_t port,
|
|
|
|
int *addr_len)
|
|
|
|
{
|
2007-04-17 22:39:57 +08:00
|
|
|
saddr->ss_family = dlm_local_addr[0]->ss_family;
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
if (saddr->ss_family == AF_INET) {
|
2006-11-03 00:19:21 +08:00
|
|
|
struct sockaddr_in *in4_addr = (struct sockaddr_in *)saddr;
|
|
|
|
in4_addr->sin_port = cpu_to_be16(port);
|
|
|
|
*addr_len = sizeof(struct sockaddr_in);
|
2007-04-17 22:39:57 +08:00
|
|
|
memset(&in4_addr->sin_zero, 0, sizeof(in4_addr->sin_zero));
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
} else {
|
2006-11-03 00:19:21 +08:00
|
|
|
struct sockaddr_in6 *in6_addr = (struct sockaddr_in6 *)saddr;
|
|
|
|
in6_addr->sin6_port = cpu_to_be16(port);
|
|
|
|
*addr_len = sizeof(struct sockaddr_in6);
|
|
|
|
}
|
2007-07-17 23:53:15 +08:00
|
|
|
memset((char *)saddr + *addr_len, 0, sizeof(struct sockaddr_storage) - *addr_len);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2021-05-22 03:08:48 +08:00
|
|
|
static void dlm_page_release(struct kref *kref)
|
|
|
|
{
|
|
|
|
struct writequeue_entry *e = container_of(kref, struct writequeue_entry,
|
|
|
|
ref);
|
|
|
|
|
|
|
|
__free_page(e->page);
|
2021-12-01 03:47:19 +08:00
|
|
|
dlm_free_writequeue(e);
|
2021-05-22 03:08:48 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static void dlm_msg_release(struct kref *kref)
|
|
|
|
{
|
|
|
|
struct dlm_msg *msg = container_of(kref, struct dlm_msg, ref);
|
|
|
|
|
|
|
|
kref_put(&msg->entry->ref, dlm_page_release);
|
2021-12-01 03:47:20 +08:00
|
|
|
dlm_free_msg(msg);
|
2021-05-22 03:08:48 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static void free_entry(struct writequeue_entry *e)
|
|
|
|
{
|
|
|
|
struct dlm_msg *msg, *tmp;
|
|
|
|
|
|
|
|
list_for_each_entry_safe(msg, tmp, &e->msgs, list) {
|
|
|
|
if (msg->orig_msg) {
|
|
|
|
msg->orig_msg->retransmit = false;
|
|
|
|
kref_put(&msg->orig_msg->ref, dlm_msg_release);
|
|
|
|
}
|
|
|
|
|
|
|
|
list_del(&msg->list);
|
|
|
|
kref_put(&msg->ref, dlm_msg_release);
|
|
|
|
}
|
|
|
|
|
|
|
|
list_del(&e->list);
|
|
|
|
atomic_dec(&e->con->writequeue_cnt);
|
|
|
|
kref_put(&e->ref, dlm_page_release);
|
|
|
|
}
|
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
static void dlm_close_sock(struct socket **sock)
|
|
|
|
{
|
|
|
|
if (*sock) {
|
|
|
|
restore_callbacks(*sock);
|
|
|
|
sock_release(*sock);
|
|
|
|
*sock = NULL;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
/* Close a remote connection and tidy up */
|
2015-08-12 06:22:21 +08:00
|
|
|
static void close_connection(struct connection *con, bool and_other,
|
|
|
|
bool tx, bool rx)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2017-09-12 16:55:50 +08:00
|
|
|
bool closing = test_and_set_bit(CF_CLOSING, &con->flags);
|
2021-05-22 03:08:48 +08:00
|
|
|
struct writequeue_entry *e;
|
2017-09-12 16:55:50 +08:00
|
|
|
|
2017-09-12 17:02:02 +08:00
|
|
|
if (tx && !closing && cancel_work_sync(&con->swork)) {
|
2015-08-12 06:22:21 +08:00
|
|
|
log_print("canceled swork for node %d", con->nodeid);
|
2017-09-12 17:02:02 +08:00
|
|
|
clear_bit(CF_WRITE_PENDING, &con->flags);
|
|
|
|
}
|
|
|
|
if (rx && !closing && cancel_work_sync(&con->rwork)) {
|
2015-08-12 06:22:21 +08:00
|
|
|
log_print("canceled rwork for node %d", con->nodeid);
|
2017-09-12 17:02:02 +08:00
|
|
|
clear_bit(CF_READ_PENDING, &con->flags);
|
|
|
|
}
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2015-08-12 06:22:21 +08:00
|
|
|
mutex_lock(&con->sock_mutex);
|
2020-11-03 09:04:25 +08:00
|
|
|
dlm_close_sock(&con->sock);
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
if (con->othercon && and_other) {
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
/* Will only re-enter once. */
|
2021-05-22 03:08:38 +08:00
|
|
|
close_connection(con->othercon, false, tx, rx);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
2007-08-02 21:58:14 +08:00
|
|
|
|
2021-05-22 03:08:48 +08:00
|
|
|
/* if we send a writequeue entry only a half way, we drop the
|
|
|
|
* whole entry because reconnection and that we not start of the
|
|
|
|
* middle of a msg which will confuse the other end.
|
|
|
|
*
|
|
|
|
* we can always drop messages because retransmits, but what we
|
|
|
|
* cannot allow is to transmit half messages which may be processed
|
|
|
|
* at the other side.
|
|
|
|
*
|
|
|
|
* our policy is to start on a clean state when disconnects, we don't
|
|
|
|
* know what's send/received on transport layer in this case.
|
|
|
|
*/
|
|
|
|
spin_lock(&con->writequeue_lock);
|
|
|
|
if (!list_empty(&con->writequeue)) {
|
|
|
|
e = list_first_entry(&con->writequeue, struct writequeue_entry,
|
|
|
|
list);
|
|
|
|
if (e->dirty)
|
|
|
|
free_entry(e);
|
|
|
|
}
|
|
|
|
spin_unlock(&con->writequeue_lock);
|
|
|
|
|
2020-09-24 22:31:26 +08:00
|
|
|
con->rx_leftover = 0;
|
2007-08-20 22:13:38 +08:00
|
|
|
con->retries = 0;
|
2021-07-17 04:22:37 +08:00
|
|
|
clear_bit(CF_APP_LIMITED, &con->flags);
|
2020-11-03 09:04:20 +08:00
|
|
|
clear_bit(CF_CONNECTED, &con->flags);
|
2021-05-22 03:08:37 +08:00
|
|
|
clear_bit(CF_DELAY_CONNECT, &con->flags);
|
|
|
|
clear_bit(CF_RECONNECT, &con->flags);
|
2021-05-22 03:08:39 +08:00
|
|
|
clear_bit(CF_EOF, &con->flags);
|
2007-08-20 22:13:38 +08:00
|
|
|
mutex_unlock(&con->sock_mutex);
|
2017-09-12 16:55:50 +08:00
|
|
|
clear_bit(CF_CLOSING, &con->flags);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2020-07-27 21:13:38 +08:00
|
|
|
static void shutdown_connection(struct connection *con)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
2021-03-02 06:05:19 +08:00
|
|
|
flush_work(&con->swork);
|
2020-07-27 21:13:38 +08:00
|
|
|
|
|
|
|
mutex_lock(&con->sock_mutex);
|
|
|
|
/* nothing to shutdown */
|
|
|
|
if (!con->sock) {
|
|
|
|
mutex_unlock(&con->sock_mutex);
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
set_bit(CF_SHUTDOWN, &con->flags);
|
|
|
|
ret = kernel_sock_shutdown(con->sock, SHUT_WR);
|
|
|
|
mutex_unlock(&con->sock_mutex);
|
|
|
|
if (ret) {
|
|
|
|
log_print("Connection %p failed to shutdown: %d will force close",
|
|
|
|
con, ret);
|
|
|
|
goto force_close;
|
|
|
|
} else {
|
|
|
|
ret = wait_event_timeout(con->shutdown_wait,
|
|
|
|
!test_bit(CF_SHUTDOWN, &con->flags),
|
|
|
|
DLM_SHUTDOWN_WAIT_TIMEOUT);
|
|
|
|
if (ret == 0) {
|
|
|
|
log_print("Connection %p shutdown timed out, will force close",
|
|
|
|
con);
|
|
|
|
goto force_close;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return;
|
|
|
|
|
|
|
|
force_close:
|
|
|
|
clear_bit(CF_SHUTDOWN, &con->flags);
|
|
|
|
close_connection(con, false, true, true);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void dlm_tcp_shutdown(struct connection *con)
|
|
|
|
{
|
|
|
|
if (con->othercon)
|
|
|
|
shutdown_connection(con->othercon);
|
|
|
|
shutdown_connection(con);
|
|
|
|
}
|
|
|
|
|
2020-09-24 22:31:26 +08:00
|
|
|
static int con_realloc_receive_buf(struct connection *con, int newlen)
|
|
|
|
{
|
|
|
|
unsigned char *newbuf;
|
|
|
|
|
|
|
|
newbuf = kmalloc(newlen, GFP_NOFS);
|
|
|
|
if (!newbuf)
|
|
|
|
return -ENOMEM;
|
|
|
|
|
|
|
|
/* copy any leftover from last receive */
|
|
|
|
if (con->rx_leftover)
|
|
|
|
memmove(newbuf, con->rx_buf, con->rx_leftover);
|
|
|
|
|
|
|
|
/* swap to new buffer space */
|
|
|
|
kfree(con->rx_buf);
|
|
|
|
con->rx_buflen = newlen;
|
|
|
|
con->rx_buf = newbuf;
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
/* Data received from remote end */
|
|
|
|
static int receive_from_sock(struct connection *con)
|
|
|
|
{
|
2020-09-24 22:31:26 +08:00
|
|
|
struct msghdr msg;
|
|
|
|
struct kvec iov;
|
|
|
|
int ret, buflen;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2007-01-24 19:17:59 +08:00
|
|
|
mutex_lock(&con->sock_mutex);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2007-02-02 00:46:33 +08:00
|
|
|
if (con->sock == NULL) {
|
|
|
|
ret = -EAGAIN;
|
|
|
|
goto out_close;
|
|
|
|
}
|
2020-09-24 22:31:26 +08:00
|
|
|
|
|
|
|
/* realloc if we get new buffer size to read out */
|
|
|
|
buflen = dlm_config.ci_buffer_size;
|
|
|
|
if (con->rx_buflen != buflen && con->rx_leftover <= buflen) {
|
|
|
|
ret = con_realloc_receive_buf(con, buflen);
|
|
|
|
if (ret < 0)
|
2006-11-03 00:19:21 +08:00
|
|
|
goto out_resched;
|
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:45 +08:00
|
|
|
for (;;) {
|
|
|
|
/* calculate new buffer parameter regarding last receive and
|
|
|
|
* possible leftover bytes
|
|
|
|
*/
|
|
|
|
iov.iov_base = con->rx_buf + con->rx_leftover;
|
|
|
|
iov.iov_len = con->rx_buflen - con->rx_leftover;
|
|
|
|
|
|
|
|
memset(&msg, 0, sizeof(msg));
|
|
|
|
msg.msg_flags = MSG_DONTWAIT | MSG_NOSIGNAL;
|
|
|
|
ret = kernel_recvmsg(con->sock, &msg, &iov, 1, iov.iov_len,
|
|
|
|
msg.msg_flags);
|
2021-11-03 03:17:16 +08:00
|
|
|
trace_dlm_recv(con->nodeid, ret);
|
2021-07-17 04:22:45 +08:00
|
|
|
if (ret == -EAGAIN)
|
|
|
|
break;
|
|
|
|
else if (ret <= 0)
|
|
|
|
goto out_close;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2021-07-17 04:22:45 +08:00
|
|
|
/* new buflen according readed bytes and leftover from last receive */
|
|
|
|
buflen = ret + con->rx_leftover;
|
|
|
|
ret = dlm_process_incoming_buffer(con->nodeid, con->rx_buf, buflen);
|
|
|
|
if (ret < 0)
|
|
|
|
goto out_close;
|
|
|
|
|
|
|
|
/* calculate leftover bytes from process and put it into begin of
|
|
|
|
* the receive buffer, so next receive we have the full message
|
|
|
|
* at the start address of the receive buffer.
|
|
|
|
*/
|
|
|
|
con->rx_leftover = buflen - ret;
|
|
|
|
if (con->rx_leftover) {
|
|
|
|
memmove(con->rx_buf, con->rx_buf + ret,
|
|
|
|
con->rx_leftover);
|
|
|
|
}
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2021-08-19 04:27:13 +08:00
|
|
|
dlm_midcomms_receive_done(con->nodeid);
|
2007-01-24 19:17:59 +08:00
|
|
|
mutex_unlock(&con->sock_mutex);
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
return 0;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
out_resched:
|
2007-01-15 22:33:34 +08:00
|
|
|
if (!test_and_set_bit(CF_READ_PENDING, &con->flags))
|
|
|
|
queue_work(recv_workqueue, &con->rwork);
|
2007-01-24 19:17:59 +08:00
|
|
|
mutex_unlock(&con->sock_mutex);
|
2007-01-22 22:51:33 +08:00
|
|
|
return -EAGAIN;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
out_close:
|
2021-05-22 03:08:37 +08:00
|
|
|
if (ret == 0) {
|
|
|
|
log_print("connection %p got EOF from %d",
|
|
|
|
con, con->nodeid);
|
2021-05-22 03:08:39 +08:00
|
|
|
|
2021-07-17 04:22:40 +08:00
|
|
|
if (dlm_proto_ops->eof_condition &&
|
|
|
|
dlm_proto_ops->eof_condition(con)) {
|
2021-05-22 03:08:39 +08:00
|
|
|
set_bit(CF_EOF, &con->flags);
|
|
|
|
mutex_unlock(&con->sock_mutex);
|
|
|
|
} else {
|
|
|
|
mutex_unlock(&con->sock_mutex);
|
|
|
|
close_connection(con, false, true, false);
|
|
|
|
|
|
|
|
/* handling for tcp shutdown */
|
|
|
|
clear_bit(CF_SHUTDOWN, &con->flags);
|
|
|
|
wake_up(&con->shutdown_wait);
|
|
|
|
}
|
|
|
|
|
2021-05-22 03:08:37 +08:00
|
|
|
/* signal to breaking receive worker */
|
|
|
|
ret = -1;
|
2021-05-22 03:08:39 +08:00
|
|
|
} else {
|
|
|
|
mutex_unlock(&con->sock_mutex);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Listening socket is busy, accept a connection */
|
2020-11-03 09:04:25 +08:00
|
|
|
static int accept_from_sock(struct listen_connection *con)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
|
|
|
int result;
|
|
|
|
struct sockaddr_storage peeraddr;
|
|
|
|
struct socket *newsock;
|
2021-05-22 03:08:35 +08:00
|
|
|
int len, idx;
|
2006-11-03 00:19:21 +08:00
|
|
|
int nodeid;
|
|
|
|
struct connection *newcon;
|
2007-01-22 22:51:33 +08:00
|
|
|
struct connection *addcon;
|
2020-09-24 22:31:23 +08:00
|
|
|
unsigned int mark;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
if (!con->sock)
|
2017-09-12 17:01:38 +08:00
|
|
|
return -ENOTCONN;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2017-09-12 17:01:38 +08:00
|
|
|
result = kernel_accept(con->sock, &newsock, O_NONBLOCK);
|
2006-11-03 00:19:21 +08:00
|
|
|
if (result < 0)
|
|
|
|
goto accept_err;
|
|
|
|
|
|
|
|
/* Get the connected socket's peer */
|
|
|
|
memset(&peeraddr, 0, sizeof(peeraddr));
|
2018-02-13 03:00:20 +08:00
|
|
|
len = newsock->ops->getname(newsock, (struct sockaddr *)&peeraddr, 2);
|
|
|
|
if (len < 0) {
|
2006-11-03 00:19:21 +08:00
|
|
|
result = -ECONNABORTED;
|
|
|
|
goto accept_err;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Get the new node's NODEID */
|
|
|
|
make_sockaddr(&peeraddr, 0, &len);
|
2021-03-02 06:05:09 +08:00
|
|
|
if (addr_to_nodeid(&peeraddr, &nodeid, &mark)) {
|
2022-01-04 22:09:47 +08:00
|
|
|
switch (peeraddr.ss_family) {
|
|
|
|
case AF_INET: {
|
|
|
|
struct sockaddr_in *sin = (struct sockaddr_in *)&peeraddr;
|
|
|
|
|
|
|
|
log_print("connect from non cluster IPv4 node %pI4",
|
|
|
|
&sin->sin_addr);
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
#if IS_ENABLED(CONFIG_IPV6)
|
|
|
|
case AF_INET6: {
|
|
|
|
struct sockaddr_in6 *sin6 = (struct sockaddr_in6 *)&peeraddr;
|
|
|
|
|
|
|
|
log_print("connect from non cluster IPv6 node %pI6c",
|
|
|
|
&sin6->sin6_addr);
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
#endif
|
|
|
|
default:
|
|
|
|
log_print("invalid family from non cluster node");
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
sock_release(newsock);
|
|
|
|
return -1;
|
|
|
|
}
|
|
|
|
|
|
|
|
log_print("got connection from %d", nodeid);
|
|
|
|
|
|
|
|
/* Check to see if we already have a connection to this node. This
|
|
|
|
* could happen if the two nodes initiate a connection at roughly
|
|
|
|
* the same time and the connections cross on the wire.
|
|
|
|
* In this case we store the incoming one in "othercon"
|
|
|
|
*/
|
2021-05-22 03:08:35 +08:00
|
|
|
idx = srcu_read_lock(&connections_srcu);
|
2009-05-15 23:50:57 +08:00
|
|
|
newcon = nodeid2con(nodeid, GFP_NOFS);
|
2006-11-03 00:19:21 +08:00
|
|
|
if (!newcon) {
|
2021-05-22 03:08:35 +08:00
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
2006-11-03 00:19:21 +08:00
|
|
|
result = -ENOMEM;
|
|
|
|
goto accept_err;
|
|
|
|
}
|
2020-11-03 09:04:25 +08:00
|
|
|
|
2021-03-02 06:05:09 +08:00
|
|
|
sock_set_mark(newsock->sk, mark);
|
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
mutex_lock(&newcon->sock_mutex);
|
2006-11-03 00:19:21 +08:00
|
|
|
if (newcon->sock) {
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
struct connection *othercon = newcon->othercon;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
|
|
|
if (!othercon) {
|
2020-08-28 03:02:49 +08:00
|
|
|
othercon = kzalloc(sizeof(*othercon), GFP_NOFS);
|
2006-11-03 00:19:21 +08:00
|
|
|
if (!othercon) {
|
2007-04-27 02:46:49 +08:00
|
|
|
log_print("failed to allocate incoming socket");
|
2007-01-24 19:17:59 +08:00
|
|
|
mutex_unlock(&newcon->sock_mutex);
|
2021-05-22 03:08:35 +08:00
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
2006-11-03 00:19:21 +08:00
|
|
|
result = -ENOMEM;
|
|
|
|
goto accept_err;
|
|
|
|
}
|
2020-09-24 22:31:26 +08:00
|
|
|
|
2020-11-03 09:04:21 +08:00
|
|
|
result = dlm_con_init(othercon, nodeid);
|
|
|
|
if (result < 0) {
|
2020-09-24 22:31:26 +08:00
|
|
|
kfree(othercon);
|
2021-03-27 16:37:04 +08:00
|
|
|
mutex_unlock(&newcon->sock_mutex);
|
2021-05-22 03:08:35 +08:00
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
2020-09-24 22:31:26 +08:00
|
|
|
goto accept_err;
|
|
|
|
}
|
|
|
|
|
2021-03-02 06:05:11 +08:00
|
|
|
lockdep_set_subclass(&othercon->sock_mutex, 1);
|
2021-05-22 03:08:36 +08:00
|
|
|
set_bit(CF_IS_OTHERCON, &othercon->flags);
|
2020-11-03 09:04:21 +08:00
|
|
|
newcon->othercon = othercon;
|
2021-05-22 03:08:37 +08:00
|
|
|
othercon->sendcon = newcon;
|
2020-07-27 21:13:37 +08:00
|
|
|
} else {
|
|
|
|
/* close other sock con if we have something new */
|
|
|
|
close_connection(othercon, false, true, false);
|
2007-08-20 22:13:38 +08:00
|
|
|
}
|
2020-07-27 21:13:37 +08:00
|
|
|
|
2021-03-02 06:05:11 +08:00
|
|
|
mutex_lock(&othercon->sock_mutex);
|
2020-07-27 21:13:37 +08:00
|
|
|
add_sock(newsock, othercon);
|
|
|
|
addcon = othercon;
|
|
|
|
mutex_unlock(&othercon->sock_mutex);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
else {
|
2016-09-24 02:23:26 +08:00
|
|
|
/* accept copies the sk after we've saved the callbacks, so we
|
|
|
|
don't want to save them a second time or comm errors will
|
|
|
|
result in calling sk_error_report recursively. */
|
2017-09-12 16:55:32 +08:00
|
|
|
add_sock(newsock, newcon);
|
2007-01-22 22:51:33 +08:00
|
|
|
addcon = newcon;
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2021-03-02 06:05:10 +08:00
|
|
|
set_bit(CF_CONNECTED, &addcon->flags);
|
2007-01-24 19:17:59 +08:00
|
|
|
mutex_unlock(&newcon->sock_mutex);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Add it to the active queue in case we got data
|
2011-03-31 09:57:33 +08:00
|
|
|
* between processing the accept adding the socket
|
2006-11-03 00:19:21 +08:00
|
|
|
* to the read_sockets list
|
|
|
|
*/
|
2007-01-22 22:51:33 +08:00
|
|
|
if (!test_and_set_bit(CF_READ_PENDING, &addcon->flags))
|
|
|
|
queue_work(recv_workqueue, &addcon->rwork);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2021-05-22 03:08:35 +08:00
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
return 0;
|
|
|
|
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
accept_err:
|
2017-09-12 17:01:38 +08:00
|
|
|
if (newsock)
|
|
|
|
sock_release(newsock);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
|
|
|
if (result != -EAGAIN)
|
2007-04-27 02:46:49 +08:00
|
|
|
log_print("error accepting connection from node: %d", result);
|
2006-11-03 00:19:21 +08:00
|
|
|
return result;
|
|
|
|
}
|
|
|
|
|
2013-06-14 17:56:13 +08:00
|
|
|
/*
|
|
|
|
* writequeue_entry_complete - try to delete and free write queue entry
|
|
|
|
* @e: write queue entry to try to delete
|
|
|
|
* @completed: bytes completed
|
|
|
|
*
|
|
|
|
* writequeue_lock must be held.
|
|
|
|
*/
|
|
|
|
static void writequeue_entry_complete(struct writequeue_entry *e, int completed)
|
|
|
|
{
|
|
|
|
e->offset += completed;
|
|
|
|
e->len -= completed;
|
2021-05-22 03:08:48 +08:00
|
|
|
/* signal that page was half way transmitted */
|
|
|
|
e->dirty = true;
|
2013-06-14 17:56:13 +08:00
|
|
|
|
2021-05-22 03:08:42 +08:00
|
|
|
if (e->len == 0 && e->users == 0)
|
2013-06-14 17:56:13 +08:00
|
|
|
free_entry(e);
|
|
|
|
}
|
|
|
|
|
2015-08-12 06:22:23 +08:00
|
|
|
/*
|
|
|
|
* sctp_bind_addrs - bind a SCTP socket to all our addresses
|
|
|
|
*/
|
2020-11-03 09:04:24 +08:00
|
|
|
static int sctp_bind_addrs(struct socket *sock, uint16_t port)
|
2015-08-12 06:22:23 +08:00
|
|
|
{
|
|
|
|
struct sockaddr_storage localaddr;
|
2020-05-29 20:09:42 +08:00
|
|
|
struct sockaddr *addr = (struct sockaddr *)&localaddr;
|
2015-08-12 06:22:23 +08:00
|
|
|
int i, addr_len, result = 0;
|
|
|
|
|
|
|
|
for (i = 0; i < dlm_local_count; i++) {
|
|
|
|
memcpy(&localaddr, dlm_local_addr[i], sizeof(localaddr));
|
|
|
|
make_sockaddr(&localaddr, port, &addr_len);
|
|
|
|
|
|
|
|
if (!i)
|
2020-11-03 09:04:24 +08:00
|
|
|
result = kernel_bind(sock, addr, addr_len);
|
2015-08-12 06:22:23 +08:00
|
|
|
else
|
2020-11-03 09:04:24 +08:00
|
|
|
result = sock_bind_add(sock->sk, addr, addr_len);
|
2015-08-12 06:22:23 +08:00
|
|
|
|
|
|
|
if (result < 0) {
|
|
|
|
log_print("Can't bind to %d addr number %d, %d.\n",
|
|
|
|
port, i + 1, result);
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return result;
|
|
|
|
}
|
|
|
|
|
2007-04-17 22:39:57 +08:00
|
|
|
/* Get local addresses */
|
|
|
|
static void init_local(void)
|
|
|
|
{
|
|
|
|
struct sockaddr_storage sas, *addr;
|
|
|
|
int i;
|
|
|
|
|
2007-04-23 23:26:21 +08:00
|
|
|
dlm_local_count = 0;
|
2012-03-21 22:18:34 +08:00
|
|
|
for (i = 0; i < DLM_MAX_ADDR_COUNT; i++) {
|
2007-04-17 22:39:57 +08:00
|
|
|
if (dlm_our_addr(&sas, i))
|
|
|
|
break;
|
|
|
|
|
2016-06-23 12:52:01 +08:00
|
|
|
addr = kmemdup(&sas, sizeof(*addr), GFP_NOFS);
|
2007-04-17 22:39:57 +08:00
|
|
|
if (!addr)
|
|
|
|
break;
|
|
|
|
dlm_local_addr[dlm_local_count++] = addr;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2020-08-28 03:02:50 +08:00
|
|
|
static void deinit_local(void)
|
|
|
|
{
|
|
|
|
int i;
|
|
|
|
|
|
|
|
for (i = 0; i < dlm_local_count; i++)
|
|
|
|
kfree(dlm_local_addr[i]);
|
|
|
|
}
|
|
|
|
|
2021-12-01 03:47:17 +08:00
|
|
|
static struct writequeue_entry *new_writequeue_entry(struct connection *con)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
|
|
|
struct writequeue_entry *entry;
|
|
|
|
|
2021-12-01 03:47:19 +08:00
|
|
|
entry = dlm_allocate_writequeue();
|
2006-11-03 00:19:21 +08:00
|
|
|
if (!entry)
|
|
|
|
return NULL;
|
|
|
|
|
2021-12-01 03:47:17 +08:00
|
|
|
entry->page = alloc_page(GFP_ATOMIC | __GFP_ZERO);
|
2006-11-03 00:19:21 +08:00
|
|
|
if (!entry->page) {
|
2021-12-01 03:47:19 +08:00
|
|
|
dlm_free_writequeue(entry);
|
2006-11-03 00:19:21 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2021-12-01 03:47:19 +08:00
|
|
|
entry->offset = 0;
|
|
|
|
entry->len = 0;
|
|
|
|
entry->end = 0;
|
|
|
|
entry->dirty = false;
|
2006-11-03 00:19:21 +08:00
|
|
|
entry->con = con;
|
2021-03-02 06:05:16 +08:00
|
|
|
entry->users = 1;
|
2021-05-22 03:08:42 +08:00
|
|
|
kref_init(&entry->ref);
|
2006-11-03 00:19:21 +08:00
|
|
|
return entry;
|
|
|
|
}
|
|
|
|
|
2021-03-02 06:05:16 +08:00
|
|
|
static struct writequeue_entry *new_wq_entry(struct connection *con, int len,
|
2021-12-01 03:47:17 +08:00
|
|
|
char **ppc, void (*cb)(void *data),
|
|
|
|
void *data)
|
2021-03-02 06:05:16 +08:00
|
|
|
{
|
|
|
|
struct writequeue_entry *e;
|
|
|
|
|
|
|
|
spin_lock(&con->writequeue_lock);
|
|
|
|
if (!list_empty(&con->writequeue)) {
|
|
|
|
e = list_last_entry(&con->writequeue, struct writequeue_entry, list);
|
|
|
|
if (DLM_WQ_REMAIN_BYTES(e) >= len) {
|
2021-05-22 03:08:42 +08:00
|
|
|
kref_get(&e->ref);
|
|
|
|
|
2021-03-02 06:05:16 +08:00
|
|
|
*ppc = page_address(e->page) + e->end;
|
2021-05-22 03:08:42 +08:00
|
|
|
if (cb)
|
2021-11-03 03:17:19 +08:00
|
|
|
cb(data);
|
2021-05-22 03:08:42 +08:00
|
|
|
|
2021-03-02 06:05:16 +08:00
|
|
|
e->end += len;
|
|
|
|
e->users++;
|
2021-12-01 03:47:17 +08:00
|
|
|
goto out;
|
2021-03-02 06:05:16 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-12-01 03:47:17 +08:00
|
|
|
e = new_writequeue_entry(con);
|
2021-03-02 06:05:16 +08:00
|
|
|
if (!e)
|
2021-12-01 03:47:17 +08:00
|
|
|
goto out;
|
2021-03-02 06:05:16 +08:00
|
|
|
|
2021-05-22 03:08:42 +08:00
|
|
|
kref_get(&e->ref);
|
2021-03-02 06:05:16 +08:00
|
|
|
*ppc = page_address(e->page);
|
|
|
|
e->end += len;
|
2021-05-22 03:08:39 +08:00
|
|
|
atomic_inc(&con->writequeue_cnt);
|
2021-05-22 03:08:42 +08:00
|
|
|
if (cb)
|
2021-11-03 03:17:19 +08:00
|
|
|
cb(data);
|
2021-05-22 03:08:42 +08:00
|
|
|
|
2021-03-02 06:05:16 +08:00
|
|
|
list_add_tail(&e->list, &con->writequeue);
|
|
|
|
|
2021-12-01 03:47:17 +08:00
|
|
|
out:
|
|
|
|
spin_unlock(&con->writequeue_lock);
|
2021-03-02 06:05:16 +08:00
|
|
|
return e;
|
|
|
|
};
|
|
|
|
|
2021-05-22 03:08:43 +08:00
|
|
|
static struct dlm_msg *dlm_lowcomms_new_msg_con(struct connection *con, int len,
|
|
|
|
gfp_t allocation, char **ppc,
|
2021-11-03 03:17:19 +08:00
|
|
|
void (*cb)(void *data),
|
|
|
|
void *data)
|
2021-05-22 03:08:43 +08:00
|
|
|
{
|
|
|
|
struct writequeue_entry *e;
|
|
|
|
struct dlm_msg *msg;
|
|
|
|
|
2021-12-01 03:47:20 +08:00
|
|
|
msg = dlm_allocate_msg(allocation);
|
2021-05-22 03:08:43 +08:00
|
|
|
if (!msg)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
kref_init(&msg->ref);
|
|
|
|
|
2021-12-01 03:47:17 +08:00
|
|
|
e = new_wq_entry(con, len, ppc, cb, data);
|
2021-05-22 03:08:43 +08:00
|
|
|
if (!e) {
|
2021-12-01 03:47:20 +08:00
|
|
|
dlm_free_msg(msg);
|
2021-05-22 03:08:43 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2021-12-01 03:47:20 +08:00
|
|
|
msg->retransmit = false;
|
|
|
|
msg->orig_msg = NULL;
|
2021-05-22 03:08:43 +08:00
|
|
|
msg->ppc = *ppc;
|
|
|
|
msg->len = len;
|
|
|
|
msg->entry = e;
|
|
|
|
|
|
|
|
return msg;
|
|
|
|
}
|
|
|
|
|
2022-04-05 04:06:37 +08:00
|
|
|
/* avoid false positive for nodes_srcu, unlock happens in
|
|
|
|
* dlm_lowcomms_commit_msg which is a must call if success
|
|
|
|
*/
|
|
|
|
#ifndef __CHECKER__
|
2021-05-22 03:08:42 +08:00
|
|
|
struct dlm_msg *dlm_lowcomms_new_msg(int nodeid, int len, gfp_t allocation,
|
2021-11-03 03:17:19 +08:00
|
|
|
char **ppc, void (*cb)(void *data),
|
|
|
|
void *data)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
|
|
|
struct connection *con;
|
2021-05-22 03:08:42 +08:00
|
|
|
struct dlm_msg *msg;
|
2021-05-22 03:08:35 +08:00
|
|
|
int idx;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2021-06-02 21:45:20 +08:00
|
|
|
if (len > DLM_MAX_SOCKET_BUFSIZE ||
|
2021-03-02 06:05:14 +08:00
|
|
|
len < sizeof(struct dlm_header)) {
|
2021-06-02 21:45:20 +08:00
|
|
|
BUILD_BUG_ON(PAGE_SIZE < DLM_MAX_SOCKET_BUFSIZE);
|
2020-11-03 09:04:18 +08:00
|
|
|
log_print("failed to allocate a buffer of size %d", len);
|
2021-03-02 06:05:14 +08:00
|
|
|
WARN_ON(1);
|
2020-11-03 09:04:18 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2021-05-22 03:08:35 +08:00
|
|
|
idx = srcu_read_lock(&connections_srcu);
|
2006-11-03 00:19:21 +08:00
|
|
|
con = nodeid2con(nodeid, allocation);
|
2021-05-22 03:08:35 +08:00
|
|
|
if (!con) {
|
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
2006-11-03 00:19:21 +08:00
|
|
|
return NULL;
|
2021-05-22 03:08:35 +08:00
|
|
|
}
|
|
|
|
|
2021-11-03 03:17:19 +08:00
|
|
|
msg = dlm_lowcomms_new_msg_con(con, len, allocation, ppc, cb, data);
|
2021-05-22 03:08:42 +08:00
|
|
|
if (!msg) {
|
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2021-05-22 03:08:35 +08:00
|
|
|
/* we assume if successful commit must called */
|
2021-05-22 03:08:42 +08:00
|
|
|
msg->idx = idx;
|
|
|
|
return msg;
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
2022-04-05 04:06:37 +08:00
|
|
|
#endif
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2021-05-22 03:08:43 +08:00
|
|
|
static void _dlm_lowcomms_commit_msg(struct dlm_msg *msg)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2021-05-22 03:08:42 +08:00
|
|
|
struct writequeue_entry *e = msg->entry;
|
2006-11-03 00:19:21 +08:00
|
|
|
struct connection *con = e->con;
|
|
|
|
int users;
|
|
|
|
|
2007-01-03 01:08:54 +08:00
|
|
|
spin_lock(&con->writequeue_lock);
|
2021-05-22 03:08:42 +08:00
|
|
|
kref_get(&msg->ref);
|
|
|
|
list_add(&msg->list, &e->msgs);
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
users = --e->users;
|
|
|
|
if (users)
|
|
|
|
goto out;
|
2021-03-02 06:05:16 +08:00
|
|
|
|
|
|
|
e->len = DLM_WQ_LENGTH_BYTES(e);
|
2006-11-03 00:19:21 +08:00
|
|
|
spin_unlock(&con->writequeue_lock);
|
|
|
|
|
2017-09-12 16:55:14 +08:00
|
|
|
queue_work(send_workqueue, &con->swork);
|
2006-11-03 00:19:21 +08:00
|
|
|
return;
|
|
|
|
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
out:
|
2006-11-03 00:19:21 +08:00
|
|
|
spin_unlock(&con->writequeue_lock);
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2022-04-05 04:06:37 +08:00
|
|
|
/* avoid false positive for nodes_srcu, lock was happen in
|
|
|
|
* dlm_lowcomms_new_msg
|
|
|
|
*/
|
|
|
|
#ifndef __CHECKER__
|
2021-05-22 03:08:43 +08:00
|
|
|
void dlm_lowcomms_commit_msg(struct dlm_msg *msg)
|
|
|
|
{
|
|
|
|
_dlm_lowcomms_commit_msg(msg);
|
|
|
|
srcu_read_unlock(&connections_srcu, msg->idx);
|
|
|
|
}
|
2022-04-05 04:06:37 +08:00
|
|
|
#endif
|
2021-05-22 03:08:43 +08:00
|
|
|
|
2021-05-22 03:08:42 +08:00
|
|
|
void dlm_lowcomms_put_msg(struct dlm_msg *msg)
|
|
|
|
{
|
|
|
|
kref_put(&msg->ref, dlm_msg_release);
|
|
|
|
}
|
|
|
|
|
2021-05-22 03:08:43 +08:00
|
|
|
/* does not held connections_srcu, usage workqueue only */
|
|
|
|
int dlm_lowcomms_resend_msg(struct dlm_msg *msg)
|
|
|
|
{
|
|
|
|
struct dlm_msg *msg_resend;
|
|
|
|
char *ppc;
|
|
|
|
|
|
|
|
if (msg->retransmit)
|
|
|
|
return 1;
|
|
|
|
|
|
|
|
msg_resend = dlm_lowcomms_new_msg_con(msg->entry->con, msg->len,
|
|
|
|
GFP_ATOMIC, &ppc, NULL, NULL);
|
|
|
|
if (!msg_resend)
|
|
|
|
return -ENOMEM;
|
|
|
|
|
|
|
|
msg->retransmit = true;
|
|
|
|
kref_get(&msg->ref);
|
|
|
|
msg_resend->orig_msg = msg;
|
|
|
|
|
|
|
|
memcpy(ppc, msg->ppc, msg->len);
|
|
|
|
_dlm_lowcomms_commit_msg(msg_resend);
|
|
|
|
dlm_lowcomms_put_msg(msg_resend);
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
/* Send a message */
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
static void send_to_sock(struct connection *con)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
|
|
|
const int msg_flags = MSG_DONTWAIT | MSG_NOSIGNAL;
|
|
|
|
struct writequeue_entry *e;
|
2021-07-17 04:22:39 +08:00
|
|
|
int len, offset, ret;
|
2010-11-13 01:15:20 +08:00
|
|
|
int count = 0;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2007-01-24 19:17:59 +08:00
|
|
|
mutex_lock(&con->sock_mutex);
|
2006-11-03 00:19:21 +08:00
|
|
|
if (con->sock == NULL)
|
|
|
|
goto out_connect;
|
|
|
|
|
|
|
|
spin_lock(&con->writequeue_lock);
|
|
|
|
for (;;) {
|
2021-07-17 04:22:39 +08:00
|
|
|
e = con_next_wq(con);
|
|
|
|
if (!e)
|
2006-11-03 00:19:21 +08:00
|
|
|
break;
|
|
|
|
|
|
|
|
len = e->len;
|
|
|
|
offset = e->offset;
|
|
|
|
BUG_ON(len == 0 && e->users == 0);
|
|
|
|
spin_unlock(&con->writequeue_lock);
|
|
|
|
|
2021-07-17 04:22:39 +08:00
|
|
|
ret = kernel_sendpage(con->sock, e->page, offset, len,
|
|
|
|
msg_flags);
|
2021-11-03 03:17:16 +08:00
|
|
|
trace_dlm_send(con->nodeid, ret);
|
2021-07-17 04:22:39 +08:00
|
|
|
if (ret == -EAGAIN || ret == 0) {
|
|
|
|
if (ret == -EAGAIN &&
|
|
|
|
test_bit(SOCKWQ_ASYNC_NOSPACE, &con->sock->flags) &&
|
|
|
|
!test_and_set_bit(CF_APP_LIMITED, &con->flags)) {
|
|
|
|
/* Notify TCP that we're limited by the
|
|
|
|
* application window size.
|
|
|
|
*/
|
|
|
|
set_bit(SOCK_NOSPACE, &con->sock->flags);
|
|
|
|
con->sock->sk->sk_write_pending++;
|
|
|
|
}
|
|
|
|
cond_resched();
|
|
|
|
goto out;
|
|
|
|
} else if (ret < 0)
|
|
|
|
goto out;
|
2010-11-13 01:15:20 +08:00
|
|
|
|
|
|
|
/* Don't starve people filling buffers */
|
|
|
|
if (++count >= MAX_SEND_MSG_COUNT) {
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
cond_resched();
|
2010-11-13 01:15:20 +08:00
|
|
|
count = 0;
|
|
|
|
}
|
2006-11-03 00:19:21 +08:00
|
|
|
|
|
|
|
spin_lock(&con->writequeue_lock);
|
2013-06-14 17:56:13 +08:00
|
|
|
writequeue_entry_complete(e, ret);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
spin_unlock(&con->writequeue_lock);
|
2021-05-22 03:08:39 +08:00
|
|
|
|
|
|
|
/* close if we got EOF */
|
|
|
|
if (test_and_clear_bit(CF_EOF, &con->flags)) {
|
|
|
|
mutex_unlock(&con->sock_mutex);
|
|
|
|
close_connection(con, false, false, true);
|
|
|
|
|
|
|
|
/* handling for tcp shutdown */
|
|
|
|
clear_bit(CF_SHUTDOWN, &con->flags);
|
|
|
|
wake_up(&con->shutdown_wait);
|
|
|
|
} else {
|
|
|
|
mutex_unlock(&con->sock_mutex);
|
|
|
|
}
|
|
|
|
|
|
|
|
return;
|
|
|
|
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
out:
|
2007-01-24 19:17:59 +08:00
|
|
|
mutex_unlock(&con->sock_mutex);
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
return;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
[DLM] Clean up lowcomms
This fixes up most of the things pointed out by akpm and Pavel Machek
with comments below indicating why some things have been left:
Andrew Morton wrote:
>
>> +static struct nodeinfo *nodeid2nodeinfo(int nodeid, gfp_t alloc)
>> +{
>> + struct nodeinfo *ni;
>> + int r;
>> + int n;
>> +
>> + down_read(&nodeinfo_lock);
>
> Given that this function can sleep, I wonder if `alloc' is useful.
>
> I see lots of callers passing in a literal "0" for `alloc'. That's in fact
> a secret (GFP_ATOMIC & ~__GFP_HIGH). I doubt if that's what you really
> meant. Particularly as the code could at least have used __GFP_WAIT (aka
> GFP_NOIO) which is much, much more reliable than "0". In fact "0" is the
> least reliable mode possible.
>
> IOW, this is all bollixed up.
When 0 is passed into nodeid2nodeinfo the function does not try to allocate a
new structure at all. it's an indication that the caller only wants the nodeinfo
struct for that nodeid if there actually is one in existance.
I've tidied the function itself so it's more obvious, (and tidier!)
>> +/* Data received from remote end */
>> +static int receive_from_sock(void)
>> +{
>> + int ret = 0;
>> + struct msghdr msg;
>> + struct kvec iov[2];
>> + unsigned len;
>> + int r;
>> + struct sctp_sndrcvinfo *sinfo;
>> + struct cmsghdr *cmsg;
>> + struct nodeinfo *ni;
>> +
>> + /* These two are marginally too big for stack allocation, but this
>> + * function is (currently) only called by dlm_recvd so static should be
>> + * OK.
>> + */
>> + static struct sockaddr_storage msgname;
>> + static char incmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> whoa. This is globally singly-threaded code??
Yes. it is only ever run in the context of dlm_recvd.
>>
>> +static void initiate_association(int nodeid)
>> +{
>> + struct sockaddr_storage rem_addr;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Another static buffer to worry about. Globally singly-threaded code?
Yes. Only ever called by dlm_sendd.
>> +
>> +/* Send a message */
>> +static int send_to_sock(struct nodeinfo *ni)
>> +{
>> + int ret = 0;
>> + struct writequeue_entry *e;
>> + int len, offset;
>> + struct msghdr outmsg;
>> + static char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
>
> Singly-threaded?
Yep.
>>
>> +static void dealloc_nodeinfo(void)
>> +{
>> + int i;
>> +
>> + for (i=1; i<=max_nodeid; i++) {
>> + struct nodeinfo *ni = nodeid2nodeinfo(i, 0);
>> + if (ni) {
>> + idr_remove(&nodeinfo_idr, i);
>
> Didn't that need locking?
Not. it's only ever called at DLM shutdown after all the other threads
have been stopped.
>>
>> +static int write_list_empty(void)
>> +{
>> + int status;
>> +
>> + spin_lock_bh(&write_nodes_lock);
>> + status = list_empty(&write_nodes);
>> + spin_unlock_bh(&write_nodes_lock);
>> +
>> + return status;
>> +}
>
> This function's return value is meaningless. As soon as the lock gets
> dropped, the return value can get out of sync with reality.
>
> Looking at the caller, this _might_ happen to be OK, but it's a nasty and
> dangerous thing. Really the locking should be moved into the caller.
It's just an optimisation to allow the caller to schedule if there is no work
to do. if something arrives immediately afterwards then it will get picked up
when the process re-awakes (and it will be woken by that arrival).
The 'accepting' atomic has gone completely. as Andrew pointed out it didn't
really achieve much anyway. I suspect it was a plaster over some other
startup or shutdown bug to be honest.
Signed-off-by: Patrick Caulfield <pcaulfie@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>
Cc: Pavel Machek <pavel@ucw.cz>
2006-12-06 23:10:37 +08:00
|
|
|
out_connect:
|
2007-01-24 19:17:59 +08:00
|
|
|
mutex_unlock(&con->sock_mutex);
|
2017-09-12 16:55:14 +08:00
|
|
|
queue_work(send_workqueue, &con->swork);
|
|
|
|
cond_resched();
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static void clean_one_writequeue(struct connection *con)
|
|
|
|
{
|
2009-01-29 02:57:40 +08:00
|
|
|
struct writequeue_entry *e, *safe;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
|
|
|
spin_lock(&con->writequeue_lock);
|
2009-01-29 02:57:40 +08:00
|
|
|
list_for_each_entry_safe(e, safe, &con->writequeue, list) {
|
2006-11-03 00:19:21 +08:00
|
|
|
free_entry(e);
|
|
|
|
}
|
|
|
|
spin_unlock(&con->writequeue_lock);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Called from recovery when it knows that a node has
|
|
|
|
left the cluster */
|
|
|
|
int dlm_lowcomms_close(int nodeid)
|
|
|
|
{
|
|
|
|
struct connection *con;
|
2012-07-27 01:44:30 +08:00
|
|
|
struct dlm_node_addr *na;
|
2021-05-22 03:08:35 +08:00
|
|
|
int idx;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
|
|
|
log_print("closing connection to node %d", nodeid);
|
2021-05-22 03:08:35 +08:00
|
|
|
idx = srcu_read_lock(&connections_srcu);
|
2006-11-03 00:19:21 +08:00
|
|
|
con = nodeid2con(nodeid, 0);
|
|
|
|
if (con) {
|
2009-08-12 05:18:23 +08:00
|
|
|
set_bit(CF_CLOSE, &con->flags);
|
2015-08-12 06:22:21 +08:00
|
|
|
close_connection(con, true, true, true);
|
2006-11-03 00:19:21 +08:00
|
|
|
clean_one_writequeue(con);
|
2020-11-03 09:04:19 +08:00
|
|
|
if (con->othercon)
|
|
|
|
clean_one_writequeue(con->othercon);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
2021-05-22 03:08:35 +08:00
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
2012-07-27 01:44:30 +08:00
|
|
|
|
|
|
|
spin_lock(&dlm_node_addrs_spin);
|
|
|
|
na = find_node_addr(nodeid);
|
|
|
|
if (na) {
|
|
|
|
list_del(&na->list);
|
|
|
|
while (na->addr_count--)
|
|
|
|
kfree(na->addr[na->addr_count]);
|
|
|
|
kfree(na);
|
|
|
|
}
|
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2007-04-17 22:39:57 +08:00
|
|
|
/* Receive workqueue function */
|
2007-01-15 22:33:34 +08:00
|
|
|
static void process_recv_sockets(struct work_struct *work)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2007-01-15 22:33:34 +08:00
|
|
|
struct connection *con = container_of(work, struct connection, rwork);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2007-01-15 22:33:34 +08:00
|
|
|
clear_bit(CF_READ_PENDING, &con->flags);
|
2021-07-17 04:22:45 +08:00
|
|
|
receive_from_sock(con);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
static void process_listen_recv_socket(struct work_struct *work)
|
|
|
|
{
|
|
|
|
accept_from_sock(&listen_con);
|
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:43 +08:00
|
|
|
static void dlm_connect(struct connection *con)
|
|
|
|
{
|
|
|
|
struct sockaddr_storage addr;
|
|
|
|
int result, addr_len;
|
|
|
|
struct socket *sock;
|
|
|
|
unsigned int mark;
|
|
|
|
|
|
|
|
/* Some odd races can cause double-connects, ignore them */
|
|
|
|
if (con->retries++ > MAX_CONNECT_RETRIES)
|
|
|
|
return;
|
|
|
|
|
|
|
|
if (con->sock) {
|
|
|
|
log_print("node %d already connected.", con->nodeid);
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
memset(&addr, 0, sizeof(addr));
|
|
|
|
result = nodeid_to_addr(con->nodeid, &addr, NULL,
|
|
|
|
dlm_proto_ops->try_new_addr, &mark);
|
|
|
|
if (result < 0) {
|
|
|
|
log_print("no address for nodeid %d", con->nodeid);
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Create a socket to communicate with */
|
|
|
|
result = sock_create_kern(&init_net, dlm_local_addr[0]->ss_family,
|
|
|
|
SOCK_STREAM, dlm_proto_ops->proto, &sock);
|
|
|
|
if (result < 0)
|
|
|
|
goto socket_err;
|
|
|
|
|
|
|
|
sock_set_mark(sock->sk, mark);
|
|
|
|
dlm_proto_ops->sockopts(sock);
|
|
|
|
|
|
|
|
add_sock(sock, con);
|
|
|
|
|
|
|
|
result = dlm_proto_ops->bind(sock);
|
|
|
|
if (result < 0)
|
|
|
|
goto add_sock_err;
|
|
|
|
|
|
|
|
log_print_ratelimited("connecting to %d", con->nodeid);
|
|
|
|
make_sockaddr(&addr, dlm_config.ci_tcp_port, &addr_len);
|
|
|
|
result = dlm_proto_ops->connect(con, sock, (struct sockaddr *)&addr,
|
|
|
|
addr_len);
|
|
|
|
if (result < 0)
|
|
|
|
goto add_sock_err;
|
|
|
|
|
|
|
|
return;
|
|
|
|
|
|
|
|
add_sock_err:
|
|
|
|
dlm_close_sock(&con->sock);
|
|
|
|
|
|
|
|
socket_err:
|
|
|
|
/*
|
|
|
|
* Some errors are fatal and this list might need adjusting. For other
|
|
|
|
* errors we try again until the max number of retries is reached.
|
|
|
|
*/
|
|
|
|
if (result != -EHOSTUNREACH &&
|
|
|
|
result != -ENETUNREACH &&
|
|
|
|
result != -ENETDOWN &&
|
|
|
|
result != -EINVAL &&
|
|
|
|
result != -EPROTONOSUPPORT) {
|
|
|
|
log_print("connect %d try %d error %d", con->nodeid,
|
|
|
|
con->retries, result);
|
|
|
|
msleep(1000);
|
|
|
|
lowcomms_connect_sock(con);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2007-04-17 22:39:57 +08:00
|
|
|
/* Send workqueue function */
|
2007-01-15 22:33:34 +08:00
|
|
|
static void process_send_sockets(struct work_struct *work)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2007-01-15 22:33:34 +08:00
|
|
|
struct connection *con = container_of(work, struct connection, swork);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2021-05-22 03:08:36 +08:00
|
|
|
WARN_ON(test_bit(CF_IS_OTHERCON, &con->flags));
|
|
|
|
|
2017-09-12 17:01:16 +08:00
|
|
|
clear_bit(CF_WRITE_PENDING, &con->flags);
|
2021-05-22 03:08:37 +08:00
|
|
|
|
2021-05-22 03:08:46 +08:00
|
|
|
if (test_and_clear_bit(CF_RECONNECT, &con->flags)) {
|
2021-05-22 03:08:37 +08:00
|
|
|
close_connection(con, false, false, true);
|
2021-05-22 03:08:46 +08:00
|
|
|
dlm_midcomms_unack_msg_resend(con->nodeid);
|
|
|
|
}
|
2021-05-22 03:08:37 +08:00
|
|
|
|
2021-07-17 04:22:43 +08:00
|
|
|
if (con->sock == NULL) {
|
2021-05-22 03:08:37 +08:00
|
|
|
if (test_and_clear_bit(CF_DELAY_CONNECT, &con->flags))
|
|
|
|
msleep(1000);
|
2021-07-17 04:22:43 +08:00
|
|
|
|
|
|
|
mutex_lock(&con->sock_mutex);
|
|
|
|
dlm_connect(con);
|
|
|
|
mutex_unlock(&con->sock_mutex);
|
2021-05-22 03:08:37 +08:00
|
|
|
}
|
2021-07-17 04:22:43 +08:00
|
|
|
|
2017-09-12 16:55:14 +08:00
|
|
|
if (!list_empty(&con->writequeue))
|
2009-08-12 05:18:23 +08:00
|
|
|
send_to_sock(con);
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2007-01-15 22:33:34 +08:00
|
|
|
static void work_stop(void)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2021-06-02 21:45:15 +08:00
|
|
|
if (recv_workqueue) {
|
2019-04-02 20:37:10 +08:00
|
|
|
destroy_workqueue(recv_workqueue);
|
2021-06-02 21:45:15 +08:00
|
|
|
recv_workqueue = NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (send_workqueue) {
|
2019-04-02 20:37:10 +08:00
|
|
|
destroy_workqueue(send_workqueue);
|
2021-06-02 21:45:15 +08:00
|
|
|
send_workqueue = NULL;
|
|
|
|
}
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2007-01-15 22:33:34 +08:00
|
|
|
static int work_start(void)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2021-06-02 21:45:17 +08:00
|
|
|
recv_workqueue = alloc_ordered_workqueue("dlm_recv", WQ_MEM_RECLAIM);
|
2010-12-14 03:42:24 +08:00
|
|
|
if (!recv_workqueue) {
|
|
|
|
log_print("can't start dlm_recv");
|
|
|
|
return -ENOMEM;
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2021-06-02 21:45:17 +08:00
|
|
|
send_workqueue = alloc_ordered_workqueue("dlm_send", WQ_MEM_RECLAIM);
|
2010-12-14 03:42:24 +08:00
|
|
|
if (!send_workqueue) {
|
|
|
|
log_print("can't start dlm_send");
|
2007-01-15 22:33:34 +08:00
|
|
|
destroy_workqueue(recv_workqueue);
|
2021-06-02 21:45:15 +08:00
|
|
|
recv_workqueue = NULL;
|
2010-12-14 03:42:24 +08:00
|
|
|
return -ENOMEM;
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-03-02 06:05:20 +08:00
|
|
|
static void shutdown_conn(struct connection *con)
|
|
|
|
{
|
2021-07-17 04:22:40 +08:00
|
|
|
if (dlm_proto_ops->shutdown_action)
|
|
|
|
dlm_proto_ops->shutdown_action(con);
|
2021-03-02 06:05:20 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
void dlm_lowcomms_shutdown(void)
|
|
|
|
{
|
2021-05-22 03:08:35 +08:00
|
|
|
int idx;
|
|
|
|
|
2021-03-02 06:05:20 +08:00
|
|
|
/* Set all the flags to prevent any
|
|
|
|
* socket activity.
|
|
|
|
*/
|
|
|
|
dlm_allow_conn = 0;
|
|
|
|
|
|
|
|
if (recv_workqueue)
|
|
|
|
flush_workqueue(recv_workqueue);
|
|
|
|
if (send_workqueue)
|
|
|
|
flush_workqueue(send_workqueue);
|
|
|
|
|
|
|
|
dlm_close_sock(&listen_con.sock);
|
|
|
|
|
2021-05-22 03:08:35 +08:00
|
|
|
idx = srcu_read_lock(&connections_srcu);
|
2021-03-02 06:05:20 +08:00
|
|
|
foreach_conn(shutdown_conn);
|
2021-05-22 03:08:35 +08:00
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
2021-03-02 06:05:20 +08:00
|
|
|
}
|
|
|
|
|
2017-09-12 16:55:40 +08:00
|
|
|
static void _stop_conn(struct connection *con, bool and_other)
|
2006-11-03 00:19:21 +08:00
|
|
|
{
|
2017-09-12 16:55:40 +08:00
|
|
|
mutex_lock(&con->sock_mutex);
|
2017-09-12 17:01:24 +08:00
|
|
|
set_bit(CF_CLOSE, &con->flags);
|
2017-09-12 16:55:40 +08:00
|
|
|
set_bit(CF_READ_PENDING, &con->flags);
|
2017-09-12 17:01:16 +08:00
|
|
|
set_bit(CF_WRITE_PENDING, &con->flags);
|
2017-09-12 17:01:55 +08:00
|
|
|
if (con->sock && con->sock->sk) {
|
2021-11-15 21:57:06 +08:00
|
|
|
lock_sock(con->sock->sk);
|
2009-01-29 02:57:40 +08:00
|
|
|
con->sock->sk->sk_user_data = NULL;
|
2021-11-15 21:57:06 +08:00
|
|
|
release_sock(con->sock->sk);
|
2017-09-12 17:01:55 +08:00
|
|
|
}
|
2017-09-12 16:55:40 +08:00
|
|
|
if (con->othercon && and_other)
|
|
|
|
_stop_conn(con->othercon, false);
|
|
|
|
mutex_unlock(&con->sock_mutex);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void stop_conn(struct connection *con)
|
|
|
|
{
|
|
|
|
_stop_conn(con, true);
|
2009-01-29 02:57:40 +08:00
|
|
|
}
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2020-09-24 22:31:26 +08:00
|
|
|
static void connection_release(struct rcu_head *rcu)
|
|
|
|
{
|
|
|
|
struct connection *con = container_of(rcu, struct connection, rcu);
|
|
|
|
|
|
|
|
kfree(con->rx_buf);
|
|
|
|
kfree(con);
|
|
|
|
}
|
|
|
|
|
2009-01-29 02:57:40 +08:00
|
|
|
static void free_conn(struct connection *con)
|
|
|
|
{
|
2015-08-12 06:22:21 +08:00
|
|
|
close_connection(con, true, true, true);
|
2020-08-28 03:02:49 +08:00
|
|
|
spin_lock(&connections_lock);
|
|
|
|
hlist_del_rcu(&con->list);
|
|
|
|
spin_unlock(&connections_lock);
|
2020-08-28 03:02:53 +08:00
|
|
|
if (con->othercon) {
|
|
|
|
clean_one_writequeue(con->othercon);
|
2020-11-03 09:04:16 +08:00
|
|
|
call_srcu(&connections_srcu, &con->othercon->rcu,
|
|
|
|
connection_release);
|
2020-08-28 03:02:53 +08:00
|
|
|
}
|
2020-08-28 03:02:52 +08:00
|
|
|
clean_one_writequeue(con);
|
2020-11-03 09:04:16 +08:00
|
|
|
call_srcu(&connections_srcu, &con->rcu, connection_release);
|
2009-01-29 02:57:40 +08:00
|
|
|
}
|
|
|
|
|
2017-09-12 16:55:40 +08:00
|
|
|
static void work_flush(void)
|
|
|
|
{
|
2021-05-22 03:08:35 +08:00
|
|
|
int ok;
|
2017-09-12 16:55:40 +08:00
|
|
|
int i;
|
|
|
|
struct connection *con;
|
|
|
|
|
|
|
|
do {
|
|
|
|
ok = 1;
|
|
|
|
foreach_conn(stop_conn);
|
2019-04-02 20:37:10 +08:00
|
|
|
if (recv_workqueue)
|
|
|
|
flush_workqueue(recv_workqueue);
|
|
|
|
if (send_workqueue)
|
|
|
|
flush_workqueue(send_workqueue);
|
2017-09-12 16:55:40 +08:00
|
|
|
for (i = 0; i < CONN_HASH_SIZE && ok; i++) {
|
2020-08-28 03:02:49 +08:00
|
|
|
hlist_for_each_entry_rcu(con, &connection_hash[i],
|
|
|
|
list) {
|
2017-09-12 16:55:40 +08:00
|
|
|
ok &= test_bit(CF_READ_PENDING, &con->flags);
|
2017-09-12 17:01:16 +08:00
|
|
|
ok &= test_bit(CF_WRITE_PENDING, &con->flags);
|
|
|
|
if (con->othercon) {
|
2017-09-12 16:55:40 +08:00
|
|
|
ok &= test_bit(CF_READ_PENDING,
|
|
|
|
&con->othercon->flags);
|
2017-09-12 17:01:16 +08:00
|
|
|
ok &= test_bit(CF_WRITE_PENDING,
|
|
|
|
&con->othercon->flags);
|
|
|
|
}
|
2017-09-12 16:55:40 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
} while (!ok);
|
|
|
|
}
|
|
|
|
|
2009-01-29 02:57:40 +08:00
|
|
|
void dlm_lowcomms_stop(void)
|
|
|
|
{
|
2021-05-22 03:08:35 +08:00
|
|
|
int idx;
|
|
|
|
|
|
|
|
idx = srcu_read_lock(&connections_srcu);
|
2017-09-12 16:55:40 +08:00
|
|
|
work_flush();
|
2016-10-08 21:14:37 +08:00
|
|
|
foreach_conn(free_conn);
|
2021-05-22 03:08:35 +08:00
|
|
|
srcu_read_unlock(&connections_srcu, idx);
|
2007-01-15 22:33:34 +08:00
|
|
|
work_stop();
|
2020-08-28 03:02:50 +08:00
|
|
|
deinit_local();
|
2021-07-17 04:22:40 +08:00
|
|
|
|
|
|
|
dlm_proto_ops = NULL;
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:41 +08:00
|
|
|
static int dlm_listen_for_all(void)
|
|
|
|
{
|
|
|
|
struct socket *sock;
|
|
|
|
int result;
|
|
|
|
|
|
|
|
log_print("Using %s for communications",
|
|
|
|
dlm_proto_ops->name);
|
|
|
|
|
2021-07-17 04:22:42 +08:00
|
|
|
result = dlm_proto_ops->listen_validate();
|
|
|
|
if (result < 0)
|
|
|
|
return result;
|
2021-07-17 04:22:41 +08:00
|
|
|
|
|
|
|
result = sock_create_kern(&init_net, dlm_local_addr[0]->ss_family,
|
|
|
|
SOCK_STREAM, dlm_proto_ops->proto, &sock);
|
|
|
|
if (result < 0) {
|
2021-11-03 03:17:10 +08:00
|
|
|
log_print("Can't create comms socket: %d", result);
|
2022-04-05 04:06:28 +08:00
|
|
|
return result;
|
2021-07-17 04:22:41 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
sock_set_mark(sock->sk, dlm_config.ci_mark);
|
|
|
|
dlm_proto_ops->listen_sockopts(sock);
|
|
|
|
|
|
|
|
result = dlm_proto_ops->listen_bind(sock);
|
|
|
|
if (result < 0)
|
|
|
|
goto out;
|
|
|
|
|
|
|
|
save_listen_callbacks(sock);
|
|
|
|
add_listen_sock(sock, &listen_con);
|
|
|
|
|
|
|
|
INIT_WORK(&listen_con.rwork, process_listen_recv_socket);
|
|
|
|
result = sock->ops->listen(sock, 5);
|
|
|
|
if (result < 0) {
|
|
|
|
dlm_close_sock(&listen_con.sock);
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
out:
|
|
|
|
sock_release(sock);
|
|
|
|
return result;
|
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:43 +08:00
|
|
|
static int dlm_tcp_bind(struct socket *sock)
|
|
|
|
{
|
|
|
|
struct sockaddr_storage src_addr;
|
|
|
|
int result, addr_len;
|
|
|
|
|
|
|
|
/* Bind to our cluster-known address connecting to avoid
|
|
|
|
* routing problems.
|
|
|
|
*/
|
|
|
|
memcpy(&src_addr, dlm_local_addr[0], sizeof(src_addr));
|
|
|
|
make_sockaddr(&src_addr, 0, &addr_len);
|
|
|
|
|
|
|
|
result = sock->ops->bind(sock, (struct sockaddr *)&src_addr,
|
|
|
|
addr_len);
|
|
|
|
if (result < 0) {
|
|
|
|
/* This *may* not indicate a critical error */
|
|
|
|
log_print("could not bind for connect: %d", result);
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int dlm_tcp_connect(struct connection *con, struct socket *sock,
|
|
|
|
struct sockaddr *addr, int addr_len)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
ret = sock->ops->connect(sock, addr, addr_len, O_NONBLOCK);
|
|
|
|
switch (ret) {
|
|
|
|
case -EINPROGRESS:
|
|
|
|
fallthrough;
|
|
|
|
case 0:
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:41 +08:00
|
|
|
static int dlm_tcp_listen_validate(void)
|
|
|
|
{
|
|
|
|
/* We don't support multi-homed hosts */
|
|
|
|
if (dlm_local_count > 1) {
|
|
|
|
log_print("TCP protocol can't handle multi-homed hosts, try SCTP");
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void dlm_tcp_sockopts(struct socket *sock)
|
|
|
|
{
|
|
|
|
/* Turn off Nagle's algorithm */
|
|
|
|
tcp_sock_set_nodelay(sock->sk);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void dlm_tcp_listen_sockopts(struct socket *sock)
|
|
|
|
{
|
|
|
|
dlm_tcp_sockopts(sock);
|
|
|
|
sock_set_reuseaddr(sock->sk);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int dlm_tcp_listen_bind(struct socket *sock)
|
|
|
|
{
|
|
|
|
int addr_len;
|
|
|
|
|
|
|
|
/* Bind to our port */
|
|
|
|
make_sockaddr(dlm_local_addr[0], dlm_config.ci_tcp_port, &addr_len);
|
|
|
|
return sock->ops->bind(sock, (struct sockaddr *)dlm_local_addr[0],
|
|
|
|
addr_len);
|
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:40 +08:00
|
|
|
static const struct dlm_proto_ops dlm_tcp_ops = {
|
2021-07-17 04:22:41 +08:00
|
|
|
.name = "TCP",
|
|
|
|
.proto = IPPROTO_TCP,
|
2021-07-17 04:22:43 +08:00
|
|
|
.connect = dlm_tcp_connect,
|
|
|
|
.sockopts = dlm_tcp_sockopts,
|
|
|
|
.bind = dlm_tcp_bind,
|
2021-07-17 04:22:41 +08:00
|
|
|
.listen_validate = dlm_tcp_listen_validate,
|
|
|
|
.listen_sockopts = dlm_tcp_listen_sockopts,
|
|
|
|
.listen_bind = dlm_tcp_listen_bind,
|
2021-07-17 04:22:40 +08:00
|
|
|
.shutdown_action = dlm_tcp_shutdown,
|
|
|
|
.eof_condition = tcp_eof_condition,
|
|
|
|
};
|
|
|
|
|
2021-07-17 04:22:43 +08:00
|
|
|
static int dlm_sctp_bind(struct socket *sock)
|
|
|
|
{
|
|
|
|
return sctp_bind_addrs(sock, 0);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int dlm_sctp_connect(struct connection *con, struct socket *sock,
|
|
|
|
struct sockaddr *addr, int addr_len)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Make sock->ops->connect() function return in specified time,
|
|
|
|
* since O_NONBLOCK argument in connect() function does not work here,
|
|
|
|
* then, we should restore the default value of this attribute.
|
|
|
|
*/
|
|
|
|
sock_set_sndtimeo(sock->sk, 5);
|
|
|
|
ret = sock->ops->connect(sock, addr, addr_len, 0);
|
|
|
|
sock_set_sndtimeo(sock->sk, 0);
|
|
|
|
if (ret < 0)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
if (!test_and_set_bit(CF_CONNECTED, &con->flags))
|
|
|
|
log_print("successful connected to node %d", con->nodeid);
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:42 +08:00
|
|
|
static int dlm_sctp_listen_validate(void)
|
|
|
|
{
|
|
|
|
if (!IS_ENABLED(CONFIG_IP_SCTP)) {
|
|
|
|
log_print("SCTP is not enabled by this kernel");
|
|
|
|
return -EOPNOTSUPP;
|
|
|
|
}
|
|
|
|
|
|
|
|
request_module("sctp");
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:41 +08:00
|
|
|
static int dlm_sctp_bind_listen(struct socket *sock)
|
|
|
|
{
|
|
|
|
return sctp_bind_addrs(sock, dlm_config.ci_tcp_port);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void dlm_sctp_sockopts(struct socket *sock)
|
|
|
|
{
|
|
|
|
/* Turn off Nagle's algorithm */
|
|
|
|
sctp_sock_set_nodelay(sock->sk);
|
|
|
|
sock_set_rcvbuf(sock->sk, NEEDED_RMEM);
|
|
|
|
}
|
|
|
|
|
2021-07-17 04:22:40 +08:00
|
|
|
static const struct dlm_proto_ops dlm_sctp_ops = {
|
2021-07-17 04:22:41 +08:00
|
|
|
.name = "SCTP",
|
|
|
|
.proto = IPPROTO_SCTP,
|
2021-07-17 04:22:43 +08:00
|
|
|
.try_new_addr = true,
|
|
|
|
.connect = dlm_sctp_connect,
|
|
|
|
.sockopts = dlm_sctp_sockopts,
|
|
|
|
.bind = dlm_sctp_bind,
|
2021-07-17 04:22:42 +08:00
|
|
|
.listen_validate = dlm_sctp_listen_validate,
|
2021-07-17 04:22:41 +08:00
|
|
|
.listen_sockopts = dlm_sctp_sockopts,
|
|
|
|
.listen_bind = dlm_sctp_bind_listen,
|
2021-07-17 04:22:40 +08:00
|
|
|
};
|
|
|
|
|
2006-11-03 00:19:21 +08:00
|
|
|
int dlm_lowcomms_start(void)
|
|
|
|
{
|
2007-04-17 22:39:57 +08:00
|
|
|
int error = -EINVAL;
|
2009-01-29 02:57:40 +08:00
|
|
|
int i;
|
|
|
|
|
|
|
|
for (i = 0; i < CONN_HASH_SIZE; i++)
|
|
|
|
INIT_HLIST_HEAD(&connection_hash[i]);
|
2006-11-03 00:19:21 +08:00
|
|
|
|
2007-04-17 22:39:57 +08:00
|
|
|
init_local();
|
|
|
|
if (!dlm_local_count) {
|
2007-04-27 02:46:49 +08:00
|
|
|
error = -ENOTCONN;
|
2006-11-03 00:19:21 +08:00
|
|
|
log_print("no local IP address has been set");
|
2012-03-31 00:46:08 +08:00
|
|
|
goto fail;
|
2006-11-03 00:19:21 +08:00
|
|
|
}
|
|
|
|
|
2020-11-03 09:04:25 +08:00
|
|
|
INIT_WORK(&listen_con.rwork, process_listen_recv_socket);
|
|
|
|
|
2012-03-31 00:46:08 +08:00
|
|
|
error = work_start();
|
|
|
|
if (error)
|
2021-06-02 21:45:15 +08:00
|
|
|
goto fail_local;
|
2012-03-31 00:46:08 +08:00
|
|
|
|
|
|
|
dlm_allow_conn = 1;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
|
|
|
/* Start listening */
|
2021-06-02 21:45:19 +08:00
|
|
|
switch (dlm_config.ci_protocol) {
|
|
|
|
case DLM_PROTO_TCP:
|
2021-07-17 04:22:40 +08:00
|
|
|
dlm_proto_ops = &dlm_tcp_ops;
|
2021-06-02 21:45:19 +08:00
|
|
|
break;
|
|
|
|
case DLM_PROTO_SCTP:
|
2021-07-17 04:22:40 +08:00
|
|
|
dlm_proto_ops = &dlm_sctp_ops;
|
2021-06-02 21:45:19 +08:00
|
|
|
break;
|
|
|
|
default:
|
|
|
|
log_print("Invalid protocol identifier %d set",
|
|
|
|
dlm_config.ci_protocol);
|
|
|
|
error = -EINVAL;
|
2021-07-17 04:22:41 +08:00
|
|
|
goto fail_proto_ops;
|
2021-06-02 21:45:19 +08:00
|
|
|
}
|
2021-07-17 04:22:41 +08:00
|
|
|
|
|
|
|
error = dlm_listen_for_all();
|
2006-11-03 00:19:21 +08:00
|
|
|
if (error)
|
2021-07-17 04:22:41 +08:00
|
|
|
goto fail_listen;
|
2006-11-03 00:19:21 +08:00
|
|
|
|
|
|
|
return 0;
|
|
|
|
|
2021-07-17 04:22:41 +08:00
|
|
|
fail_listen:
|
|
|
|
dlm_proto_ops = NULL;
|
|
|
|
fail_proto_ops:
|
2012-03-31 00:46:08 +08:00
|
|
|
dlm_allow_conn = 0;
|
2020-11-03 09:04:25 +08:00
|
|
|
dlm_close_sock(&listen_con.sock);
|
2021-06-02 21:45:15 +08:00
|
|
|
work_stop();
|
|
|
|
fail_local:
|
|
|
|
deinit_local();
|
2012-03-31 00:46:08 +08:00
|
|
|
fail:
|
2006-11-03 00:19:21 +08:00
|
|
|
return error;
|
|
|
|
}
|
2012-07-27 01:44:30 +08:00
|
|
|
|
|
|
|
void dlm_lowcomms_exit(void)
|
|
|
|
{
|
|
|
|
struct dlm_node_addr *na, *safe;
|
|
|
|
|
|
|
|
spin_lock(&dlm_node_addrs_spin);
|
|
|
|
list_for_each_entry_safe(na, safe, &dlm_node_addrs, list) {
|
|
|
|
list_del(&na->list);
|
|
|
|
while (na->addr_count--)
|
|
|
|
kfree(na->addr[na->addr_count]);
|
|
|
|
kfree(na);
|
|
|
|
}
|
|
|
|
spin_unlock(&dlm_node_addrs_spin);
|
|
|
|
}
|