On 24/02/2022 13:03, Konstantin Meskhidze wrote:

2/24/2022 12:55 PM, Mickaël Salaün пишет:

quoted

On 24/02/2022 04:18, Konstantin Meskhidze wrote:

quoted

2/1/2022 9:31 PM, Mickaël Salaün пишет:

quoted

On 24/01/2022 09:02, Konstantin Meskhidze wrote:

quoted

Support 4 tests for bind and connect networks actions:

Good to see such tests!

quoted

1. bind() a socket with no landlock restrictions.
2. bind() sockets with landllock restrictions.

You can leverage the FIXTURE_VARIANT helpers to factor out this kind 
of tests (see ptrace_test.c).

quoted

3. connect() a socket to listening one with no landlock restricitons.
4. connect() sockets with landlock restrictions.

Same here, you can factor out code. I guess you could create helpers 
for client and server parts.

We also need to test with IPv4, IPv6 and the AF_UNSPEC tricks.

Please provide the kernel test coverage and explain why the 
uncovered code cannot be covered: 
https://www.kernel.org/doc/html/latest/dev-tools/gcov.html

  Hi Mickaёl!
  Could you please provide the example of your test coverage build
  process? Cause as I undersatand there is no need to get coverage data
  for the entire kernel, just for landlock files.

You just need to follow the documentation:
- start the VM with the kernel appropriately configured for coverage;
- run all the Landlock tests;
- gather the coverage and shutdown the VM;
- use lcov and genhtml to create the web pages;
- look at the coverage for security/landlock/

It would be interesting to know the coverage for security/landlock/ 
before and after your changes, and also specifically for 
security/landlock.net.c

quoted

    Thank you so much!

    One more questuoin - Is it possible to run Landlock tests in QEMU and
    and gather coverage info or I need to change kernel for the whole VM?

You need to gather the coverage info on the same system that ran the 
tests, so with the same kernel supporting both Landlock and gcov. You 
can then generate the web pages elsewhere.