Fix bug of creating metadata for custom attribute SAI_ACL_ENTRY/TABLE_ATTR_(FIELD/ACTION)

[iamjustadd]

1. Fix bug in saisanitycheck that there is no check for range
   SAI_ACL_ENTRY_ATTR(/_FIELD/_ACTION)_CUSTOM_RANGE_START to
   SAI_ACL_ENTRY_ATTR(/_FIELD/_ACTION)_CUSTOM_RANGE_END
2. Remove all SAI_*_CUSTOM_RANGE_START/END, using SAI_*_CUSTOM_RANGE_BASE instead.
3. Adjust metadata generator to compitable with above changes.

[iamjustadd]

@kcudnik , it's a new commit.

1. Didn't create any custom headers, only have a example custom ACL header in doc/ .
2. Replace CUSTOM_RANGE_START/END with CUSTOM_BASE.
   Reason: In custom headers, create new attribute won't modify the value of CUSTOM_RANGE_END. So it's better to define START/END at custom headers for customer's need.
3. Create ACL_ENTRY_ACTION/FIELD_CUSTOM_RANGE_BASE = ACL_ENTRY_CUSTOM_RANGE_BASE + 0x1000 to align with non-custom attributes range.
4. Modify saisanitycheck.c and test.pm to compitable with above changes.

[tjchadaga]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[kcudnik]

this is pointless, nothing happens here

[iamjustadd]

Why? I think here is to move the index to the first custom attribute of SAI_ACL_ENTRY_ATTR_ACTION_*

[kcudnik]

add a comment then whats happening, and check if index gets at the last element then log error

[iamjustadd]

I added comment for this loop.

This function is to check whether the number of attributes acl_entry_action_* is matched with sai_acl_action_type , in another word, assert that user defined attribute acl_entry_action which must have a correspond acl_action_tyoe.
At line 5136 have this assertion:
META_ASSERT_TRUE(enum_index == sai_metadata_enum_sai_acl_action_type_t.valuescount,
"number of acl entry action mismatch vs number of enums in sai_acl_action_type_t");

If User not create custom attribute of ACL_ENTRY_ACTION, then here will loop till the
meta_acl_entry[index] == NULL(hit the last).
This is not an ERROR.

[iamjustadd]

@tjchadaga @kcudnik Hi gentlemen, Any other comments?