Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-07 Thread Balbir Singh
> How about running 2.6.24-rc4 with _only_ the patch to this driver to > convert it to krefs? Does that combination cause problems? > > The patch is at: > > http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/gregkh-01-driver/kobject-convert-hvcs-to-use-kref-not-kobject.patch

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-07 Thread Balbir Singh
How about running 2.6.24-rc4 with _only_ the patch to this driver to convert it to krefs? Does that combination cause problems? The patch is at: http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/gregkh-01-driver/kobject-convert-hvcs-to-use-kref-not-kobject.patch This

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Greg KH
On Fri, Dec 07, 2007 at 08:32:26AM +0530, Kamalesh Babulal wrote: > Greg KH wrote: > > On Thu, Dec 06, 2007 at 03:54:51PM -0800, Badari Pulavarty wrote: > >> On Thu, 2007-12-06 at 12:31 -0800, Greg KH wrote: > >>> On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: > Greg KH wrote:

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Kamalesh Babulal
Greg KH wrote: > On Thu, Dec 06, 2007 at 03:54:51PM -0800, Badari Pulavarty wrote: >> On Thu, 2007-12-06 at 12:31 -0800, Greg KH wrote: >>> On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: Greg KH wrote: >> Why release the spinlock here? It's done after the count is

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Balbir Singh
Badari Pulavarty wrote: > On Fri, 2007-12-07 at 00:28 +0530, Balbir Singh wrote: >> Greg KH wrote: >> Why release the spinlock here? It's done after the count is incremented. This patch does not seem correct. >>> Doh, you are correct, I'll make sure that I fix this up before applying >>>

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Greg KH
On Thu, Dec 06, 2007 at 03:54:51PM -0800, Badari Pulavarty wrote: > On Thu, 2007-12-06 at 12:31 -0800, Greg KH wrote: > > On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: > > > Greg KH wrote: > > > > > > >> Why release the spinlock here? It's done after the count is > > > >>

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Badari Pulavarty
On Thu, 2007-12-06 at 12:31 -0800, Greg KH wrote: > On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: > > Greg KH wrote: > > > > >> Why release the spinlock here? It's done after the count is incremented. > > >> This patch does not seem correct. > > > > > > Doh, you are correct, I'll

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Greg KH
On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: > Greg KH wrote: > > >> Why release the spinlock here? It's done after the count is incremented. > >> This patch does not seem correct. > > > > Doh, you are correct, I'll make sure that I fix this up before applying > > it. > > > >

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Greg KH
On Thu, Dec 06, 2007 at 11:49:59PM +0530, Balbir Singh wrote: > Kamalesh Babulal wrote: > > Hi Andrew, > > > > The 2.6.24-rc4-mm1 kernel build fails with build failure, > > > > CC drivers/char/hvcs.o > > drivers/char/hvcs.c: In function ‘hvcs_open’: > > drivers/char/hvcs.c:1180: error:

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Badari Pulavarty
On Fri, 2007-12-07 at 00:28 +0530, Balbir Singh wrote: > Greg KH wrote: > > >> Why release the spinlock here? It's done after the count is incremented. > >> This patch does not seem correct. > > > > Doh, you are correct, I'll make sure that I fix this up before applying > > it. > > > > thanks,

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Balbir Singh
Greg KH wrote: >> Why release the spinlock here? It's done after the count is incremented. >> This patch does not seem correct. > > Doh, you are correct, I'll make sure that I fix this up before applying > it. > > thanks, > > greg k-h Hi, Greg, I ran some tests with the fixed up version of

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Kamalesh Babulal
Greg KH wrote: > On Thu, Dec 06, 2007 at 11:49:59PM +0530, Balbir Singh wrote: >> Kamalesh Babulal wrote: >>> Hi Andrew, >>> >>> The 2.6.24-rc4-mm1 kernel build fails with build failure, >>> >>> CC drivers/char/hvcs.o >>> drivers/char/hvcs.c: In function ‘hvcs_open’: >>>

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Balbir Singh
Kamalesh Babulal wrote: > Hi Andrew, > > The 2.6.24-rc4-mm1 kernel build fails with build failure, > > CC drivers/char/hvcs.o > drivers/char/hvcs.c: In function ‘hvcs_open’: > drivers/char/hvcs.c:1180: error: wrong type argument to unary exclamation mark > make[2]: ***

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Balbir Singh
Kamalesh Babulal wrote: Hi Andrew, The 2.6.24-rc4-mm1 kernel build fails with build failure, CC drivers/char/hvcs.o drivers/char/hvcs.c: In function ‘hvcs_open’: drivers/char/hvcs.c:1180: error: wrong type argument to unary exclamation mark make[2]: *** [drivers/char/hvcs.o] Error

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Kamalesh Babulal
Greg KH wrote: On Thu, Dec 06, 2007 at 11:49:59PM +0530, Balbir Singh wrote: Kamalesh Babulal wrote: Hi Andrew, The 2.6.24-rc4-mm1 kernel build fails with build failure, CC drivers/char/hvcs.o drivers/char/hvcs.c: In function ‘hvcs_open’: drivers/char/hvcs.c:1180: error: wrong type

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Balbir Singh
Greg KH wrote: Why release the spinlock here? It's done after the count is incremented. This patch does not seem correct. Doh, you are correct, I'll make sure that I fix this up before applying it. thanks, greg k-h Hi, Greg, I ran some tests with the fixed up version of this patch

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Badari Pulavarty
On Fri, 2007-12-07 at 00:28 +0530, Balbir Singh wrote: Greg KH wrote: Why release the spinlock here? It's done after the count is incremented. This patch does not seem correct. Doh, you are correct, I'll make sure that I fix this up before applying it. thanks, greg k-h

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Greg KH
On Thu, Dec 06, 2007 at 11:49:59PM +0530, Balbir Singh wrote: Kamalesh Babulal wrote: Hi Andrew, The 2.6.24-rc4-mm1 kernel build fails with build failure, CC drivers/char/hvcs.o drivers/char/hvcs.c: In function ‘hvcs_open’: drivers/char/hvcs.c:1180: error: wrong type

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Greg KH
On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: Greg KH wrote: Why release the spinlock here? It's done after the count is incremented. This patch does not seem correct. Doh, you are correct, I'll make sure that I fix this up before applying it. thanks, greg

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Badari Pulavarty
On Thu, 2007-12-06 at 12:31 -0800, Greg KH wrote: On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: Greg KH wrote: Why release the spinlock here? It's done after the count is incremented. This patch does not seem correct. Doh, you are correct, I'll make sure that I

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Greg KH
On Thu, Dec 06, 2007 at 03:54:51PM -0800, Badari Pulavarty wrote: On Thu, 2007-12-06 at 12:31 -0800, Greg KH wrote: On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: Greg KH wrote: Why release the spinlock here? It's done after the count is incremented. This

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Balbir Singh
Badari Pulavarty wrote: On Fri, 2007-12-07 at 00:28 +0530, Balbir Singh wrote: Greg KH wrote: Why release the spinlock here? It's done after the count is incremented. This patch does not seem correct. Doh, you are correct, I'll make sure that I fix this up before applying it. thanks,

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Kamalesh Babulal
Greg KH wrote: On Thu, Dec 06, 2007 at 03:54:51PM -0800, Badari Pulavarty wrote: On Thu, 2007-12-06 at 12:31 -0800, Greg KH wrote: On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: Greg KH wrote: Why release the spinlock here? It's done after the count is incremented. This patch

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-06 Thread Greg KH
On Fri, Dec 07, 2007 at 08:32:26AM +0530, Kamalesh Babulal wrote: Greg KH wrote: On Thu, Dec 06, 2007 at 03:54:51PM -0800, Badari Pulavarty wrote: On Thu, 2007-12-06 at 12:31 -0800, Greg KH wrote: On Fri, Dec 07, 2007 at 12:28:58AM +0530, Balbir Singh wrote: Greg KH wrote: Why release

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-05 Thread Greg KH
On Wed, Dec 05, 2007 at 07:42:02PM +0530, Kamalesh Babulal wrote: > Hi Andrew, > > The 2.6.24-rc4-mm1 kernel build fails with build failure, > > CC drivers/char/hvcs.o > drivers/char/hvcs.c: In function ‘hvcs_open’: > drivers/char/hvcs.c:1180: error: wrong type argument to unary

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-05 Thread Kamalesh Babulal
Hi Andrew, The 2.6.24-rc4-mm1 kernel build fails with build failure, CC drivers/char/hvcs.o drivers/char/hvcs.c: In function ‘hvcs_open’: drivers/char/hvcs.c:1180: error: wrong type argument to unary exclamation mark make[2]: *** [drivers/char/hvcs.o] Error 1 make[1]: *** [drivers/char]

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-05 Thread Greg KH
On Wed, Dec 05, 2007 at 07:42:02PM +0530, Kamalesh Babulal wrote: Hi Andrew, The 2.6.24-rc4-mm1 kernel build fails with build failure, CC drivers/char/hvcs.o drivers/char/hvcs.c: In function ‘hvcs_open’: drivers/char/hvcs.c:1180: error: wrong type argument to unary exclamation mark

Re: 2.6.24-rc4-mm1 kobject changes broken with hvcs driver on powerpc

2007-12-05 Thread Kamalesh Babulal
Hi Andrew, The 2.6.24-rc4-mm1 kernel build fails with build failure, CC drivers/char/hvcs.o drivers/char/hvcs.c: In function ‘hvcs_open’: drivers/char/hvcs.c:1180: error: wrong type argument to unary exclamation mark make[2]: *** [drivers/char/hvcs.o] Error 1 make[1]: *** [drivers/char]