> > typedef struct compress_param compress_param; > > > > +enum { > > + DONE, > > + START, > > +}; > > + > > Do you really need any more than a 'bool busy' ?
Good ideal. > > struct decompress_param { > > /* To be done */ > > }; > > typedef struct decompress_param decompress_param; > > > > static compress_param *comp_param; > > +static QemuMutex *mutex; > > +static QemuCond *cond; > > Those need better names and a comment; If I'm reading it correctly, this > cond is used to wake up the parent thread when one of the workers has > finished it's task? Yes, it is.