All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: qemu-devel@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>,
	Stefan Hajnoczi <stefanha@redhat.com>,
	Jeff Cody <jcody@redhat.com>, Alexander Graf <agraf@suse.de>,
	Anthony Liguori <anthony@codemonkey.ws>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Jason Baron <jbaron@redhat.com>
Subject: [Qemu-devel] KVM Forum 2012 Block BoF minutes
Date: Thu, 15 Nov 2012 14:31:53 +0100	[thread overview]
Message-ID: <87pq3fat6e.fsf@blackfin.pond.sub.org> (raw)

Attendees:

Kevin Wolf <kwolf@redhat.com>
Stefan Hajnoczi <stefanha@redhat.com>
Jeff Cody <jcody@redhat.com>
Markus Armbruster <armbru@redhat.com>

and a few people dropping in and out.

Minutes are basically a TODO list.  Could be a bit terse in places; if
anything's unclear, please ask.


Block layer data structure cleanup:

- Split block backend off BlockDriverState

- Turn block backend into a proper ADT (besides other good things, this
  makes it possible to replace its implementation by a test harness for
  unit testing of its customers)

- Get rid of BlockDriverState opaque (driver private state), inherit the
  common part into driver state

Convert open to QemuOpts (or perhaps QDict?)

Avoid double open for probing (update: patches posted)

Block jobs:

- Rate limiting is broken in general (it works for the single
  BlockDriverState case)

- Block jobs should be jobs: generally available instead of tied to a
  (single) BlockDriverState; get rid of the block job pointer in
  BlockDriverState

- Migration should probably be a job

Block migration needs to die after its replacement is ready

BlockDriverState member in_use and DriveInfo member refcount are a mess:

- in_use is used to avoid running certain things concurrently, but the
  rules are unclear, except they're clearly incomplete; possible rules
  could be about need for consistent views of image contents

- refcount is only used together with in_use, and appears to be for
  avoiding premature deletion

AHCI:

- Same IDE device models connect to different kinds of controllers just
  fine

- Use links to connect controller device model and drive device models;
  the link serves as generic address replacing bus, unit, index

- Make -drive if!=none true sugar for suitable -device (details depend
  on machine)

- Checking legacy bus, unit, index needs to be delayed until machine's
  limits are known

- Add "enough" -hdX to cover common usage (got 4 now, want 6 for q35)

- Some AHCI controllers can optionally mimick a legacy IDE controller,
  and expose some of their ports there (controlled by BIOS for real
  hardware, by controller qdev property for us)

- Convenience machine option to control that qdev property of an onboard
  controller would be nice (should not apply to additional controllers
  plugged in with -device)

- if=ahci is not necessary

             reply	other threads:[~2012-11-15 13:32 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-15 13:31 Markus Armbruster [this message]
2012-11-18  5:04 ` [Qemu-devel] KVM Forum 2012 Block BoF minutes Marcelo Tosatti
2012-11-19 17:03   ` [Qemu-devel] BlockDriverState member in_use (was: KVM Forum 2012 Block BoF minutes) Markus Armbruster
2012-11-19 20:13     ` Marcelo Tosatti

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87pq3fat6e.fsf@blackfin.pond.sub.org \
    --to=armbru@redhat.com \
    --cc=agraf@suse.de \
    --cc=anthony@codemonkey.ws \
    --cc=jbaron@redhat.com \
    --cc=jcody@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.