Re: [PATCH v2] android: binder: fix dangling pointer comparison

On Thu, Aug 18, 2016 at 7:30 PM, ZhaoJunmin Zhao(Junmin) wrote: > > > 在 2016/8/18 23:23, Greg Kroah-Hartman 写道: >> >> On Tue, Aug 16, 2016 at 07:44:59PM -0700, Arve Hjønnevåg wrote: >>> >>> On Mon, Aug 15, 2016 at 7:58 AM, Greg Kroah-Hartman >>>

Re: [PATCH v2] android: binder: fix dangling pointer comparison

在 2016/8/18 23:23, Greg Kroah-Hartman 写道: On Tue, Aug 16, 2016 at 07:44:59PM -0700, Arve Hjønnevåg wrote: On Mon, Aug 15, 2016 at 7:58 AM, Greg Kroah-Hartman wrote: On Thu, Jun 16, 2016 at 12:45:33AM +0200, Jann Horn wrote: If /dev/binder is opened and the

[PATCH] Staging: android: ion: fix parenthesis alignment

This fixes remaining checkpatch.pl "Alignment should match open parenthesis" issues. Signed-off-by: Ben LeMasurier --- drivers/staging/android/ion/ion.c | 42 +- drivers/staging/android/ion/ion_chunk_heap.c | 12

[PATCH 2/2] staging: dgnc: refactor the dgnc_maxcps_room() to improve

The whole code in dgnc_maxcps_room() function surrounds with one if-statement for checking channel's maxcps and buffer size. I tried to separate the logic for this function from if-condition. Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_tty.c | 46

[PATCH 1/2] staging: dgnc: change a parameter from tty_struct to

The dgnc_maxcps_room() function must be called only for print device. The if-statement for checking print device checks before calling maxcps_room() and also this function doesn't need to have any data except channel_t. Signed-off-by: Daeseok Youn ---

Partnership Investment OFFER

ARE YOU LOOKING TO START A NEW BUSINESS? ARE YOU LOOKING TO DEVELOP AN EXISTING BUSINESS? ARE YOU LOOKING FOR FINANCE FOR A PROJECT OR BUSINESS? ARE YOU LOOKING SEEKING FINANCE FOR A REAL-ESTATE DEVELOPMENT? ARE YOU SEEKING FINANCE TO PURCHASE OR INVEST IN PROPERTIES? OR YOU SEEK A 100%

[PATCH 1/2] Drivers: hv: utils: Continue to poll VSS channel after handling requests.

From: Alex Ng Multiple VSS_OP_HOT_BACKUP requests may arrive in quick succession, even though the host only signals once. The driver wass handling the first request while ignoring the others in the ring buffer. We should poll the VSS channel after handling a request to

[PATCH 2/2] Drivers: hv: utils: Check VSS daemon is listening before a hot backup

From: Alex Ng Hyper-V host will send a VSS_OP_HOT_BACKUP request to check if guest is ready for a live backup/snapshot. The driver should respond to the check only if the daemon is running and listening to requests. This allows the host to fallback to standard

[PATCH 0/2] Drivers: hv: util: Some fixes to the backup driver

From: K. Y. Srinivasan Some fixes to the backup driver. Alex Ng (2): Drivers: hv: utils: Continue to poll VSS channel after handling requests. Drivers: hv: utils: Check VSS daemon is listening before a hot backup drivers/hv/hv_snapshot.c | 92

Re: [PATCH v2] android: binder: fix dangling pointer comparison

On Tue, Aug 16, 2016 at 07:44:59PM -0700, Arve Hjønnevåg wrote: > On Mon, Aug 15, 2016 at 7:58 AM, Greg Kroah-Hartman > wrote: > > On Thu, Jun 16, 2016 at 12:45:33AM +0200, Jann Horn wrote: > >> If /dev/binder is opened and the opener process then e.g. calls execve, >

Re: [PATCH 2/2] staging: most: aim-cdev: report error returned by alloc_chrdev_region

On Thu, Aug 18, 2016 at 02:05:27PM +0200, Christian Gromm wrote: > This patch passes the error code returned by function > alloc_chrdev_region(). It is needed to stop the module > from hiding the actual cause of failure. > > Signed-off-by: Christian Gromm > --- >

Re: [PATCH 1/2] staging: most: aim-cdev: fix reported error codes

On Thu, Aug 18, 2016 at 02:05:26PM +0200, Christian Gromm wrote: > This patch is needed to have the module report appropriate error codes. what do you mean "appropriate"? This patch description doesn't make sense, sorry, please fix up and resend the series. thanks, greg k-h

Re: [PATCH 0/8] staging: most: fix hdm-usb issues

On Tue, Aug 16, 2016 at 04:10:26PM +0200, Christian Gromm wrote: > This patch set is needed to fix issues of the hdm-usb module. You sent me 2 patch series for this same driver right now. One 9 patches and one 8 patches. Which go first? Why 2 different sets? What am I supposed to do, guess?

[PATCH 0/2] staging: most: fix issues of hdm-dim2

This patch set is needed to fix issues of module hdm-dim2 of the MOST driver. Andrey Shvetsov (2): dim2-hdm: initialize DBR memory of synchronous channels dim2-hdm: introduce function dim2_transfer_madr drivers/staging/most/hdm-dim2/dim2_hal.c | 34 ++-- 1 file

[PATCH 2/2] staging: most: dim2-hdm: introduce function dim2_transfer_madr

This patch removes duplicated code by putting it into the new function dim2_transfer_madr. Signed-off-by: Andrey Shvetsov Signed-off-by: Christian Gromm --- drivers/staging/most/hdm-dim2/dim2_hal.c | 38 +--- 1

[PATCH 1/2] staging: most: dim2-hdm: initialize DBR memory of synchronous channels

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: Andrey

Undelivered Mail Returned to Sender

This is the mail system at host localhost.localdomain. I'm sorry to have to inform you that your message could not be delivered to one or more recipients. It's attached below. For further assistance, please send mail to postmaster. If you do so, please include this problem report. You can

[PATCH] staging: most: aim-network: fix interrupt unsafe spinlocks

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 Signed-off-by: Christian

Re: [PATCH] staging/android: add Doc for SW_SYNC ioctl interface

On Thu 2016-08-11 13:45:53, Gustavo Padovan wrote: > From: Gustavo Padovan > > This interface is hidden from kernel headers and it is intended for use > only for testing. So testers would have to add the ioctl information > internally. This is to prevent misuse

[PATCH 2/2] staging: most: aim-cdev: report error returned by alloc_chrdev_region

This patch passes the error code returned by function alloc_chrdev_region(). It is needed to stop the module from hiding the actual cause of failure. Signed-off-by: Christian Gromm --- drivers/staging/most/aim-cdev/cdev.c | 5 +++-- 1 file changed, 3

[PATCH 1/2] staging: most: aim-cdev: fix reported error codes

This patch is needed to have the module report appropriate error codes. Signed-off-by: Christian Gromm --- drivers/staging/most/aim-cdev/cdev.c | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) diff --git

[PATCH 0/2] staging: most: aim-cdev: fix error handling

This patch set provides two fixes regarding the exception processing of module aim-cdev. Christian Gromm (2): staging: most: aim-cdev: fix reported error codes staging: most: aim-cdev: report error returned by alloc_chrdev_region drivers/staging/most/aim-cdev/cdev.c | 22