Thread (53 messages) 53 messages, 6 authors, 2020-04-09

Re: [PATCH v3 00/26] Introduce common headers for vDSO

From: Thomas Gleixner <hidden>
Date: 2020-03-15 10:03:05
Also in: linux-arch, linux-mips, lkml

Dmitry Safonov [off-list ref] writes:
I like the idea, but I'm wondering if we could have less-grained
headers? Like, AFAICS the patches create headers < 10 lines and even
mostly < 5 lines.. I like that header's names perfectly describe what's
inside, but I'm not sure how effective to have a lot of extra-small
includes.
If that goes all into a big header then the headers from where the bits and
pieces are split out would have all to include this big header which
might result in other include dependency nightmares.
quoted
 create mode 100644 include/vdso/time.h
 create mode 100644 include/vdso/time32.h
 create mode 100644 include/vdso/time64.h
Maybe we could made them less-grained?

I.e, time32 + time64 + time.h => time.h?
Then you end up with ifdeffery. I like the clear separation.

Thanks,

        tglx

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help