On Tue, Mar 19, 2024 at 05:36:01PM +0100, Stefano Garzarella wrote:
> On Mon, Feb 19, 2024 at 02:56:04AM +0800, Zhu Lingshan wrote:
> > This commits reports write zeroes configuration of
> > virtio-block devices to user space, includes:
> > 1)maximum write zeroes sectors size
> > 2)maximum write zeroes segment number
> >
> > Signed-off-by: Zhu Lingshan <lingshan.zhu@xxxxxxxxx>
> > ---
> > drivers/vdpa/vdpa.c | 23 +++++++++++++++++++++++
> > include/uapi/linux/vdpa.h | 2 ++
> > 2 files changed, 25 insertions(+)
> >
> > diff --git a/drivers/vdpa/vdpa.c b/drivers/vdpa/vdpa.c
> > index bf3fa90908a7..adbcbc7b18b2 100644
> > --- a/drivers/vdpa/vdpa.c
> > +++ b/drivers/vdpa/vdpa.c
> > @@ -1064,6 +1064,26 @@ static int vdpa_dev_blk_discard_config_fill(struct sk_buff *msg, u64 features,
> > return 0;
> > }
> >
> > +static int
> > +vdpa_dev_blk_write_zeroes_config_fill(struct sk_buff *msg, u64 features,
> > + const struct virtio_blk_config *config)
> > +{
> > + u32 val_u32;
> > +
> > + if ((features & BIT_ULL(VIRTIO_BLK_F_WRITE_ZEROES)) == 0)
> > + return 0;
> > +
> > + val_u32 = __virtio32_to_cpu(true, config->max_write_zeroes_sectors);
> > + if (nla_put_u32(msg, VDPA_ATTR_DEV_BLK_CFG_MAX_WRITE_ZEROES_SEC, val_u32))
> > + return -EMSGSIZE;
> > +
> > + val_u32 = __virtio32_to_cpu(true, config->max_write_zeroes_seg);
> > + if (nla_put_u32(msg, VDPA_ATTR_DEV_BLK_CFG_MAX_WRITE_ZEROES_SEG, val_u32))
> > + return -EMSGSIZE;
>
> Should we cover also `config->write_zeroes_may_unmap` ?
This review came too late I applied the patch. Generally
if I don't see any response in about
a week I assume there is not interest.
> > +
> > + return 0;
> > +}
> > +
> > static int vdpa_dev_blk_config_fill(struct vdpa_device *vdev,
> > struct sk_buff *msg)
> > {
> > @@ -1099,6 +1119,9 @@ static int vdpa_dev_blk_config_fill(struct vdpa_device *vdev,
> > if (vdpa_dev_blk_discard_config_fill(msg, features_device, &config))
> > return -EMSGSIZE;
> >
> > + if (vdpa_dev_blk_write_zeroes_config_fill(msg, features_device, &config))
> > + return -EMSGSIZE;
> > +
> > return 0;
> > }
> >
> > diff --git a/include/uapi/linux/vdpa.h b/include/uapi/linux/vdpa.h
> > index 30887931a220..797d5708492f 100644
> > --- a/include/uapi/linux/vdpa.h
> > +++ b/include/uapi/linux/vdpa.h
> > @@ -68,6 +68,8 @@ enum vdpa_attr {
> > VDPA_ATTR_DEV_BLK_CFG_MAX_DISCARD_SEC, /* u32 */
> > VDPA_ATTR_DEV_BLK_CFG_MAX_DISCARD_SEG, /* u32 */
> > VDPA_ATTR_DEV_BLK_CFG_DISCARD_SEC_ALIGN,/* u32 */
> > + VDPA_ATTR_DEV_BLK_CFG_MAX_WRITE_ZEROES_SEC, /* u32 */
> > + VDPA_ATTR_DEV_BLK_CFG_MAX_WRITE_ZEROES_SEG, /* u32 */
> >
> > /* new attributes must be added above here */
> > VDPA_ATTR_MAX,
> > --
> > 2.39.3
> >
> >
