[PATCH V2 2/3] lnvm: introduce chunk-log command for chunk info

[minwoo.im.dev@gmail.com (Minwoo Im)]

Hi Matias,

> > +static inline const char *print_chunk_type(__u8 ct)
> > +{
> > +	switch (ct) {
> > +	case 1 << 0:	return "SEQWRITE_REQ";
> > +	case 1 << 1:	return "RANDWRITE_ALLOWED";
> > +	case 1 << 4:	return "DEVIATED";
> > +	default:	return "UNKNOWN";
> > +	}
> > +}
> 
> The Chunk Type is divided into two value ranges - one for the chunk type
> (seq, req) and one that describes the properties of the zone. I would move
> the "Deviated" into being expressed as an attribute (or similar). Then they
> are clearly separated. An OCSSD implementation can both have the SEQ write
> bit set, together with bit 4 for dynamic chunk capacity. The above doesn't
> communicate that in the right way.

Ah sorry, I think my brain was not working well in the dawn ;).  I'll
provide "Deviated" things as an attribute first regardless the chunk
types.

Thanks for the review.  Will prepare the next version.