fs: dlm: use GFP_ZERO for page buffer
authorAlexander Aring <aahringo@redhat.com>
Mon, 1 Mar 2021 22:05:15 +0000 (17:05 -0500)
committerDavid Teigland <teigland@redhat.com>
Tue, 9 Mar 2021 14:56:42 +0000 (08:56 -0600)
This patch uses GFP_ZERO for allocate a page for the internal dlm
sending buffer allocator instead of calling memset zero after every
allocation. An already allocated space will never be reused again.

Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
fs/dlm/lock.c
fs/dlm/lowcomms.c
fs/dlm/rcom.c

index 002123efc6b052346ee8c7093789fbbb08929bd1..b93df39d09153d1c0617a5687017fc6aab0ae131 100644 (file)
@@ -3541,8 +3541,6 @@ static int _create_message(struct dlm_ls *ls, int mb_len,
        if (!mh)
                return -ENOBUFS;
 
-       memset(mb, 0, mb_len);
-
        ms = (struct dlm_message *) mb;
 
        ms->m_header.h_version = (DLM_HEADER_MAJOR | DLM_HEADER_MINOR);
index 206bd30feb29ac58be879743795974090ad66db8..97d0c93dd64475cacc48e066c2463778c65987e7 100644 (file)
@@ -1355,7 +1355,7 @@ static struct writequeue_entry *new_writequeue_entry(struct connection *con,
        if (!entry)
                return NULL;
 
-       entry->page = alloc_page(allocation);
+       entry->page = alloc_page(allocation | __GFP_ZERO);
        if (!entry->page) {
                kfree(entry);
                return NULL;
index 73ddee5159d79cebcc15f896f8d099a53ac835db..f5b1bd65728d57130ca8653a388e54e9b55eccc5 100644 (file)
@@ -41,7 +41,6 @@ static int create_rcom(struct dlm_ls *ls, int to_nodeid, int type, int len,
                          to_nodeid, type, len);
                return -ENOBUFS;
        }
-       memset(mb, 0, mb_len);
 
        rc = (struct dlm_rcom *) mb;
 
@@ -462,7 +461,6 @@ int dlm_send_ls_not_ready(int nodeid, struct dlm_rcom *rc_in)
        mh = dlm_lowcomms_get_buffer(nodeid, mb_len, GFP_NOFS, &mb);
        if (!mh)
                return -ENOBUFS;
-       memset(mb, 0, mb_len);
 
        rc = (struct dlm_rcom *) mb;