[ovs-dev] [PATCH ovn v6 1/3] northd: refactor and split some IPAM functions

0-day Robot robot at bytheb.org
Tue Jan 12 19:00:15 UTC 2021


Bleep bloop.  Greetings Mark Michelson, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
WARNING: Comment with 'xxx' marker
#80 FILE: northd/ipam.c:28:
    /* XXX Since we only accept /64 addresses, why do we even bother

WARNING: Line is 81 characters long (recommended limit is 79)
#204 FILE: northd/ipam.c:152:
init_ipam_info(struct ipam_info *info, const struct smap *config, const char *id)

WARNING: Comment with 'xxx' marker
#842 FILE: northd/ovn-northd.c:12613:
    /* XXX Having to explicitly clean up macam here

Lines checked: 855, Warnings: 3, Errors: 0


Please check this out.  If you feel there has been an error, please email aconole at redhat.com

Thanks,
0-day Robot


More information about the dev mailing list