[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 0/8] blockdev: Further BlockBackend work
From: |
Eric Blake |
Subject: |
Re: [Qemu-block] [PATCH v2 0/8] blockdev: Further BlockBackend work |
Date: |
Mon, 9 Nov 2015 20:45:16 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
[side question]
On 11/09/2015 08:27 PM, Max Reitz wrote:
> Overall, this series adds more uses for BlockBackends and makes the code
> follow the "reference theory" more closely, in that any BlockBackend
> created (through -drive or blockdev-add) has a reference count of 1, and
> that reference should be held by the monitor. This is reflected here by
> introducing an explicit list of monitor-owned BlockBackends, which in
> turn allows us to remove bdrv_states, and, perhaps most importantly,
> blk_hide_on_behalf_of_do_drive_del().
>
> Although I don't suppose anyone will care much, here's that
> backport-diff against v1:
>
> Key:
> [----] : patches are identical
> [####] : number of functional differences between upstream/downstream patch
> [down] : patch is downstream-only
> The flags [FC] indicate (F)unctional and (C)ontextual differences,
> respectively
>
> 001/8:[----] [-C] 'block: Add blk_name_taken()'
> 002/8:[----] [-C] 'block: Add blk_next_inserted()'
> 003/8:[----] [-C] 'block: Add blk_commit_all() and blk_invalidate_cache_all()'
> 004/8:[0073] [FC] 'block: Use BlockBackend more'
> 005/8:[0004] [FC] 'blockdev: Add list of monitor-owned BlockBackends'
> 006/8:[down] 'blockdev: Remove blk_hide_on_behalf_of_hmp_drive_del()'
> (renamed from "..._on_behalf_of_do_drive_del()")
This is a nice note. Is there any way to make 'git backport-diff'
automatically do this, or does it have to be done manually when renaming
patches?
> 007/8:[0213] [FC] 'block: Move some bdrv_*_all() functions to BB'
> 008/8:[0027] [FC] 'block: Remove bdrv_states'
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-block] [PATCH v2 1/8] block: Add blk_name_taken(), (continued)
- [Qemu-block] [PATCH v2 1/8] block: Add blk_name_taken(), Max Reitz, 2015/11/09
- [Qemu-block] [PATCH v2 2/8] block: Add blk_next_inserted(), Max Reitz, 2015/11/09
- [Qemu-block] [PATCH v2 3/8] block: Add blk_commit_all() and blk_invalidate_cache_all(), Max Reitz, 2015/11/09
- [Qemu-block] [PATCH v2 5/8] blockdev: Add list of monitor-owned BlockBackends, Max Reitz, 2015/11/09
- [Qemu-block] [PATCH v2 4/8] block: Use BlockBackend more, Max Reitz, 2015/11/09
- [Qemu-block] [PATCH v2 6/8] blockdev: Remove blk_hide_on_behalf_of_hmp_drive_del(), Max Reitz, 2015/11/09
- [Qemu-block] [PATCH v2 8/8] block: Remove bdrv_states, Max Reitz, 2015/11/09
- [Qemu-block] [PATCH v2 7/8] block: Move some bdrv_*_all() functions to BB, Max Reitz, 2015/11/09
- Re: [Qemu-block] [PATCH v2 0/8] blockdev: Further BlockBackend work,
Eric Blake <=