2017-12-21 14:35:30 +03:00
// SPDX-License-Identifier: GPL-2.0
2010-01-06 11:47:10 +03:00
/*
* padata . c - generic interface to process data streams in parallel
*
2012-03-28 10:42:34 +04:00
* See Documentation / padata . txt for an api documentation .
*
2010-01-06 11:47:10 +03:00
* Copyright ( C ) 2008 , 2009 secunet Security Networks AG
* Copyright ( C ) 2008 , 2009 Steffen Klassert < steffen . klassert @ secunet . com >
*
* This program is free software ; you can redistribute it and / or modify it
* under the terms and conditions of the GNU General Public License ,
* version 2 , as published by the Free Software Foundation .
*
* This program is distributed in the hope it will be useful , but WITHOUT
* ANY WARRANTY ; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE . See the GNU General Public License for
* more details .
*
* You should have received a copy of the GNU General Public License along with
* this program ; if not , write to the Free Software Foundation , Inc . ,
* 51 Franklin St - Fifth Floor , Boston , MA 02110 - 1301 USA .
*/
2011-05-23 22:51:41 +04:00
# include <linux/export.h>
2010-01-06 11:47:10 +03:00
# include <linux/cpumask.h>
# include <linux/err.h>
# include <linux/cpu.h>
# include <linux/padata.h>
# include <linux/mutex.h>
# include <linux/sched.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 11:04:11 +03:00
# include <linux/slab.h>
2010-07-14 14:33:08 +04:00
# include <linux/sysfs.h>
2010-01-06 11:47:10 +03:00
# include <linux/rcupdate.h>
2016-09-06 20:04:49 +03:00
# include <linux/module.h>
2010-01-06 11:47:10 +03:00
2010-04-29 16:37:32 +04:00
# define MAX_OBJ_NUM 1000
2010-01-06 11:47:10 +03:00
2019-11-19 08:17:31 +03:00
static void padata_free_pd ( struct parallel_data * pd ) ;
2010-01-06 11:47:10 +03:00
static int padata_index_to_cpu ( struct parallel_data * pd , int cpu_index )
{
int cpu , target_cpu ;
2010-07-14 14:31:57 +04:00
target_cpu = cpumask_first ( pd - > cpumask . pcpu ) ;
2010-01-06 11:47:10 +03:00
for ( cpu = 0 ; cpu < cpu_index ; cpu + + )
2010-07-14 14:31:57 +04:00
target_cpu = cpumask_next ( target_cpu , pd - > cpumask . pcpu ) ;
2010-01-06 11:47:10 +03:00
return target_cpu ;
}
2019-09-06 04:40:28 +03:00
static int padata_cpu_hash ( struct parallel_data * pd , unsigned int seq_nr )
2010-01-06 11:47:10 +03:00
{
/*
* Hash the sequence numbers to the cpus by taking
* seq_nr mod . number of cpus in use .
*/
2019-09-06 04:40:28 +03:00
int cpu_index = seq_nr % cpumask_weight ( pd - > cpumask . pcpu ) ;
2010-01-06 11:47:10 +03:00
return padata_index_to_cpu ( pd , cpu_index ) ;
}
2010-07-14 14:31:57 +04:00
static void padata_parallel_worker ( struct work_struct * parallel_work )
2010-01-06 11:47:10 +03:00
{
2010-07-14 14:31:57 +04:00
struct padata_parallel_queue * pqueue ;
2010-01-06 11:47:10 +03:00
LIST_HEAD ( local_list ) ;
local_bh_disable ( ) ;
2010-07-14 14:31:57 +04:00
pqueue = container_of ( parallel_work ,
struct padata_parallel_queue , work ) ;
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
spin_lock ( & pqueue - > parallel . lock ) ;
list_replace_init ( & pqueue - > parallel . list , & local_list ) ;
spin_unlock ( & pqueue - > parallel . lock ) ;
2010-01-06 11:47:10 +03:00
while ( ! list_empty ( & local_list ) ) {
struct padata_priv * padata ;
padata = list_entry ( local_list . next ,
struct padata_priv , list ) ;
list_del_init ( & padata - > list ) ;
padata - > parallel ( padata ) ;
}
local_bh_enable ( ) ;
}
2010-05-19 07:44:27 +04:00
/**
2010-01-06 11:47:10 +03:00
* padata_do_parallel - padata parallelization function
*
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
* @ ps : padatashell
2010-01-06 11:47:10 +03:00
* @ padata : object to be parallelized
2019-09-06 04:40:24 +03:00
* @ cb_cpu : pointer to the CPU that the serialization callback function should
* run on . If it ' s not in the serial cpumask of @ pinst
* ( i . e . cpumask . cbcpu ) , this function selects a fallback CPU and if
* none found , returns - EINVAL .
2010-01-06 11:47:10 +03:00
*
* The parallelization callback function will run with BHs off .
* Note : Every object which is parallelized by padata_do_parallel
* must be seen by padata_do_serial .
*/
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
int padata_do_parallel ( struct padata_shell * ps ,
2019-09-06 04:40:24 +03:00
struct padata_priv * padata , int * cb_cpu )
2010-01-06 11:47:10 +03:00
{
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
struct padata_instance * pinst = ps - > pinst ;
2019-09-06 04:40:24 +03:00
int i , cpu , cpu_index , target_cpu , err ;
2010-07-14 14:31:57 +04:00
struct padata_parallel_queue * queue ;
2010-01-06 11:47:10 +03:00
struct parallel_data * pd ;
rcu_read_lock_bh ( ) ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
pd = rcu_dereference_bh ( ps - > pd ) ;
2010-01-06 11:47:10 +03:00
2010-07-07 17:32:02 +04:00
err = - EINVAL ;
2010-07-20 10:51:25 +04:00
if ( ! ( pinst - > flags & PADATA_INIT ) | | pinst - > flags & PADATA_INVALID )
2010-01-06 11:47:10 +03:00
goto out ;
2019-09-06 04:40:24 +03:00
if ( ! cpumask_test_cpu ( * cb_cpu , pd - > cpumask . cbcpu ) ) {
if ( ! cpumask_weight ( pd - > cpumask . cbcpu ) )
goto out ;
/* Select an alternate fallback CPU and notify the caller. */
cpu_index = * cb_cpu % cpumask_weight ( pd - > cpumask . cbcpu ) ;
cpu = cpumask_first ( pd - > cpumask . cbcpu ) ;
for ( i = 0 ; i < cpu_index ; i + + )
cpu = cpumask_next ( cpu , pd - > cpumask . cbcpu ) ;
* cb_cpu = cpu ;
}
2010-01-06 11:47:10 +03:00
err = - EBUSY ;
if ( ( pinst - > flags & PADATA_RESET ) )
goto out ;
if ( atomic_read ( & pd - > refcnt ) > = MAX_OBJ_NUM )
goto out ;
2010-07-07 17:32:02 +04:00
err = 0 ;
2010-01-06 11:47:10 +03:00
atomic_inc ( & pd - > refcnt ) ;
padata - > pd = pd ;
2019-09-06 04:40:24 +03:00
padata - > cb_cpu = * cb_cpu ;
2010-01-06 11:47:10 +03:00
2019-09-06 04:40:28 +03:00
padata - > seq_nr = atomic_inc_return ( & pd - > seq_nr ) ;
target_cpu = padata_cpu_hash ( pd , padata - > seq_nr ) ;
2017-09-08 21:57:11 +03:00
padata - > cpu = target_cpu ;
2010-07-14 14:31:57 +04:00
queue = per_cpu_ptr ( pd - > pqueue , target_cpu ) ;
2010-01-06 11:47:10 +03:00
spin_lock ( & queue - > parallel . lock ) ;
list_add_tail ( & padata - > list , & queue - > parallel . list ) ;
spin_unlock ( & queue - > parallel . lock ) ;
2019-09-06 04:40:28 +03:00
queue_work ( pinst - > parallel_wq , & queue - > work ) ;
2010-01-06 11:47:10 +03:00
out :
rcu_read_unlock_bh ( ) ;
return err ;
}
EXPORT_SYMBOL ( padata_do_parallel ) ;
2010-05-19 07:44:27 +04:00
/*
2019-09-06 04:40:28 +03:00
* padata_find_next - Find the next object that needs serialization .
2010-05-19 07:44:27 +04:00
*
* Return values are :
*
* A pointer to the control struct of the next object that needs
* serialization , if present in one of the percpu reorder queues .
*
2019-09-06 04:40:28 +03:00
* NULL , if the next object that needs serialization will
2010-05-19 07:44:27 +04:00
* be parallel processed by another cpu and is not yet present in
* the cpu ' s reorder queue .
*/
2019-09-06 04:40:28 +03:00
static struct padata_priv * padata_find_next ( struct parallel_data * pd ,
bool remove_object )
2010-01-06 11:47:10 +03:00
{
2012-12-06 13:16:23 +04:00
struct padata_parallel_queue * next_queue ;
2010-01-06 11:47:10 +03:00
struct padata_priv * padata ;
struct padata_list * reorder ;
2019-07-18 18:01:46 +03:00
int cpu = pd - > cpu ;
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
next_queue = per_cpu_ptr ( pd - > pqueue , cpu ) ;
2010-01-06 11:47:10 +03:00
reorder = & next_queue - > reorder ;
padata: avoid race in reordering
Under extremely heavy uses of padata, crashes occur, and with list
debugging turned on, this happens instead:
[87487.298728] WARNING: CPU: 1 PID: 882 at lib/list_debug.c:33
__list_add+0xae/0x130
[87487.301868] list_add corruption. prev->next should be next
(ffffb17abfc043d0), but was ffff8dba70872c80. (prev=ffff8dba70872b00).
[87487.339011] [<ffffffff9a53d075>] dump_stack+0x68/0xa3
[87487.342198] [<ffffffff99e119a1>] ? console_unlock+0x281/0x6d0
[87487.345364] [<ffffffff99d6b91f>] __warn+0xff/0x140
[87487.348513] [<ffffffff99d6b9aa>] warn_slowpath_fmt+0x4a/0x50
[87487.351659] [<ffffffff9a58b5de>] __list_add+0xae/0x130
[87487.354772] [<ffffffff9add5094>] ? _raw_spin_lock+0x64/0x70
[87487.357915] [<ffffffff99eefd66>] padata_reorder+0x1e6/0x420
[87487.361084] [<ffffffff99ef0055>] padata_do_serial+0xa5/0x120
padata_reorder calls list_add_tail with the list to which its adding
locked, which seems correct:
spin_lock(&squeue->serial.lock);
list_add_tail(&padata->list, &squeue->serial.list);
spin_unlock(&squeue->serial.lock);
This therefore leaves only place where such inconsistency could occur:
if padata->list is added at the same time on two different threads.
This pdata pointer comes from the function call to
padata_get_next(pd), which has in it the following block:
next_queue = per_cpu_ptr(pd->pqueue, cpu);
padata = NULL;
reorder = &next_queue->reorder;
if (!list_empty(&reorder->list)) {
padata = list_entry(reorder->list.next,
struct padata_priv, list);
spin_lock(&reorder->lock);
list_del_init(&padata->list);
atomic_dec(&pd->reorder_objects);
spin_unlock(&reorder->lock);
pd->processed++;
goto out;
}
out:
return padata;
I strongly suspect that the problem here is that two threads can race
on reorder list. Even though the deletion is locked, call to
list_entry is not locked, which means it's feasible that two threads
pick up the same padata object and subsequently call list_add_tail on
them at the same time. The fix is thus be hoist that lock outside of
that block.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Acked-by: Steffen Klassert <steffen.klassert@secunet.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2017-03-23 14:24:43 +03:00
spin_lock ( & reorder - > lock ) ;
2019-09-06 04:40:28 +03:00
if ( list_empty ( & reorder - > list ) ) {
spin_unlock ( & reorder - > lock ) ;
return NULL ;
}
2010-01-06 11:47:10 +03:00
2019-09-06 04:40:28 +03:00
padata = list_entry ( reorder - > list . next , struct padata_priv , list ) ;
2010-01-06 11:47:10 +03:00
2019-09-06 04:40:28 +03:00
/*
* Checks the rare case where two or more parallel jobs have hashed to
* the same CPU and one of the later ones finishes first .
*/
if ( padata - > seq_nr ! = pd - > processed ) {
padata: avoid race in reordering
Under extremely heavy uses of padata, crashes occur, and with list
debugging turned on, this happens instead:
[87487.298728] WARNING: CPU: 1 PID: 882 at lib/list_debug.c:33
__list_add+0xae/0x130
[87487.301868] list_add corruption. prev->next should be next
(ffffb17abfc043d0), but was ffff8dba70872c80. (prev=ffff8dba70872b00).
[87487.339011] [<ffffffff9a53d075>] dump_stack+0x68/0xa3
[87487.342198] [<ffffffff99e119a1>] ? console_unlock+0x281/0x6d0
[87487.345364] [<ffffffff99d6b91f>] __warn+0xff/0x140
[87487.348513] [<ffffffff99d6b9aa>] warn_slowpath_fmt+0x4a/0x50
[87487.351659] [<ffffffff9a58b5de>] __list_add+0xae/0x130
[87487.354772] [<ffffffff9add5094>] ? _raw_spin_lock+0x64/0x70
[87487.357915] [<ffffffff99eefd66>] padata_reorder+0x1e6/0x420
[87487.361084] [<ffffffff99ef0055>] padata_do_serial+0xa5/0x120
padata_reorder calls list_add_tail with the list to which its adding
locked, which seems correct:
spin_lock(&squeue->serial.lock);
list_add_tail(&padata->list, &squeue->serial.list);
spin_unlock(&squeue->serial.lock);
This therefore leaves only place where such inconsistency could occur:
if padata->list is added at the same time on two different threads.
This pdata pointer comes from the function call to
padata_get_next(pd), which has in it the following block:
next_queue = per_cpu_ptr(pd->pqueue, cpu);
padata = NULL;
reorder = &next_queue->reorder;
if (!list_empty(&reorder->list)) {
padata = list_entry(reorder->list.next,
struct padata_priv, list);
spin_lock(&reorder->lock);
list_del_init(&padata->list);
atomic_dec(&pd->reorder_objects);
spin_unlock(&reorder->lock);
pd->processed++;
goto out;
}
out:
return padata;
I strongly suspect that the problem here is that two threads can race
on reorder list. Even though the deletion is locked, call to
list_entry is not locked, which means it's feasible that two threads
pick up the same padata object and subsequently call list_add_tail on
them at the same time. The fix is thus be hoist that lock outside of
that block.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Acked-by: Steffen Klassert <steffen.klassert@secunet.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2017-03-23 14:24:43 +03:00
spin_unlock ( & reorder - > lock ) ;
2019-09-06 04:40:28 +03:00
return NULL ;
2010-01-06 11:47:10 +03:00
}
2019-09-06 04:40:28 +03:00
if ( remove_object ) {
list_del_init ( & padata - > list ) ;
atomic_dec ( & pd - > reorder_objects ) ;
+ + pd - > processed ;
pd - > cpu = cpumask_next_wrap ( cpu , pd - > cpumask . pcpu , - 1 , false ) ;
2010-01-06 11:47:10 +03:00
}
2019-09-06 04:40:28 +03:00
spin_unlock ( & reorder - > lock ) ;
2010-01-06 11:47:10 +03:00
return padata ;
}
static void padata_reorder ( struct parallel_data * pd )
{
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
struct padata_instance * pinst = pd - > ps - > pinst ;
2012-03-09 10:20:12 +04:00
int cb_cpu ;
2010-01-06 11:47:10 +03:00
struct padata_priv * padata ;
2010-07-14 14:31:57 +04:00
struct padata_serial_queue * squeue ;
2019-07-18 18:01:46 +03:00
struct padata_parallel_queue * next_queue ;
2010-01-06 11:47:10 +03:00
2010-05-19 07:44:27 +04:00
/*
* We need to ensure that only one cpu can work on dequeueing of
* the reorder queue the time . Calculating in which percpu reorder
* queue the next object will arrive takes some time . A spinlock
* would be highly contended . Also it is not clear in which order
* the objects arrive to the reorder queues . So a cpu could wait to
* get the lock just to notice that there is nothing to do at the
* moment . Therefore we use a trylock and let the holder of the lock
* care for all the objects enqueued during the holdtime of the lock .
*/
2010-01-06 11:47:10 +03:00
if ( ! spin_trylock_bh ( & pd - > lock ) )
2010-05-19 07:43:14 +04:00
return ;
2010-01-06 11:47:10 +03:00
while ( 1 ) {
2019-09-06 04:40:28 +03:00
padata = padata_find_next ( pd , true ) ;
2010-01-06 11:47:10 +03:00
2010-05-19 07:44:27 +04:00
/*
2017-04-12 11:40:19 +03:00
* If the next object that needs serialization is parallel
* processed by another cpu and is still on it ' s way to the
* cpu ' s reorder queue , nothing to do for now .
2010-05-19 07:44:27 +04:00
*/
2019-09-06 04:40:28 +03:00
if ( ! padata )
2010-01-06 11:47:10 +03:00
break ;
2012-03-09 10:20:12 +04:00
cb_cpu = padata - > cb_cpu ;
squeue = per_cpu_ptr ( pd - > squeue , cb_cpu ) ;
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
spin_lock ( & squeue - > serial . lock ) ;
list_add_tail ( & padata - > list , & squeue - > serial . list ) ;
spin_unlock ( & squeue - > serial . lock ) ;
2010-01-06 11:47:10 +03:00
2019-09-06 04:40:27 +03:00
queue_work_on ( cb_cpu , pinst - > serial_wq , & squeue - > work ) ;
2010-01-06 11:47:10 +03:00
}
spin_unlock_bh ( & pd - > lock ) ;
2010-05-19 07:44:27 +04:00
/*
* The next object that needs serialization might have arrived to
2019-07-18 18:01:46 +03:00
* the reorder queues in the meantime .
2019-07-16 19:32:53 +03:00
*
2019-07-18 18:01:46 +03:00
* Ensure reorder queue is read after pd - > lock is dropped so we see
* new objects from another task in padata_do_serial . Pairs with
2019-07-16 19:32:53 +03:00
* smp_mb__after_atomic in padata_do_serial .
2010-05-19 07:44:27 +04:00
*/
2019-07-16 19:32:53 +03:00
smp_mb ( ) ;
2010-01-06 11:47:10 +03:00
2019-07-18 18:01:46 +03:00
next_queue = per_cpu_ptr ( pd - > pqueue , pd - > cpu ) ;
2019-09-06 04:40:28 +03:00
if ( ! list_empty ( & next_queue - > reorder . list ) & &
padata_find_next ( pd , false ) )
2019-09-06 04:40:27 +03:00
queue_work ( pinst - > serial_wq , & pd - > reorder_work ) ;
2010-01-06 11:47:10 +03:00
}
2017-09-08 21:57:10 +03:00
static void invoke_padata_reorder ( struct work_struct * work )
{
struct parallel_data * pd ;
local_bh_disable ( ) ;
2019-07-18 18:01:46 +03:00
pd = container_of ( work , struct parallel_data , reorder_work ) ;
2017-09-08 21:57:10 +03:00
padata_reorder ( pd ) ;
local_bh_enable ( ) ;
}
2010-07-14 14:31:57 +04:00
static void padata_serial_worker ( struct work_struct * serial_work )
2010-01-06 11:47:10 +03:00
{
2010-07-14 14:31:57 +04:00
struct padata_serial_queue * squeue ;
2010-01-06 11:47:10 +03:00
struct parallel_data * pd ;
LIST_HEAD ( local_list ) ;
2019-11-19 08:17:31 +03:00
int cnt ;
2010-01-06 11:47:10 +03:00
local_bh_disable ( ) ;
2010-07-14 14:31:57 +04:00
squeue = container_of ( serial_work , struct padata_serial_queue , work ) ;
pd = squeue - > pd ;
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
spin_lock ( & squeue - > serial . lock ) ;
list_replace_init ( & squeue - > serial . list , & local_list ) ;
spin_unlock ( & squeue - > serial . lock ) ;
2010-01-06 11:47:10 +03:00
2019-11-19 08:17:31 +03:00
cnt = 0 ;
2010-01-06 11:47:10 +03:00
while ( ! list_empty ( & local_list ) ) {
struct padata_priv * padata ;
padata = list_entry ( local_list . next ,
struct padata_priv , list ) ;
list_del_init ( & padata - > list ) ;
padata - > serial ( padata ) ;
2019-11-19 08:17:31 +03:00
cnt + + ;
2010-01-06 11:47:10 +03:00
}
local_bh_enable ( ) ;
2019-11-19 08:17:31 +03:00
if ( atomic_sub_and_test ( cnt , & pd - > refcnt ) )
padata_free_pd ( pd ) ;
2010-01-06 11:47:10 +03:00
}
2010-05-19 07:44:27 +04:00
/**
2010-01-06 11:47:10 +03:00
* padata_do_serial - padata serialization function
*
* @ padata : object to be serialized .
*
* padata_do_serial must be called for every parallelized object .
* The serialization callback function will run with BHs off .
*/
void padata_do_serial ( struct padata_priv * padata )
{
2019-07-19 22:04:44 +03:00
struct parallel_data * pd = padata - > pd ;
struct padata_parallel_queue * pqueue = per_cpu_ptr ( pd - > pqueue ,
padata - > cpu ) ;
2019-09-06 04:40:28 +03:00
struct padata_priv * cur ;
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
spin_lock ( & pqueue - > reorder . lock ) ;
2019-09-06 04:40:28 +03:00
/* Sort in ascending order of sequence number. */
list_for_each_entry_reverse ( cur , & pqueue - > reorder . list , list )
if ( cur - > seq_nr < padata - > seq_nr )
break ;
list_add ( & padata - > list , & cur - > list ) ;
2019-07-18 18:01:46 +03:00
atomic_inc ( & pd - > reorder_objects ) ;
2010-07-14 14:31:57 +04:00
spin_unlock ( & pqueue - > reorder . lock ) ;
2010-01-06 11:47:10 +03:00
2019-07-16 19:32:53 +03:00
/*
2019-07-18 18:01:46 +03:00
* Ensure the addition to the reorder list is ordered correctly
2019-07-16 19:32:53 +03:00
* with the trylock of pd - > lock in padata_reorder . Pairs with smp_mb
* in padata_reorder .
*/
smp_mb__after_atomic ( ) ;
2019-07-18 18:01:46 +03:00
padata_reorder ( pd ) ;
2010-01-06 11:47:10 +03:00
}
EXPORT_SYMBOL ( padata_do_serial ) ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
static int padata_setup_cpumasks ( struct padata_instance * pinst )
2010-01-06 11:47:10 +03:00
{
2019-09-06 04:40:28 +03:00
struct workqueue_attrs * attrs ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
int err ;
attrs = alloc_workqueue_attrs ( ) ;
if ( ! attrs )
return - ENOMEM ;
/* Restrict parallel_wq workers to pd->cpumask.pcpu. */
cpumask_copy ( attrs - > cpumask , pinst - > cpumask . pcpu ) ;
err = apply_workqueue_attrs ( pinst - > parallel_wq , attrs ) ;
free_workqueue_attrs ( attrs ) ;
return err ;
}
static int pd_setup_cpumasks ( struct parallel_data * pd ,
const struct cpumask * pcpumask ,
const struct cpumask * cbcpumask )
{
2019-09-06 04:40:28 +03:00
int err = - ENOMEM ;
2010-01-06 11:47:10 +03:00
2019-09-06 04:40:28 +03:00
if ( ! alloc_cpumask_var ( & pd - > cpumask . pcpu , GFP_KERNEL ) )
goto out ;
if ( ! alloc_cpumask_var ( & pd - > cpumask . cbcpu , GFP_KERNEL ) )
goto free_pcpu_mask ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
cpumask_copy ( pd - > cpumask . pcpu , pcpumask ) ;
cpumask_copy ( pd - > cpumask . cbcpu , cbcpumask ) ;
2019-09-06 04:40:28 +03:00
2010-07-14 14:31:57 +04:00
return 0 ;
2019-09-06 04:40:28 +03:00
free_pcpu_mask :
free_cpumask_var ( pd - > cpumask . pcpu ) ;
out :
return err ;
2010-07-14 14:31:57 +04:00
}
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
static void __padata_list_init ( struct padata_list * pd_list )
{
INIT_LIST_HEAD ( & pd_list - > list ) ;
spin_lock_init ( & pd_list - > lock ) ;
}
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
/* Initialize all percpu queues used by serial workers */
static void padata_init_squeues ( struct parallel_data * pd )
{
int cpu ;
struct padata_serial_queue * squeue ;
2010-04-29 16:41:36 +04:00
2010-07-14 14:31:57 +04:00
for_each_cpu ( cpu , pd - > cpumask . cbcpu ) {
squeue = per_cpu_ptr ( pd - > squeue , cpu ) ;
squeue - > pd = pd ;
__padata_list_init ( & squeue - > serial ) ;
INIT_WORK ( & squeue - > work , padata_serial_worker ) ;
}
}
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
/* Initialize all percpu queues used by parallel workers */
static void padata_init_pqueues ( struct parallel_data * pd )
{
2019-09-06 04:40:29 +03:00
int cpu ;
2010-07-14 14:31:57 +04:00
struct padata_parallel_queue * pqueue ;
2010-01-06 11:47:10 +03:00
2019-09-06 04:40:29 +03:00
for_each_cpu ( cpu , pd - > cpumask . pcpu ) {
2010-07-14 14:31:57 +04:00
pqueue = per_cpu_ptr ( pd - > pqueue , cpu ) ;
2017-09-08 21:57:09 +03:00
2010-07-14 14:31:57 +04:00
__padata_list_init ( & pqueue - > reorder ) ;
__padata_list_init ( & pqueue - > parallel ) ;
INIT_WORK ( & pqueue - > work , padata_parallel_worker ) ;
atomic_set ( & pqueue - > num_obj , 0 ) ;
2010-01-06 11:47:10 +03:00
}
2010-07-14 14:31:57 +04:00
}
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
/* Allocate and initialize the internal cpumask dependend resources. */
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
static struct parallel_data * padata_alloc_pd ( struct padata_shell * ps )
2010-07-14 14:31:57 +04:00
{
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
struct padata_instance * pinst = ps - > pinst ;
const struct cpumask * cbcpumask ;
const struct cpumask * pcpumask ;
2010-07-14 14:31:57 +04:00
struct parallel_data * pd ;
2010-01-06 11:47:10 +03:00
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
cbcpumask = pinst - > rcpumask . cbcpu ;
pcpumask = pinst - > rcpumask . pcpu ;
2010-07-14 14:31:57 +04:00
pd = kzalloc ( sizeof ( struct parallel_data ) , GFP_KERNEL ) ;
if ( ! pd )
goto err ;
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
pd - > pqueue = alloc_percpu ( struct padata_parallel_queue ) ;
if ( ! pd - > pqueue )
goto err_free_pd ;
pd - > squeue = alloc_percpu ( struct padata_serial_queue ) ;
if ( ! pd - > squeue )
goto err_free_pqueue ;
2019-09-06 04:40:28 +03:00
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
pd - > ps = ps ;
if ( pd_setup_cpumasks ( pd , pcpumask , cbcpumask ) )
2010-07-14 14:31:57 +04:00
goto err_free_squeue ;
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
padata_init_pqueues ( pd ) ;
padata_init_squeues ( pd ) ;
2013-10-25 14:14:15 +04:00
atomic_set ( & pd - > seq_nr , - 1 ) ;
2010-01-06 11:47:10 +03:00
atomic_set ( & pd - > reorder_objects , 0 ) ;
2019-11-19 08:17:31 +03:00
atomic_set ( & pd - > refcnt , 1 ) ;
2010-01-06 11:47:10 +03:00
spin_lock_init ( & pd - > lock ) ;
2019-08-08 19:05:35 +03:00
pd - > cpu = cpumask_first ( pd - > cpumask . pcpu ) ;
2019-07-18 18:01:46 +03:00
INIT_WORK ( & pd - > reorder_work , invoke_padata_reorder ) ;
2010-01-06 11:47:10 +03:00
return pd ;
2010-07-14 14:31:57 +04:00
err_free_squeue :
free_percpu ( pd - > squeue ) ;
err_free_pqueue :
free_percpu ( pd - > pqueue ) ;
2010-01-06 11:47:10 +03:00
err_free_pd :
kfree ( pd ) ;
err :
return NULL ;
}
static void padata_free_pd ( struct parallel_data * pd )
{
2010-07-14 14:31:57 +04:00
free_cpumask_var ( pd - > cpumask . pcpu ) ;
free_cpumask_var ( pd - > cpumask . cbcpu ) ;
free_percpu ( pd - > pqueue ) ;
free_percpu ( pd - > squeue ) ;
2010-01-06 11:47:10 +03:00
kfree ( pd ) ;
}
2010-07-07 17:30:10 +04:00
static void __padata_start ( struct padata_instance * pinst )
{
pinst - > flags | = PADATA_INIT ;
}
2010-07-07 17:30:47 +04:00
static void __padata_stop ( struct padata_instance * pinst )
{
if ( ! ( pinst - > flags & PADATA_INIT ) )
return ;
pinst - > flags & = ~ PADATA_INIT ;
synchronize_rcu ( ) ;
}
2011-03-31 05:57:33 +04:00
/* Replace the internal control structure with a new one. */
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
static int padata_replace_one ( struct padata_shell * ps )
2010-01-06 11:47:10 +03:00
{
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
struct parallel_data * pd_new ;
2010-01-06 11:47:10 +03:00
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
pd_new = padata_alloc_pd ( ps ) ;
if ( ! pd_new )
return - ENOMEM ;
2010-01-06 11:47:10 +03:00
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
ps - > opd = rcu_dereference_protected ( ps - > pd , 1 ) ;
rcu_assign_pointer ( ps - > pd , pd_new ) ;
2010-01-06 11:47:10 +03:00
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
return 0 ;
}
2019-12-03 22:31:10 +03:00
static int padata_replace ( struct padata_instance * pinst )
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
{
int notification_mask = 0 ;
struct padata_shell * ps ;
int err ;
pinst - > flags | = PADATA_RESET ;
2010-01-06 11:47:10 +03:00
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
cpumask_copy ( pinst - > omask , pinst - > rcpumask . pcpu ) ;
cpumask_and ( pinst - > rcpumask . pcpu , pinst - > cpumask . pcpu ,
cpu_online_mask ) ;
if ( ! cpumask_equal ( pinst - > omask , pinst - > rcpumask . pcpu ) )
2010-07-14 14:31:57 +04:00
notification_mask | = PADATA_CPU_PARALLEL ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
cpumask_copy ( pinst - > omask , pinst - > rcpumask . cbcpu ) ;
cpumask_and ( pinst - > rcpumask . cbcpu , pinst - > cpumask . cbcpu ,
cpu_online_mask ) ;
if ( ! cpumask_equal ( pinst - > omask , pinst - > rcpumask . cbcpu ) )
2010-07-14 14:31:57 +04:00
notification_mask | = PADATA_CPU_SERIAL ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
list_for_each_entry ( ps , & pinst - > pslist , list ) {
err = padata_replace_one ( ps ) ;
if ( err )
break ;
}
synchronize_rcu ( ) ;
list_for_each_entry_continue_reverse ( ps , & pinst - > pslist , list )
if ( atomic_dec_and_test ( & ps - > opd - > refcnt ) )
padata_free_pd ( ps - > opd ) ;
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
if ( notification_mask )
blocking_notifier_call_chain ( & pinst - > cpumask_change_notifier ,
2010-07-27 09:15:50 +04:00
notification_mask ,
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
& pinst - > cpumask ) ;
2010-01-06 11:47:10 +03:00
pinst - > flags & = ~ PADATA_RESET ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
return err ;
2010-01-06 11:47:10 +03:00
}
2010-05-19 07:44:27 +04:00
/**
2010-07-14 14:31:57 +04:00
* padata_register_cpumask_notifier - Registers a notifier that will be called
* if either pcpu or cbcpu or both cpumasks change .
2010-01-06 11:47:10 +03:00
*
2010-07-14 14:31:57 +04:00
* @ pinst : A poineter to padata instance
* @ nblock : A pointer to notifier block .
2010-01-06 11:47:10 +03:00
*/
2010-07-14 14:31:57 +04:00
int padata_register_cpumask_notifier ( struct padata_instance * pinst ,
struct notifier_block * nblock )
2010-01-06 11:47:10 +03:00
{
2010-07-14 14:31:57 +04:00
return blocking_notifier_chain_register ( & pinst - > cpumask_change_notifier ,
nblock ) ;
}
EXPORT_SYMBOL ( padata_register_cpumask_notifier ) ;
/**
* padata_unregister_cpumask_notifier - Unregisters cpumask notifier
* registered earlier using padata_register_cpumask_notifier
*
* @ pinst : A pointer to data instance .
* @ nlock : A pointer to notifier block .
*/
int padata_unregister_cpumask_notifier ( struct padata_instance * pinst ,
struct notifier_block * nblock )
{
return blocking_notifier_chain_unregister (
& pinst - > cpumask_change_notifier ,
nblock ) ;
}
EXPORT_SYMBOL ( padata_unregister_cpumask_notifier ) ;
2010-07-07 17:31:26 +04:00
/* If cpumask contains no active cpu, we mark the instance as invalid. */
static bool padata_validate_cpumask ( struct padata_instance * pinst ,
const struct cpumask * cpumask )
{
2012-03-28 10:43:21 +04:00
if ( ! cpumask_intersects ( cpumask , cpu_online_mask ) ) {
2010-07-07 17:31:26 +04:00
pinst - > flags | = PADATA_INVALID ;
return false ;
}
pinst - > flags & = ~ PADATA_INVALID ;
return true ;
}
2010-07-27 09:15:06 +04:00
static int __padata_set_cpumasks ( struct padata_instance * pinst ,
cpumask_var_t pcpumask ,
cpumask_var_t cbcpumask )
{
int valid ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
int err ;
2010-07-27 09:15:06 +04:00
valid = padata_validate_cpumask ( pinst , pcpumask ) ;
if ( ! valid ) {
__padata_stop ( pinst ) ;
goto out_replace ;
}
valid = padata_validate_cpumask ( pinst , cbcpumask ) ;
if ( ! valid )
__padata_stop ( pinst ) ;
out_replace :
cpumask_copy ( pinst - > cpumask . pcpu , pcpumask ) ;
cpumask_copy ( pinst - > cpumask . cbcpu , cbcpumask ) ;
2019-12-03 22:31:10 +03:00
err = padata_setup_cpumasks ( pinst ) ? : padata_replace ( pinst ) ;
2010-07-27 09:15:06 +04:00
if ( valid )
__padata_start ( pinst ) ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
return err ;
2010-07-27 09:15:06 +04:00
}
2010-07-14 14:31:57 +04:00
/**
* padata_set_cpumask : Sets specified by @ cpumask_type cpumask to the value
* equivalent to @ cpumask .
2010-01-06 11:47:10 +03:00
*
* @ pinst : padata instance
2010-07-14 14:31:57 +04:00
* @ cpumask_type : PADATA_CPU_SERIAL or PADATA_CPU_PARALLEL corresponding
* to parallel and serial cpumasks respectively .
2010-01-06 11:47:10 +03:00
* @ cpumask : the cpumask to use
*/
2010-07-14 14:31:57 +04:00
int padata_set_cpumask ( struct padata_instance * pinst , int cpumask_type ,
cpumask_var_t cpumask )
{
struct cpumask * serial_mask , * parallel_mask ;
2010-07-27 09:15:06 +04:00
int err = - EINVAL ;
mutex_lock ( & pinst - > lock ) ;
2010-04-29 16:42:30 +04:00
get_online_cpus ( ) ;
2010-07-14 14:31:57 +04:00
switch ( cpumask_type ) {
case PADATA_CPU_PARALLEL :
serial_mask = pinst - > cpumask . cbcpu ;
parallel_mask = cpumask ;
break ;
case PADATA_CPU_SERIAL :
parallel_mask = pinst - > cpumask . pcpu ;
serial_mask = cpumask ;
break ;
default :
2010-07-27 09:15:06 +04:00
goto out ;
2010-01-06 11:47:10 +03:00
}
2010-07-27 09:15:06 +04:00
err = __padata_set_cpumasks ( pinst , parallel_mask , serial_mask ) ;
2010-01-06 11:47:10 +03:00
out :
2010-04-29 16:42:30 +04:00
put_online_cpus ( ) ;
2010-01-06 11:47:10 +03:00
mutex_unlock ( & pinst - > lock ) ;
return err ;
}
EXPORT_SYMBOL ( padata_set_cpumask ) ;
2016-05-20 03:09:59 +03:00
/**
* padata_start - start the parallel processing
*
* @ pinst : padata instance to start
*/
int padata_start ( struct padata_instance * pinst )
{
int err = 0 ;
mutex_lock ( & pinst - > lock ) ;
if ( pinst - > flags & PADATA_INVALID )
err = - EINVAL ;
2018-11-09 16:16:39 +03:00
__padata_start ( pinst ) ;
2016-05-20 03:09:59 +03:00
mutex_unlock ( & pinst - > lock ) ;
return err ;
}
EXPORT_SYMBOL ( padata_start ) ;
/**
* padata_stop - stop the parallel processing
*
* @ pinst : padata instance to stop
*/
void padata_stop ( struct padata_instance * pinst )
{
mutex_lock ( & pinst - > lock ) ;
__padata_stop ( pinst ) ;
mutex_unlock ( & pinst - > lock ) ;
}
EXPORT_SYMBOL ( padata_stop ) ;
# ifdef CONFIG_HOTPLUG_CPU
2010-01-06 11:47:10 +03:00
static int __padata_add_cpu ( struct padata_instance * pinst , int cpu )
{
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
int err = 0 ;
2010-01-06 11:47:10 +03:00
2012-03-28 10:43:21 +04:00
if ( cpumask_test_cpu ( cpu , cpu_online_mask ) ) {
2019-12-03 22:31:10 +03:00
err = padata_replace ( pinst ) ;
2010-07-07 17:31:26 +04:00
2010-07-14 14:31:57 +04:00
if ( padata_validate_cpumask ( pinst , pinst - > cpumask . pcpu ) & &
padata_validate_cpumask ( pinst , pinst - > cpumask . cbcpu ) )
2010-07-07 17:31:26 +04:00
__padata_start ( pinst ) ;
2010-01-06 11:47:10 +03:00
}
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
return err ;
2010-01-06 11:47:10 +03:00
}
static int __padata_remove_cpu ( struct padata_instance * pinst , int cpu )
{
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
int err = 0 ;
2010-01-06 11:47:10 +03:00
2019-12-03 22:31:10 +03:00
if ( ! cpumask_test_cpu ( cpu , cpu_online_mask ) ) {
2010-07-14 14:31:57 +04:00
if ( ! padata_validate_cpumask ( pinst , pinst - > cpumask . pcpu ) | |
2010-07-20 10:49:20 +04:00
! padata_validate_cpumask ( pinst , pinst - > cpumask . cbcpu ) )
2010-07-07 17:31:26 +04:00
__padata_stop ( pinst ) ;
2019-12-03 22:31:10 +03:00
err = padata_replace ( pinst ) ;
2010-01-06 11:47:10 +03:00
}
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
return err ;
2010-01-06 11:47:10 +03:00
}
2010-07-14 14:31:57 +04:00
static inline int pinst_has_cpu ( struct padata_instance * pinst , int cpu )
{
return cpumask_test_cpu ( cpu , pinst - > cpumask . pcpu ) | |
cpumask_test_cpu ( cpu , pinst - > cpumask . cbcpu ) ;
}
2016-09-06 20:04:49 +03:00
static int padata_cpu_online ( unsigned int cpu , struct hlist_node * node )
2010-01-06 11:47:10 +03:00
{
struct padata_instance * pinst ;
2016-09-06 20:04:49 +03:00
int ret ;
2010-01-06 11:47:10 +03:00
2016-09-06 20:04:49 +03:00
pinst = hlist_entry_safe ( node , struct padata_instance , node ) ;
if ( ! pinst_has_cpu ( pinst , cpu ) )
return 0 ;
2010-01-06 11:47:10 +03:00
2016-09-06 20:04:49 +03:00
mutex_lock ( & pinst - > lock ) ;
ret = __padata_add_cpu ( pinst , cpu ) ;
mutex_unlock ( & pinst - > lock ) ;
return ret ;
}
2010-01-06 11:47:10 +03:00
2019-12-03 22:31:10 +03:00
static int padata_cpu_dead ( unsigned int cpu , struct hlist_node * node )
2016-09-06 20:04:49 +03:00
{
struct padata_instance * pinst ;
int ret ;
pinst = hlist_entry_safe ( node , struct padata_instance , node ) ;
if ( ! pinst_has_cpu ( pinst , cpu ) )
return 0 ;
2010-01-06 11:47:10 +03:00
2016-09-06 20:04:49 +03:00
mutex_lock ( & pinst - > lock ) ;
ret = __padata_remove_cpu ( pinst , cpu ) ;
mutex_unlock ( & pinst - > lock ) ;
return ret ;
2010-01-06 11:47:10 +03:00
}
2016-09-06 20:04:49 +03:00
static enum cpuhp_state hp_online ;
2010-04-29 16:40:10 +04:00
# endif
2010-01-06 11:47:10 +03:00
2010-07-14 14:33:08 +04:00
static void __padata_free ( struct padata_instance * pinst )
{
# ifdef CONFIG_HOTPLUG_CPU
2019-12-03 22:31:10 +03:00
cpuhp_state_remove_instance_nocalls ( CPUHP_PADATA_DEAD , & pinst - > node ) ;
2016-09-06 20:04:49 +03:00
cpuhp_state_remove_instance_nocalls ( hp_online , & pinst - > node ) ;
2010-07-14 14:33:08 +04:00
# endif
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
WARN_ON ( ! list_empty ( & pinst - > pslist ) ) ;
2010-07-14 14:33:08 +04:00
padata_stop ( pinst ) ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
free_cpumask_var ( pinst - > omask ) ;
free_cpumask_var ( pinst - > rcpumask . cbcpu ) ;
free_cpumask_var ( pinst - > rcpumask . pcpu ) ;
2010-07-14 14:33:08 +04:00
free_cpumask_var ( pinst - > cpumask . pcpu ) ;
free_cpumask_var ( pinst - > cpumask . cbcpu ) ;
2019-09-06 04:40:27 +03:00
destroy_workqueue ( pinst - > serial_wq ) ;
destroy_workqueue ( pinst - > parallel_wq ) ;
2010-07-14 14:33:08 +04:00
kfree ( pinst ) ;
}
# define kobj2pinst(_kobj) \
container_of ( _kobj , struct padata_instance , kobj )
# define attr2pentry(_attr) \
container_of ( _attr , struct padata_sysfs_entry , attr )
static void padata_sysfs_release ( struct kobject * kobj )
{
struct padata_instance * pinst = kobj2pinst ( kobj ) ;
__padata_free ( pinst ) ;
}
struct padata_sysfs_entry {
struct attribute attr ;
ssize_t ( * show ) ( struct padata_instance * , struct attribute * , char * ) ;
ssize_t ( * store ) ( struct padata_instance * , struct attribute * ,
const char * , size_t ) ;
} ;
static ssize_t show_cpumask ( struct padata_instance * pinst ,
struct attribute * attr , char * buf )
{
struct cpumask * cpumask ;
ssize_t len ;
mutex_lock ( & pinst - > lock ) ;
if ( ! strcmp ( attr - > name , " serial_cpumask " ) )
cpumask = pinst - > cpumask . cbcpu ;
else
cpumask = pinst - > cpumask . pcpu ;
2015-02-14 01:38:05 +03:00
len = snprintf ( buf , PAGE_SIZE , " %*pb \n " ,
nr_cpu_ids , cpumask_bits ( cpumask ) ) ;
2010-07-14 14:33:08 +04:00
mutex_unlock ( & pinst - > lock ) ;
2015-02-14 01:38:05 +03:00
return len < PAGE_SIZE ? len : - EINVAL ;
2010-07-14 14:33:08 +04:00
}
static ssize_t store_cpumask ( struct padata_instance * pinst ,
struct attribute * attr ,
const char * buf , size_t count )
{
cpumask_var_t new_cpumask ;
ssize_t ret ;
int mask_type ;
if ( ! alloc_cpumask_var ( & new_cpumask , GFP_KERNEL ) )
return - ENOMEM ;
ret = bitmap_parse ( buf , count , cpumask_bits ( new_cpumask ) ,
nr_cpumask_bits ) ;
if ( ret < 0 )
goto out ;
mask_type = ! strcmp ( attr - > name , " serial_cpumask " ) ?
PADATA_CPU_SERIAL : PADATA_CPU_PARALLEL ;
ret = padata_set_cpumask ( pinst , mask_type , new_cpumask ) ;
if ( ! ret )
ret = count ;
out :
free_cpumask_var ( new_cpumask ) ;
return ret ;
}
# define PADATA_ATTR_RW(_name, _show_name, _store_name) \
static struct padata_sysfs_entry _name # # _attr = \
__ATTR ( _name , 0644 , _show_name , _store_name )
# define PADATA_ATTR_RO(_name, _show_name) \
static struct padata_sysfs_entry _name # # _attr = \
__ATTR ( _name , 0400 , _show_name , NULL )
PADATA_ATTR_RW ( serial_cpumask , show_cpumask , store_cpumask ) ;
PADATA_ATTR_RW ( parallel_cpumask , show_cpumask , store_cpumask ) ;
/*
* Padata sysfs provides the following objects :
* serial_cpumask [ RW ] - cpumask for serial workers
* parallel_cpumask [ RW ] - cpumask for parallel workers
*/
static struct attribute * padata_default_attrs [ ] = {
& serial_cpumask_attr . attr ,
& parallel_cpumask_attr . attr ,
NULL ,
} ;
2019-04-02 05:51:47 +03:00
ATTRIBUTE_GROUPS ( padata_default ) ;
2010-07-14 14:33:08 +04:00
static ssize_t padata_sysfs_show ( struct kobject * kobj ,
struct attribute * attr , char * buf )
{
struct padata_instance * pinst ;
struct padata_sysfs_entry * pentry ;
ssize_t ret = - EIO ;
pinst = kobj2pinst ( kobj ) ;
pentry = attr2pentry ( attr ) ;
if ( pentry - > show )
ret = pentry - > show ( pinst , attr , buf ) ;
return ret ;
}
static ssize_t padata_sysfs_store ( struct kobject * kobj , struct attribute * attr ,
const char * buf , size_t count )
{
struct padata_instance * pinst ;
struct padata_sysfs_entry * pentry ;
ssize_t ret = - EIO ;
pinst = kobj2pinst ( kobj ) ;
pentry = attr2pentry ( attr ) ;
if ( pentry - > show )
ret = pentry - > store ( pinst , attr , buf , count ) ;
return ret ;
}
static const struct sysfs_ops padata_sysfs_ops = {
. show = padata_sysfs_show ,
. store = padata_sysfs_store ,
} ;
static struct kobj_type padata_attr_type = {
. sysfs_ops = & padata_sysfs_ops ,
2019-04-02 05:51:47 +03:00
. default_groups = padata_default_groups ,
2010-07-14 14:33:08 +04:00
. release = padata_sysfs_release ,
} ;
2010-07-14 14:31:57 +04:00
/**
2010-07-27 09:14:28 +04:00
* padata_alloc - allocate and initialize a padata instance and specify
* cpumasks for serial and parallel workers .
2010-01-06 11:47:10 +03:00
*
2019-09-06 04:40:21 +03:00
* @ name : used to identify the instance
2010-07-14 14:31:57 +04:00
* @ pcpumask : cpumask that will be used for padata parallelization
* @ cbcpumask : cpumask that will be used for padata serialization
2010-01-06 11:47:10 +03:00
*/
2019-09-06 04:40:21 +03:00
static struct padata_instance * padata_alloc ( const char * name ,
2017-05-24 11:15:17 +03:00
const struct cpumask * pcpumask ,
const struct cpumask * cbcpumask )
2010-01-06 11:47:10 +03:00
{
struct padata_instance * pinst ;
pinst = kzalloc ( sizeof ( struct padata_instance ) , GFP_KERNEL ) ;
if ( ! pinst )
goto err ;
2019-09-06 04:40:28 +03:00
pinst - > parallel_wq = alloc_workqueue ( " %s_parallel " , WQ_UNBOUND , 0 ,
name ) ;
2019-09-06 04:40:27 +03:00
if ( ! pinst - > parallel_wq )
2010-01-06 11:47:10 +03:00
goto err_free_inst ;
2019-09-06 04:40:21 +03:00
2019-09-06 04:40:26 +03:00
get_online_cpus ( ) ;
2019-09-06 04:40:27 +03:00
pinst - > serial_wq = alloc_workqueue ( " %s_serial " , WQ_MEM_RECLAIM |
WQ_CPU_INTENSIVE , 1 , name ) ;
if ( ! pinst - > serial_wq )
2019-09-06 04:40:26 +03:00
goto err_put_cpus ;
2019-09-06 04:40:27 +03:00
if ( ! alloc_cpumask_var ( & pinst - > cpumask . pcpu , GFP_KERNEL ) )
goto err_free_serial_wq ;
2010-07-14 14:31:57 +04:00
if ( ! alloc_cpumask_var ( & pinst - > cpumask . cbcpu , GFP_KERNEL ) ) {
free_cpumask_var ( pinst - > cpumask . pcpu ) ;
2019-09-06 04:40:27 +03:00
goto err_free_serial_wq ;
2010-07-07 17:31:26 +04:00
}
2010-07-14 14:31:57 +04:00
if ( ! padata_validate_cpumask ( pinst , pcpumask ) | |
! padata_validate_cpumask ( pinst , cbcpumask ) )
goto err_free_masks ;
2010-01-06 11:47:10 +03:00
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
if ( ! alloc_cpumask_var ( & pinst - > rcpumask . pcpu , GFP_KERNEL ) )
2010-07-14 14:31:57 +04:00
goto err_free_masks ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
if ( ! alloc_cpumask_var ( & pinst - > rcpumask . cbcpu , GFP_KERNEL ) )
goto err_free_rcpumask_pcpu ;
if ( ! alloc_cpumask_var ( & pinst - > omask , GFP_KERNEL ) )
goto err_free_rcpumask_cbcpu ;
2010-03-04 08:30:22 +03:00
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
INIT_LIST_HEAD ( & pinst - > pslist ) ;
2010-01-06 11:47:10 +03:00
2010-07-14 14:31:57 +04:00
cpumask_copy ( pinst - > cpumask . pcpu , pcpumask ) ;
cpumask_copy ( pinst - > cpumask . cbcpu , cbcpumask ) ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
cpumask_and ( pinst - > rcpumask . pcpu , pcpumask , cpu_online_mask ) ;
cpumask_and ( pinst - > rcpumask . cbcpu , cbcpumask , cpu_online_mask ) ;
if ( padata_setup_cpumasks ( pinst ) )
goto err_free_omask ;
2010-01-06 11:47:10 +03:00
pinst - > flags = 0 ;
2010-07-14 14:31:57 +04:00
BLOCKING_INIT_NOTIFIER_HEAD ( & pinst - > cpumask_change_notifier ) ;
2010-07-14 14:33:08 +04:00
kobject_init ( & pinst - > kobj , & padata_attr_type ) ;
2010-01-06 11:47:10 +03:00
mutex_init ( & pinst - > lock ) ;
2013-08-23 15:12:33 +04:00
# ifdef CONFIG_HOTPLUG_CPU
2017-05-24 11:15:18 +03:00
cpuhp_state_add_instance_nocalls_cpuslocked ( hp_online , & pinst - > node ) ;
2019-12-03 22:31:10 +03:00
cpuhp_state_add_instance_nocalls_cpuslocked ( CPUHP_PADATA_DEAD ,
& pinst - > node ) ;
2013-08-23 15:12:33 +04:00
# endif
2019-09-06 04:40:26 +03:00
put_online_cpus ( ) ;
2010-01-06 11:47:10 +03:00
return pinst ;
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
err_free_omask :
free_cpumask_var ( pinst - > omask ) ;
err_free_rcpumask_cbcpu :
free_cpumask_var ( pinst - > rcpumask . cbcpu ) ;
err_free_rcpumask_pcpu :
free_cpumask_var ( pinst - > rcpumask . pcpu ) ;
2010-07-14 14:31:57 +04:00
err_free_masks :
free_cpumask_var ( pinst - > cpumask . pcpu ) ;
free_cpumask_var ( pinst - > cpumask . cbcpu ) ;
2019-09-06 04:40:27 +03:00
err_free_serial_wq :
destroy_workqueue ( pinst - > serial_wq ) ;
2019-09-06 04:40:26 +03:00
err_put_cpus :
put_online_cpus ( ) ;
2019-09-06 04:40:27 +03:00
destroy_workqueue ( pinst - > parallel_wq ) ;
2010-01-06 11:47:10 +03:00
err_free_inst :
kfree ( pinst ) ;
err :
return NULL ;
}
2017-05-24 11:15:17 +03:00
/**
* padata_alloc_possible - Allocate and initialize padata instance .
* Use the cpu_possible_mask for serial and
* parallel workers .
*
2019-09-06 04:40:21 +03:00
* @ name : used to identify the instance
2017-05-24 11:15:17 +03:00
*/
2019-09-06 04:40:21 +03:00
struct padata_instance * padata_alloc_possible ( const char * name )
2017-05-24 11:15:17 +03:00
{
2019-09-06 04:40:21 +03:00
return padata_alloc ( name , cpu_possible_mask , cpu_possible_mask ) ;
2017-05-24 11:15:17 +03:00
}
EXPORT_SYMBOL ( padata_alloc_possible ) ;
2010-05-19 07:44:27 +04:00
/**
2010-01-06 11:47:10 +03:00
* padata_free - free a padata instance
*
2010-05-19 07:44:27 +04:00
* @ padata_inst : padata instance to free
2010-01-06 11:47:10 +03:00
*/
void padata_free ( struct padata_instance * pinst )
{
2010-07-14 14:33:08 +04:00
kobject_put ( & pinst - > kobj ) ;
2010-01-06 11:47:10 +03:00
}
EXPORT_SYMBOL ( padata_free ) ;
2016-09-06 20:04:49 +03:00
crypto: pcrypt - Avoid deadlock by using per-instance padata queues
If the pcrypt template is used multiple times in an algorithm, then a
deadlock occurs because all pcrypt instances share the same
padata_instance, which completes requests in the order submitted. That
is, the inner pcrypt request waits for the outer pcrypt request while
the outer request is already waiting for the inner.
This patch fixes this by allocating a set of queues for each pcrypt
instance instead of using two global queues. In order to maintain
the existing user-space interface, the pinst structure remains global
so any sysfs modifications will apply to every pcrypt instance.
Note that when an update occurs we have to allocate memory for
every pcrypt instance. Should one of the allocations fail we
will abort the update without rolling back changes already made.
The new per-instance data structure is called padata_shell and is
essentially a wrapper around parallel_data.
Reproducer:
#include <linux/if_alg.h>
#include <sys/socket.h>
#include <unistd.h>
int main()
{
struct sockaddr_alg addr = {
.salg_type = "aead",
.salg_name = "pcrypt(pcrypt(rfc4106-gcm-aesni))"
};
int algfd, reqfd;
char buf[32] = { 0 };
algfd = socket(AF_ALG, SOCK_SEQPACKET, 0);
bind(algfd, (void *)&addr, sizeof(addr));
setsockopt(algfd, SOL_ALG, ALG_SET_KEY, buf, 20);
reqfd = accept(algfd, 0, 0);
write(reqfd, buf, 32);
read(reqfd, buf, 16);
}
Reported-by: syzbot+56c7151cad94eec37c521f0e47d2eee53f9361c4@syzkaller.appspotmail.com
Fixes: 5068c7a883d1 ("crypto: pcrypt - Add pcrypt crypto parallelization wrapper")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
2019-11-26 10:58:45 +03:00
/**
* padata_alloc_shell - Allocate and initialize padata shell .
*
* @ pinst : Parent padata_instance object .
*/
struct padata_shell * padata_alloc_shell ( struct padata_instance * pinst )
{
struct parallel_data * pd ;
struct padata_shell * ps ;
ps = kzalloc ( sizeof ( * ps ) , GFP_KERNEL ) ;
if ( ! ps )
goto out ;
ps - > pinst = pinst ;
get_online_cpus ( ) ;
pd = padata_alloc_pd ( ps ) ;
put_online_cpus ( ) ;
if ( ! pd )
goto out_free_ps ;
mutex_lock ( & pinst - > lock ) ;
RCU_INIT_POINTER ( ps - > pd , pd ) ;
list_add ( & ps - > list , & pinst - > pslist ) ;
mutex_unlock ( & pinst - > lock ) ;
return ps ;
out_free_ps :
kfree ( ps ) ;
out :
return NULL ;
}
EXPORT_SYMBOL ( padata_alloc_shell ) ;
/**
* padata_free_shell - free a padata shell
*
* @ ps : padata shell to free
*/
void padata_free_shell ( struct padata_shell * ps )
{
struct padata_instance * pinst = ps - > pinst ;
mutex_lock ( & pinst - > lock ) ;
list_del ( & ps - > list ) ;
padata_free_pd ( rcu_dereference_protected ( ps - > pd , 1 ) ) ;
mutex_unlock ( & pinst - > lock ) ;
kfree ( ps ) ;
}
EXPORT_SYMBOL ( padata_free_shell ) ;
2016-09-06 20:04:49 +03:00
# ifdef CONFIG_HOTPLUG_CPU
static __init int padata_driver_init ( void )
{
int ret ;
ret = cpuhp_setup_state_multi ( CPUHP_AP_ONLINE_DYN , " padata:online " ,
2019-12-03 22:31:10 +03:00
padata_cpu_online , NULL ) ;
2016-09-06 20:04:49 +03:00
if ( ret < 0 )
return ret ;
hp_online = ret ;
2019-12-03 22:31:10 +03:00
ret = cpuhp_setup_state_multi ( CPUHP_PADATA_DEAD , " padata:dead " ,
NULL , padata_cpu_dead ) ;
if ( ret < 0 ) {
cpuhp_remove_multi_state ( hp_online ) ;
return ret ;
}
2016-09-06 20:04:49 +03:00
return 0 ;
}
module_init ( padata_driver_init ) ;
static __exit void padata_driver_exit ( void )
{
2019-12-03 22:31:10 +03:00
cpuhp_remove_multi_state ( CPUHP_PADATA_DEAD ) ;
2016-09-06 20:04:49 +03:00
cpuhp_remove_multi_state ( hp_online ) ;
}
module_exit ( padata_driver_exit ) ;
# endif