Skip to content
This repository has been archived by the owner on Oct 12, 2020. It is now read-only.

util.c: include poll.h instead of sys/poll.h #20

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

maxinbjohn
Copy link

This fixes a compile warning when building with musl:

In file included from util.c:27:0:
|
qemux86-64/usr/include/sys/poll.h:1:2:
error: #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
[-Werror=cpp]
| #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
| ^

Signed-off-by: Maxin B. John maxin.john@intel.com

This fixes a compile warning when building with musl:

 In file included from util.c:27:0:
 |
 qemux86-64/usr/include/sys/poll.h:1:2:
 error: #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
 [-Werror=cpp]
 |  #warning redirecting incorrect #include <sys/poll.h> to <poll.h>
 |   ^

Signed-off-by: Maxin B. John <maxin.john@intel.com>
Fixes this build error:

| In file included from mdadm.c:28:0:
| mdadm.h:142:0: error: "bswap_16" redefined [-Werror]
|  #define bswap_16(x) (((x) & 0x00ffU) << 8 | \
|  ^

Signed-off-by: Maxin B. John <maxin.john@intel.com>
With x32 toolchain, this code caused the below listed error:

| Monitor.c: In function 'check_array':
| Monitor.c:545:16: error: comparison between signed and unsigned
integer expressions [-Werror=sign-compare]
|   if (st->utime == array.utime &&

Signed-off-by: Maxin B. John <maxin.john@intel.com>
@Alukardd
Copy link

Alukardd commented Feb 5, 2016

Neil isn't maintainer of the mdadm from February.
mdadm repo now on kernel.org — git://git.kernel.org/pub/scm/utils/mdadm/mdadm.git
New co-ordinator of mdadm is Jes Serensen.

And as I understand all PR or issues should be send via email to dm-devel@redhat.com or linux-raid@vger.kernel.org.

@neilbrown, I think you should change description of this repo according to actual mdadm maintainer.

@maxinbjohn
Copy link
Author

@Alukardd Thanks for the info. I will send these patches to linux-raid@vger.kernel.org

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

Successfully merging this pull request may close these issues.

2 participants