[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 14/42] job: Add reference counting
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH 14/42] job: Add reference counting |
Date: |
Wed, 16 May 2018 22:56:26 +0200 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
Am 16.05.2018 um 20:17 hat Eric Blake geschrieben:
> On 05/09/2018 11:26 AM, Kevin Wolf wrote:
> > This moves reference counting from BlockJob to Job.
> >
> > In order to keep calling the BlockJob cleanup code when the job is
> > deleted via job_unref(), introduce a new JobDriver.free callback. Every
> > block job must use block_job_free() for this callback, this is asserted
> > in block_job_create().
> >
> > Signed-off-by: Kevin Wolf <address@hidden>
> > ---
>
> > +++ b/job.c
>
> > +
> > +void job_unref(Job *job)
> > +{
> > + if (--job->refcnt == 0) {
>
> Should this be free()-like and allow an incoming job == NULL as a no-op?
This behaves like block_job_unref() always behavec, and I don't see a
single caller having a NULL check before calling job_unref(), so is it
worth it?
Kevin