lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 23 Jan 2019 17:25:00 -0500 From: "Michael S. Tsirkin" <mst@...hat.com> To: Christoph Hellwig <hch@....de> Cc: Joerg Roedel <joro@...tes.org>, Jason Wang <jasowang@...hat.com>, Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>, Jens Axboe <axboe@...nel.dk>, virtualization@...ts.linux-foundation.org, linux-block@...r.kernel.org, linux-kernel@...r.kernel.org, iommu@...ts.linux-foundation.org, jfehlig@...e.com, jon.grimm@....com, brijesh.singh@....com, jroedel@...e.de, pbonzini@...hat.com, jsnow@...hat.com Subject: Re: [PATCH 5/5] virtio-blk: Consider virtio_max_dma_size() for maximum segment size On Wed, Jan 23, 2019 at 10:31:39PM +0100, Christoph Hellwig wrote: > On Wed, Jan 23, 2019 at 05:30:49PM +0100, Joerg Roedel wrote: > > + max_size = virtio_max_dma_size(vdev); > > + > > /* Host can optionally specify maximum segment size and number of > > * segments. */ > > err = virtio_cread_feature(vdev, VIRTIO_BLK_F_SIZE_MAX, > > struct virtio_blk_config, size_max, &v); > > if (!err) > > - blk_queue_max_segment_size(q, v); > > - else > > - blk_queue_max_segment_size(q, -1U); > > + max_size = min(max_size, v); > > + > > + blk_queue_max_segment_size(q, max_size); > > I wonder if we should just move the dma max segment size check > into blk_queue_max_segment_size so that all block drivers benefit > from it. Even if not I think at least the SCSI midlayer should > be updated to support it. > > Btw, I wonder why virtio-scsi sticks to the default segment size, > unlike virtio-blk. Well no one bothered exposing that through that device. Why does virtio block have it? Hard for me to say. First driver version had it already but QEMU does not seem to use it and it seems that it never did. -- MST
Powered by blists - more mailing lists