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

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 > --- > arch/x86/include/asm/kvm_host

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

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 2/6] staging: most: aim-cdev: remove labels

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 --- drivers/staging/most/aim-cdev/cdev.c | 15 +-- 1 file changed, 5 insertions

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

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 --- drivers/staging/most/mostcore/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2

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

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 st

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

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 --- drivers/staging/most/aim-v4l2/video.c| 2 +- drivers/staging/most/hdm-dim2/dim2_hdm.c | 4 ++-- drivers/stagi

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

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 streaming channels, before it sub

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

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 re

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

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 changed

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

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 b/drivers/staging/greybus/aud

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

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 b/drivers/staging/greybus/a

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

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 insertion(+), 1 deletion(-) Any

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

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 b/drivers/staging/greybus/a

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

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 b/drivers/staging/greybus/aud

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

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 since the hardware provides

[PATCH 0/7] Tests for sync infrastructure

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 userspa

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

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 + tools/testing/selftests/sync/.gitig

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

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 + tools/testing/selftests/sync/sync_wait.c | 91

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

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 +++ tools/testing/sel

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

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 ++ tools/testing/self

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

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 + .../testing/selftests/sync/sync_stress_consumer.c | 185

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

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 + .../selftests/sync/sync_stress_parallelism.c | 111

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

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 +++ tools/testi

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

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 > --- > drivers/staging/greybus/audio_

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

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 > --- > drivers/staging/greybus/audio_codec.c | 2 +- > 1 file changed, 1 insertion

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

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 a/drivers/staging/rtl8712/rtl871x_ioctl_li

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

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 depen

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

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 | 6 -- 2 files change

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

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 | 3 --- 1 file changed, 3

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

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 +-- 1 file changed,

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

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 + 1 file

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

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 deletions(-) diff --git a/

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

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 insertions(+), 2 deletio

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

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 a/drivers/staging/dgnc/dgnc_driver.c

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

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 b/drivers/staging/dgnc/dgnc_driver.c

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

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 ++-- drivers/staging/dgnc/dgnc_tty.h| 2 +

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

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 dgnc_cleanup_

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

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 --- drivers/staging/dgnc/dgnc_tty.c | 25 +++

[PATCH] staging: ion: Fix checkpatch.pl coding style issues

One warning was left in place that wanted a typedef (ion_user_handle_t) to be removed. I think the typedef makes sense so I left it intact. Signed-off-by: Antti Keränen --- drivers/staging/android/ion/hisilicon/hi6220_ion.c | 4 ++-- drivers/staging/android/uapi/ion.h | 2 +- 2

Re: [PATCH] staging: ion: Fix checkpatch.pl coding style issues

On Thu, Sep 22, 2016 at 09:08:01AM +0300, Antti Keränen wrote: > One warning was left in place that wanted a typedef (ion_user_handle_t) to be > removed. > I think the typedef makes sense so I left it intact. This does not describe at all what type of changes you made, or why you made them, only

[PATCH] staging: ion: Fix checkpatch.pl coding style issues

This patch fixes two coding style issues found with checkpatch.pl. Both of them are warnings. uapi/ion.h had an unaligned block comment. ion/hisilicon/hi6220_ion.c had unnecessary braces around a single statement block. One warning in uapi/ion.h was left in place that wanted a typedef (ion_user_h