Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

5.2(-rc1) build issues #8770

Closed
nivedita76 opened this issue May 20, 2019 · 4 comments
Closed

5.2(-rc1) build issues #8770

nivedita76 opened this issue May 20, 2019 · 4 comments

Comments

@nivedita76
Copy link
Contributor

System information

Type Version/Name
Distribution Name gentoo
Distribution Version
Linux Kernel 5.2-rc1
Architecture amd64
ZFS Version git head
SPL Version

Describe the problem you're observing

In 5.2-rc1 some of the RWSEM defines seem to have been hidden, I guess because no users in the kernel tree (RWSEM_ACTIVE_READ_BIAS and RWSEM_ACTIVE_WRITE_BIAS). spl-rwlock.c no longer compiles, I had to comment out the logic in __rwsem_tryupgrade to get it to build.

Describe how to reproduce the problem

Include any warning/errors/backtraces from the system logs

@kusumi
Copy link
Member

kusumi commented May 21, 2019

Haven't looked into details, but it looks ok to define them in include/spl/sys/rwlock.h.
Btw, the original commit is torvalds/linux@46ad084.
e.g.

     39 #else
     40 /* Linux 5.2 moved RWSEM_* macros to kernel/locking/rwsem.h. */
     41 #if !defined(RWSEM_ACTIVE_MASK)
     42 #ifdef CONFIG_64BIT
     43 #define RWSEM_ACTIVE_MASK               0xffffffffL
     44 #else
     45 #define RWSEM_ACTIVE_MASK               0x0000ffffL
     46 #endif
     47 #endif
     48 #if !defined(RWSEM_ACTIVE_BIAS)
     49 #define RWSEM_ACTIVE_BIAS               0x00000001L
     50 #endif
     51 #if !defined(RWSEM_WAITING_BIAS)
     52 #define RWSEM_WAITING_BIAS              (-RWSEM_ACTIVE_MASK-1)
     53 #endif
     54 #if !defined(RWSEM_ACTIVE_READ_BIAS)
     55 #define RWSEM_ACTIVE_READ_BIAS          RWSEM_ACTIVE_BIAS
     56 #endif
     57 #if !defined(RWSEM_ACTIVE_WRITE_BIAS)
     58 #define RWSEM_ACTIVE_WRITE_BIAS         (RWSEM_WAITING_BIAS + RWSEM_ACTIVE_BIAS)
     59 #endif
     60 #define SPL_RWSEM_SINGLE_READER_VALUE   (RWSEM_ACTIVE_READ_BIAS)
     61 #define SPL_RWSEM_SINGLE_WRITER_VALUE   (RWSEM_ACTIVE_WRITE_BIAS)
     62 #endif

@kusumi
Copy link
Member

kusumi commented May 21, 2019

Btw, this should be addressed once #8730 is merged. It's been there for a few weeks in pr queue.

@nivedita76
Copy link
Contributor Author

Ok thanks

@behlendorf
Copy link
Contributor

I'll get the build issue PRs merged soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants