Hi Tony, Thank you for your review!
On 07/24/2018 01:01 PM, Tony Luck wrote: > On Thu, Jul 19, 2018 at 7:07 AM, Masayoshi Mizuma <msys.miz...@gmail.com> > wrote: >> From: Masayoshi Mizuma <m.miz...@jp.fujitsu.com> >> >> KASAN reported the following slab-out-of-bounds when sb_edac >> module was loaded on Broadwell machine which has two PCI segments. > > Although you found this with KASAN as an out of bounds array reference, > the real problem is that the sb_edac.c driver didn't know about systems > with segmented PCI busses. > > So the Subject: for the e-mail (and thus the commit message) should > be: > > [PATCH] EDAC, sb_edac: Add support for systems with segmented PCI busses You are right. I will modify the subject and commit message, and resend it. > > Otherwise this looks fine. > > Reviewed-by: Tony Luck <tony.l...@intel.com> Thanks! Masa > > -Tony >