> > My plan was to have something like bs->job->job_type-
> >{before,after}_write.
> >
> >    int coroutine_fn (*before_write)(BlockDriverState *bs,
> >         int64_t sector_num, int nb_sectors, QEMUIOVector *qiov,
> >         void **cookie);
> >    int coroutine_fn (*after_write)(BlockDriverState *bs,
> >         int64_t sector_num, int nb_sectors, QEMUIOVector *qiov,
> >         void *cookie);
> 
> I don't think that job is the right place. Instead I would put a list of 
> filters into
> BDS:

Well, I can also add it to job_type. Just tell me what you prefer, and I will 
write the patch.


Reply via email to