"Darrick J. Wong" <djwong@xxxxxxxxxx> writes: > Ritesh: Dump the enum; "because btrfs does it" is not sufficient > justification. The rest is good enough, I'll put it in iomap-for-next > along with willy's thing as soon as 6.5-rc1 closes, and if you all have > further complaints, send your own patches. Sure, Darrick. I agree that enum created more confusion than help (sorry aobut that). I will dump it in the next revision & address Matthew's review comment on the variable naming. I agree to lock the current set of changes as functionality wise those are looking good from everyone. Any further optimizations/improvements can be handled seperately so that we don't complicate this one. Thanks once again Darrick and all the other reviewers!! -ritesh I am pulled into something else for next couple of days. Post that I will work on it to get next revision out.