Re: [PATCH -next] xen/pvcalls: use GFP_ATOMIC under spin lock

[Boris Ostrovsky]

On 12/27/2017 10:46 PM, Wei Yongjun wrote:
> A spin lock is taken here so we should use GFP_ATOMIC.
>
> Fixes: 9774c6cca266 ("xen/pvcalls: implement accept command")
> Signed-off-by: Wei Yongjun 

Applied to for-linus-4.15

-boris

Re: [PATCH -next] xen/pvcalls: use GFP_ATOMIC under spin lock

[Juergen Gross]

On 28/12/17 04:46, Wei Yongjun wrote:
> A spin lock is taken here so we should use GFP_ATOMIC.
> 
> Fixes: 9774c6cca266 ("xen/pvcalls: implement accept command")
> Signed-off-by: Wei Yongjun 

Reviewed-by: Juergen Gross 


Juergen

[PATCH -next] xen/pvcalls: use GFP_ATOMIC under spin lock

[Wei Yongjun]

A spin lock is taken here so we should use GFP_ATOMIC.

Fixes: 9774c6cca266 ("xen/pvcalls: implement accept command")
Signed-off-by: Wei Yongjun 
---
 drivers/xen/pvcalls-front.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/xen/pvcalls-front.c b/drivers/xen/pvcalls-front.c
index 0c1ec68..dfd00d9 100644
--- a/drivers/xen/pvcalls-front.c
+++ b/drivers/xen/pvcalls-front.c
@@ -807,7 +807,7 @@ int pvcalls_front_accept(struct socket *sock, struct socket 
*newsock, int flags)
pvcalls_exit();
return ret;
}
-   map2 = kzalloc(sizeof(*map2), GFP_KERNEL);
+   map2 = kzalloc(sizeof(*map2), GFP_ATOMIC);
if (map2 == NULL) {
clear_bit(PVCALLS_FLAG_ACCEPT_INFLIGHT,
  (void *)&map->passive.flags);