On Thu, Jan 11, 2024 at 08:50:45AM -0800, Nicolin Chen wrote: > On Wed, Jan 10, 2024 at 08:10:13PM -0800, Yi Liu wrote: > > +#define test_cmd_hwpt_invalidate(hwpt_id, reqs, data_type, lreq, nreqs) \ > > + ({ \ > > + ASSERT_EQ(0, \ > > + _test_cmd_hwpt_invalidate(self->fd, hwpt_id, reqs, \ > > + data_type, \ > > + lreq, nreqs)); \ > > + }) > > +#define test_err_hwpt_invalidate(_errno, hwpt_id, reqs, data_type, lreq, \ > > + nreqs) \ > > + ({ \ > > + EXPECT_ERRNO(_errno, \ > > + _test_cmd_hwpt_invalidate(self->fd, hwpt_id, \ > > + reqs, data_type, \ > > + lreq, nreqs)); \ > > + }) > > Nit: spaces at the end of the four lines above that have string > "data_type" are all misaligned, probably because of the previous > replacement of "req_type". I will fix it Jason