Fix required properties in schemas

With invalid properties not showing up in the output, they are not
required in the schema anymore.

Note: UTs test with all properties enabled in the cper-generato

Fix required properties in schemas

With invalid properties not showing up in the output, they are not
required in the schema anymore.

Note: UTs test with all properties enabled in the cper-generator.

Tested by running example cpers against schemas

Change-Id: I54dcda2ae00587e4a7d59e0365eb41ac1669ba46
Signed-off-by: Aushim Nagarkatti <anagarkatti@nvidia.com>

show more ...

Remove validation bits

Discard invalid properties from json decode. JSON output should only
contain valid properties. This saves time in preventing post
processing of output for valid fields.

Ensur

Remove validation bits

Discard invalid properties from json decode. JSON output should only
contain valid properties. This saves time in preventing post
processing of output for valid fields.

Ensure round trip validity with validation bits removed and required
properties populated.

Fix bugs in json decode.

Overhaul unit tests to use valijson. Add tests with static examples
to validate against schema. Use and nlohmann for better schema
validation over intrinsic libcper validation.

Example json output before:
{
"ValidationBits": {
"LevelValid": false,
"CorrectedValid": true
},
"Level": 1,
"Corrected": true
}

After:
{
"Corrected": true
}

Change-Id: I188bdc2827a57d938c22a431238fadfcdc939ab8
Signed-off-by: Aushim Nagarkatti <anagarkatti@nvidia.com>

show more ...

Change include style to use system includes

The libcper header files in `libcper` are installed to
`usr/include/libcper`. Use that system includes in `libcper` source
instead of using the project in

Change include style to use system includes

The libcper header files in `libcper` are installed to
`usr/include/libcper`. Use that system includes in `libcper` source
instead of using the project includes.

Change-Id: I596edc2c754dae4829844f535a1e34caa246fb43
Signed-off-by: Thu Nguyen <thu@os.amperecomputing.com>

show more ...

Format meson files

Format with

meson format -i $(git ls-files | grep "meson.*$")

Tested: code compiles. Whitespace only.

Change-Id: I63f2e78aeb7924f9560c3b45ba3b5c4059c47e12
Signed-off-by: Ed Ta

Format meson files

Format with

meson format -i $(git ls-files | grep "meson.*$")

Tested: code compiles. Whitespace only.

Change-Id: I63f2e78aeb7924f9560c3b45ba3b5c4059c47e12
Signed-off-by: Ed Tanous <ed@tanous.net>

show more ...

Generate pkgconfig dependency for parser library

Had to rename overloaded variables & change tests/meson.build to match

Change-Id: Id92a71e6ff4c7edcdc0b64de90e078936921b4d5
Signed-off-by: Karthik R

Generate pkgconfig dependency for parser library

Had to rename overloaded variables & change tests/meson.build to match

Change-Id: Id92a71e6ff4c7edcdc0b64de90e078936921b4d5
Signed-off-by: Karthik Rajagopalan <krajagopalan@nvidia.com>

show more ...

Remove json-schema install

This code just copies the json-schema from the source dir to the compile
dir. It doesn't actually accomplish anything.

Move the source dir to a compile flag, and let uni

Remove json-schema install

This code just copies the json-schema from the source dir to the compile
dir. It doesn't actually accomplish anything.

Move the source dir to a compile flag, and let unit tests read in the
schema directly.

Note: Copy was actually done in two places.

Change-Id: I26aee9edbac5b253ca426d32118b71cc5f905e10
Signed-off-by: Ed Tanous <ed@tanous.net>

show more ...

Fix meson warnings

Clean up some variable naming conventions, remove some unused variables.

Change-Id: I87fc4645853b9ce33968863ece0f73cf4bae71fc
Signed-off-by: Ed Tanous <ed@tanous.net>

Move to embedded base64

Base64 encode/decode is a relatively simple algorithm, and currently
libcper takes a dependency on libb64 for this. libb64 does not have
methods for determining the encoded

Move to embedded base64

Base64 encode/decode is a relatively simple algorithm, and currently
libcper takes a dependency on libb64 for this. libb64 does not have
methods for determining the encoded size or decoded size, and rely on
the user to provide the right buffer sizes, which libcper currently
approximates as 2X the input size (which is incorrect).

This commit removes the libb64 dependency entirely, and inlines a
libcper specific base64 encoder and decoder, using EDK2-allowed types.

The implementation itself is unique to libcper and makes the following
design decisions.
1. Malloc is performed within the base64_<> functions. This reduces the
number of malloc calls total, and removes the need for separately
determining the output size.
2. Arguments are passed in by EDK2-types under the assumption that this
will keep compatibility with EDK2 implementations.
3. Incremental parsing is not supported. CPER records are expected to
be algorithmically small, and buffered such that the entire value
fits in memory. This was already an assumption, but dropping the
support for incremental encoding significantly reduces the amount of
code to support it. It could be added back in the future if needed.

Change-Id: Idb010db105067ea317dbee05c2663511ab3c6611
Signed-off-by: Ed Tanous <ed@tanous.net>

show more ...

Reformat test meson.build

Using muon fmt

Change-Id: I01142cb05dd9d3901413e40f2218a7dc72b3f8d7
Signed-off-by: Ed Tanous <etanous@nvidia.com>

Move to meson build configuration

cmake is Deprecated

Signed-off-by: John Chung <john.chung@arm.com>
Change-Id: I9d08b087828aa5a16f601785dcb7868704ec73a3