On 4/8/21 7:06 AM, David Howells wrote: > Zheng Zengkai <zhengzeng...@huawei.com> wrote: > >> static int afs_dir_set_page_dirty(struct page *page) >> { >> BUG(); /* This should never happen. */ >> + return 0; >> } > > That shouldn't be necessary. BUG() should be marked as 'no return' to the > compiler. What arch and compiler are you using?
How do mark a #define BUG() as __noreturn? Several arch-es use #define for BUG() instead of using a function. -- ~Randy