GitHub PushEvent at 2014-11-26 11:34:22
Repository: renesas
Actor: khiemnguyen
Payload: {"shas":[["86a570c577b7975f31c0539f6a724194892ebc43","a809e197ca11609a4d74ffc18b0d9eb231aabd56@gmail.com","ASoC: fsl_asrc: Add reg_defaults for regmap to fix kernel dump\n\nKernel dump (WARN_ON) ocurred during system boot-up inside regmap_write():\n\n------------[ cut here ]------------\nWARNING: CPU: 0 PID: 47 at kernel/locking/lockdep.c:2744 lockdep_trace_alloc+0xe8/0x108()\nDEBUG_LOCKS_WARN_ON(irqs_disabled_flags(flags))\nModules linked in:\nCPU: 0 PID: 47 Comm: kworker/u2:2 Not tainted 3.18.0-rc1-10245-gb75d289-dirty #56\nWorkqueue: deferwq deferred_probe_work_func\nBacktrace:\n[<80012294>] (dump_backtrace) from [<80012578>] (show_stack+0x18/0x1c)\n r6:8097c73c r5:8097c73c r4:00000000 r3:be33ba80\n[<80012560>] (show_stack) from [<806aac48>] (dump_stack+0x8c/0xa4)\n[<806aabbc>] (dump_stack) from [<8002a694>] (warn_slowpath_common+0x70/0x94)\n r6:80062838 r5:00000009 r4:bd827b30 r3:be33ba80\n[<8002a624>] (warn_slowpath_common) from [<8002a6f0>] (warn_slowpath_fmt+0x38/0x40)\n r8:00000004 r7:00000001 r6:000080d0 r5:60000193 r4:bd826010\n[<8002a6bc>] (warn_slowpath_fmt) from [<80062838>] (lockdep_trace_alloc+0xe8/0x108)\n r3:80831590 r2:8082e160\n[<80062750>] (lockdep_trace_alloc) from [<800ea5dc>] (kmem_cache_alloc+0x28/0x134)\n r5:000080d0 r4:be001f00\n[<800ea5b4>] (kmem_cache_alloc) from [<8038d72c>] (regcache_rbtree_write+0x15c/0x648)\n r10:00000000 r9:0000001c r8:00000004 r7:00000001 r6:00000000 r5:bd819a00\n r4:00000000 r3:811aea88\n[<8038d5d0>] (regcache_rbtree_write) from [<8038c4d8>] (regcache_write+0x5c/0x64)\n r10:be3f9f88 r9:00000000 r8:00000004 r7:00000001 r6:00000000 r5:00000001\n r4:bd819a00\n[<8038c47c>] (regcache_write) from [<8038b0dc>] (_regmap_raw_write+0x134/0x5f4)\n r6:be3f9f84 r5:00000001 r4:bd819a00 r3:00000001\n[<8038afa8>] (_regmap_raw_write) from [<8038b610>] (_regmap_bus_raw_write+0x74/0x94)\n r10:00000000 r9:00000001 r8:be3fb080 r7:bd819a00 r6:00000001 r5:00000000\n r4:bd819a00\n[<8038b59c>] (_regmap_bus_raw_write) from [<8038a8b4>] (_regmap_write+0x60/0x9c)\n r6:00000001 r5:00000000 r4:bd819a00 r3:8038b59c\n[<8038a854>] (_regmap_write) from [<8038ba24>] (regmap_write+0x48/0x68)\n r7:bd81ad80 r6:00000001 r5:00000000 r4:bd819a00\n[<8038b9dc>] (regmap_write) from [<80528f30>] (fsl_asrc_dai_probe+0x34/0x104)\n r6:bd888628 r5:be3fb080 r4:be3b4410 r3:be3b442c\n------------[ dump end ]------------\n\n=============================================================================\n2741         /*\n2742          * Oi! Can't be having __GFP_FS allocations with IRQs disabled.\n2743          */\n2744         if (DEBUG_LOCKS_WARN_ON(irqs_disabled_flags(flags)))\n2745                 return;\n=============================================================================\n\nBy looking at 2744 line, we can get that it's because regcache_rbtree_write()\nwould call kmalloc() with GFP flag if it couldn't find an existing block to\ninsert nodes while this kmalloc() call is inside a spin_lock_irq_save pair,\ni.e. IRQs disabled.\n\nEven though this may be a bug that should be fixed, I still try to send this\npatch as a quick fix (work around) since it does no harm to assign default\nvalues of every registers when using regcache.\n\nSigned-off-by: Nicolin Chen <nicoleotsuka@gmail.com>\nSigned-off-by: Mark Brown <broonie@kernel.org>","Nicolin Chen",true],["b2a9a3b818db727479cc2b9c2924e4ab9a4bbb07","b6ffd6973e972cb999e8e535ab74da7fee0c035f@ingics.com","ASoC: es8328-i2c: Fix i2c_device_id name field in es8328_id\n\nThe convention for i2c_device_id name does not need to have company prefix.\n\nSigned-off-by: Axel Lin <axel.lin@ingics.com>\nSigned-off-by: Mark Brown <broonie@kernel.org>","Axel Lin",true],["885e7b0e181c14e4d0ddd26c688bad2b84c1ada9","0d270388f2f92757a5de0f4bd891d3b392c44c4f@purestorage.com","target: Don't call TFO->write_pending if data_length == 0\n\nIf an initiator sends a zero-length command (e.g. TEST UNIT READY) but\nsets the transfer direction in the transport layer to indicate a\ndata-out phase, we still shouldn't try to transfer data.  At best it's\na NOP, and depending on the transport, we might crash on an\nuninitialized sg list.\n\nReported-by: Craig Watson <craig.watson@vanguard-rugged.com>\nSigned-off-by: Roland Dreier <roland@purestorage.com>\nCc: <stable@vger.kernel.org> # 3.1\nSigned-off-by: Nicholas Bellinger <nab@linux-iscsi.org>","Roland Dreier",true],["ab8edab132829b26dd13db6caca3c242cce35dc1","978acd1567d5598152161fdf8bf3ca568f950c9b@linux-iscsi.org","vhost-scsi: Take configfs group dependency during VHOST_SCSI_SET_ENDPOINT\n\nThis patch addresses a bug where individual vhost-scsi configfs endpoint\ngroups can be removed from below while active exports to QEMU userspace\nstill exist, resulting in an OOPs.\n\nIt adds a configfs_depend_item() in vhost_scsi_set_endpoint() to obtain\nan explicit dependency on se_tpg->tpg_group in order to prevent individual\nvhost-scsi WWPN endpoints from being released via normal configfs methods\nwhile an QEMU ioctl reference still exists.\n\nAlso, add matching configfs_undepend_item() in vhost_scsi_clear_endpoint()\nto release the dependency, once QEMU's reference to the individual group\nat /sys/kernel/config/target/vhost/$WWPN/$TPGT is released.\n\n(Fix up vhost_scsi_clear_endpoint() error path - DanC)\n\nCc: Michael S. Tsirkin <mst@redhat.com>\nCc: Paolo Bonzini <pbonzini@redhat.com>\nCc: Stefan Hajnoczi <stefanha@redhat.com>\nCc: <stable@vger.kernel.org> # 3.6+\nSigned-off-by: Nicholas Bellinger <nab@linux-iscsi.org>","Nicholas Bellinger",true],["f2774f430ea65fddc068865d364bb254d2816648","c3a0a03c0c59635d5ad58ea9cf606c1aa814fe9e@purestorage.com","iscsi-target: return the correct port in SendTargets\n\nThe fact that a target is published on the any address has no bearing on\nwhich port(s) it is published. SendTargets should always send the\nportal's port, not the port used for discovery.\n\nSigned-off-by: Steven Allen <steven.allen@purestorage.com>\nSigned-off-by: Nicholas Bellinger <nab@linux-iscsi.org>","Steven Allen",true],["ab477c1ff5e0a744c072404bf7db51bfe1f05b6e","89ed62d80e76c0eb24ee0d6433b48a91c2273b5e@acm.org","srp-target: Retry when QP creation fails with ENOMEM\n\nIt is not guaranteed to that srp_sq_size is supported\nby the HCA. So if we failed to create the QP with ENOMEM,\ntry with a smaller srp_sq_size. Keep it up until we hit\nMIN_SRPT_SQ_SIZE, then fail the connection.\n\nReported-by: Mark Lehrer <lehrer@gmail.com>\nSigned-off-by: Bart Van Assche <bvanassche@acm.org>\nSigned-off-by: Sagi Grimberg <sagig@mellanox.com>\nCc: <stable@vger.kernel.org> # 3.4+\nSigned-off-by: Nicholas Bellinger <nab@linux-iscsi.org>","Bart Van Assche",true],["29f95bd76f6ec1eff88eec6a04191104a11a7f97","271e1a51254e937f0b3e037325a6f6e4dbf91810@rock-chips.com","ASoC: rockchip-i2s: fix infinite loop in rockchip_snd_rxctrl\n\nWe can get into an infinite loop if the I2S_CLR register fails to\nclear due to a missing break statement, so add that.\n\nSigned-off-by: Jianqun <jay.xu@rock-chips.com>\nSigned-off-by: Mark Brown <broonie@kernel.org>\nCc: stable@vger.kernel.org","Jianqun",true],["df078d291d9b50e3ccfb8f030f85de701696a5ad","64970d799dae4b072c7e9f48aacc13707983c54b@intel.com","ASoC: rt5645: Mark RT5645_TDM_CTRL_3 as readable\n\namixer query fails due to it is not readable reigster\n\nSigned-off-by: Fang, Yang A <yang.a.fang@intel.com>\nAcked-by: Bard Liao <bardliao@realtek.com>\nSigned-off-by: Mark Brown <broonie@kernel.org>","Fang, Yang A",true],["c1b9b9b1ad2df6144ca3fbe6989f7bd9ea5c5562","a3b51ccb87d18302c1692defcfbf83319d1737eb@renesas.com","ASoC: fsi: remove unsupported PAUSE flag\n\nFSI doesn't support PAUSE.\nRemove SNDRV_PCM_INFO_PAUSE flags from snd_pcm_hardware info\n\nSigned-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>\nSigned-off-by: Mark Brown <broonie@kernel.org>\nCc: stable@vger.kernel.org","Kuninori Morimoto",true],["706c66213e5e623e23f521b1acbd8171af7a3549","a3b51ccb87d18302c1692defcfbf83319d1737eb@renesas.com","ASoC: rsnd: remove unsupported PAUSE flag\n\nR-Car sound doesn't support PAUSE.\nRemove SNDRV_PCM_INFO_PAUSE flags from snd_pcm_hardware info\n\nSigned-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>\nSigned-off-by: Mark Brown <broonie@kernel.org>\nCc: stable@vger.kernel.org","Kuninori Morimoto",true],["1ffae3612fe53dd18b92e696ab4f29df319a508d","3318dc5ce3e4fb7c28a0b841b6801c884e1d0896@metafoo.de","ASoC: Fix snd_soc_find_dai() matching component by name\n\nCommit 14621c7e5e72 (\"ASoC: Consolidate CPU and CODEC DAI lookup\")\nconsolidated the lookup of CPU DAIs and CODEC DAIs into a single function.\nWhen matching a component by name for CODEC DAIs the code previous to the\npatch compared the name in the DAI link table with component->name. For CPU\nDAIs the code compared to dev_name(component->dev). The newly introduced\nfunction ended up using the later as well.\n\nFor most components dev_name(component->dev) and component->name are the\nsame. The main notable exception are I2C devices where the driver name and\nthe device name are concatenated to form the component name. By using\ndev_name(component->dev) instead of component->name the patch broke the\nmatching of I2C CODECs by name.\n\nThis patch restores the original behavior by using component->name instead\nof dev_name(component->dev). This will be safe even for CPU DAIs since for\nCPU DAIs both are the same.\n\nFixes: 14621c7e5e72 (\"ASoC: Consolidate CPU and CODEC DAI lookup\")\nReported-by: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>\nSigned-off-by: Lars-Peter Clausen <lars@metafoo.de>\nSigned-off-by: Mark Brown <broonie@kernel.org>","Lars-Peter Clausen",true],["f57915cfa5b2b14c1cffa2e83c034f55e3f0e70d","de1b678a59344538ef9f2ddb34a5f9486544e2f9@mellanox.com","ib_isert: Add max_send_sge=2 minimum for control PDU responses\n\nThis patch adds a max_send_sge=2 minimum in isert_conn_setup_qp()\nto ensure outgoing control PDU responses with tx_desc->num_sge=2\nare able to function correctly.\n\nThis addresses a bug with RDMA hardware using dev_attr.max_sge=3,\nthat in the original code with the ConnectX-2 work-around would\nresult in isert_conn->max_sge=1 being negotiated.\n\nOriginally reported by Chris with ocrdma driver.\n\nReported-by: Chris Moore <Chris.Moore@emulex.com>\nTested-by: Chris Moore <Chris.Moore@emulex.com>\nSigned-off-by: Or Gerlitz <ogerlitz@mellanox.com>\nCc: <stable@vger.kernel.org> # 3.10+\nSigned-off-by: Nicholas Bellinger <nab@linux-iscsi.org>","Or Gerlitz",true],["3b726ae2de02a406cc91903f80132daee37b6f1b","380ce2a82d3d08e67ab163b51dc6d30288ad3f13@dev.mellanox.co.il","iser-target: Handle DEVICE_REMOVAL event on network portal listener correctly\n\nIn this case the cm_id->context is the isert_np, and the cm_id->qp\nis NULL, so use that to distinct the cases.\n\nSince we don't expect any other events on this cm_id we can\njust return -1 for explicit termination of the cm_id by the\ncma layer.\n\nSigned-off-by: Sagi Grimberg <sagig@mellanox.com>\nCc: <stable@vger.kernel.org> # 3.10+\nSigned-off-by: Nicholas Bellinger <nab@linux-iscsi.org>","Sagi Grimberg",true],["b6932ee35f1c1364dcea0e691b2feb912a6777e5","c3a0a03c0c59635d5ad58ea9cf606c1aa814fe9e@purestorage.com","target: return CONFLICT only when SA key unmatched\n\nPREEMPT (and PREEMPT AND ABORT) should return CONFLICT iff a specified\nSERVICE ACTION RESERVATION KEY is specified and matches no existing\npersistent reservation.\n\nWithout this patch, a PREEMPT will return CONFLICT if either all\nreservations are held by the initiator (self preemption) or there is\nnothing to preempt. According to the spec, both of these cases should\nsucceed.\n\nSigned-off-by: Steven Allen <steven.allen@purestorage.com>\nSigned-off-by: Nicholas Bellinger <nab@linux-iscsi.org>","Steven Allen",true],["ce5686d4ed12158599d2042a6c8659254ed263ce","3fddac958924aef220f202ca567388ddab3f14a8@infradead.org","perf/x86: Fix embarrasing typo\n\nBecause we're all human and typing sucks..\n\nFixes: 7fb0f1de49fc (\"perf/x86: Fix compile warnings for intel_uncore\")\nReported-by: Andi Kleen <ak@linux.intel.com>\nSigned-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>\nCc: Linus Torvalds <torvalds@linux-foundation.org>\nCc: x86@kernel.org\nLink: http://lkml.kernel.org/n/tip-be0bftjh8yfm4uvmvtf3yi87@git.kernel.org\nSigned-off-by: Ingo Molnar <mingo@kernel.org>","Peter Zijlstra (Intel)",true],["f7b8a47da17c9ee4998f2ca2018fcc424e953c0e","e5b39230c99b4d3ebd6aa54f6d823f95134bc39d@parallels.com","sched: Remove lockdep check in sched_move_task()\n\nsched_move_task() is the only interface to change sched_task_group:\ncpu_cgrp_subsys methods and autogroup_move_group() use it.\n\nEverything is synchronized by task_rq_lock(), so cpu_cgroup_attach()\nis ordered with other users of sched_move_task(). This means we do no\nneed RCU here: if we've dereferenced a tg here, the .attach method\nhasn't been called for it yet.\n\nThus, we should pass \"true\" to task_css_check() to silence lockdep\nwarnings.\n\nFixes: eeb61e53ea19 (\"sched: Fix race between task_group and sched_task_group\")\nReported-by: Oleg Nesterov <oleg@redhat.com>\nReported-by: Fengguang Wu <fengguang.wu@intel.com>\nSigned-off-by: Kirill Tkhai <ktkhai@parallels.com>\nSigned-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>\nCc: Linus Torvalds <torvalds@linux-foundation.org>\nLink: http://lkml.kernel.org/r/1414473874.8574.2.camel@tkhai\nSigned-off-by: Ingo Molnar <mingo@kernel.org>","Kirill Tkhai",true],["ea9d0d771fcd32cd56070819749477d511ec9117","4596b3305151c7ee743192a95d394341e3d3b644@suse.de","ASoC: dpcm: Fix race between FE/BE updates and trigger\n\nDPCM can update the FE/BE connection states totally asynchronously\nfrom the FE's PCM state.  Most of FE/BE state changes are protected by\nmutex, so that they won't race, but there are still some actions that\nare uncovered.  For example, suppose to switch a BE while a FE's\nstream is running.  This would call soc_dpcm_runtime_update(), which\nsets FE's runtime_update flag, then sets up and starts BEs, and clears\nFE's runtime_update flag again.\n\nWhen a device emits XRUN during this operation, the PCM core triggers\nsnd_pcm_stop(XRUN).  Since the trigger action is an atomic ops, this\nisn't blocked by the mutex, thus it kicks off DPCM's trigger action.\nIt eventually updates and clears FE's runtime_update flag while\nsoc_dpcm_runtime_update() is running concurrently, and it results in\nconfusion.\n\nUsually, for avoiding such a race, we take a lock.  There is a PCM\nstream lock for that purpose.  However, as already mentioned, the\ntrigger action is atomic, and we can't take the lock for the whole\nsoc_dpcm_runtime_update() or other operations that include the lengthy\njobs like hw_params or prepare.\n\nThis patch provides an alternative solution.  This adds a way to defer\nthe conflicting trigger callback to be executed at the end of FE/BE\nstate changes.  For doing it, two things are introduced:\n\n- Each runtime_update state change of FEs is protected via PCM stream\n  lock.\n- The FE's trigger callback checks the runtime_update flag.  If it's\n  not set, the trigger action is executed there.  If set, mark the\n  pending trigger action and returns immediately.\n- At the exit of runtime_update state change, it checks whether the\n  pending trigger is present.  If yes, it executes the trigger action\n  at this point.\n\nReported-and-tested-by: Qiao Zhou <zhouqiao@marvell.com>\nSigned-off-by: Takashi Iwai <tiwai@suse.de>\nAcked-by: Liam Girdwood <liam.r.girdwood@linux.intel.com>\nSigned-off-by: Mark Brown <broonie@kernel.org>\nCc: stable@vger.kernel.org","Takashi Iwai",true],["ece509c10985ba93ccc8c68f808a9e767250041c","c791bfe44996c1eed9c7149a0c5f12a86d58eb85@chromium.org","ASoC: max98090: Correct pclk divisor settings\n\nThe Baytrail-based chromebooks have a 20MHz mclk, the code was setting\nthe divisor incorrectly in this case.  According to the 98090\ndatasheet, the divisor should be set to DIV1 for 10 <= mclk <= 20.\nCorrect this and the surrounding clock ranges as well to match the\ndatasheet.\n\nSigned-off-by: Dylan Reid <dgreid@chromium.org>\nSigned-off-by: Mark Brown <broonie@kernel.org>","Dylan Reid",true],["62e6a3b6f4e1f9b96fa66ab1cdf2ffd8594df9e9","e2dd33448c0d502d92fa832c8b393532a3b3f74b@suse.de","ASoC: samsung: Add MODULE_DEVICE_TABLE for Snow\n\nThis enables the snd_soc_snow module to be auto-loaded.\n\nSigned-off-by: Andreas Färber <afaerber@suse.de>\nSigned-off-by: Mark Brown <broonie@kernel.org>\nCc: stable@vger.kernel.org","Andreas Färber",true],["ac87f22147098da9cf83a0b413ef7dda42277d85","44f95c252e5bdc8ed4413e18dbc09adedb90bea7@realtek.com","ASoC: rt5670: correct the incorrect default values\n\nThe patch corrects the incorrect default register values.\n\nSigned-off-by: Bard Liao <bardliao@realtek.com>\nSigned-off-by: Mark Brown <broonie@kernel.org>\nCc: stable@vger.kernel.org","Bard Liao",true]],"size":248,"ref":"refs/heads/devel","head":"f9a19ed87a695136abdf63a3f286513e4dc59d26"}