diff options
| author | Hanna Reitz <hreitz@redhat.com> | 2021-08-12 10:41:43 +0200 |
|---|---|---|
| committer | Hanna Reitz <hreitz@redhat.com> | 2021-09-15 15:54:06 +0200 |
| commit | 33ff4c9e081acdded2c90c8c1963f3403e16082b (patch) | |
| tree | 4a240f99fbe0439248bdd8a92f9ad3e4815bfaff /include | |
| parent | e24154d878c11688202628755f8a47e3cba0f52a (diff) | |
| download | qemu-33ff4c9e081acdded2c90c8c1963f3403e16082b.zip qemu-33ff4c9e081acdded2c90c8c1963f3403e16082b.tar.gz qemu-33ff4c9e081acdded2c90c8c1963f3403e16082b.tar.bz2 | |
block: Drop BDS comment regarding bdrv_append()
There is a comment above the BDS definition stating care must be taken
to consider handling newly added fields in bdrv_append().
Actually, this comment should have said "bdrv_swap()" as of 4ddc07cac
(nine years ago), and in any case, bdrv_swap() was dropped in
8e419aefa (six years ago). So no such care is necessary anymore.
Signed-off-by: Hanna Reitz <hreitz@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Message-Id: <20210812084148.14458-2-hreitz@redhat.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/block/block_int.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/include/block/block_int.h b/include/block/block_int.h index f1a54db..12e5750 100644 --- a/include/block/block_int.h +++ b/include/block/block_int.h @@ -839,12 +839,6 @@ struct BdrvChild { QLIST_ENTRY(BdrvChild) next_parent; }; -/* - * Note: the function bdrv_append() copies and swaps contents of - * BlockDriverStates, so if you add new fields to this struct, please - * inspect bdrv_append() to determine if the new fields need to be - * copied as well. - */ struct BlockDriverState { /* Protected by big QEMU lock or read-only after opening. No special * locking needed during I/O... |