GitHub PushEvent at 2014-09-24 06:20:46
Repository: linux-wpan-next
Actor: alexaring
Payload: {"shas":[["4b70ac5fd9b58bfaa5f25b4ea48f528aefbf3308","20b70f0af00562e63758b9ee42012ecc96c58590@redhat.com","aio: change exit_aio() to load mm->ioctx_table once and avoid rcu_read_lock()\n\nOn 04/30, Benjamin LaHaise wrote:\n>\n> > -\t\tctx->mmap_size = 0;\n> > -\n> > -\t\tkill_ioctx(mm, ctx, NULL);\n> > +\t\tif (ctx) {\n> > +\t\t\tctx->mmap_size = 0;\n> > +\t\t\tkill_ioctx(mm, ctx, NULL);\n> > +\t\t}\n>\n> Rather than indenting and moving the two lines changing mmap_size and the\n> kill_ioctx() call, why not just do \"if (!ctx) ... continue;\"?  That reduces\n> the number of lines changed and avoid excessive indentation.\n\nOK. To me the code looks better/simpler with \"if (ctx)\", but this is subjective\nof course, I won't argue.\n\nThe patch still removes the empty line between mmap_size = 0 and kill_ioctx(),\nwe reset mmap_size only for kill_ioctx(). But feel free to remove this change.\n\n-------------------------------------------------------------------------------\nSubject: [PATCH v3 1/2] aio: change exit_aio() to load mm->ioctx_table once and avoid rcu_read_lock()\n\n1. We can read ->ioctx_table only once and we do not read rcu_read_lock()\n   or even rcu_dereference().\n\n   This mm has no users, nobody else can play with ->ioctx_table. Otherwise\n   the code is buggy anyway, if we need rcu_read_lock() in a loop because\n   ->ioctx_table can be updated then kfree(table) is obviously wrong.\n\n2. Update the comment. \"exit_mmap(mm) is coming\" is the good reason to avoid\n   munmap(), but another reason is that we simply can't do vm_munmap() unless\n   current->mm == mm and this is not true in general, the caller is mmput().\n\n3. We do not really need to nullify mm->ioctx_table before return, probably\n   the current code does this to catch the potential problems. But in this\n   case RCU_INIT_POINTER(NULL) looks better.\n\nSigned-off-by: Oleg Nesterov <oleg@redhat.com>\nSigned-off-by: Benjamin LaHaise <bcrl@kvack.org>","Oleg Nesterov",true],["855ef0dec7271ff7be7381feaaf3f4aed80bd503","20b70f0af00562e63758b9ee42012ecc96c58590@redhat.com","aio: kill the misleading rcu read locks in ioctx_add_table() and kill_ioctx()\n\nioctx_add_table() is the writer, it does not need rcu_read_lock() to\nprotect ->ioctx_table. It relies on mm->ioctx_lock and rcu locks just\nadd the confusion.\n\nAnd it doesn't need rcu_dereference() by the same reason, it must see\nany updates previously done under the same ->ioctx_lock. We could use\nrcu_dereference_protected() but the patch uses rcu_dereference_raw(),\nthe function is simple enough.\n\nThe same for kill_ioctx(), although it does not update the pointer.\n\nSigned-off-by: Oleg Nesterov <oleg@redhat.com>\nSigned-off-by: Benjamin LaHaise <bcrl@kvack.org>","Oleg Nesterov",true],["3482f2c52b77bf6596e24aae82e204a0603eba66","9069e6f5a2b566e2674a0ba1e2bf39c12c195fad@linaro.org","of: Create of_console_check() for selecting a console specified in /chosen\n\nThe devicetree has a binding for specifying the console device in the\n/chosen node, but the kernel doesn't use it consistently. This change\nadds an API for testing if a device node is a console, and adds a\npreferred console entry if it is.\n\nAt the same time this patch removes the of_device_is_stdout_path() API\nsince it is unused.\n\nSigned-off-by: Grant Likely <grant.likely@linaro.org>\nTested-by: Sascha Hauer <s.hauer@pengutronix.de>","Grant Likely",true],["a208ffd251d08ed7ba6bdf3ae1e423373fb12d3d","9069e6f5a2b566e2674a0ba1e2bf39c12c195fad@linaro.org","of: Enable console on serial ports specified by /chosen/stdout-path\n\nIf the devicetree specifies a serial port as a stdout device, then the\nkernel can use it as the default console if nothing else was selected on\nthe command line. For any serial port that uses the uart_add_one_port()\nfeature, the uart_add_one_port() has all the information needed to\nautomatically enable the console device, which is what this patch does.\n\nWith this change applied, a device tree platform can be booted without\nany console= parameters on the command line and the kernel will still be\nable to determine its console.\n\nTested on QEMU Versatile model and i.MX\n\nSigned-off-by: Grant Likely <grant.likely@linaro.org>\nTested-by: Sascha Hauer <s.hauer@pengutronix.de>","Grant Likely",true],["003882180340a2808151d320dedd853f4cff5e4b","9069e6f5a2b566e2674a0ba1e2bf39c12c195fad@linaro.org","arm/versatile: Add the uart as the stdout device.\n\nAdd a stdout-path property to the Versatile devicetree so that automatic\nconsole selection works without needing a console= line on the kernel\ncommand line.\n\nSigned-off-by: Grant Likely <grant.likely@linaro.org>","Grant Likely",true],["a752ee56ad84bf9a35b8323af1ad22b03c1df2c4","9069e6f5a2b566e2674a0ba1e2bf39c12c195fad@linaro.org","tty: Update hypervisor tty drivers to use core stdout parsing code.\n\nThe evh_bytechan, hvc_opal and hvc_vio drivers all open code the parsing\nof the stdout node in the device tree. This patch simplifies the driver\nby removing the duplicated functionality.\n\nSigned-off-by: Grant Likely <grant.likely@linaro.org>","Grant Likely",true],["ccdb8ed3b3c739fe99a6f2f474f7ffad3203485d","9069e6f5a2b566e2674a0ba1e2bf39c12c195fad@linaro.org","of: Migrate of_find_node_by_name() users to for_each_node_by_name()\n\nThere are a bunch of users open coding the for_each_node_by_name() by\ncalling of_find_node_by_name() directly instead of using the macro. This\nis getting in the way of some cleanups, and the possibility of removing\nof_find_node_by_name() entirely. Clean it up so that all the users are\nconsistent.\n\nSigned-off-by: Grant Likely <grant.likely@linaro.org>\nCc: Rob Herring <robh+dt@kernel.org>\nCc: Benjamin Herrenschmidt <benh@kernel.crashing.org>\nCc: \"Rafael J. Wysocki\" <rjw@rjwysocki.net>\nCc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>\nCc: Takashi Iwai <tiwai@suse.de>","Grant Likely",true],["9e3754461507147ce5bffba5faaea7c182d64667","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","sb_edac: make memory type detection per memory controller\n\nHaswell has different register, offset to determine memory type and supports\nDDR4 in some models. This patch makes it easier to have a different method\ndepending on the memory controller type.\n\nCc: Tony Luck <tony.luck@intel.com>\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["f14d6892e4a6086d48fa77ac8e59003d3f4f8e16","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","sb_edac: make node id retrieval per model\n\nHaswell has a different way to retrieve the node id, make so this procedure\ncan be reimplemented.\n\nCc: Tony Luck <tony.luck@intel.com>\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["b976bcf249f4d3b28963e67b5194bc2690da260c","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","sb_edac: make RIR limit retrieval per model\n\nHaswell has a different way to retrieve RIR limits, make this procedure per\nmodel.\n\nCc: Tony Luck <tony.luck@intel.com>\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["dbc954ddddaaff0183a21dbd736ec10ca01e9f1a","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","sb_edac: search devices using product id\n\nThis patch changes the way devices are searched by using product id instead of\ndevice/function numbers. Tested in a Sandy Bridge and a Ivy Bridge machine to\nmake sure everything works properly.\n\nCc: Tony Luck <tony.luck@intel.com>\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["66ca72749166163c4ec9e5ab5416548a40d902d3","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","sb_edac: update Kconfig description\n\nKconfig wasn't updated when Ivy Bridge support was added.\n\nCc: Tony Luck <tony.luck@intel.com>\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["2ff3a308b5c776bdf4641311b87c843fe1bc3785","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","sb_edac: fix socket detection on Ivy Bridge controllers\n\nThis patch fixes the obvious bug while handling the socket/HA bitmask used in\nIvy Bridge memory controllers.\n\nCc: Tony Luck <tony.luck@intel.com>\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["d7c660b7dcb2f3c3835432cf733fba03507f4387","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","sb_edac: make minimal use of channel_mask\n\nchannel_mask will be used in the future to determine which group of memory\nmodules is causing the errors since when mirroring, lockstep and close page\nare enabled you can't. While that doesn't happen, use the channel_mask to\ndetermine the channel instead of relying on the MC event/exception.\n\nCc: Tony Luck <tony.luck@intel.com>\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["adc61bcd91aa8364efaa496e7a4699950c362c21","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","sb_edac: remove bogus assumption on mc ordering\n\nWhen a MC is handled, the correct sbridge_dev is searched based on the node,\nchecking again later with the assumption the first memory controller found is\nthe first socket's memory controller is a bogus assumption. Get rid of it.\n\nCc: Tony Luck <tony.luck@intel.com>\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["7b8278358cc2b453ca6e75eedb3741cdb7e97236","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","edac: add DDR4 and RDDR4\n\nHaswell memory controller can make use of DDR4 and Registered DDR4\n\nCc: tony.luck@intel.com\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["c41afdca29acfdc78b89f936cdcb8cabc6b08aae","0cae1d1e981e84d16b82ca3d17be8a7f826608d3@samsung.com","sb_edac: Fix mix tab/spaces alignments\n\nWe should not have spaces before ^I on alignments.\n\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Mauro Carvalho Chehab",true],["50d1bb93672fa2f42cec6e06ce799fbe864f57e9","78e305742ac743ecf22d8139502576ac8709e0ad@redhat.com","sb_edac: add support for Haswell based systems\n\nHaswell memory controllers are very similar to Ivy Bridge and Sandy Bridge\nones. This patch adds support to Haswell based systems.\n\n[m.chehab@samsung.com: Fix CodingStyle issues]\nCc: Tony Luck <tony.luck@intel.com>\nSigned-off-by: Aristeu Rozanski <aris@redhat.com>\nSigned-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>","Aristeu Rozanski",true],["17737d3b5997ac9f810967f0c6014d124ec39490","cd8c6775e60d6f67a6984377324e5290df3d5358@fb.com","Merge branch 'for-3.17' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/percpu into for-3.17/core\n\nMerge the percpu_ref changes from Tejun, he says they are stable now.","Jens Axboe",true],["531ed6261e7466907418b1a9971a5c71d7d250e4","546b05909706652891a87f7bfe385ae147f61f91@kernel.org","blk-mq: fix a memory ordering bug in blk_mq_queue_enter()\n\nblk-mq uses a percpu_counter to keep track of how many usages are in\nflight.  The percpu_counter is drained while freezing to ensure that\nno usage is left in-flight after freezing is complete.\n\nblk_mq_queue_enter/exit() and blk_mq_[un]freeze_queue() implement this\nper-cpu gating mechanism; unfortunately, it contains a subtle bug -\nsmp_wmb() in blk_mq_queue_enter() doesn't prevent prevent the cpu from\nfetching @q->bypass_depth before incrementing @q->mq_usage_counter and\nif freezing happens inbetween the caller can slip through and freezing\ncan be complete while there are active users.\n\nUse smp_mb() instead so that bypass_depth and mq_usage_counter\nmodifications and tests are properly interlocked.\n\nSigned-off-by: Tejun Heo <tj@kernel.org>\nCc: Jens Axboe <axboe@kernel.dk>\nCc: Nicholas A. Bellinger <nab@linux-iscsi.org>\nSigned-off-by: Jens Axboe <axboe@fb.com>","Tejun Heo",true]],"size":988,"ref":"refs/heads/master","head":"63105244c0e1a1b2ba02a1d3c67229d7e24cbf08"}