Re: [PATCH ndctl RESEND 1/2] ndctl/test: Add destroy region test

2023-12-05 Thread Verma, Vishal L
On Tue, 2023-12-05 at 13:22 -0800, Ira Weiny wrote: > Verma, Vishal L wrote: > > [snip] > > > > > > > > > > Correct, the set -e will cause the script to abort with an error exit > > code whenever a command fails. > > > > I do wonder if we need this new test - with Dave's patch here[1], > >

Re: [PATCH ndctl RESEND 1/2] ndctl/test: Add destroy region test

2023-12-05 Thread Ira Weiny
Verma, Vishal L wrote: [snip] > > > > +# Find a memory device to create regions on to test the destroy > > > > +readarray -t mems < <("$CXL" list -b cxl_test -M | jq -r '.[].memdev') > > > > +for mem in ${mems[@]}; do > > > > +    ramsize=$($CXL list -m $mem | jq -r '.[].ram_size') > > > > +