Re: [PATCH] block: ioprio: Replace GFP_ATOMIC with GFP_KERNEL in set_task_ioprio()

From: Christoph Hellwig
Date: Mon Jul 23 2018 - 03:18:13 EST

Next message: Jia-Ju Bai: "Re: [REPORT] Possible unnecessary usages of GFP_ATOMIC in crypto/ablkcipher.c"
Previous message: Stephen Rothwell: "linux-next: manual merge of the xarray tree with the nvdimm tree"
In reply to: Jia-Ju Bai: "[PATCH] block: ioprio: Replace GFP_ATOMIC with GFP_KERNEL in set_task_ioprio()"
Next in thread: Jia-Ju Bai: "Re: [PATCH] block: ioprio: Replace GFP_ATOMIC with GFP_KERNEL in set_task_ioprio()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

But to me it seems that btrfs and maybe ext4 as well would be much
better off just setting a per-IO priority in the bio.

Any chance yo could look into that?

Next message: Jia-Ju Bai: "Re: [REPORT] Possible unnecessary usages of GFP_ATOMIC in crypto/ablkcipher.c"
Previous message: Stephen Rothwell: "linux-next: manual merge of the xarray tree with the nvdimm tree"
In reply to: Jia-Ju Bai: "[PATCH] block: ioprio: Replace GFP_ATOMIC with GFP_KERNEL in set_task_ioprio()"
Next in thread: Jia-Ju Bai: "Re: [PATCH] block: ioprio: Replace GFP_ATOMIC with GFP_KERNEL in set_task_ioprio()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]