Re: [RFC PATCH v1 02/28] kvm: svm: Add kvm_fast_pio_in support

2016-09-21 Thread Borislav Petkov
On Mon, Aug 22, 2016 at 07:23:54PM -0400, Brijesh Singh wrote: > From: Tom Lendacky > > Update the I/O interception support to add the kvm_fast_pio_in function > to speed up the in instruction similar to the out instruction. > > Signed-off-by: Tom Lendacky

[PATCH 6/6] staging: most: replace MOST_CH_ISOC_AVP with MOST_CH_ISOC

2016-09-21 Thread Christian Gromm
From: Andrey Shvetsov This patch replaces the enum value MOST_CH_ISOC_AVP with the more appropriate MOST_CH_ISOC. Signed-off-by: Andrey Shvetsov Signed-off-by: Christian Gromm ---

[PATCH 3/6] staging: most: aim-cdev: make syscall write accept buffers of arbitrary size

2016-09-21 Thread Christian Gromm
From: Andrey Shvetsov This patch allows to call the write() function for synchronous and isochronous channels with buffers of any size. The AIM simply waits for data to fill up the MOST buffer object according to the network interface controller specification for

[PATCH 1/6] staging: most: make function most_submit_mbo return void

2016-09-21 Thread Christian Gromm
Function most_submit_mbo() causes an exception only if either the pointer mbo or mbo->context equals NULL. From the underlying architecture's point of view both cases must _not_ come true and would happen only, if something has tampered with the pointers. This would render runtime code unable to

[PATCH 4/6] staging: most: core: remove trailing zero from text property

2016-09-21 Thread Christian Gromm
From: Andrey Shvetsov This patch removes trailing zeros from the strings returned by the attributes available_datatypes and available_directions. Signed-off-by: Andrey Shvetsov Signed-off-by: Christian Gromm ---

[PATCH 2/6] staging: most: aim-cdev: remove labels

2016-09-21 Thread Christian Gromm
From: Andrey Shvetsov This patch removes the labels 'put_mbo' and 'unlock' and relocates the code accordingly making the code look simpler. Signed-off-by: Andrey Shvetsov Signed-off-by: Christian Gromm ---

[PATCH 0/6] staging: most: fix driver modules

2016-09-21 Thread Christian Gromm
This patch set is needed to fix up issues of the driver modules. Andrey Shvetsov (4): staging: most: aim-cdev: remove labels staging: most: aim-cdev: make syscall write accept buffers of arbitrary size staging: most: core: remove trailing zero from text property staging: most: replace

[PATCH 5/6] staging: most: clean up configuration strings

2016-09-21 Thread Christian Gromm
This patch adds the strings 'rx', 'tx' and 'isoc' to the list of accepted identifiers when setting up a channel configuration. To keep consistency it removes the prefix "dir_" from strings returned by the attributes set_direction and available_directions and it removes the suffix "_avp" from the

Re: [PATCH 1/2] staging: greybus: audio_codec.c: space required before the open brace

2016-09-21 Thread Greg KH
On Wed, Sep 21, 2016 at 05:11:22PM +0200, Richard Groux wrote: > Minor error spotted by checkpatch.pl in greybus > space required before the open brace '{' > > Signed-off-by: Richard Groux > --- > drivers/staging/greybus/audio_codec.c | 2 +- > 1 file changed, 1

[PATCH 0/2] staging: greybus: audio_codec.c: minor error spotted by checkpatch.pl

2016-09-21 Thread Richard Groux
This patchset removes few error spotted by checkpatch.pl Richard Groux (2): staging: greybus: audio_codec.c: space required before the open brace staging: greybus: audio_codec.c: code indent should use tabs where possible drivers/staging/greybus/audio_codec.c | 4 ++-- 1 file

[PATCH 1/2] staging: greybus: audio_codec.c: space required before the open brace

2016-09-21 Thread Richard Groux
Minor error spotted by checkpatch.pl in greybus space required before the open brace '{' Signed-off-by: Richard Groux --- drivers/staging/greybus/audio_codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.c

[PATCH 2/2] staging: greybus: audio_codec.c: code indent should use tabs where possible

2016-09-21 Thread Richard Groux
Minor error spotted by checkpatch.pl in greybus code indent should use tabs where possible Signed-off-by: Richard Groux --- drivers/staging/greybus/audio_codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.c

Re: [RFC PATCH v1 03/28] kvm: svm: Use the hardware provided GPA instead of page walk

2016-09-21 Thread Borislav Petkov
On Mon, Aug 22, 2016 at 07:24:07PM -0400, Brijesh Singh wrote: > From: Tom Lendacky > > When a guest causes a NPF which requires emulation, KVM sometimes walks > the guest page tables to translate the GVA to a GPA. This is unnecessary > most of the time on AMD hardware

[PATCH 1/2] staging: greybus: audio_codec.c: space required before the open brace

2016-09-21 Thread Richard Groux
Minor error spotted by checkpatch.pl in greybus space required before the open brace '{' Signed-off-by: Richard Groux --- drivers/staging/greybus/audio_codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.c

[PATCH 2/2] staging: greybus: audio_codec.c: code indent should use tabs where possible

2016-09-21 Thread Richard Groux
Minor error spotted by checkpatch.pl in greybus code indent should use tabs where possible Signed-off-by: Richard Groux --- drivers/staging/greybus/audio_codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/audio_codec.c

[PATCH 07/11] staging: dgnc: rename dgnc_finalize_board_init() to

2016-09-21 Thread Daeseok Youn
The dgnc_finalize_board_init() function has only job for requesting the IRQ. It should be renamed to dgnc_request_irq() Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH 10/11] staging: dgnc: remove useless variables

2016-09-21 Thread Daeseok Youn
The dgnc_major_serial_registered and dgnc_major_serial_registered do not need to use to check whether the tty driver is registered or not. These variables are used only in dgnc_cleanup_tty() function, This function will be called normally with initialized board structure. It means the

[PATCH 06/11] staging: dgnc: introduce the dgnc_tty_unregister()

2016-09-21 Thread Daeseok Youn
The dgnc_tty_unregister() will be called when the dgnc_tty_register() is failed. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.c | 7 +-- drivers/staging/dgnc/dgnc_tty.c| 8 drivers/staging/dgnc/dgnc_tty.h| 1 + 3 files changed, 14

[PATCH 09/11] staging: dgnc: rename dgnc_tty_uninit() to

2016-09-21 Thread Daeseok Youn
The dgnc_tty_uninit() doesn't match with dgnc_tty_init() at all. And also the dgnc_cleanup_tty() is only called for exiting the module. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.c | 2 +- drivers/staging/dgnc/dgnc_tty.c| 4 ++--

[PATCH 08/11] staging: dgnc: introduce the dgnc_free_irq()

2016-09-21 Thread Daeseok Youn
The dgnc_free_irq() will free the requested IRQ from the dgnc_request_irq(). Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_driver.c

[PATCH 02/11] staging: dgnc: remove useless message buffer

2016-09-21 Thread Daeseok Youn
There is a temporary message buffer for the boot message in dgnc_found_board() but the buffer was not used anywhere in dgnc driver. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.c | 28 drivers/staging/dgnc/dgnc_driver.h |

[PATCH 00/11] staging: dgnc: cleanup the function on dgnc driver

2016-09-21 Thread Daeseok Youn
This series of patches will cleanup the functions in dgnc driver. Rename it and introduce new functions to split big function into smaller one. Please take a look at my patches to update the dgnc driver and provide some comments if there are something wrong. This series of patches maybe have

[PATCH 01/11] staging: dgnc: remove redundant initialization for

2016-09-21 Thread Daeseok Youn
The channel array in board_t was initialized in dgnc_found_board() with NULL. But the channel is going to initialize in dgnc_tty_init(). So the channel array doesn't need to set NULL for initailization. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.c |

[PATCH 03/11] staging: dgnc: missing NULL check for ioremap in

2016-09-21 Thread Daeseok Youn
The ioremap() function can be failed, so it need to have error handling in dgnc_do_remap(). And also the return type of dgnc_do_remap() should be changed from "void" to "int" Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.c | 31

[PATCH 04/11] staging: dgnc: kfree for board structure in

2016-09-21 Thread Daeseok Youn
The board structure should be freed when any function was failed in dgnc_found_board(). And the board strucure will be stored into dgnc_board array when the dgnc_found_board() function has no error. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.c | 17

[PATCH 05/11] staging: dgnc: move functions unrelated with

2016-09-21 Thread Daeseok Youn
The functions related with tty device initialization are needed to be moved from dgnc_found_board() to dgnc_init_one(). Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.c | 81 -- 1 file changed, 43 insertions(+), 38

[PATCH 11/11] staging: dgnc: introduce find_board_by_major()

2016-09-21 Thread Daeseok Youn
It was used to get a board structure with dgnc_BoardsByMajor array. But this driver already has the array for managing initialized board as dgap_board[]. It can be used for searching the board structure by major number. Signed-off-by: Daeseok Youn ---

[PATCH 0/7] Tests for sync infrastructure

2016-09-21 Thread Emilio López
Hello everyone, This is a series of tests to exercise the sync kernel infrastructure. It is meant to be a test suite for the work Gustavo has been doing to destage it. These tests were originally part of a battery of tests shipping with Android's libsync that were rewritten to use the new

[PATCH 1/7] selftest: sync: basic tests for sw_sync framework

2016-09-21 Thread Emilio López
These tests are based on the libsync test suite from Android. This commit lays the ground for future tests, as well as includes tests for a variety of basic allocation commands. Signed-off-by: Emilio López --- tools/testing/selftests/Makefile | 1 +

[PATCH 5/7] selftest: sync: stress test for parallelism

2016-09-21 Thread Emilio López
This test is based on the libsync test suite from Android. This commit includes a stress test that invokes operations in parallel. Signed-off-by: Emilio López --- tools/testing/selftests/sync/Makefile | 1 +

[PATCH 6/7] selftest: sync: stress consumer/producer test

2016-09-21 Thread Emilio López
This test is based on the libsync test suite from Android. This commit includes a stress test that replicates a consumer/producer pattern. Signed-off-by: Emilio López --- tools/testing/selftests/sync/Makefile | 1 +

[PATCH] rtl871x: use tabs with indentation instead of spaces

2016-09-21 Thread Christopher Pezley
There is a line with space indentation instead of tabs. I've changed it to use tabs for indentation. Signed-off-by: Christopher H. Pezley --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 4/7] selftest: sync: wait tests for sw_sync framework

2016-09-21 Thread Emilio López
These tests are based on the libsync test suite from Android. This commit includes tests for waiting on fences. Signed-off-by: Emilio López --- tools/testing/selftests/sync/Makefile| 1 + tools/testing/selftests/sync/sync_test.c | 1 +

[PATCH 3/7] selftest: sync: merge tests for sw_sync framework

2016-09-21 Thread Emilio López
These tests are based on the libsync test suite from Android. This commit includes tests for basic merge operations. Signed-off-by: Emilio López --- tools/testing/selftests/sync/Makefile | 1 + tools/testing/selftests/sync/sync_merge.c | 60

[PATCH 2/7] selftest: sync: fence tests for sw_sync framework

2016-09-21 Thread Emilio López
These tests are based on the libsync test suite from Android. This commit includes tests for basic fence creation. Signed-off-by: Emilio López --- tools/testing/selftests/sync/Makefile | 1 + tools/testing/selftests/sync/sync_fence.c | 132

Re: [PATCH 1/2] staging: greybus: audio_codec.c: space required before the open brace

2016-09-21 Thread Alex Elder
On 09/21/2016 12:05 PM, Richard Groux wrote: > Minor error spotted by checkpatch.pl in greybus > space required before the open brace '{' > > Signed-off-by: Richard Groux Sure, looks good. Greg will have to apply these patches. Reviewed-by: Alex Elder

[PATCH 7/7] selftest: sync: stress test for merges

2016-09-21 Thread Emilio López
This test is based on the libsync test suite from Android. This commit includes a test to stress merge operations. Signed-off-by: Emilio López --- tools/testing/selftests/sync/Makefile| 1 + tools/testing/selftests/sync/sync_stress_merge.c | 116

Re: [PATCH 2/2] staging: greybus: audio_codec.c: code indent should use tabs where possible

2016-09-21 Thread Alex Elder
On 09/21/2016 12:05 PM, Richard Groux wrote: > Minor error spotted by checkpatch.pl in greybus > code indent should use tabs where possible > > Signed-off-by: Richard Groux Looks good. Reviewed-by: Alex Elder > --- >