Re: [PATCH net-next v4 2/2] netcons: Add udp send fail statistics to netconsole

On Wed, Oct 30, 2024 at 9:49 PM Jakub Kicinski [off-list ref] wrote:

On Sun, 27 Oct 2024 12:59:42 -0700 Maksym Kutsevol wrote:

quoted

+struct netconsole_target_stats  {
+     u64_stats_t xmit_drop_count;
+     u64_stats_t enomem_count;
+     struct u64_stats_sync syncp;
+} __aligned(2 * sizeof(u64));

Why the alignment?

Hi Jakub,

Thanks for looking into this!

Parroting examples, e.g.
struct pcpu_lstats {
u64_stats_t packets;
u64_stats_t bytes;
struct u64_stats_sync syncp;
} __aligned(2 * sizeof(u64));

in netdevice.h https://github.com/torvalds/linux/blob/master/include/linux/netdevice.h#L2743-L2747
I don't have any strongly held opinion about this. I'd appreciate an
explanation (a link would suffice) why this is a bad idea.

quoted

+static void netpoll_send_udp_count_errs(struct netconsole_target *nt,
+                                     const char *msg, int len)

This is defined in the netconsole driver, it should not use the
netpoll_ prefix for the function name.'

netconsole_send_udp_count_errs sounds better?

Regards,
Maksym