Re: [RFC][PATCH 0/3] Process events biarch bug: Intro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 27, 2006 at 04:47:01AM -0700, Matt Helsley ([email protected]) wrote:
> The events sent by Process Events from a 64-bit kernel are not binary compatible
> with what a 32-bit userspace program would expect to recieve because the timespec
> struct (used to send a timestamp) is not the same. This means that fields stored
> after the timestamp are offset and programs that don't take this into account
> break under these circumstances.
> 
> This is a problem for 32-bit userspace programs running with 64-bit kernels on
> ppc64, s390, x86-64.. any "biarch" system.
> 
> This series:
> 
> 1 - Gives a name to the union of the process events structure so it may be used
> 	to work around the problem from userspace.
> 2 - Comments on the bug and describes a userspace workaround in
> 	Documentation/connector/process_events.txt
> 3 - Implements a second connector interface without the problem
> 	(Removing the old interface or changing the definition would break
> 	 binary compatibility)

If you are sure binary compatibility on this stage of process
notification connector is really major issue, then I agree that above is
correct, otherwise I would recommend to just replace broken code with fixed size objects.

Btw, __u64 is not the best choice for some arches too due to it's
alignment (64bit code requires u64 to be aligned to 64 bit, while 32bit
code will only align it to 32 bits), so you will need 
attribute ((aligned(8)))) for your own ___u64.

-- 
	Evgeniy Polyakov
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux