From: Michael Mueller <mimu@linux.ibm.com>
To: Halil Pasic <pasic@linux.ibm.com>,
kvm@vger.kernel.org, linux-s390@vger.kernel.org,
Cornelia Huck <cohuck@redhat.com>,
Sebastian Ott <sebott@linux.ibm.com>,
Heiko Carstens <heiko.carstens@de.ibm.com>
Cc: virtualization@lists.linux-foundation.org,
"Michael S. Tsirkin" <mst@redhat.com>,
Christoph Hellwig <hch@infradead.org>,
Thomas Huth <thuth@redhat.com>,
Christian Borntraeger <borntraeger@de.ibm.com>,
Viktor Mihajlovski <mihajlov@linux.ibm.com>,
Vasily Gorbik <gor@linux.ibm.com>,
Janosch Frank <frankja@linux.ibm.com>,
Claudio Imbrenda <imbrenda@linux.ibm.com>,
Farhan Ali <alifm@linux.ibm.com>,
Eric Farman <farman@linux.ibm.com>,
"Jason J. Herne" <jjherne@linux.ibm.com>
Subject: Re: [PATCH v5 6/8] virtio/s390: add indirection to indicators access
Date: Thu, 13 Jun 2019 10:29:32 +0200 [thread overview]
Message-ID: <caea5a6e-f174-7019-ce73-e9b52e20d50a@linux.ibm.com> (raw)
In-Reply-To: <20190612111236.99538-7-pasic@linux.ibm.com>
On 12.06.19 13:12, Halil Pasic wrote:
> This will come in handy soon when we pull out the indicators from
> virtio_ccw_device to a memory area that is shared with the hypervisor
> (in particular for protected virtualization guests).
>
> Signed-off-by: Halil Pasic <pasic@linux.ibm.com>
> Reviewed-by: Pierre Morel <pmorel@linux.ibm.com>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> ---
> drivers/s390/virtio/virtio_ccw.c | 40 ++++++++++++++++++++------------
> 1 file changed, 25 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/s390/virtio/virtio_ccw.c b/drivers/s390/virtio/virtio_ccw.c
> index 1da7430f94c8..e96a8cc56ec2 100644
> --- a/drivers/s390/virtio/virtio_ccw.c
> +++ b/drivers/s390/virtio/virtio_ccw.c
> @@ -68,6 +68,16 @@ struct virtio_ccw_device {
> void *airq_info;
> };
>
> +static inline unsigned long *indicators(struct virtio_ccw_device *vcdev)
> +{
> + return &vcdev->indicators;
> +}
> +
> +static inline unsigned long *indicators2(struct virtio_ccw_device *vcdev)
> +{
> + return &vcdev->indicators2;
> +}
> +
> struct vq_info_block_legacy {
> __u64 queue;
> __u32 align;
> @@ -338,17 +348,17 @@ static void virtio_ccw_drop_indicator(struct virtio_ccw_device *vcdev,
> ccw->cda = (__u32)(unsigned long) thinint_area;
> } else {
> /* payload is the address of the indicators */
> - indicatorp = kmalloc(sizeof(&vcdev->indicators),
> + indicatorp = kmalloc(sizeof(indicators(vcdev)),
> GFP_DMA | GFP_KERNEL);
> if (!indicatorp)
> return;
> *indicatorp = 0;
> ccw->cmd_code = CCW_CMD_SET_IND;
> - ccw->count = sizeof(&vcdev->indicators);
> + ccw->count = sizeof(indicators(vcdev));
> ccw->cda = (__u32)(unsigned long) indicatorp;
> }
> /* Deregister indicators from host. */
> - vcdev->indicators = 0;
> + *indicators(vcdev) = 0;
> ccw->flags = 0;
> ret = ccw_io_helper(vcdev, ccw,
> vcdev->is_thinint ?
> @@ -657,10 +667,10 @@ static int virtio_ccw_find_vqs(struct virtio_device *vdev, unsigned nvqs,
> * We need a data area under 2G to communicate. Our payload is
> * the address of the indicators.
> */
> - indicatorp = kmalloc(sizeof(&vcdev->indicators), GFP_DMA | GFP_KERNEL);
> + indicatorp = kmalloc(sizeof(indicators(vcdev)), GFP_DMA | GFP_KERNEL);
> if (!indicatorp)
> goto out;
> - *indicatorp = (unsigned long) &vcdev->indicators;
> + *indicatorp = (unsigned long) indicators(vcdev);
> if (vcdev->is_thinint) {
> ret = virtio_ccw_register_adapter_ind(vcdev, vqs, nvqs, ccw);
> if (ret)
> @@ -669,21 +679,21 @@ static int virtio_ccw_find_vqs(struct virtio_device *vdev, unsigned nvqs,
> }
> if (!vcdev->is_thinint) {
> /* Register queue indicators with host. */
> - vcdev->indicators = 0;
> + *indicators(vcdev) = 0;
> ccw->cmd_code = CCW_CMD_SET_IND;
> ccw->flags = 0;
> - ccw->count = sizeof(&vcdev->indicators);
> + ccw->count = sizeof(indicators(vcdev));
> ccw->cda = (__u32)(unsigned long) indicatorp;
> ret = ccw_io_helper(vcdev, ccw, VIRTIO_CCW_DOING_SET_IND);
> if (ret)
> goto out;
> }
> /* Register indicators2 with host for config changes */
> - *indicatorp = (unsigned long) &vcdev->indicators2;
> - vcdev->indicators2 = 0;
> + *indicatorp = (unsigned long) indicators2(vcdev);
> + *indicators2(vcdev) = 0;
> ccw->cmd_code = CCW_CMD_SET_CONF_IND;
> ccw->flags = 0;
> - ccw->count = sizeof(&vcdev->indicators2);
> + ccw->count = sizeof(indicators2(vcdev));
> ccw->cda = (__u32)(unsigned long) indicatorp;
> ret = ccw_io_helper(vcdev, ccw, VIRTIO_CCW_DOING_SET_CONF_IND);
> if (ret)
> @@ -1093,17 +1103,17 @@ static void virtio_ccw_int_handler(struct ccw_device *cdev,
> vcdev->err = -EIO;
> }
> virtio_ccw_check_activity(vcdev, activity);
> - for_each_set_bit(i, &vcdev->indicators,
> - sizeof(vcdev->indicators) * BITS_PER_BYTE) {
> + for_each_set_bit(i, indicators(vcdev),
> + sizeof(*indicators(vcdev)) * BITS_PER_BYTE) {
> /* The bit clear must happen before the vring kick. */
> - clear_bit(i, &vcdev->indicators);
> + clear_bit(i, indicators(vcdev));
> barrier();
> vq = virtio_ccw_vq_by_ind(vcdev, i);
> vring_interrupt(0, vq);
> }
> - if (test_bit(0, &vcdev->indicators2)) {
> + if (test_bit(0, indicators2(vcdev))) {
> virtio_config_changed(&vcdev->vdev);
> - clear_bit(0, &vcdev->indicators2);
> + clear_bit(0, indicators2(vcdev));
> }
> }
>
>
Reviewed-by: Michael Mueller <mimu@linux.ibm.com>
Michael
next prev parent reply other threads:[~2019-06-13 16:31 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-12 11:12 [PATCH v5 0/8] s390: virtio: support protected virtualization Halil Pasic
2019-06-12 11:12 ` [PATCH v5 1/8] s390/mm: force swiotlb for " Halil Pasic
2019-06-13 8:09 ` Michael Mueller
2019-07-11 21:48 ` Thiago Jung Bauermann
2019-06-12 11:12 ` [PATCH v5 2/8] s390/cio: introduce DMA pools to cio Halil Pasic
2019-06-12 14:23 ` Cornelia Huck
2019-06-13 8:13 ` Michael Mueller
2019-06-12 11:12 ` [PATCH v5 3/8] s390/cio: add basic protected virtualization support Halil Pasic
2019-06-13 8:21 ` Michael Mueller
2019-06-12 11:12 ` [PATCH v5 4/8] s390/airq: use DMA memory for adapter interrupts Halil Pasic
2019-06-12 14:35 ` Cornelia Huck
2019-06-12 15:11 ` Halil Pasic
2019-06-13 8:25 ` Michael Mueller
2019-06-12 11:12 ` [PATCH v5 5/8] virtio/s390: use cacheline aligned airq bit vectors Halil Pasic
2019-06-13 8:27 ` Michael Mueller
2019-06-12 11:12 ` [PATCH v5 6/8] virtio/s390: add indirection to indicators access Halil Pasic
2019-06-13 8:29 ` Michael Mueller [this message]
2019-06-12 11:12 ` [PATCH v5 7/8] virtio/s390: use DMA memory for ccw I/O and classic notifiers Halil Pasic
2019-06-13 8:32 ` Michael Mueller
2019-06-12 11:12 ` [PATCH v5 8/8] virtio/s390: make airq summary indicators DMA Halil Pasic
2019-06-13 8:35 ` Michael Mueller
2019-06-13 9:11 ` [PATCH v5 0/8] s390: virtio: support protected virtualization Michael Mueller
2019-06-13 11:14 ` Halil Pasic
2019-06-13 11:17 ` Michael Mueller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=caea5a6e-f174-7019-ce73-e9b52e20d50a@linux.ibm.com \
--to=mimu@linux.ibm.com \
--cc=alifm@linux.ibm.com \
--cc=borntraeger@de.ibm.com \
--cc=cohuck@redhat.com \
--cc=farman@linux.ibm.com \
--cc=frankja@linux.ibm.com \
--cc=gor@linux.ibm.com \
--cc=hch@infradead.org \
--cc=heiko.carstens@de.ibm.com \
--cc=imbrenda@linux.ibm.com \
--cc=jjherne@linux.ibm.com \
--cc=kvm@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
--cc=mihajlov@linux.ibm.com \
--cc=mst@redhat.com \
--cc=pasic@linux.ibm.com \
--cc=sebott@linux.ibm.com \
--cc=thuth@redhat.com \
--cc=virtualization@lists.linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).