qemu-pr-helper: mark more coroutine_fns
do_sgio can suspend via the coroutine function thread_pool_submit_co, so it has to be coroutine_fn as well---and the same is true of all its direct and indirect callers. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Message-Id: <20230309084456.304669-7-pbonzini@redhat.com> Reviewed-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
parent
c3b21fb189
commit
70bd77bea5
@ -177,8 +177,8 @@ static int do_sgio_worker(void *opaque)
|
||||
return status;
|
||||
}
|
||||
|
||||
static int do_sgio(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
uint8_t *buf, int *sz, int dir)
|
||||
static int coroutine_fn do_sgio(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
uint8_t *buf, int *sz, int dir)
|
||||
{
|
||||
int r;
|
||||
|
||||
@ -319,7 +319,7 @@ static SCSISense mpath_generic_sense(int r)
|
||||
}
|
||||
}
|
||||
|
||||
static int mpath_reconstruct_sense(int fd, int r, uint8_t *sense)
|
||||
static int coroutine_fn mpath_reconstruct_sense(int fd, int r, uint8_t *sense)
|
||||
{
|
||||
switch (r) {
|
||||
case MPATH_PR_SUCCESS:
|
||||
@ -371,8 +371,8 @@ static int mpath_reconstruct_sense(int fd, int r, uint8_t *sense)
|
||||
}
|
||||
}
|
||||
|
||||
static int multipath_pr_in(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
uint8_t *data, int sz)
|
||||
static int coroutine_fn multipath_pr_in(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
uint8_t *data, int sz)
|
||||
{
|
||||
int rq_servact = cdb[1];
|
||||
struct prin_resp resp;
|
||||
@ -426,8 +426,8 @@ static int multipath_pr_in(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
return mpath_reconstruct_sense(fd, r, sense);
|
||||
}
|
||||
|
||||
static int multipath_pr_out(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
const uint8_t *param, int sz)
|
||||
static int coroutine_fn multipath_pr_out(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
const uint8_t *param, int sz)
|
||||
{
|
||||
int rq_servact = cdb[1];
|
||||
int rq_scope = cdb[2] >> 4;
|
||||
@ -544,8 +544,8 @@ static int multipath_pr_out(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
}
|
||||
#endif
|
||||
|
||||
static int do_pr_in(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
uint8_t *data, int *resp_sz)
|
||||
static int coroutine_fn do_pr_in(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
uint8_t *data, int *resp_sz)
|
||||
{
|
||||
#ifdef CONFIG_MPATH
|
||||
if (is_mpath(fd)) {
|
||||
@ -562,8 +562,8 @@ static int do_pr_in(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
SG_DXFER_FROM_DEV);
|
||||
}
|
||||
|
||||
static int do_pr_out(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
const uint8_t *param, int sz)
|
||||
static int coroutine_fn do_pr_out(int fd, const uint8_t *cdb, uint8_t *sense,
|
||||
const uint8_t *param, int sz)
|
||||
{
|
||||
int resp_sz;
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user