Re: [PATCH 1/2] vhost/scsi: Use GFP_ATOMIC with percpu_ida_alloc for obtaining tag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Sep 24, 2013 at 3:10 AM, Nicholas A. Bellinger
<nab@xxxxxxxxxxxxxxx> wrote:
> From: Nicholas Bellinger <nab@xxxxxxxxxxxxxxx>
>
> Fix GFP_KERNEL -> GFP_ATOMIC usage of percpu_ida_alloc() within
> vhost_scsi_get_tag(), as this code is expected to be called directly
> from interrupt context.
>
> Cc: Michael S. Tsirkin <mst@xxxxxxxxxx>
> Cc: Asias He <asias@xxxxxxxxxx>
> Signed-off-by: Nicholas Bellinger <nab@xxxxxxxxxxxxxxx>
> ---

Acked-by: Asias He <asias@xxxxxxxxxx>

>  drivers/vhost/scsi.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/vhost/scsi.c b/drivers/vhost/scsi.c
> index 4b79a1f..532820c 100644
> --- a/drivers/vhost/scsi.c
> +++ b/drivers/vhost/scsi.c
> @@ -728,7 +728,7 @@ vhost_scsi_get_tag(struct vhost_virtqueue *vq,
>         }
>         se_sess = tv_nexus->tvn_se_sess;
>
> -       tag = percpu_ida_alloc(&se_sess->sess_tag_pool, GFP_KERNEL);
> +       tag = percpu_ida_alloc(&se_sess->sess_tag_pool, GFP_ATOMIC);
>         cmd = &((struct tcm_vhost_cmd *)se_sess->sess_cmd_map)[tag];
>         sg = cmd->tvc_sgl;
>         pages = cmd->tvc_upages;
> --
> 1.7.10.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe target-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Asias
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SCSI]     [Kernel Newbies]     [Linux SCSI Target Infrastructure]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Device Mapper]

  Powered by Linux