On Wed, Oct 30, 2024 at 9:49 PM Jakub Kicinski <kuba@xxxxxxxxxx> wrote:
>
> On Sun, 27 Oct 2024 12:59:42 -0700 Maksym Kutsevol wrote:
> > +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.
> > +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
