| Message ID | 1458072268-53705-2-git-send-email-vsementsov@virtuozzo.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 15.03.2016 23:04, Vladimir Sementsov-Ogievskiy wrote: > From: Fam Zheng <famz@redhat.com> > > For dirty bitmap users to get the size and the name of a > BdrvDirtyBitmap. > > Signed-off-by: Fam Zheng <famz@redhat.com> > Reviewed-by: John Snow <jsnow@redhat.com> > Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> it's an accidental s.o.b., actually there are no changes by me. > --- > block/dirty-bitmap.c | 10 ++++++++++ > include/block/dirty-bitmap.h | 2 ++ > 2 files changed, 12 insertions(+) > > diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c > index 556e1d1..45cfa3b 100644 > --- a/block/dirty-bitmap.c > +++ b/block/dirty-bitmap.c > @@ -97,6 +97,16 @@ BdrvDirtyBitmap *bdrv_create_dirty_bitmap(BlockDriverState *bs, > return bitmap; > } > > +int64_t bdrv_dirty_bitmap_size(const BdrvDirtyBitmap *bitmap) > +{ > + return bitmap->size; > +} > + > +const char *bdrv_dirty_bitmap_name(const BdrvDirtyBitmap *bitmap) > +{ > + return bitmap->name; > +} > + > bool bdrv_dirty_bitmap_frozen(BdrvDirtyBitmap *bitmap) > { > return bitmap->successor; > diff --git a/include/block/dirty-bitmap.h b/include/block/dirty-bitmap.h > index 80afe60..4dc8750 100644 > --- a/include/block/dirty-bitmap.h > +++ b/include/block/dirty-bitmap.h > @@ -29,6 +29,8 @@ uint32_t bdrv_get_default_bitmap_granularity(BlockDriverState *bs); > uint32_t bdrv_dirty_bitmap_granularity(BdrvDirtyBitmap *bitmap); > bool bdrv_dirty_bitmap_enabled(BdrvDirtyBitmap *bitmap); > bool bdrv_dirty_bitmap_frozen(BdrvDirtyBitmap *bitmap); > +const char *bdrv_dirty_bitmap_name(const BdrvDirtyBitmap *bitmap); > +int64_t bdrv_dirty_bitmap_size(const BdrvDirtyBitmap *bitmap); > DirtyBitmapStatus bdrv_dirty_bitmap_status(BdrvDirtyBitmap *bitmap); > int bdrv_get_dirty(BlockDriverState *bs, BdrvDirtyBitmap *bitmap, > int64_t sector);
On 03/15/2016 02:08 PM, Vladimir Sementsov-Ogievskiy wrote: > On 15.03.2016 23:04, Vladimir Sementsov-Ogievskiy wrote: >> From: Fam Zheng <famz@redhat.com> >> >> For dirty bitmap users to get the size and the name of a >> BdrvDirtyBitmap. >> >> Signed-off-by: Fam Zheng <famz@redhat.com> >> Reviewed-by: John Snow <jsnow@redhat.com> >> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> > > it's an accidental s.o.b., actually there are no changes by me. But if you are emailing the current version of the series, you should at least have your name associated with the commit message - if not s.o.b, then Reviewed-by.
On 03/15/2016 04:08 PM, Vladimir Sementsov-Ogievskiy wrote: > On 15.03.2016 23:04, Vladimir Sementsov-Ogievskiy wrote: >> From: Fam Zheng <famz@redhat.com> >> >> For dirty bitmap users to get the size and the name of a >> BdrvDirtyBitmap. >> >> Signed-off-by: Fam Zheng <famz@redhat.com> >> Reviewed-by: John Snow <jsnow@redhat.com> >> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> > > it's an accidental s.o.b., actually there are no changes by me. > You can add an S-o-B even when you don't make changes to indicate a "chain of custody" for the patch. You are certifying that the patch is, to the best of your knowledge, properly credited and licensed. It's not required for you to add your /own/ S-o-B to every patch you send, but it's not hurting anything here either. At least, that was my understanding of it. I know as a maintainer I add my S-o-B to everything that goes through my tree even if I don't modify it just to signify where the patch has been and who has handled it. --js >> --- >> block/dirty-bitmap.c | 10 ++++++++++ >> include/block/dirty-bitmap.h | 2 ++ >> 2 files changed, 12 insertions(+) >> >> diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c >> index 556e1d1..45cfa3b 100644 >> --- a/block/dirty-bitmap.c >> +++ b/block/dirty-bitmap.c >> @@ -97,6 +97,16 @@ BdrvDirtyBitmap >> *bdrv_create_dirty_bitmap(BlockDriverState *bs, >> return bitmap; >> } >> +int64_t bdrv_dirty_bitmap_size(const BdrvDirtyBitmap *bitmap) >> +{ >> + return bitmap->size; >> +} >> + >> +const char *bdrv_dirty_bitmap_name(const BdrvDirtyBitmap *bitmap) >> +{ >> + return bitmap->name; >> +} >> + >> bool bdrv_dirty_bitmap_frozen(BdrvDirtyBitmap *bitmap) >> { >> return bitmap->successor; >> diff --git a/include/block/dirty-bitmap.h b/include/block/dirty-bitmap.h >> index 80afe60..4dc8750 100644 >> --- a/include/block/dirty-bitmap.h >> +++ b/include/block/dirty-bitmap.h >> @@ -29,6 +29,8 @@ uint32_t >> bdrv_get_default_bitmap_granularity(BlockDriverState *bs); >> uint32_t bdrv_dirty_bitmap_granularity(BdrvDirtyBitmap *bitmap); >> bool bdrv_dirty_bitmap_enabled(BdrvDirtyBitmap *bitmap); >> bool bdrv_dirty_bitmap_frozen(BdrvDirtyBitmap *bitmap); >> +const char *bdrv_dirty_bitmap_name(const BdrvDirtyBitmap *bitmap); >> +int64_t bdrv_dirty_bitmap_size(const BdrvDirtyBitmap *bitmap); >> DirtyBitmapStatus bdrv_dirty_bitmap_status(BdrvDirtyBitmap *bitmap); >> int bdrv_get_dirty(BlockDriverState *bs, BdrvDirtyBitmap *bitmap, >> int64_t sector); > >
diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c index 556e1d1..45cfa3b 100644 --- a/block/dirty-bitmap.c +++ b/block/dirty-bitmap.c @@ -97,6 +97,16 @@ BdrvDirtyBitmap *bdrv_create_dirty_bitmap(BlockDriverState *bs, return bitmap; } +int64_t bdrv_dirty_bitmap_size(const BdrvDirtyBitmap *bitmap) +{ + return bitmap->size; +} + +const char *bdrv_dirty_bitmap_name(const BdrvDirtyBitmap *bitmap) +{ + return bitmap->name; +} + bool bdrv_dirty_bitmap_frozen(BdrvDirtyBitmap *bitmap) { return bitmap->successor; diff --git a/include/block/dirty-bitmap.h b/include/block/dirty-bitmap.h index 80afe60..4dc8750 100644 --- a/include/block/dirty-bitmap.h +++ b/include/block/dirty-bitmap.h @@ -29,6 +29,8 @@ uint32_t bdrv_get_default_bitmap_granularity(BlockDriverState *bs); uint32_t bdrv_dirty_bitmap_granularity(BdrvDirtyBitmap *bitmap); bool bdrv_dirty_bitmap_enabled(BdrvDirtyBitmap *bitmap); bool bdrv_dirty_bitmap_frozen(BdrvDirtyBitmap *bitmap); +const char *bdrv_dirty_bitmap_name(const BdrvDirtyBitmap *bitmap); +int64_t bdrv_dirty_bitmap_size(const BdrvDirtyBitmap *bitmap); DirtyBitmapStatus bdrv_dirty_bitmap_status(BdrvDirtyBitmap *bitmap); int bdrv_get_dirty(BlockDriverState *bs, BdrvDirtyBitmap *bitmap, int64_t sector);