* [Qemu-devel] [PATCH] block: Remove unused s->hd in various drivers
@ 2010-11-16 17:56 Kevin Wolf
2010-11-16 20:29 ` Stefan Hajnoczi
2010-11-16 22:36 ` Christoph Hellwig
0 siblings, 2 replies; 3+ messages in thread
From: Kevin Wolf @ 2010-11-16 17:56 UTC (permalink / raw)
To: qemu-devel; +Cc: kwolf
All drivers use bs->file instead of s->hd fpr quite a while now, so it's time
to remove s->hd.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
---
block/qcow.c | 1 -
block/qcow2.h | 1 -
block/vdi.c | 1 -
block/vmdk.c | 1 -
block/vpc.c | 2 --
5 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/block/qcow.c b/block/qcow.c
index 9cd547d..f67d3d3 100644
--- a/block/qcow.c
+++ b/block/qcow.c
@@ -54,7 +54,6 @@ typedef struct QCowHeader {
#define L2_CACHE_SIZE 16
typedef struct BDRVQcowState {
- BlockDriverState *hd;
int cluster_bits;
int cluster_size;
int cluster_sectors;
diff --git a/block/qcow2.h b/block/qcow2.h
index 2d22e5e..5217bea 100644
--- a/block/qcow2.h
+++ b/block/qcow2.h
@@ -79,7 +79,6 @@ typedef struct QCowSnapshot {
} QCowSnapshot;
typedef struct BDRVQcowState {
- BlockDriverState *hd;
int cluster_bits;
int cluster_size;
int cluster_sectors;
diff --git a/block/vdi.c b/block/vdi.c
index 3b51e53..ab8f70f 100644
--- a/block/vdi.c
+++ b/block/vdi.c
@@ -186,7 +186,6 @@ typedef struct {
} VdiHeader;
typedef struct {
- BlockDriverState *hd;
/* The block map entries are little endian (even in memory). */
uint32_t *bmap;
/* Size of block (bytes). */
diff --git a/block/vmdk.c b/block/vmdk.c
index f505303..2298a75 100644
--- a/block/vmdk.c
+++ b/block/vmdk.c
@@ -61,7 +61,6 @@ typedef struct {
#define L2_CACHE_SIZE 16
typedef struct BDRVVmdkState {
- BlockDriverState *hd;
int64_t l1_table_offset;
int64_t l1_backup_table_offset;
uint32_t *l1_table;
diff --git a/block/vpc.c b/block/vpc.c
index 416f489..21e2a68 100644
--- a/block/vpc.c
+++ b/block/vpc.c
@@ -110,8 +110,6 @@ struct vhd_dyndisk_header {
};
typedef struct BDRVVPCState {
- BlockDriverState *hd;
-
uint8_t footer_buf[HEADER_SIZE];
uint64_t free_data_block_offset;
int max_table_entries;
--
1.7.2.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] block: Remove unused s->hd in various drivers
2010-11-16 17:56 [Qemu-devel] [PATCH] block: Remove unused s->hd in various drivers Kevin Wolf
@ 2010-11-16 20:29 ` Stefan Hajnoczi
2010-11-16 22:36 ` Christoph Hellwig
1 sibling, 0 replies; 3+ messages in thread
From: Stefan Hajnoczi @ 2010-11-16 20:29 UTC (permalink / raw)
To: Kevin Wolf; +Cc: qemu-devel
On Tue, Nov 16, 2010 at 5:56 PM, Kevin Wolf <kwolf@redhat.com> wrote:
> All drivers use bs->file instead of s->hd fpr quite a while now, so it's time
> to remove s->hd.
>
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
> block/qcow.c | 1 -
> block/qcow2.h | 1 -
> block/vdi.c | 1 -
> block/vmdk.c | 1 -
> block/vpc.c | 2 --
> 5 files changed, 0 insertions(+), 6 deletions(-)
Reviewed-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] block: Remove unused s->hd in various drivers
2010-11-16 17:56 [Qemu-devel] [PATCH] block: Remove unused s->hd in various drivers Kevin Wolf
2010-11-16 20:29 ` Stefan Hajnoczi
@ 2010-11-16 22:36 ` Christoph Hellwig
1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2010-11-16 22:36 UTC (permalink / raw)
To: Kevin Wolf; +Cc: qemu-devel
On Tue, Nov 16, 2010 at 06:56:49PM +0100, Kevin Wolf wrote:
> All drivers use bs->file instead of s->hd fpr quite a while now, so it's time
> to remove s->hd.
Looks good.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-11-16 22:36 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-11-16 17:56 [Qemu-devel] [PATCH] block: Remove unused s->hd in various drivers Kevin Wolf
2010-11-16 20:29 ` Stefan Hajnoczi
2010-11-16 22:36 ` Christoph Hellwig
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.