On Thu, Mar 21, 2024 at 09:26:58AM +0100, Laurent Vivier wrote:
> On 3/21/24 02:26, David Gibson wrote:
> > On Wed, Mar 20, 2024 at 05:31:46PM +0100, Laurent Vivier wrote:
> > > To be able to provide pointers to TCP headers and IP headers without
> > > worrying about alignment in the structure, split the structure into
> > > several arrays and point to each part of the frame using an iovec array.
> > > 
> > > Using iovec also allows us to simply ignore the first entry when the
> > > vnet length header is not needed.
> > > 
> > > Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> > 
> > [snip]
> > > +static_assert(MSS4 <= sizeof(tcp4_payload[0].data));
> > 
> > This generates a clang-tidy warning, because apparently the C11
> > version of static_assert() requires a message - making it optional is
> > a C2x extension.
> > 
> > Laurent, I know you're having trouble getting the full testsuite to
> > run (but some ideas on that later today, I hope).  But could you
> > please add a "make cppcheck" and "make clang-tidy" to your pre-post
> > routine.
> > 
> 
> I'll do.
> Thank you to have checked that.
> Are there any other commands to run before to send?
> Something like scripts/chekpatch.pl we have in QEMU?
> 
> I have always the same problem with the test suite, even after a "make
> realclean" in test (see attachment)
> How to debug?

Huh.  Guest kernel can't find root.  That's not a fault I was
expecting (and alas, won't be helped by the patch I have in the
works).

What's really weird is I don't see any signs of loading the initrd
before that failure, and with the mbuto images we're using we should
never *leave* the initrd.  Which kind of makes sense - if it's not
seeing the initrd, then there is indeed no root to mount.

Can you gran the full qemu command line it's trying to use?

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson