* [PATCH] kvm tools: Rename pci_device to pci_hdr for clarity
@ 2011-05-07 10:50 Sasha Levin
2011-05-09 10:35 ` Asias He
0 siblings, 1 reply; 3+ messages in thread
From: Sasha Levin @ 2011-05-07 10:50 UTC (permalink / raw)
To: penberg; +Cc: mingo, asias.hejun, gorcunov, prasadjoshi124, kvm, Sasha Levin
Signed-off-by: Sasha Levin <levinsasha928@gmail.com>
---
tools/kvm/virtio/blk.c | 14 +++++++-------
1 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/tools/kvm/virtio/blk.c b/tools/kvm/virtio/blk.c
index accfc3e..cc3dc78 100644
--- a/tools/kvm/virtio/blk.c
+++ b/tools/kvm/virtio/blk.c
@@ -45,7 +45,7 @@ struct blk_dev {
struct virt_queue vqs[NUM_VIRT_QUEUES];
struct blk_dev_job jobs[NUM_VIRT_QUEUES];
- struct pci_device_header pci_device;
+ struct pci_device_header pci_hdr;
};
static struct blk_dev *bdevs[VIRTIO_BLK_MAX_DEV];
@@ -103,7 +103,7 @@ static bool virtio_blk_pci_io_in(struct kvm *self, u16 port, void *data, int siz
break;
case VIRTIO_PCI_ISR:
ioport__write8(data, 0x1);
- kvm__irq_line(self, bdev->pci_device.irq_line, 0);
+ kvm__irq_line(self, bdev->pci_hdr.irq_line, 0);
break;
case VIRTIO_MSI_CONFIG_VECTOR:
ioport__write16(data, bdev->config_vector);
@@ -167,7 +167,7 @@ static void virtio_blk_do_io(struct kvm *kvm, void *param)
while (virt_queue__available(vq))
virtio_blk_do_io_request(kvm, bdev, vq);
- kvm__irq_line(kvm, bdev->pci_device.irq_line, 1);
+ kvm__irq_line(kvm, bdev->pci_hdr.irq_line, 1);
}
static bool virtio_blk_pci_io_out(struct kvm *self, u16 port, void *data, int size, u32 count)
@@ -283,7 +283,7 @@ void virtio_blk__init(struct kvm *self, struct disk_image *disk)
.blk_config = (struct virtio_blk_config) {
.capacity = disk->size / SECTOR_SIZE,
},
- .pci_device = (struct pci_device_header) {
+ .pci_hdr = (struct pci_device_header) {
.vendor_id = PCI_VENDOR_ID_REDHAT_QUMRANET,
.device_id = PCI_DEVICE_ID_VIRTIO_BLK,
.header_type = PCI_HEADER_TYPE_NORMAL,
@@ -298,10 +298,10 @@ void virtio_blk__init(struct kvm *self, struct disk_image *disk)
if (irq__register_device(PCI_DEVICE_ID_VIRTIO_BLK, &dev, &pin, &line) < 0)
return;
- bdev->pci_device.irq_pin = pin;
- bdev->pci_device.irq_line = line;
+ bdev->pci_hdr.irq_pin = pin;
+ bdev->pci_hdr.irq_line = line;
- pci__register(&bdev->pci_device, dev);
+ pci__register(&bdev->pci_hdr, dev);
ioport__register(blk_dev_base_addr, &virtio_blk_io_ops, IOPORT_VIRTIO_BLK_SIZE);
}
--
1.7.5.rc3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] kvm tools: Rename pci_device to pci_hdr for clarity
2011-05-07 10:50 [PATCH] kvm tools: Rename pci_device to pci_hdr for clarity Sasha Levin
@ 2011-05-09 10:35 ` Asias He
2011-05-09 11:23 ` Sasha Levin
0 siblings, 1 reply; 3+ messages in thread
From: Asias He @ 2011-05-09 10:35 UTC (permalink / raw)
To: Sasha Levin; +Cc: penberg, mingo, gorcunov, prasadjoshi124, kvm
On 05/07/2011 06:50 PM, Sasha Levin wrote:
> Signed-off-by: Sasha Levin <levinsasha928@gmail.com>
> ---
> tools/kvm/virtio/blk.c | 14 +++++++-------
> 1 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/tools/kvm/virtio/blk.c b/tools/kvm/virtio/blk.c
> index accfc3e..cc3dc78 100644
> --- a/tools/kvm/virtio/blk.c
> +++ b/tools/kvm/virtio/blk.c
> @@ -45,7 +45,7 @@ struct blk_dev {
>
> struct virt_queue vqs[NUM_VIRT_QUEUES];
> struct blk_dev_job jobs[NUM_VIRT_QUEUES];
> - struct pci_device_header pci_device;
> + struct pci_device_header pci_hdr;
> };
>
> static struct blk_dev *bdevs[VIRTIO_BLK_MAX_DEV];
> @@ -103,7 +103,7 @@ static bool virtio_blk_pci_io_in(struct kvm *self, u16 port, void *data, int siz
> break;
> case VIRTIO_PCI_ISR:
> ioport__write8(data, 0x1);
> - kvm__irq_line(self, bdev->pci_device.irq_line, 0);
> + kvm__irq_line(self, bdev->pci_hdr.irq_line, 0);
> break;
> case VIRTIO_MSI_CONFIG_VECTOR:
> ioport__write16(data, bdev->config_vector);
> @@ -167,7 +167,7 @@ static void virtio_blk_do_io(struct kvm *kvm, void *param)
> while (virt_queue__available(vq))
> virtio_blk_do_io_request(kvm, bdev, vq);
>
> - kvm__irq_line(kvm, bdev->pci_device.irq_line, 1);
> + kvm__irq_line(kvm, bdev->pci_hdr.irq_line, 1);
> }
>
> static bool virtio_blk_pci_io_out(struct kvm *self, u16 port, void *data, int size, u32 count)
> @@ -283,7 +283,7 @@ void virtio_blk__init(struct kvm *self, struct disk_image *disk)
> .blk_config = (struct virtio_blk_config) {
> .capacity = disk->size / SECTOR_SIZE,
> },
> - .pci_device = (struct pci_device_header) {
> + .pci_hdr = (struct pci_device_header) {
> .vendor_id = PCI_VENDOR_ID_REDHAT_QUMRANET,
> .device_id = PCI_DEVICE_ID_VIRTIO_BLK,
> .header_type = PCI_HEADER_TYPE_NORMAL,
> @@ -298,10 +298,10 @@ void virtio_blk__init(struct kvm *self, struct disk_image *disk)
> if (irq__register_device(PCI_DEVICE_ID_VIRTIO_BLK, &dev, &pin, &line) < 0)
> return;
>
> - bdev->pci_device.irq_pin = pin;
> - bdev->pci_device.irq_line = line;
> + bdev->pci_hdr.irq_pin = pin;
> + bdev->pci_hdr.irq_line = line;
>
> - pci__register(&bdev->pci_device, dev);
> + pci__register(&bdev->pci_hdr, dev);
>
> ioport__register(blk_dev_base_addr, &virtio_blk_io_ops, IOPORT_VIRTIO_BLK_SIZE);
> }
Does this renaming apply to other devices as well? Mind to send followup
patches?
--
Best Regards,
Asias He
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] kvm tools: Rename pci_device to pci_hdr for clarity
2011-05-09 10:35 ` Asias He
@ 2011-05-09 11:23 ` Sasha Levin
0 siblings, 0 replies; 3+ messages in thread
From: Sasha Levin @ 2011-05-09 11:23 UTC (permalink / raw)
To: Asias He; +Cc: penberg, mingo, gorcunov, prasadjoshi124, kvm
On Mon, 2011-05-09 at 18:35 +0800, Asias He wrote:
> On 05/07/2011 06:50 PM, Sasha Levin wrote:
> > Signed-off-by: Sasha Levin <levinsasha928@gmail.com>
> > ---
> > tools/kvm/virtio/blk.c | 14 +++++++-------
> > 1 files changed, 7 insertions(+), 7 deletions(-)
> >
> > diff --git a/tools/kvm/virtio/blk.c b/tools/kvm/virtio/blk.c
> > index accfc3e..cc3dc78 100644
> > --- a/tools/kvm/virtio/blk.c
> > +++ b/tools/kvm/virtio/blk.c
> > @@ -45,7 +45,7 @@ struct blk_dev {
> >
> > struct virt_queue vqs[NUM_VIRT_QUEUES];
> > struct blk_dev_job jobs[NUM_VIRT_QUEUES];
> > - struct pci_device_header pci_device;
> > + struct pci_device_header pci_hdr;
> > };
> >
> > static struct blk_dev *bdevs[VIRTIO_BLK_MAX_DEV];
> > @@ -103,7 +103,7 @@ static bool virtio_blk_pci_io_in(struct kvm *self, u16 port, void *data, int siz
> > break;
> > case VIRTIO_PCI_ISR:
> > ioport__write8(data, 0x1);
> > - kvm__irq_line(self, bdev->pci_device.irq_line, 0);
> > + kvm__irq_line(self, bdev->pci_hdr.irq_line, 0);
> > break;
> > case VIRTIO_MSI_CONFIG_VECTOR:
> > ioport__write16(data, bdev->config_vector);
> > @@ -167,7 +167,7 @@ static void virtio_blk_do_io(struct kvm *kvm, void *param)
> > while (virt_queue__available(vq))
> > virtio_blk_do_io_request(kvm, bdev, vq);
> >
> > - kvm__irq_line(kvm, bdev->pci_device.irq_line, 1);
> > + kvm__irq_line(kvm, bdev->pci_hdr.irq_line, 1);
> > }
> >
> > static bool virtio_blk_pci_io_out(struct kvm *self, u16 port, void *data, int size, u32 count)
> > @@ -283,7 +283,7 @@ void virtio_blk__init(struct kvm *self, struct disk_image *disk)
> > .blk_config = (struct virtio_blk_config) {
> > .capacity = disk->size / SECTOR_SIZE,
> > },
> > - .pci_device = (struct pci_device_header) {
> > + .pci_hdr = (struct pci_device_header) {
> > .vendor_id = PCI_VENDOR_ID_REDHAT_QUMRANET,
> > .device_id = PCI_DEVICE_ID_VIRTIO_BLK,
> > .header_type = PCI_HEADER_TYPE_NORMAL,
> > @@ -298,10 +298,10 @@ void virtio_blk__init(struct kvm *self, struct disk_image *disk)
> > if (irq__register_device(PCI_DEVICE_ID_VIRTIO_BLK, &dev, &pin, &line) < 0)
> > return;
> >
> > - bdev->pci_device.irq_pin = pin;
> > - bdev->pci_device.irq_line = line;
> > + bdev->pci_hdr.irq_pin = pin;
> > + bdev->pci_hdr.irq_line = line;
> >
> > - pci__register(&bdev->pci_device, dev);
> > + pci__register(&bdev->pci_hdr, dev);
> >
> > ioport__register(blk_dev_base_addr, &virtio_blk_io_ops, IOPORT_VIRTIO_BLK_SIZE);
> > }
>
> Does this renaming apply to other devices as well? Mind to send followup
> patches?
>
No, It was virtio-blk specific.
I named the var pci_device when I've added multiple virtio-blk device
support.
--
Sasha.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-05-09 11:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-07 10:50 [PATCH] kvm tools: Rename pci_device to pci_hdr for clarity Sasha Levin
2011-05-09 10:35 ` Asias He
2011-05-09 11:23 ` Sasha Levin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).