On 08/17/2017 06:27 AM, Julien Grall wrote: > Hi Boris, > > On 16/08/17 19:33, Boris Ostrovsky wrote: >> +static void check_one_page(struct page_info *pg) >> +{ >> +#ifdef CONFIG_SCRUB_DEBUG >> + mfn_t mfn = _mfn(page_to_mfn(pg)); >> + const uint64_t *ptr; >> + unsigned int i; >> + >> + if ( !boot_scrub_done ) >> + return; >> + >> + ptr = map_domain_page(mfn); >> + for ( i = 0; i < PAGE_SIZE / sizeof (*ptr); i++ ) >> + ASSERT(ptr[i] == SCRUB_PATTERN); > > ASSERT will be turned into a NOP on non-debug build. However, it is > possible to select SCRUB_DEBUG on non-debug build when expert mode is > enabled. > > So I would turn this into a BUG_ON() to make it work in all > configuration.
Yes, good point, thanks. I will wait for Jan's review and if there are no more comments then, given that this is the last patch in the series and so the change can cause no conflicts, perhaps he can fix it during commit. Or I can resend. -boris _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel