On Thu, Mar 02, 2023 at 01:23:48AM +0000, Tian, Kevin wrote: > > { > > - if (self->domain_id) { > > + if (self->stdev_id) { > > /* IOAS cannot be freed while a domain is on it */ > > EXPECT_ERRNO(EBUSY, > > _test_ioctl_destroy(self->fd, self->ioas_id)); > > it's easier to understand this comment. I changed this comment to use the right language: if (self->stdev_id) { /* IOAS cannot be freed while a device has a HWPT using it */ EXPECT_ERRNO(EBUSY, _test_ioctl_destroy(self->fd, self->ioas_id)); Thanks, Jason