Re: [gentoo-dev] linux/dma-buf.h mysteriously missing

2017-03-30 Thread Matt Turner 
On Thu, Mar 30, 2017 at 7:44 AM, Mike Gilbert  wrote:
> On Wed, Mar 29, 2017 at 1:51 PM, Matt Turner  wrote:
>> It's a bug that has since been fixed by kernel commit
>> 2220fc1ab363e6fab1f321430d69be17a8b92bd7 ("uapi: add missing install
>> of dma-buf.h"). The header was originally added in 4.10, and the fix
>> commit is in 4.11-rc1.
>>
>> I guess we just need to hack around it in sys-kernel/linux-headers-4.10.
>>
>
> git says the file was added in 4.6, not 4.10.

Yes, you are right. My mistake.

Re: [gentoo-dev] linux/dma-buf.h mysteriously missing

2017-03-30 Thread Kristian Fiskerstrand 
On 03/30/2017 04:44 PM, Mike Gilbert wrote:
> On Wed, Mar 29, 2017 at 1:51 PM, Matt Turner  wrote:
>> It's a bug that has since been fixed by kernel commit
>> 2220fc1ab363e6fab1f321430d69be17a8b92bd7 ("uapi: add missing install
>> of dma-buf.h"). The header was originally added in 4.10, and the fix
>> commit is in 4.11-rc1.
>>
>> I guess we just need to hack around it in sys-kernel/linux-headers-4.10.
>>
> 
> git says the file was added in 4.6, not 4.10.
> 

That seems more consistent with the original posting, containing
#if LINUX_VERSION_CODE < KERNEL_VERSION(4, 6, 0) clause for definitions

-- 
Kristian Fiskerstrand
OpenPGP keyblock reachable at hkp://pool.sks-keyservers.net
fpr:94CB AFDD 3034 5109 5618 35AA 0B7F 8B60 E3ED FAE3



signature.asc
Description: OpenPGP digital signature

Re: [gentoo-dev] linux/dma-buf.h mysteriously missing

2017-03-30 Thread Mike Gilbert 
On Wed, Mar 29, 2017 at 1:51 PM, Matt Turner  wrote:
> It's a bug that has since been fixed by kernel commit
> 2220fc1ab363e6fab1f321430d69be17a8b92bd7 ("uapi: add missing install
> of dma-buf.h"). The header was originally added in 4.10, and the fix
> commit is in 4.11-rc1.
>
> I guess we just need to hack around it in sys-kernel/linux-headers-4.10.
>

git says the file was added in 4.6, not 4.10.

Re: [gentoo-dev] linux/dma-buf.h mysteriously missing

2017-03-30 Thread Paweł Hajdan , Jr . 
On 29/03/2017 19:51, Matt Turner wrote:
> It's a bug that has since been fixed by kernel commit
> 2220fc1ab363e6fab1f321430d69be17a8b92bd7 ("uapi: add missing install
> of dma-buf.h"). The header was originally added in 4.10, and the fix
> commit is in 4.11-rc1.
> 
> I guess we just need to hack around it in sys-kernel/linux-headers-4.10.

Thanks for the pointer!

I filed 

Paweł




signature.asc
Description: OpenPGP digital signature

Re: [gentoo-dev] linux/dma-buf.h mysteriously missing

2017-03-29 Thread Matt Turner 
It's a bug that has since been fixed by kernel commit
2220fc1ab363e6fab1f321430d69be17a8b92bd7 ("uapi: add missing install
of dma-buf.h"). The header was originally added in 4.10, and the fix
commit is in 4.11-rc1.

I guess we just need to hack around it in sys-kernel/linux-headers-4.10.

Re: [gentoo-dev] linux/dma-buf.h mysteriously missing

2017-03-29 Thread Mike Gilbert 
On Wed, Mar 29, 2017 at 12:34 PM, Paweł Hajdan, Jr.
 wrote:
> I was packaging chromium-59.0.3053.3 . I worked around the problem
> described here, but I'd like to find the right long term solution.
>
> I was hitting the following compile error:
>
> ../../ui/gfx/linux/client_native_pixmap_dmabuf.cc:39:27: fatal error:
> linux/dma-buf.h: No such file or directory
>  #include 
>
> Despite having sys-kernel/linux-headers-4.10 installed, I do not have
> that header. Interestingly, it is present in the tarball
> (gentoo-headers-base-4.10.tar.xz), but doesn't seem to get installed.
>
> This is the workaround I applied:
> 
>
> This is part of the chromium code in question:
>
> 
>
> #if LINUX_VERSION_CODE < KERNEL_VERSION(4, 6, 0)
> #include 
>
> struct local_dma_buf_sync {
>   __u64 flags;
> };
>
> #define LOCAL_DMA_BUF_SYNC_READ (1 << 0)
> #define LOCAL_DMA_BUF_SYNC_WRITE (2 << 0)
> #define LOCAL_DMA_BUF_SYNC_RW \
>   (LOCAL_DMA_BUF_SYNC_READ | LOCAL_DMA_BUF_SYNC_WRITE)
> #define LOCAL_DMA_BUF_SYNC_START (0 << 2)
> #define LOCAL_DMA_BUF_SYNC_END (1 << 2)
>
> #define LOCAL_DMA_BUF_BASE 'b'
> #define LOCAL_DMA_BUF_IOCTL_SYNC \
>   _IOW(LOCAL_DMA_BUF_BASE, 0, struct local_dma_buf_sync)
>
> #else
> #include 
> #endif

It seems the file was not added to include/uapi/linux/Kbuild. This was
likely an oversight by the kernel developer who added the header file.

I would suggest filing a bug.

[gentoo-dev] linux/dma-buf.h mysteriously missing

2017-03-29 Thread Paweł Hajdan , Jr . 
I was packaging chromium-59.0.3053.3 . I worked around the problem
described here, but I'd like to find the right long term solution.

I was hitting the following compile error:

../../ui/gfx/linux/client_native_pixmap_dmabuf.cc:39:27: fatal error:
linux/dma-buf.h: No such file or directory
 #include 

Despite having sys-kernel/linux-headers-4.10 installed, I do not have
that header. Interestingly, it is present in the tarball
(gentoo-headers-base-4.10.tar.xz), but doesn't seem to get installed.

This is the workaround I applied:


This is part of the chromium code in question:



#if LINUX_VERSION_CODE < KERNEL_VERSION(4, 6, 0)
#include 

struct local_dma_buf_sync {
  __u64 flags;
};

#define LOCAL_DMA_BUF_SYNC_READ (1 << 0)
#define LOCAL_DMA_BUF_SYNC_WRITE (2 << 0)
#define LOCAL_DMA_BUF_SYNC_RW \
  (LOCAL_DMA_BUF_SYNC_READ | LOCAL_DMA_BUF_SYNC_WRITE)
#define LOCAL_DMA_BUF_SYNC_START (0 << 2)
#define LOCAL_DMA_BUF_SYNC_END (1 << 2)

#define LOCAL_DMA_BUF_BASE 'b'
#define LOCAL_DMA_BUF_IOCTL_SYNC \
  _IOW(LOCAL_DMA_BUF_BASE, 0, struct local_dma_buf_sync)

#else
#include 
#endif

Paweł



signature.asc
Description: OpenPGP digital signature