Re: CURRENT: buildworld failure due to nvmecontrol source error

From: NGie Cooper <yaneurabeya_at_gmail.com>
Date: Sat, 30 Jan 2016 09:12:49 -0800
> On Jan 30, 2016, at 04:10, Outback Dingo <outbackdingo_at_gmail.com> wrote:
> 
> On Sat, Jan 30, 2016 at 12:39 PM, O. Hartmann <ohartman_at_zedat.fu-berlin.de>
> wrote:
> 
>> Buildworld (r295070) fails in building nvmecontrol patches correctly:
>> 
>> [...]
>> (cd /usr/src/lib/libc/tests/gen &&  DEPENDFILE=.depend.raise_test
>> NO_SUBDIR=1 make
>> -f /usr/src/lib/libc/tests/gen/Makefile _RECURSING_PROGS=t
>> PROG=raise_test ) ---
>> all_subdir_share --- ===> share/i18n/csmapper/APPLE (all)
>> --- all_subdir_usr.sbin ---
>> ===> usr.sbin/acpi/iasl (all)
>> --- all_subdir_sbin ---
>> /usr/src/sbin/nvmecontrol/power.c:44:16: error: invalid application of
>> 'sizeof' to an
>> incomplete type 'struct nvme_power_state' _Static_assert(sizeof(struct
>> nvme_power_state)
>> == 256 / NBBY, ^     ~~~~~~~~~~~~~~~~~~~~~~~~~
>> /usr/src/sbin/nvmecontrol/power.c:44:30: note: forward declaration of
>> 'struct
>> nvme_power_state' _Static_assert(sizeof(struct nvme_power_state) == 256 /
>> NBBY,
>>                             ^
>> /usr/src/sbin/nvmecontrol/power.c:60:14: error: incomplete definition of
>> type 'struct
>> nvme_power_state' mpower = nps->mp;
>>                 ~~~^
>> /usr/src/sbin/nvmecontrol/power.c:44:30: note: forward declaration of
>> 'struct
>> nvme_power_state' _Static_assert(sizeof(struct nvme_power_state) == 256 /
>> NBBY,
>>                             ^
>> /usr/src/sbin/nvmecontrol/power.c:61:9: error: incomplete definition of
>> type 'struct
>> nvme_power_state' if (nps->mps == 0)
> 
> same here to confirm
> cc  -pg  -O2 -pipe
> -I/backup/freebsd/kerberos5/lib/libroken/../../../crypto/heimdal/lib/roken
> -I.   -DHAVE_CONFIG_H
> -I/backup/freebsd/kerberos5/lib/libroken/../../include -std=gnu99 -fstack
> -protector-strong    -Qunused-arguments -c
> /backup/freebsd/kerberos5/lib/libroken/../../../crypto/heimdal/lib/roken/eread.c
> -o eread.po
> --- all_subdir_secure ---
> cc  -pg  -O2 -pipe
>  -I/backup/freebsd/secure/lib/libcrypto/../../../crypto/openssl -DTERMIOS
> -DANSI_SOURCE -DOPENSSL_THREADS -DDSO_DLFCN -DHAVE_DLFCN_H -DL_ENDIAN
> -DOPENSSL_IA32_SSE2 -DAES
> _ASM -DBSAES_ASM -DVPAES_ASM -DECP_NISTZ256_ASM -DOPENSSL_BN_ASM_MONT
> -DOPENSSL_BN_ASM_MONT5 -DOPENSSL_BN_ASM_GF2m -DMD5_ASM -DGHASH_ASM
> -DSHA1_ASM -DSHA256_ASM -DSHA512_ASM -DWHIRLPOOL_ASM
> -I/usr/obj/backup/freebsd/secure/lib/libcrypto
> -I/backup/freebsd/secure/lib/libcrypto/../../../crypto/openssl/crypto
> -I/backup/freebsd/secure/lib/libcrypto/../../../crypto/openssl/crypto/a
> sn1
> -I/backup/freebsd/secure/lib/libcrypto/../../../crypto/openssl/crypto/evp
> -I/backup/freebsd/secure/lib/libcrypto/../../../crypto/openssl/crypto/modes
> -std=gnu89 -fstack-protector-strong
> -Wno-pointer-sign -Wno-empty-body -Wno-string-plus-int
> -Wno-unused-const-variable -Wno-tautological-compare -Wno-unused-value
> -Wno-parentheses-equality -Wno-unused-function -Wno-enum-conve
> rsion -Wno-unused-local-typedef -Wno-switch -Wno-switch-enum
> -Wno-knr-promoted-parameter -Wno-parentheses  -Qunused-arguments -c
> /backup/freebsd/secure/lib/libcrypto/../../../crypto/openssl
> /crypto/evp/m_md5.c -o m_md5.po
> --- all_subdir_sbin ---
> /backup/freebsd/sbin/nvmecontrol/power.c:44:16: error: invalid application
> of 'sizeof' to an incomplete type 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>              ^     ~~~~~~~~~~~~~~~~~~~~~~~~~
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:60:14: error: incomplete
> definition of type 'struct nvme_power_state'
>       mpower = nps->mp;
>                ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:61:9: error: incomplete definition
> of type 'struct nvme_power_state'
>       if (nps->mps == 0)
>           ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:63:14: error: incomplete
> definition of type 'struct nvme_power_state'
>       ipower = nps->idlp;
>                ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:64:9: error: incomplete definition
> of type 'struct nvme_power_state'
>       if (nps->ips == 1)
>           ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:66:14: error: incomplete
> definition of type 'struct nvme_power_state'
> --- all_subdir_rescue ---
> --- dhclient_make ---
> --- packet.o ---
> --- all_subdir_sbin ---
>       apower = nps->actp;
>                ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:67:9: error: incomplete definition
> of type 'struct nvme_power_state'
>       if (nps->aps == 1)
>           ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:71:12: error: incomplete
> definition of type 'struct nvme_power_state'
>              nps->nops ? '*' : ' ', nps->enlat / 1000, nps->enlat % 1000,
>              ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:71:35: error: incomplete
> definition of type 'struct nvme_power_state'
>              nps->nops ? '*' : ' ', nps->enlat / 1000, nps->enlat % 1000,
>                                     ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:71:54: error: incomplete
> definition of type 'struct nvme_power_state'
>              nps->nops ? '*' : ' ', nps->enlat / 1000, nps->enlat % 1000,
>                                                        ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:72:12: error: incomplete
> definition of type 'struct nvme_power_state'
>              nps->exlat / 1000, nps->exlat % 1000, nps->rrt, nps->rrl,
>              ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:72:31: error: incomplete
> definition of type 'struct nvme_power_state'
>              nps->exlat / 1000, nps->exlat % 1000, nps->rrt, nps->rrl,
>                                 ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:72:50: error: incomplete
> definition of type 'struct nvme_power_state'
>              nps->exlat / 1000, nps->exlat % 1000, nps->rrt, nps->rrl,
>                                                    ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:72:60: error: incomplete
> definition of type 'struct nvme_power_state'
>              nps->exlat / 1000, nps->exlat % 1000, nps->rrt, nps->rrl,
>                                                              ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:73:12: error: incomplete
> definition of type 'struct nvme_power_state'
>              nps->rwt, nps->rwl, ipower / 10000, ipower % 10000,
>              ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:73:22: error: incomplete
> definition of type 'struct nvme_power_state'
>              nps->rwt, nps->rwl, ipower / 10000, ipower % 10000,
>                        ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:74:44: error: incomplete
> definition of type 'struct nvme_power_state'
>              apower / 10000, apower % 10000, nps->apw);
>                                              ~~~^
> /backup/freebsd/sbin/nvmecontrol/power.c:44:30: note: forward declaration
> of 'struct nvme_power_state'
> _Static_assert(sizeof(struct nvme_power_state) == 256 / NBBY,
>                            ^
> /backup/freebsd/sbin/nvmecontrol/power.c:86:29: error: no member named
> 'power_state' in 'struct nvme_controller_data'
>               power_list_one(i, &cdata->power_state[i]);
>                                  ~~~~~  ^
> --- all_subdir_rescue ---
> cc  -O2 -pipe   -DRESCUE -std=gnu99 -fstack-protector-strong
> -Wsystem-headers -Werror -Wall -Wno-format-y2k -Wno-uninitialized
> -Wno-pointer-sign -Wno-empty-body -Wno-string-plus-int -Wno-un
> used-const-variable -Wno-tautological-compare -Wno-unused-value
> -Wno-parentheses-equality -Wno-unused-function -Wno-enum-conversion
> -Wno-unused-local-typedef -Wno-switch -Wno-switch-enum -W
> no-knr-promoted-parameter  -Qunused-arguments -c
> /backup/freebsd/sbin/dhclient/packet.c -o packet.o
> --- all_subdir_sbin ---
> 18 errors generated.
> *** [power.o] Error code 1
> 
> make[4]: stopped in /backup/freebsd/sbin/nvmecontrol

Hi,
	Thank you for the report. I’ve reverted the commit in r295076, because it broke buildworld and the missing code wasn’t apparent based on the commit done to nvmecontrol(8).
Thanks!
-NGie
Received on Sat Jan 30 2016 - 16:12:54 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:02 UTC