1. 02 Oct, 2016 1 commit
    • popcornmix's avatar
      staging: add bcm2708 vchiq driver · 71bad7f0
      popcornmix authored
      Signed-off-by: default avatarpopcornmix <popcornmix@gmail.com>
      
      vchiq: create_pagelist copes with vmalloc memory
      Signed-off-by: default avatarDaniel Stone <daniels@collabora.com>
      
      vchiq: fix the shim message release
      Signed-off-by: default avatarDaniel Stone <daniels@collabora.com>
      
      vchiq: export additional symbols
      Signed-off-by: default avatarDaniel Stone <daniels@collabora.com>
      
      VCHIQ: Make service closure fully synchronous (drv)
      
      This is one half of a two-part patch, the other half of which is to
      the vchiq_lib user library. With these patches, calls to
      vchiq_close_service and vchiq_remove_service won't return until any
      associated callbacks have been delivered to the callback thread.
      
      VCHIQ: Add per-service tracing
      
      The new service option VCHIQ_SERVICE_OPTION_TRACE is a boolean that
      toggles tracing for the specified service.
      
      This commit also introduces vchi_service_set_option and the associated
      option VCHI_SERVICE_OPTION_TRACE.
      
      vchiq: Make the synchronous-CLOSE logic more tolerant
      
      vchiq: Move logging control into debugfs
      
      vchiq: Take care of a corner case tickled by VCSM
      
      Closing a connection that isn't fully open requires care, since one
      side does not know the other side's port number. Code was present to
      handle the case where a CLOSE is sent immediately after an OPEN, i.e.
      before the OPENACK has been received, but this was incorrectly being
      used when an OPEN from a client using port 0 was rejected.
      
      (In the observed failure, the host was attempting to use the VCSM
      service, which isn't present in the 'cutdown' firmware. The failure
      was intermittent because sometimes the keepalive service would
      grab port 0.)
      
      This case can be distinguished because the client's remoteport will
      still be VCHIQ_PORT_FREE, and the srvstate will be OPENING. Either
      condition is sufficient to differentiate it from the special case
      described above.
      
      vchiq: Avoid high load when blocked and unkillable
      
      vchiq: Include SIGSTOP and SIGCONT in list of signals not-masked by vchiq to allow gdb to work
      
      vchiq_arm: Complete support for SYNCHRONOUS mode
      
      vchiq: Remove inline from suspend/resume
      
      vchiq: Allocation does not need to be atomic
      
      vchiq: Fix wrong condition check
      
      The log level is checked from within the log call. Remove the check in the call.
      Signed-off-by: default avatarPranith Kumar <bobby.prani@gmail.com>
      
      BCM270x: Add vchiq device to platform file and Device Tree
      
      Prepare to turn the vchiq module into a driver.
      Signed-off-by: default avatarNoralf Trønnes <noralf@tronnes.org>
      
      bcm2708: vchiq: Add Device Tree support
      
      Turn vchiq into a driver and stop hardcoding resources.
      Use devm_* functions in probe path to simplify cleanup.
      A global variable is used to hold the register address. This is done
      to keep this patch as small as possible.
      Also make available on ARCH_BCM2835.
      Based on work by Lubomir Rintel.
      Signed-off-by: default avatarNoralf Trønnes <noralf@tronnes.org>
      
      vchiq: Change logging level for inbound data
      
      vchiq_arm: Two cacheing fixes
      
      1) Make fragment size vary with cache line size
      Without this patch, non-cache-line-aligned transfers may corrupt
      (or be corrupted by) adjacent data structures.
      
      Both ARM and VC need to be updated to enable this feature. This is
      ensured by having the loader apply a new DT parameter -
      cache-line-size. The existence of this parameter guarantees that the
      kernel is capable, and the parameter will only be modified from the
      safe default if the loader is capable.
      
      2) Flush/invalidate vmalloc'd memory, and invalidate after reads
      
      vchiq: fix NULL pointer dereference when closing driver
      
      The following code run as root will cause a null pointer dereference oops:
      
              int fd = open("/dev/vc-cma", O_RDONLY);
              if (fd < 0)
                      err(1, "open failed");
              (void)close(fd);
      
      [ 1704.877721] Unable to handle kernel NULL pointer dereference at virtual address 00000000
      [ 1704.877725] pgd = b899c000
      [ 1704.877736] [00000000] *pgd=37fab831, *pte=00000000, *ppte=00000000
      [ 1704.877748] Internal error: Oops: 817 [#1] PREEMPT SMP ARM
      [ 1704.877765] Modules linked in: evdev i2c_bcm2708 uio_pdrv_genirq uio
      [ 1704.877774] CPU: 2 PID: 3656 Comm: stress-ng-fstat Not tainted 3.19.1-12-generic-bcm2709 #12-Ubuntu
      [ 1704.877777] Hardware name: BCM2709
      [ 1704.877783] task: b8ab9b00 ti: b7e68000 task.ti: b7e68000
      [ 1704.877798] PC is at __down_interruptible+0x50/0xec
      [ 1704.877806] LR is at down_interruptible+0x5c/0x68
      [ 1704.877813] pc : [<80630ee8>]    lr : [<800704b0>]    psr: 60080093
      sp : b7e69e50  ip : b7e69e88  fp : b7e69e84
      [ 1704.877817] r10: b88123c8  r9 : 00000010  r8 : 00000001
      [ 1704.877822] r7 : b8ab9b00  r6 : 7fffffff  r5 : 80a1cc34  r4 : 80a1cc34
      [ 1704.877826] r3 : b7e69e50  r2 : 00000000  r1 : 00000000  r0 : 80a1cc34
      [ 1704.877833] Flags: nZCv  IRQs off  FIQs on  Mode SVC_32  ISA ARM  Segment user
      [ 1704.877838] Control: 10c5387d  Table: 3899c06a  DAC: 00000015
      [ 1704.877843] Process do-oops (pid: 3656, stack limit = 0xb7e68238)
      [ 1704.877848] Stack: (0xb7e69e50 to 0xb7e6a000)
      [ 1704.877856] 9e40:                                     80a1cc3c 00000000 00000010 b88123c8
      [ 1704.877865] 9e60: b7e69e84 80a1cc34 fff9fee9 ffffffff b7e68000 00000009 b7e69ea4 b7e69e88
      [ 1704.877874] 9e80: 800704b0 80630ea4 fff9fee9 60080013 80a1cc28 fff9fee9 b7e69edc b7e69ea8
      [ 1704.877884] 9ea0: 8040f558 80070460 fff9fee9 ffffffff 00000000 00000000 00000009 80a1cb7c
      [ 1704.877893] 9ec0: 00000000 80a1cb7c 00000000 00000010 b7e69ef4 b7e69ee0 803e1ba4 8040f514
      [ 1704.877902] 9ee0: 00000e48 80a1cb7c b7e69f14 b7e69ef8 803e1c9c 803e1b74 b88123c0 b92acb18
      [ 1704.877911] 9f00: b8812790 b8d815d8 b7e69f24 b7e69f18 803e2250 803e1bc8 b7e69f5c b7e69f28
      [ 1704.877921] 9f20: 80167bac 803e222c 00000000 00000000 b7e69f54 b8ab9ffc 00000000 8098c794
      [ 1704.877930] 9f40: b8ab9b00 8000efc4 b7e68000 00000000 b7e69f6c b7e69f60 80167d6c 80167b28
      [ 1704.877939] 9f60: b7e69f8c b7e69f70 80047d38 80167d60 b7e68000 b7e68010 8000efc4 b7e69fb0
      [ 1704.877949] 9f80: b7e69fac b7e69f90 80012820 80047c84 01155490 011549a8 00000001 00000006
      [ 1704.877957] 9fa0: 00000000 b7e69fb0 8000ee5c 80012790 00000000 353d8c0f 7efc4308 00000000
      [ 1704.877966] 9fc0: 01155490 011549a8 00000001 00000006 00000000 00000000 76cf3ba0 00000003
      [ 1704.877975] 9fe0: 00000000 7efc42e4 0002272f 76e2ed66 60080030 00000003 00000000 00000000
      [ 1704.877998] [<80630ee8>] (__down_interruptible) from [<800704b0>] (down_interruptible+0x5c/0x68)
      [ 1704.878015] [<800704b0>] (down_interruptible) from [<8040f558>] (vchiu_queue_push+0x50/0xd8)
      [ 1704.878032] [<8040f558>] (vchiu_queue_push) from [<803e1ba4>] (send_worker_msg+0x3c/0x54)
      [ 1704.878045] [<803e1ba4>] (send_worker_msg) from [<803e1c9c>] (vc_cma_set_reserve+0xe0/0x1c4)
      [ 1704.878057] [<803e1c9c>] (vc_cma_set_reserve) from [<803e2250>] (vc_cma_release+0x30/0x38)
      [ 1704.878069] [<803e2250>] (vc_cma_release) from [<80167bac>] (__fput+0x90/0x1e0)
      [ 1704.878082] [<80167bac>] (__fput) from [<80167d6c>] (____fput+0x18/0x1c)
      [ 1704.878094] [<80167d6c>] (____fput) from [<80047d38>] (task_work_run+0xc0/0xf8)
      [ 1704.878109] [<80047d38>] (task_work_run) from [<80012820>] (do_work_pending+0x9c/0xc4)
      [ 1704.878123] [<80012820>] (do_work_pending) from [<8000ee5c>] (work_pending+0xc/0x20)
      [ 1704.878133] Code: e50b1034 e3a01000 e50b2030 e580300c (e5823000)
      
      ..the fix is to ensure that we have actually initialized the queue before we attempt
      to push any items onto it.  This occurs if we do an open() followed by a close() without
      any activity in between.
      Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
      
      vchiq_arm: Sort out the vmalloc case
      
      See: https://github.com/raspberrypi/linux/issues/1055
      
      vchiq: hack: Add include depecated dma include file
      
      [gregkh] added dependancy on CONFIG_BROKEN to make things sane for now.
      
      Cc: Eric Anholt <eric@anholt.net>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      71bad7f0
  2. 28 Sep, 2016 14 commits
  3. 27 Sep, 2016 25 commits