Currently the acl and aerl are set to values 3 and 5. as specificaton's recommendation, these values are 4.
There isn't a limit on the number of concurrent aborts in the implementation. So one arbitrary value is as good as another, but I'd suggest removing the comment.
The wording in the specification is:
Supporting 5 (aerl = 4) would meet their recommendation, wouldn't you agree?
You are right, currently the Abort Command is not processed by the controller. I'd like the value to be more normal as real nvme controllers use the specification recommended value. I will remove the comment later.
Yes, a value higher than 4 is totally OK as specification says. but the value of 5 is a little un-normal. I don't know why we pick up this value at first. as 4 is the value that other controllers do as far as my observation.