- 21 Aug, 2016 3 commits
-
-
wang di authored
Group function prototypes together related to dir.c. Move ll_release_page to be with function declarations. Signed-off-by:
wang di <di.wang@intel.com> Reviewed-on: http://review.whamcloud.com/7043 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3531Reviewed-by:
John L. Hammond <john.hammond@intel.com> Reviewed-by:
Jinshan Xiong <jinshan.xiong@intel.com> Reviewed-by:
Andreas Dilger <andreas.dilger@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
wang di authored
Remove comment about fixing swabbing that is not needed. Signed-off-by:
-
wang di authored
Pass in struct md_op_data for ll_get_dir_page function. Signed-off-by:
-
- 18 Aug, 2016 5 commits
-
-
Christian Gromm authored
The networking AIM does not use the *_irqsave and *_irqrestore flavored spinlock functions. The rx_completion callback, however, can be called from an interrupt context. This patch is needed to fix this problem. Signed-off-by:
Andrey Shvetsov <andrey.shvetsov@k2l.de> Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Signed-off-by:
-
Ben LeMasurier authored
This fixes the checkpatch.pl "Alignment should match open parenthesis" issues in ion_test.c. Signed-off-by:
Ben LeMasurier <ben@crypt.ly> Signed-off-by:
-
Ben LeMasurier authored
This fixes the checkpatch.pl "Alignment should match open parenthesis" issues in ion_heap.c. Signed-off-by:
-
Christian Gromm authored
This patch removes duplicated code by putting it into the new function dim2_transfer_madr. Signed-off-by:
-
Christian Gromm authored
In the time after a synchronous DIM2 channel is enabled and before the first buffer travels this channel, the DIM2 macro is babbling random random data to the network. This patch initializes the internal DBR memory of the DIM2 that is used to transfer the synchronous data. Signed-off-by:
-
- 15 Aug, 2016 32 commits
-
-
Tim Sell authored
Since lock_visor_dev is a binary semaphore it makes more sense to use a mutex lock. Signed-off-by:
Tim Sell <Timothy.Sell@unisys.com> Signed-off-by:
David Kershner <david.kershner@unisys.com> Acked-By:
Neil Horman <nhorman@tuxdriver.com> Reviewed-by:
Thomas Gleixner <tglx@linutronix.de> Signed-off-by:
-
Tim Sell authored
Ensure we properly lock between visorinput_channel_interrupt(), visorinput_open(), and devdata_create(). We now guarantee that: * interrupts will be disabled and remain disabled during device creation, by setting 'paused = true' across device creation * we canNOT get into visorinput_open() until the device structure is totally initialized, by delaying the input_register_device() until the end of device initialization We also now ensure that lock_visor_dev is held across updates of devdata state, to ensure state consistency. Signed-off-by:
-
David Kershner authored
Only visorbus needs this header file so move it to visorbus directory. Signed-off-by:
-
Tim Sell authored
This patch ONLY touches comment lines, i.e., NO executable code is affected. Comments were fixed in visorbus_private.h: * Minor typos were corrected. * Useless comments were removed. Signed-off-by:
-
David Binder authored
Renames visorchipset_device_pause_response to device_pause_response, thereby following the convention that other responder functions follow. Signed-off-by:
David Binder <david.binder@unisys.com> Signed-off-by:
-
David Binder authored
When this functionality was first implemented, visorchipset and visorbus were separate drivers, which necessitated a registration mechanism for them to communicate. More-recently, visorchipset and visorbus were combined into a single driver, and now exist as separate source files within the same driver, known as 'visorbus'. This eliminated the need for a registration mechanism, but it has remained nevertheless until now. For the sake of simplification, this registration mechanism is now being removed. Signed-off-by:
-
David Binder authored
Fixes the kerneldoc comment for struct visor_device - the struct members were not listed with the appropriate @ prefix. Signed-off-by:
-
David Binder authored
Moves function prototypes that are unique to visorbus from include/visorbus.h to visorbus/visorbus_private.h. Signed-off-by:
-
David Binder authored
Adds kerneldoc formatting to appropriate functions. Other multi-line comments now use proper formatting. Signed-off-by:
-
David Binder authored
This patch ONLY touches comment lines, i.e., NO executable code is affected. Comments were fixed in visorchannel.c: * All functions worthy of documenting now use standard kerneldoc formatting. * Multi-line comments were tweaked so as to use appropriate conventions. * Minor typos were corrected. * Useless comments were removed. Signed-off-by:
-
David Binder authored
This patch ONLY touches comment lines, i.e., NO executable code is affected. * All functions worthy of documenting now use standard kerneldoc formatting. * Improper uses of kerneldoc formatting were converted to standard multi-line comments. * Multi-line comments were tweaked so as to use appropriate conventions. Signed-off-by:
-
David Binder authored
This patch ONLY touches comment lines, i.e., NO executable code is affected. * All functions worthy of documenting now use standard kerneldoc formatting. * Multi-line comments were tweaked so as to use appropriate conventions. * Minor typos were corrected. Signed-off-by:
-
Tim Sell authored
The off parameter to visorchannel_create_guts() was never used, so it was removed. Signed-off-by:
-
Bryan Thompson authored
The STANDALONE_CLIENT define is no longer used by Unisys driver code. Signed-off-by:
Bryan Thompson <bryan.thompson@unisys.com> Signed-off-by:
-
Bryan Thompson authored
Remove visorbus_clear_channel, visorchannel_signalqueue_slots_avail, visorchannel_signalqueue_max_slots, visorchannel_clear, and visorchannel_debug which are no longer called by any driver. Signed-off-by:
-
Bryan Thompson authored
The driver that is now visorbus started out as multiple separate drivers, and when they were merged the EXPORT_SYMBOL statements that were required for separate drivers were left in the code. This patch removes those now unnecessary exports. Signed-off-by:
-
Bryan Thompson authored
visordriver_callback_lock is just a binary semaphore that logically makes more sense as a mutex. Signed-off-by:
-
Tim Sell authored
These files were made no-longer-necessary by recent commits. Signed-off-by:
-
Tim Sell authored
A kernel timer is now used as the vehicle to periodically call the channel_interrupt function of registered visor drivers, instead of a workqueue. This simplifies a lot of things by making periodic_work.c and periodic_work.h no longer necessary. This change also means that the channel_interrupt() callbacks registered by visor drivers (via visorbus_register_visor_driver()) will now be called in atomic context (i.e., canNOT sleep) rather than kernel thread context (CAN sleep). Fortunately this did NOT necessitate any change to the existing channel_interrupt() callbacks, because none of them ever perform any operations that would be invalid in atomic context. Signed-off-by:
-
Tim Sell authored
Locking in the _interrupt() function is NOT necessary so long as we ensure that interrupts have been stopped whenever we need to pause or resume the device, which we now do. While a device is paused, we ensure that interrupts stay disabled, i.e. that the _interrupt() function will NOT be called, yet remember the desired state in devdata->interrupts_enabled if open() or close() are called are called while the device is paused. Then when the device is resumed, we restore the actual state of interrupts (i.e., whether _interrupt() is going to be called or not) to the desired state in devdata->interrupts_enabled. Signed-off-by:
-
Tim Sell authored
periodic_test_workqueue was an unused relic from the past, and was removed. Signed-off-by:
-
David Binder authored
Code relating to ktheads was previously removed from s-Par driver code. This patch cleans up lingering remnants of kthreads by removing thread- related enum types. Signed-off-by:
-
David Binder authored
Fixes a comment spelling mistake in visornic. Signed-off-by:
-
David Binder authored
Modifies the format string of snprintf to expect an unsigned int instead of a signed one, per the supplied argument. Signed-off-by:
-
David Binder authored
Removes unused struct definition, channel_size_info, in response to findings by SonarQube. Signed-off-by:
-
David Binder authored
Removes unused module parameters from visorbus_main.c, in response to findings by SonarQube. Signed-off-by:
-
Erik Arfvidson authored
This patch modifies the caller of parahotplug_request_complete() to check the return value and return appropriate result. Signed-off-by:
Erik Arfvidson <erik.arfvidson@unisys.com> Signed-off-by:
-
Erik Arfvidson authored
This patch removes the return values from the write_vbus_* channel functions. Nobody was checking the return values and the value stored into the vbus info is not critical to the functioning of the device. Signed-off-by:
-
Christian Gromm authored
This patch reduces context of the list_del mutex for the function hdm_enqueue(). Signed-off-by:
-
Christian Gromm authored
This patch gives the struct device variable of the core a more meaningful name. Signed-off-by:
-
Christian Gromm authored
This patch removes the debug print in case an HDM ran out of rx buffers. This is not needed since the status is reflected in the sysfs channel_starving flag. Additionally, it prevents the system form slowing down. Signed-off-by:
-
Christian Gromm authored
If a DIM2 channel is being closed while it is busy, the channels's status flag could possibly stay active for this very channel. This causes the kernel to freeze by the time the channel is opened again. This patch fixes the problem. Signed-off-by:
-
