• Kevin Hao's avatar
    crypto: talitos: init the priv->alg_list more earlier in talitos_probe() · f3de9cb1
    Kevin Hao authored
    In function talitos_probe(), it will jump to err_out when getting an
    error in talitos_probe_irq(). Then the uninitialized list head
    priv->alg_list will be used in function talitos_remove(). In this case
    we would get a call trace like the following. So move up the
    initialization of priv->alg_list.
    
      Unable to handle kernel paging request for data at address 0x00000000
      Faulting instruction address: 0xc0459ff4
      Oops: Kernel access of bad area, sig: 11 [#1]
      SMP NR_CPUS=8 P1020 RDB
      Modules linked in:
      CPU: 1 PID: 1 Comm: swapper/0 Tainted: G        W    3.13.0-08789-g54c0a4b4 #33
      task: cf050000 ti: cf04c000 task.ti: cf04c000
      NIP: c0459ff4 LR: c0459fd4 CTR: c02f2438
      REGS: cf04dcb0 TRAP: 0300   Tainted: G        W     (3.13.0-08789-g54c0a4b4)
      MSR: 00029000 <CE,EE,ME>  CR: 82000028  XER: 20000000
      DEAR: 00000000 ESR: 00000000
      GPR00: c045ac28 cf04dd60 cf050000 cf2579c0 00021000 00000000 c02f35b0 0000014e
      GPR08: c07e702c cf104300 c07e702c 0000014e 22000024 00000000 c0002a3c 00000000
      GPR16: 00000000 00000000 00000000 00000000 00000000 00000000 c082e4e0 000000df
      GPR24: 00000000 00100100 00200200 cf257a2c cf0efe10 cf2579c0 cf0efe10 00000000
      NIP [c0459ff4] talitos_remove+0x3c/0x1c8
      LR [c0459fd4] talitos_remove+0x1c/0x1c8
      Call Trace:
      [cf04dd60] [c07485d8] __func__.13331+0x1241c8/0x1391c0 (unreliable)
      [cf04dd90] [c045ac28] talitos_probe+0x244/0x998
      [cf04dde0] [c0306a74] platform_drv_probe+0x28/0x68
      [cf04ddf0] [c0304d38] really_probe+0x78/0x250
      [cf04de10] [c030505c] __driver_attach+0xc8/0xcc
      [cf04de30] [c0302e98] bus_for_each_dev+0x6c/0xb8
      [cf04de60] [c03043cc] bus_add_driver+0x168/0x220
      [cf04de80] [c0305798] driver_register+0x88/0x130
      [cf04de90] [c0002458] do_one_initcall+0x14c/0x198
      [cf04df00] [c079f904] kernel_init_freeable+0x138/0x1d4
      [cf04df30] [c0002a50] kernel_init+0x14/0x124
      [cf04df40] [c000ec40] ret_from_kernel_thread+0x5c/0x64
    Signed-off-by: default avatarKevin Hao <haokexin@gmail.com>
    Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
    f3de9cb1
talitos.c 79 KB