* Re: linux-next: Tree for Aug 27 (kunit) [not found] <20190827190526.6f27e763@canb.auug.org.au> @ 2019-08-27 15:29 ` Randy Dunlap 2019-08-27 16:09 ` Brendan Higgins 0 siblings, 1 reply; 3+ messages in thread From: Randy Dunlap @ 2019-08-27 15:29 UTC (permalink / raw) To: Stephen Rothwell, Linux Next Mailing List Cc: Linux Kernel Mailing List, Shuah Khan, linux-kselftest, Brendan Higgins On 8/27/19 2:05 AM, Stephen Rothwell wrote: > Hi all, > > Changes since 20190826: > on i386: # CONFIG_PRINTK is not set ../kunit/test.c: In function ‘kunit_vprintk_emit’: ../kunit/test.c:21:9: error: implicit declaration of function ‘vprintk_emit’; did you mean ‘vprintk’? [-Werror=implicit-function-declaration] return vprintk_emit(0, level, NULL, 0, fmt, args); ^~~~~~~~~~~~ vprintk -- ~Randy ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: linux-next: Tree for Aug 27 (kunit) 2019-08-27 15:29 ` linux-next: Tree for Aug 27 (kunit) Randy Dunlap @ 2019-08-27 16:09 ` Brendan Higgins 2019-08-27 16:12 ` shuah 0 siblings, 1 reply; 3+ messages in thread From: Brendan Higgins @ 2019-08-27 16:09 UTC (permalink / raw) To: Randy Dunlap Cc: Stephen Rothwell, Linux Next Mailing List, Linux Kernel Mailing List, Shuah Khan, open list:KERNEL SELFTEST FRAMEWORK On Tue, Aug 27, 2019 at 8:29 AM Randy Dunlap <rdunlap@infradead.org> wrote: > > On 8/27/19 2:05 AM, Stephen Rothwell wrote: > > Hi all, > > > > Changes since 20190826: > > > > on i386: > # CONFIG_PRINTK is not set > > > ../kunit/test.c: In function ‘kunit_vprintk_emit’: > ../kunit/test.c:21:9: error: implicit declaration of function ‘vprintk_emit’; did you mean ‘vprintk’? [-Werror=implicit-function-declaration] > return vprintk_emit(0, level, NULL, 0, fmt, args); > ^~~~~~~~~~~~ > vprintk Ooops, it never occurred to me to test the situation where I wouldn't be able to see test results :-) It seems to me that the right thing to do here is to do what dev_printk and friends do and to ifdef this out if CONFIG_PRINTK is unavailable. Does that seem reasonable? Also, do you want me to resend my patches with the fix or do you want me to send a new patch with this fix? (Sorry for the newbie question.) Thanks! ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: linux-next: Tree for Aug 27 (kunit) 2019-08-27 16:09 ` Brendan Higgins @ 2019-08-27 16:12 ` shuah 0 siblings, 0 replies; 3+ messages in thread From: shuah @ 2019-08-27 16:12 UTC (permalink / raw) To: Brendan Higgins, Randy Dunlap Cc: Stephen Rothwell, Linux Next Mailing List, Linux Kernel Mailing List, open list:KERNEL SELFTEST FRAMEWORK, shuah On 8/27/19 10:09 AM, Brendan Higgins wrote: > On Tue, Aug 27, 2019 at 8:29 AM Randy Dunlap <rdunlap@infradead.org> wrote: >> >> On 8/27/19 2:05 AM, Stephen Rothwell wrote: >>> Hi all, >>> >>> Changes since 20190826: >>> >> >> on i386: >> # CONFIG_PRINTK is not set >> >> >> ../kunit/test.c: In function ‘kunit_vprintk_emit’: >> ../kunit/test.c:21:9: error: implicit declaration of function ‘vprintk_emit’; did you mean ‘vprintk’? [-Werror=implicit-function-declaration] >> return vprintk_emit(0, level, NULL, 0, fmt, args); >> ^~~~~~~~~~~~ >> vprintk > Thanks Randy for catching this. > Ooops, it never occurred to me to test the situation where I wouldn't > be able to see test results :-) > > It seems to me that the right thing to do here is to do what > dev_printk and friends do and to ifdef this out if CONFIG_PRINTK is > unavailable. Does that seem reasonable? > > Also, do you want me to resend my patches with the fix or do you want > me to send a new patch with this fix? (Sorry for the newbie question.) > Brendan, I would like to apply the fix on top of the series. I don't want to revert. thanks, -- Shuah ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-08-27 16:12 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20190827190526.6f27e763@canb.auug.org.au> 2019-08-27 15:29 ` linux-next: Tree for Aug 27 (kunit) Randy Dunlap 2019-08-27 16:09 ` Brendan Higgins 2019-08-27 16:12 ` shuah
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).