Commit 030b0772 authored by Dan Williams's avatar Dan Williams

async_pq: rename scribble page

The global scribble page is used as a temporary destination buffer when
disabling the P or Q result is requested.  The local scribble buffer
contains memory for performing address conversions.  Rename the global
variable to avoid confusion.
Signed-off-by: default avatarDan Williams <dan.j.williams@intel.com>

parent 5676470f
...@@ -26,9 +26,10 @@ ...@@ -26,9 +26,10 @@
#include <linux/async_tx.h> #include <linux/async_tx.h>
/** /**
* scribble - space to hold throwaway P buffer for synchronous gen_syndrome * pq_scribble_page - space to hold throwaway P or Q buffer for
* synchronous gen_syndrome
*/ */
static struct page *scribble; static struct page *pq_scribble_page;
/* the struct page *blocks[] parameter passed to async_gen_syndrome() /* the struct page *blocks[] parameter passed to async_gen_syndrome()
* and async_syndrome_val() contains the 'P' destination address at * and async_syndrome_val() contains the 'P' destination address at
...@@ -226,11 +227,11 @@ async_gen_syndrome(struct page **blocks, unsigned int offset, int disks, ...@@ -226,11 +227,11 @@ async_gen_syndrome(struct page **blocks, unsigned int offset, int disks,
async_tx_quiesce(&submit->depend_tx); async_tx_quiesce(&submit->depend_tx);
if (!P(blocks, disks)) { if (!P(blocks, disks)) {
P(blocks, disks) = scribble; P(blocks, disks) = pq_scribble_page;
BUG_ON(len + offset > PAGE_SIZE); BUG_ON(len + offset > PAGE_SIZE);
} }
if (!Q(blocks, disks)) { if (!Q(blocks, disks)) {
Q(blocks, disks) = scribble; Q(blocks, disks) = pq_scribble_page;
BUG_ON(len + offset > PAGE_SIZE); BUG_ON(len + offset > PAGE_SIZE);
} }
do_sync_gen_syndrome(blocks, offset, disks, len, submit); do_sync_gen_syndrome(blocks, offset, disks, len, submit);
...@@ -384,9 +385,9 @@ EXPORT_SYMBOL_GPL(async_syndrome_val); ...@@ -384,9 +385,9 @@ EXPORT_SYMBOL_GPL(async_syndrome_val);
static int __init async_pq_init(void) static int __init async_pq_init(void)
{ {
scribble = alloc_page(GFP_KERNEL); pq_scribble_page = alloc_page(GFP_KERNEL);
if (scribble) if (pq_scribble_page)
return 0; return 0;
pr_err("%s: failed to allocate required spare page\n", __func__); pr_err("%s: failed to allocate required spare page\n", __func__);
...@@ -396,7 +397,7 @@ static int __init async_pq_init(void) ...@@ -396,7 +397,7 @@ static int __init async_pq_init(void)
static void __exit async_pq_exit(void) static void __exit async_pq_exit(void)
{ {
put_page(scribble); put_page(pq_scribble_page);
} }
module_init(async_pq_init); module_init(async_pq_init);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment