On 3/4/2019 4:04 PM, Nicolas Boichat wrote:
On Wed, Feb 13, 2019 at 4:56 PM Nicolas Boichat <drinkcat@xxxxxxxxxxxx> wrote:
On Tue, Jan 22, 2019 at 4:46 PM Prateek Patel <prpatel@xxxxxxxxxx> wrote:
On 11/10/2018 2:58 AM, Rob Herring wrote:
On Fri, Nov 9, 2018 at 1:09 AM Prateek Patel <prpatel@xxxxxxxxxx> wrote:
From: Sri Krishna chowdary <schowdary@xxxxxxxxxx>
Memory reserved with "nomap" DT property in of_reserved_mem.c
removes the memory block. The removed memory blocks don't have
VA to PA mapping created in kernel page table. Kmemleak scan on
removed memory blocks is causing page faults and leading to
kernel panic. So, Disable kmemleak scan on the removed memory
blocks.
Following is the observed crash log:
[ 154.846370] Unable to handle kernel paging request at virtual address ffffffc070a00000
<1>[ 154.846576] Mem abort info:
<1>[ 154.846635] Exception class = DABT (current EL), IL = 32 bits
<1>[ 154.846737] SET = 0, FnV = 0
<1>[ 154.846796] EA = 0, S1PTW = 0
<1>[ 154.846859] Data abort info:
<1>[ 154.846913] ISV = 0, ISS = 0x00000006
<1>[ 154.846983] CM = 0, WnR = 0
<1>[ 154.847053] swapper pgtable: 4k pages, 39-bit VAs, pgd = ffffff8009df7000
<1>[ 154.847228] [ffffffc070a00000] *pgd=000000087fff5803, *pud=000000087fff5803, *pmd=0000000000000000
<0>[ 154.847408] Internal error: Oops: 96000006 [#1] PREEMPT SMP
<4>[ 154.847511] Modules linked in: nvs_led_test nvs_bmi160 nvs_cm3218 nvs_bh1730fvc nvi_bmpX80 nvi_ak89xx nvi_mpu cdc_acm uas lr388k7_ts imx268 imx318 imx204 imx274 imx185 lc898212 ov23850 ov10823 ov9281 ov5693 tc358840 pca9570 nvs snd_soc_tegra_machine_driver_mobile lp855x_bl spidev input_cfboost pwm_tegra tegra_cryptodev tegra_se_nvhost tegra_se_elp tegra_se ghash_ce sha2_ce sha1_ce aes_ce_ccm cryptd nvgpu cpufreq_userspace snd_soc_tegra186_alt_dspk snd_soc_tegra186_alt_asrc snd_soc_tegra186_alt_arad snd_soc_tegra210_alt_ope snd_soc_tegra210_alt_mvc snd_soc_tegra210_alt_dmic snd_soc_tegra210_alt_amx snd_soc_tegra210_alt_adx snd_soc_tegra210_alt_afc snd_soc_tegra210_alt_mixer snd_soc_tegra210_alt_i2s snd_soc_tegra210_alt_sfc snd_soc_tegra210_alt_adsp snd_soc_tegra210_alt_admaif snd_soc_tegra210_alt_xbar
<4>[ 154.882606] snd_soc_tegra_alt_utils snd_hda_tegra
<4>[ 154.888133] CPU: 2 PID: 8079 Comm: sh Not tainted 4.14.53-tegra-05132-g9c33465 #2
<4>[ 154.895983] Hardware name: e3360_1099 (DT)
<4>[ 154.900447] task: ffffffc7d62dda00 task.stack: ffffff800e2b0000
<4>[ 154.906502] PC is at scan_block+0x7c/0x148
<4>[ 154.911234] LR is at scan_block+0x78/0x148
<4>[ 154.915689] pc : [<ffffff8008271724>] lr : [<ffffff8008271720>] pstate: 804000c9
<4>[ 154.923290] sp : ffffff800e2b3b80
<4>[ 154.927228] x29: ffffff800e2b3b80 x28: ffffffc7d62dda00
<4>[ 154.932999] x27: ffffff8009aaa000 x26: ffffffc070c00000
<4>[ 154.938769] x25: 00000000000000c0 x24: ffffff8009d90608
<4>[ 154.944287] x23: ffffffc7dc6c6000 x22: ffffff8009d90000
<4>[ 154.950320] x21: ffffff8009aeb320 x20: ffffffc070a00ff9
<4>[ 154.955919] x19: ffffffc070a00000 x18: 00000000bec4c3f2
<4>[ 154.961438] x17: 0000002224777924 x16: ffffff80080bb0e0
<4>[ 154.967124] x15: 0000000000000000 x14: 0000000000000f75
<4>[ 154.973069] x13: 000fffffffffffff x12: ffffffbf1e9f4240
<4>[ 154.978670] x11: 0000000000000040 x10: 0000000000000ad0
<4>[ 154.984107] x9 : ffffff800e2b3ab0 x8 : ffffffc7d62de530
<4>[ 154.989958] x7 : 0000000780000000 x6 : 0000000000000018
<4>[ 154.995645] x5 : 0000000000000000 x4 : 0000000000000000
<4>[ 155.001245] x3 : ffffff8009aaa000 x2 : 00000047f6712000
<4>[ 155.006846] x1 : ffffffc7d1ae6900 x0 : 0000000000000000
[snip]
For reference, there is another patch series that fixes the problem,
by Mike Rapoport (https://lore.kernel.org/patchwork/patch/1041790/),
already in linux-next (through mmotm):
42c9c0ac24393c of: fix kmemleak crash caused by imbalance in early
memory reservation
48ec6d51f7d3f2 of: fix parameters order for call to memblock_find_in_range()
If anyone is interested, I backported these patch to our 4.19 tree
here, and they fix the issue:
https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1496707
https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1496708
Thanks,
Thanks Nicolas for the reference,
My acked-by is already present in the patch
(https://lore.kernel.org/patchwork/patch/1041790/)
