Re: [RFC PATCH 1/4] kmemtrace: Core implementation.

From: Pekka J Enberg
Date: Fri Jul 18 2008 - 04:48:35 EST

Hi Eduard-Gabriel,

On Thu, 17 Jul 2008, Eduard - Gabriel Munteanu wrote:
> > > +struct kmemtrace_event {
> >
> > So why don't we have the ABI version embedded here like blktrace has
> > so that user-space can check if the format matches its expectations?
> > That should be future-proof as well: as long as y ou keep the existing
> > fields where they're at now, you can always add new fields at the end
> > of the struct.
> You can't add fields at the end, because the struct size will change and
> reads will be erroneous. Also, stamping every 'packet' with ABI version
> looks like a huge waste of space.

It's an ABI so you want to make it backwards compatible and extensible.
Yes, it's just for debugging, so the rules are bit more relaxed here but
that's not an excuse for not designing the ABI properly.

I really wish we would follow the example set by blktrace here. It uses a
fixed-length header that knows the length of the rest of the packet.

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at