opaque types instead of union epoll_data

From	Sodagudi Prasad <psodagud@codeaurora.org>
Newsgroups	linux.kernel
Subject	opaque types instead of union epoll_data
Date	2017-03-07 13:40 +0100
Message-ID	<timzf-7wm-1@gated-at.bofh.it> (permalink)
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi All,

uapi structs epoll_data are more opaque than user space expects. kernel 
have defined as __u64 instead of the union epoll_data.
Because of this libc have redefined struct epoll_event with union data 
member.

https://android.googlesource.com/platform/bionic.git/+/master/libc/include/sys/epoll.h
typedef union epoll_data {
   void* ptr;
   int fd;
   uint32_t u32;
   uint64_t u64;
} epoll_data_t;
struct epoll_event {
   uint32_t events;
   epoll_data_t data;
}

Kernel UAPI header defined as "include/uapi/linux/eventpoll.h"
struct epoll_event {
         __u32 events;
         __u64 data;  =====>opaque type instead of union epoll_data
} EPOLL_PACKED;


Because of this we are landing into some issues as we copy kernel 
headers. Will it be going to be addressed?

-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora 
Forum,
Linux Foundation Collaborative Project

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

opaque types instead of union epoll_data Sodagudi Prasad <psodagud@codeaurora.org> - 2017-03-07 13:40 +0100
  Re: opaque types instead of union epoll_data Carlos O'Donell <carlos@redhat.com> - 2017-03-07 18:40 +0100
    Re: opaque types instead of union epoll_data Greg KH <gregkh@linuxfoundation.org> - 2017-03-07 19:30 +0100
      Re: opaque types instead of union epoll_data Carlos O'Donell <carlos@redhat.com> - 2017-03-08 18:40 +0100
        Re: opaque types instead of union epoll_data Greg KH <gregkh@linuxfoundation.org> - 2017-03-08 19:10 +0100

csiph-web