Re: [PATCH v2 4/5] bug: Provide toggle for BUG on data corruption

2016-08-17 Thread Steven Rostedt
On Tue, 16 Aug 2016 17:20:28 -0700 Kees Cook wrote: > EXPORT_SYMBOL(__list_add_valid); > @@ -46,26 +41,18 @@ bool __list_del_entry_valid(struct list_head *entry) > prev = entry->prev; > next = entry->next; > > - if (unlikely(next == LIST_POISON1)) { > -

Re: [PATCH v2 4/5] bug: Provide toggle for BUG on data corruption

2016-08-17 Thread Steven Rostedt
On Tue, 16 Aug 2016 17:20:28 -0700 Kees Cook wrote: > EXPORT_SYMBOL(__list_add_valid); > @@ -46,26 +41,18 @@ bool __list_del_entry_valid(struct list_head *entry) > prev = entry->prev; > next = entry->next; > > - if (unlikely(next == LIST_POISON1)) { > - WARN(1,

Re: [PATCH v2 4/5] bug: Provide toggle for BUG on data corruption

2016-08-16 Thread Kees Cook
On Tue, Aug 16, 2016 at 5:26 PM, Joe Perches wrote: > On Tue, 2016-08-16 at 17:20 -0700, Kees Cook wrote: >> The kernel checks for cases of data structure corruption under some >> CONFIGs (e.g. CONFIG_DEBUG_LIST). When corruption is detected, some >> systems may want to BUG()

Re: [PATCH v2 4/5] bug: Provide toggle for BUG on data corruption

2016-08-16 Thread Kees Cook
On Tue, Aug 16, 2016 at 5:26 PM, Joe Perches wrote: > On Tue, 2016-08-16 at 17:20 -0700, Kees Cook wrote: >> The kernel checks for cases of data structure corruption under some >> CONFIGs (e.g. CONFIG_DEBUG_LIST). When corruption is detected, some >> systems may want to BUG() immediately instead

Re: [PATCH v2 4/5] bug: Provide toggle for BUG on data corruption

2016-08-16 Thread Joe Perches
On Tue, 2016-08-16 at 17:20 -0700, Kees Cook wrote: > The kernel checks for cases of data structure corruption under some > CONFIGs (e.g. CONFIG_DEBUG_LIST). When corruption is detected, some > systems may want to BUG() immediately instead of letting the system run > with known corruption.  

Re: [PATCH v2 4/5] bug: Provide toggle for BUG on data corruption

2016-08-16 Thread Joe Perches
On Tue, 2016-08-16 at 17:20 -0700, Kees Cook wrote: > The kernel checks for cases of data structure corruption under some > CONFIGs (e.g. CONFIG_DEBUG_LIST). When corruption is detected, some > systems may want to BUG() immediately instead of letting the system run > with known corruption.