RE: [PATCH 05/10] staging: unisys: visorbus: Clarify reason for pointer check in bus_destroy()

2017-02-02 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Thursday, February 02, 2017 7:14 AM > To: Kershner, David A > Cc: driverdev-devel@linuxdriverproject.org; *S-Par-Maintainer > ; jes.soren...@gmail.com;

RE: [PATCH 00/13] staging: unisys: Remove POSTCODE macros

2016-12-01 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Thursday, December 01, 2016 2:34 AM > To: Kershner, David A > Cc: driverdev-devel@linuxdriverproject.org; *S-Par-Maintainer > ; jes.soren...@redhat.com >

RE: [PATCH v2] staging: unisys: visorbus: visorchannel: Refactor locking code to be statically deterministic.

2016-10-19 Thread Sell, Timothy C
> -Original Message- > From: Kershner, David A > Sent: Wednesday, October 19, 2016 8:04 PM > To: Cathal Mullaney > Subject: RE: [PATCH v2] staging: unisys: visorbus: visorchannel: Refactor > locking code to be statically deterministic. > > > -Original

RE: [PATCH] staging: unisys: visorbus: visorchannel: Refactor locking code to be statically deterministic.

2016-10-19 Thread Sell, Timothy C
On Wednesday, October 19, 2016 7:31 AM, Cathal Mullaney wrote: > This patch makes locking in visorchannel_signalempty statically deterministic. > As a result this patch fixes the sparse warning: > Context imbalance in 'visorchannel_signalempty' - different lock contexts for > basic block. > > The

RE: [PATCH 3/3] drivers: Add visorbus to the drivers/virt directory

2016-09-11 Thread Sell, Timothy C
On Sun, 11 Sep 2016 02:17:10 -0700, Greg KH wrote: > On Tue, Aug 30, 2016 at 04:29:07PM +0000, Sell, Timothy C wrote: >> E.g., so even though no obvious error-recovery occurs above in-response to >> kzalloc() failures, the fact that -CONTROLVM_RESP_ERROR_KMALLOC_FAILED is >> p

RE: [PATCH 3/3] drivers: Add visorbus to the drivers/virt directory

2016-08-30 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Sunday, August 21, 2016 2:05 PM > To: Kershner, David A > > On Fri, Jun 10, 2016 at 11:23:56PM -0400, David Kershner wrote: > > visorbus is currently located at

RE: [PATCH 3/3] drivers: Add visorbus to the drivers/virt directory

2016-08-22 Thread Sell, Timothy C
> -Original Message- > From: 'Greg KH' [mailto:gre...@linuxfoundation.org] > Sent: Monday, August 22, 2016 4:16 AM > To: Sell, Timothy C <timothy.s...@unisys.com> > Cc: cor...@lwn.net; t...@linutronix.de; mi...@redhat.com; > h...@zytor.com; Arfvidson, Erik &l

RE: [PATCH 3/3] drivers: Add visorbus to the drivers/virt directory

2016-08-21 Thread Sell, Timothy C
t;erik.arfvid...@unisys.com>; Sell, Timothy > C <timothy.s...@unisys.com>; hof...@osadl.org; dzic...@redhat.com; > jes.soren...@redhat.com; Curtin, Alexander Paul > <alexander.cur...@unisys.com>; janani.rvchn...@gmail.com; > sudipm.mukher...@gmail.com; pra...@redhat.com; B

RE: [PATCH] staging: unisys: visorbus: Replace semaphore with mutex

2016-06-20 Thread Sell, Timothy C
> -Original Message- > From: Binoy Jayan [mailto:binoy.ja...@linaro.org] > Sent: Monday, June 20, 2016 1:07 AM > To: Kershner, David A; Greg Kroah-Hartman > Cc: Arnd Bergmann; *S-Par-Maintainer; de...@driverdev.osuosl.org; linux- > ker...@vger.kernel.org; Binoy Jayan > Subject: [PATCH]

RE: [PATCH v4 09/29] staging: unisys: visorinput: remove unnecessary locking

2016-06-10 Thread Sell, Timothy C
> > > > From: Thomas Gleixner [mailto:t...@linutronix.de] > > > > > > > > I think I asked this before, but I might have missed the answer. > > > > > > > > Why is this a rw_sempahore? It's never taken with down_read and > > looking > > > > at the usage sites it's simply a mutex, right? > > > > > >

RE: [PATCH v4 09/29] staging: unisys: visorinput: remove unnecessary locking

2016-06-09 Thread Sell, Timothy C
> -Original Message- > From: Thomas Gleixner [mailto:t...@linutronix.de] > Sent: Thursday, June 09, 2016 3:56 PM > To: Sell, Timothy C > Cc: cor...@lwn.net; mi...@redhat.com; h...@zytor.com; > gre...@linuxfoundation.org; Arfvidson, Erik; hof...@osadl.org; > dzic...@

RE: [PATCH v4 09/29] staging: unisys: visorinput: remove unnecessary locking

2016-06-09 Thread Sell, Timothy C
> -Original Message- > From: Thomas Gleixner [mailto:t...@linutronix.de] > Sent: Thursday, June 09, 2016 5:01 AM > To: Kershner, David A > Cc: cor...@lwn.net; mi...@redhat.com; h...@zytor.com; > gre...@linuxfoundation.org; Arfvidson, Erik; Sell, Timothy C; > h

RE: [PATCH v3 11/30] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-07 Thread Sell, Timothy C
> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Tuesday, June 07, 2016 9:40 AM > To: Kershner, David A > Cc: cor...@lwn.net; t...@linutronix.de; mi...@redhat.com; > h...@zytor.com; gre...@linuxfoundation.org; Arfvidson, Erik; Sell, Timothy >

RE: [PATCH v2 10/27] staging: unisys: visorinput: remove unnecessary locking

2016-06-02 Thread Sell, Timothy C
> -Original Message- > From: Thomas Gleixner [mailto:t...@linutronix.de] > Sent: Wednesday, June 01, 2016 2:41 AM > To: Kershner, David A > Cc: cor...@lwn.net; mi...@redhat.com; h...@zytor.com; > gre...@linuxfoundation.org; Arfvidson, Erik; Sell, Timothy C; > h

RE: [PATCH v2 10/27] staging: unisys: visorinput: remove unnecessary locking

2016-06-01 Thread Sell, Timothy C
> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Wednesday, June 01, 2016 2:43 PM > To: Sell, Timothy C > Cc: Kershner, David A; cor...@lwn.net; t...@linutronix.de; > mi...@redhat.com; h...@zytor.com; gre...@linuxfoundation.org; >

RE: [PATCH v2 10/27] staging: unisys: visorinput: remove unnecessary locking

2016-06-01 Thread Sell, Timothy C
> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Wednesday, June 01, 2016 10:18 AM > To: Kershner, David A > Cc: cor...@lwn.net; t...@linutronix.de; mi...@redhat.com; > h...@zytor.com; gre...@linuxfoundation.org; Arfvidson, Erik; Sell

RE: new driver for drivers/virt/?

2016-05-19 Thread Sell, Timothy C
> -Original Message- > From: Sell, Timothy C > Sent: Wednesday, May 18, 2016 6:25 PM > To: 'Thomas Gleixner' > Cc: mi...@kernel.org; dave.han...@linux.intel.com; > ti...@freescale.com; ga...@kernel.crashing.org; Kershner, David A; > cor...@lwn.net; mi...@redha

RE: new driver for drivers/virt/?

2016-05-19 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Wednesday, May 18, 2016 6:25 PM > To: Thomas Gleixner > Cc: Sell, Timothy C; mi...@kernel.org; dave.han...@linux.intel.com; > ti...@freescale.com; ga...@kernel.crashing.org; Kershner, David A

RE: new driver for drivers/virt/?

2016-05-18 Thread Sell, Timothy C
> -Original Message- > From: Thomas Gleixner [mailto:t...@linutronix.de] > Sent: Wednesday, May 18, 2016 6:12 PM > To: Sell, Timothy C > Cc: mi...@kernel.org; dave.han...@linux.intel.com; > ti...@freescale.com; ga...@kernel.crashing.org; Kershner, David A; &g

new driver for drivers/virt/?

2016-05-18 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Wednesday, May 18, 2016 1:27 PM > To: Jes Sorensen > Cc: Kershner, David A; cor...@lwn.net; t...@linutronix.de; > mi...@redhat.com; h...@zytor.com; Arfvidson, Erik; Sell, Timothy C; > h

RE: Time for a code audit?

2016-05-02 Thread Sell, Timothy C
> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Monday, May 02, 2016 8:55 AM > To: Kershner, David A > Cc: gre...@linuxfoundation.org; driverdev-devel@linuxdriverproject.org; > Sell, Timothy C; *S-Par-Maintainer; Binder, David Ant

RE: Staging: unisys/verisonic: Correct double unlock

2016-04-05 Thread Sell, Timothy C
> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Tuesday, April 05, 2016 10:58 AM > To: Sell, Timothy C > Cc: Iban Rodriguez; Kershner, David A; Greg Kroah-Hartman; Benjamin > Romer; *S-Par-Maintainer; de...@driverdev.osuosl.org; linux- > k

RE: Staging: unisys/verisonic: Correct double unlock

2016-04-04 Thread Sell, Timothy C
> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Monday, April 04, 2016 10:35 AM > To: Sell, Timothy C > Cc: Iban Rodriguez; Kershner, David A; Greg Kroah-Hartman; Benjamin > Romer; *S-Par-Maintainer; de...@driverdev.osuosl.org; linux- > k

RE: Staging: unisys/verisonic: Correct double unlock

2016-04-02 Thread Sell, Timothy C
> -Original Message- > From: Iban Rodriguez [mailto:iban.rodrig...@ono.com] > Sent: Saturday, April 02, 2016 1:47 PM > To: Kershner, David A; Greg Kroah-Hartman; Benjamin Romer; Sell, Timothy > C; Neil Horman > Cc: *S-Par-Maintainer; de...@driverdev.osuosl

RE: [PATCH 3/6] staging: unisys: visorbus: git rid of gotos in devmajorminor_create_file

2016-03-09 Thread Sell, Timothy C
> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Wednesday, March 09, 2016 1:48 PM > To: Sell, Timothy C > Cc: external-ges-uni...@redhat.com; gre...@linuxfoundation.org; > driverdev-devel@linuxdriverproject.org; *S-Par-Maintainer > Su

RE: [PATCH 3/6] staging: unisys: visorbus: git rid of gotos in devmajorminor_create_file

2016-03-09 Thread Sell, Timothy C
> -Original Message- > From: Neil Horman [mailto:nhor...@redhat.com] > Sent: Wednesday, March 09, 2016 11:09 AM > To: external-ges-uni...@redhat.com > Cc: gre...@linuxfoundation.org; driverdev-devel@linuxdriverproject.org; > *S-Par-Maintainer > Subject: Re: [PATCH 3/6] staging: unisys:

RE: Time for a code audit?

2016-02-15 Thread Sell, Timothy C
ed be what is at the core of your argument against it. > -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Friday, February 12, 2016 5:02 PM > To: Romer, Benjamin M > Cc: gre...@linuxfoundation.org; driverdev-devel@linuxdriverproject.org;

RE: [PATCH v2 0/7] staging: unisys: visorinput fixes and enhancements

2015-11-17 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Tuesday, November 17, 2015 5:18 PM > To: Romer, Benjamin M > Cc: *S-Par-Maintainer; driverdev-devel@linuxdriverproject.org > Subject: Re: [PATCH v2 0/7] staging: unisys: visorinput fixes and > enhancements >

RE: [PATCH 3/9] staging: unisys: visorinput: use kref ref-counting for device data struct

2015-11-12 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Thursday, November 12, 2015 4:28 PM > To: Sell, Timothy C > Cc: Romer, Benjamin M; driverdev-devel@linuxdriverproject.org; *S-Par- > Maintainer > Subject: Re: [PATCH 3/9] staging: unisys:

RE: [PATCH 3/9] staging: unisys: visorinput: use kref ref-counting for device data struct

2015-11-12 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Thursday, November 12, 2015 4:12 PM > To: Sell, Timothy C > Cc: Romer, Benjamin M; driverdev-devel@linuxdriverproject.org; *S-Par- > Maintainer > Subject: Re: [PATCH 3/9] staging: unisys:

RE: [PATCH 3/9] staging: unisys: visorinput: use kref ref-counting for device data struct

2015-11-12 Thread Sell, Timothy C
hat removing "FB" was hard. ;-( Removing FB is at the crux of this patchset. Rarely a week passes where I don't get a complaint from the community about the fact that I need to add "INPUT" to that line to fix errors when building with randconfig. Thanks. Tim Sell > -Ori

RE: [PATCH 3/9] staging: unisys: visorinput: use kref ref-counting for device data struct

2015-10-17 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Saturday, October 17, 2015 12:51 PM > To: Sell, Timothy C > Cc: Romer, Benjamin M; driverdev-devel@linuxdriverproject.org; *S-Par- > Maintainer > Subject: Re: [PATCH 3/9] staging: unisys:

RE: [PATCH 3/9] staging: unisys: visorinput: use kref ref-counting for device data struct

2015-10-17 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Saturday, October 17, 2015 1:41 PM > To: Sell, Timothy C > Cc: Romer, Benjamin M; driverdev-devel@linuxdriverproject.org; *S-Par- > Maintainer > Subject: Re: [PATCH 3/9] staging: unisys:

RE: [PATCH 3/9] staging: unisys: visorinput: use kref ref-counting for device data struct

2015-10-17 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Saturday, October 17, 2015 1:59 AM > To: Romer, Benjamin M > Cc: *S-Par-Maintainer; driverdev-devel@linuxdriverproject.org; Sell, > Timothy C > Subject: Re: [PATCH 3/9] staging: unisys:

RE: [PATCH 3/9] staging: unisys: visorinput: use kref ref-counting for device data struct

2015-10-17 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Saturday, October 17, 2015 11:42 AM > To: Sell, Timothy C > Cc: Romer, Benjamin M; driverdev-devel@linuxdriverproject.org; *S-Par- > Maintainer > Subject: Re: [PATCH 3/9] staging: unisys:

RE: [PATCH 3/9] staging: unisys: visorinput: use kref ref-counting for device data struct

2015-10-16 Thread Sell, Timothy C
> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Friday, October 16, 2015 9:03 PM > To: Romer, Benjamin M > Cc: gre...@linuxfoundation.org; *S-Par-Maintainer; driverdev- > de...@linuxdriverproject.org; Sell, Timothy C > Subject: Re

RE: randconfig build error with next-20151013, in drivers/staging/unisys/visorinput

2015-10-13 Thread Sell, Timothy C
We have a patch in the pipeline to add "INPUT" to the "depends on" in drivers/staging/unisys/visorinput/Kconfig, which will fix this problem. Thanks Jim. - Tim Sell > -Original Message- > From: Jim Davis [mailto:jim.ep...@gmail.com] > Sent: Tuesday, October 13, 2015 1:34 PM > To:

RE: randconfig build error with next-20150923, in drivers/staging/unisys/visorhid

2015-09-23 Thread Sell, Timothy C
> -Original Message- > From: Jim Davis [mailto:jim.ep...@gmail.com] > Sent: Wednesday, September 23, 2015 1:29 PM > To: Stephen Rothwell; linux-next; linux-kernel; Greg Kroah-Hartman; > Romer, Benjamin M; Kershner, David A; *S-Par-Maintainer; devel > Subject: randconfig build error with

RE: [PATCH 2/6] staging: unisys: visornic - prevent lock recursion after IO recovery

2015-07-29 Thread Sell, Timothy C
-Original Message- From: Dan Carpenter [mailto:dan.carpen...@oracle.com] Sent: Wednesday, July 29, 2015 4:28 AM To: Romer, Benjamin M Cc: gre...@linuxfoundation.org; jes.soren...@redhat.com; *S-Par- Maintainer; driverdev-devel@linuxdriverproject.org; Sell, Timothy C Subject: Re

RE: [PATCH 5/5] staging: unisys: add error messages to visornic

2015-06-16 Thread Sell, Timothy C
-Original Message- From: Greg KH [mailto:gre...@linuxfoundation.org] Sent: Tuesday, June 16, 2015 5:36 PM To: Romer, Benjamin M Cc: driverdev-devel@linuxdriverproject.org; jes.soren...@redhat.com; *S- Par-Maintainer; Sell, Timothy C Subject: Re: [PATCH 5/5] staging: unisys: add