* Linux v2.6.18-rc1
@ 2006-07-06 4:26 Linus Torvalds
2006-07-06 9:44 ` Matt Keenan
` (6 more replies)
0 siblings, 7 replies; 36+ messages in thread
From: Linus Torvalds @ 2006-07-06 4:26 UTC (permalink / raw)
To: Linux Kernel Mailing List
Ok,
the merge window for 2.6.18 is closed, and -rc1 is out there (git trees
updated, the tar-ball and patches are still uploading over my pitiful DSL
line - and as usual it may take a short while before mirroring takes
place and distributes things across the globe).
The changes are too big for the mailing list, even just the shortlog. As
usual, lots of stuff happened. Most architectures got updated, ACPI
updates, networking, SCSI and sound, IDE, infiniband, input, DVB etc etc
etc.
There's also a fair amount of basic infrastructure updates here, with
things like the generic IRQ layer, the lockdep (oh, and priority-
inheritance mutex support) stuff by Ingo &co, some generic
timekeeping infrastructure ("clocksource") stuff, memory hotplug
(and page migration) support, etc etc.
Git users should generally just select the part they are interested in,
and do something like
git log v2.6.17.. -- drivers/usb/ | git shortlog | less -S
to get a better and more focused view of what has changed.
Have fun,
Linus
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 4:26 Linux v2.6.18-rc1 Linus Torvalds
@ 2006-07-06 9:44 ` Matt Keenan
2006-07-06 12:49 ` Jonathan Corbet
2006-07-06 16:22 ` Linus Torvalds
2006-07-06 12:34 ` Alistair John Strachan
` (5 subsequent siblings)
6 siblings, 2 replies; 36+ messages in thread
From: Matt Keenan @ 2006-07-06 9:44 UTC (permalink / raw)
To: Linus Torvalds; +Cc: LKML
Linus Torvalds wrote:
[snip snip]
> The changes are too big for the mailing list, even just the shortlog. As
> usual, lots of stuff happened. Most architectures got updated, ACPI
> updates, networking, SCSI and sound, IDE, infiniband, input, DVB etc etc
> etc.
>
>
[snip snip]
> Git users should generally just select the part they are interested in,
> and do something like
>
> git log v2.6.17.. -- drivers/usb/ | git shortlog | less -S
>
> to get a better and more focused view of what has changed.
>
[snip snip]
Is it possible to get a URL to a shortlog on a web git somewhere? Has
this information been posted before and I have missed it?
Matt
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 4:26 Linux v2.6.18-rc1 Linus Torvalds
2006-07-06 9:44 ` Matt Keenan
@ 2006-07-06 12:34 ` Alistair John Strachan
2006-07-07 0:11 ` Andrew Morton
2006-07-06 12:44 ` Nigel Cunningham
` (4 subsequent siblings)
6 siblings, 1 reply; 36+ messages in thread
From: Alistair John Strachan @ 2006-07-06 12:34 UTC (permalink / raw)
To: Linus Torvalds; +Cc: Linux Kernel Mailing List, linux-acpi, Len Brown
On Thursday 06 July 2006 05:26, Linus Torvalds wrote:
> Ok,
> the merge window for 2.6.18 is closed, and -rc1 is out there (git trees
> updated, the tar-ball and patches are still uploading over my pitiful DSL
> line - and as usual it may take a short while before mirroring takes
> place and distributes things across the globe).
>
> The changes are too big for the mailing list, even just the shortlog. As
> usual, lots of stuff happened. Most architectures got updated, ACPI
> updates, networking, SCSI and sound, IDE, infiniband, input, DVB etc etc
> etc.
ACPI problem here. Doesn't seem to actively break anything, but the messages
look bad (HP NC6000 notebook). Haven't tried suspending. The error popped
up roughly 90 minutes after booting. Laptop has been on AC power throughout.
ACPI Error (exmutex-0248): Cannot release Mutex [C0E8], not acquired [20060623]
ACPI Error (psparse-0537): Method parse/execution failed [\_SB_.C044.C057.C0E7.C12F] (Node c1aeca40), AE_AML_MUTEX_NOT_ACQUIRED
ACPI Error (psparse-0537): Method parse/execution failed [\_SB_.C12F] (Node c1aeecfc), AE_AML_MUTEX_NOT_ACQUIRED
ACPI Error (psparse-0537): Method parse/execution failed [\_SB_.C137._BST] (Node c1aeec84), AE_AML_MUTEX_NOT_ACQUIRED
ACPI Exception (acpi_battery-0206): AE_AML_MUTEX_NOT_ACQUIRED, Evaluating _BST [20060623]
--
Cheers,
Alistair.
Third year Computer Science undergraduate.
1F2 55 South Clerk Street, Edinburgh, UK.
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 4:26 Linux v2.6.18-rc1 Linus Torvalds
2006-07-06 9:44 ` Matt Keenan
2006-07-06 12:34 ` Alistair John Strachan
@ 2006-07-06 12:44 ` Nigel Cunningham
2006-07-06 19:44 ` David R
` (3 subsequent siblings)
6 siblings, 0 replies; 36+ messages in thread
From: Nigel Cunningham @ 2006-07-06 12:44 UTC (permalink / raw)
To: Linus Torvalds; +Cc: Linux Kernel Mailing List
[-- Attachment #1: Type: text/plain, Size: 811 bytes --]
Hi.
On Thursday 06 July 2006 14:26, Linus Torvalds wrote:
> Ok,
> the merge window for 2.6.18 is closed, and -rc1 is out there (git trees
> updated, the tar-ball and patches are still uploading over my pitiful DSL
> line - and as usual it may take a short while before mirroring takes
> place and distributes things across the globe).
On my amd64 based laptop, I had a failures with both suspend implementations
(swsusp and suspend2) when I had iommu debugging on. The same config worked
fine with 2.6.17. I guess something in the delta didn't like iommu usage
being forced on a Turion. I'm not sure how much time I'll find for further
debugging, so I'll report it now.
Regards,
Nigel
--
Nigel, Michelle and Alisdair Cunningham
5 Mitchell Street
Cobden 3266
Victoria, Australia
[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 9:44 ` Matt Keenan
@ 2006-07-06 12:49 ` Jonathan Corbet
2006-07-06 16:22 ` Linus Torvalds
1 sibling, 0 replies; 36+ messages in thread
From: Jonathan Corbet @ 2006-07-06 12:49 UTC (permalink / raw)
To: Matt Keenan; +Cc: linux-kernel
> Is it possible to get a URL to a shortlog on a web git somewhere? Has this
> information been posted before and I have missed it?
Dunno if there's a web git version anywhere, but I did put a copy of the
shortlog with the LWN announcement: http://lwn.net/Articles/190301/
jon
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 9:44 ` Matt Keenan
2006-07-06 12:49 ` Jonathan Corbet
@ 2006-07-06 16:22 ` Linus Torvalds
1 sibling, 0 replies; 36+ messages in thread
From: Linus Torvalds @ 2006-07-06 16:22 UTC (permalink / raw)
To: Matt Keenan; +Cc: LKML
On Thu, 6 Jul 2006, Matt Keenan wrote:
>
> > Git users should generally just select the part they are interested in, and
> > do something like
> >
> > git log v2.6.17.. -- drivers/usb/ | git shortlog | less -S
> >
> > to get a better and more focused view of what has changed.
> >
> [snip snip]
>
> Is it possible to get a URL to a shortlog on a web git somewhere? Has this
> information been posted before and I have missed it?
The most recent version of "gitweb" can actually do this kind of thing for
you on the web, but the version currently installed on kernel.org only
does it per-file, not per directory, making it less useful.
I suspect it will get upgraded one of these days, and when it does, you
can generate the above kind of listing without even having git installed
by going to
http://www.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git
and then selecting the "tree" view (it defaults to "summary" - see the top
line that says "summary"/"shortlog"/"log"/"commit"/"commitdiff"/"tree",
and select "tree" from there). Then you can walk down the tree, and when
you find the file (or, eventually, directory) you are interested in, click
on "history".
So even without git, you can get a lot of this kind of information.
However, with just the web interface, you do lose a lot of browsing
capability, like browsing multiple files/directories at the same time, and
doing things like limiting the output since just a particular version.
So you could just get git on your own, and read the docs, and play with
it. The place to start is probably the git homepage:
http://git.or.cz/
(also look up the Wiki there - click on "Wiki" in the header to see some
Wiki resources that probably get updated a bit more often than the home
page itself).
Linus
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 4:26 Linux v2.6.18-rc1 Linus Torvalds
` (2 preceding siblings ...)
2006-07-06 12:44 ` Nigel Cunningham
@ 2006-07-06 19:44 ` David R
2006-07-06 22:17 ` Greg KH
2006-07-07 1:05 ` Hervé Fache
2006-07-07 15:41 ` Steve Fox
` (2 subsequent siblings)
6 siblings, 2 replies; 36+ messages in thread
From: David R @ 2006-07-06 19:44 UTC (permalink / raw)
To: Linus Torvalds; +Cc: Linux Kernel Mailing List
[-- Attachment #1: Type: text/plain, Size: 954 bytes --]
Linus Torvalds wrote:
> Ok,
> the merge window for 2.6.18 is closed, and -rc1 is out there (git trees
Most things seem fine here with rc1, but I do see a permissions issue with my
USB scanner.
In 2.6.17
david@davidux:/dev/bus/usb/001 # l
total 0
drwxr-xr-x 2 root root 100 2006-07-06 20:19 ./
drwxr-xr-x 4 root root 80 2006-07-06 20:19 ../
crw-r--r-- 1 root root 189, 0 2006-07-06 20:19 001
crw-r--r-- 1 david root 189, 1 2006-07-06 20:19 002
crw-r--r-- 1 root root 189, 4 2006-07-06 20:19 005
but with 2.6.18
david@davidux:/dev/bus/usb/001> l
total 0
drwxr-xr-x 2 root root 100 2006-07-06 20:24 ./
drwxr-xr-x 4 root root 80 2006-07-06 20:24 ../
crw-r--r-- 1 root root 189, 0 2006-07-06 20:24 001
crw-r--r-- 1 root root 189, 1 2006-07-06 20:24 002
crw-r--r-- 1 root root 189, 4 2006-07-06 20:24 005
Does something need tweaking with udev scripts maybe? This is a SuSE 10.1 system.
Cheers
David
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 252 bytes --]
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 19:44 ` David R
@ 2006-07-06 22:17 ` Greg KH
2006-07-07 21:11 ` David R
2006-07-08 14:44 ` Andi Kleen
2006-07-07 1:05 ` Hervé Fache
1 sibling, 2 replies; 36+ messages in thread
From: Greg KH @ 2006-07-06 22:17 UTC (permalink / raw)
To: David R; +Cc: Linus Torvalds, Linux Kernel Mailing List
On Thu, Jul 06, 2006 at 08:44:11PM +0100, David R wrote:
> Linus Torvalds wrote:
> > Ok,
> > the merge window for 2.6.18 is closed, and -rc1 is out there (git trees
>
> Most things seem fine here with rc1, but I do see a permissions issue with my
> USB scanner.
>
> In 2.6.17
>
> david@davidux:/dev/bus/usb/001 # l
> total 0
> drwxr-xr-x 2 root root 100 2006-07-06 20:19 ./
> drwxr-xr-x 4 root root 80 2006-07-06 20:19 ../
> crw-r--r-- 1 root root 189, 0 2006-07-06 20:19 001
> crw-r--r-- 1 david root 189, 1 2006-07-06 20:19 002
> crw-r--r-- 1 root root 189, 4 2006-07-06 20:19 005
>
> but with 2.6.18
>
> david@davidux:/dev/bus/usb/001> l
> total 0
> drwxr-xr-x 2 root root 100 2006-07-06 20:24 ./
> drwxr-xr-x 4 root root 80 2006-07-06 20:24 ../
> crw-r--r-- 1 root root 189, 0 2006-07-06 20:24 001
> crw-r--r-- 1 root root 189, 1 2006-07-06 20:24 002
> crw-r--r-- 1 root root 189, 4 2006-07-06 20:24 005
>
> Does something need tweaking with udev scripts maybe? This is a SuSE 10.1 system.
Perhaps, that is odd. The scanner should default to the logged in user,
right? Please file a bug at bugzilla.novell.com and the SuSE people can
work on it there.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 12:34 ` Alistair John Strachan
@ 2006-07-07 0:11 ` Andrew Morton
0 siblings, 0 replies; 36+ messages in thread
From: Andrew Morton @ 2006-07-07 0:11 UTC (permalink / raw)
To: Alistair John Strachan; +Cc: torvalds, linux-kernel, linux-acpi, len.brown
Alistair John Strachan <s0348365@sms.ed.ac.uk> wrote:
>
> On Thursday 06 July 2006 05:26, Linus Torvalds wrote:
> > Ok,
> > the merge window for 2.6.18 is closed, and -rc1 is out there (git trees
> > updated, the tar-ball and patches are still uploading over my pitiful DSL
> > line - and as usual it may take a short while before mirroring takes
> > place and distributes things across the globe).
> >
> > The changes are too big for the mailing list, even just the shortlog. As
> > usual, lots of stuff happened. Most architectures got updated, ACPI
> > updates, networking, SCSI and sound, IDE, infiniband, input, DVB etc etc
> > etc.
>
> ACPI problem here. Doesn't seem to actively break anything, but the messages
> look bad (HP NC6000 notebook). Haven't tried suspending. The error popped
> up roughly 90 minutes after booting. Laptop has been on AC power throughout.
>
> ACPI Error (exmutex-0248): Cannot release Mutex [C0E8], not acquired [20060623]
> ACPI Error (psparse-0537): Method parse/execution failed [\_SB_.C044.C057.C0E7.C12F] (Node c1aeca40), AE_AML_MUTEX_NOT_ACQUIRED
> ACPI Error (psparse-0537): Method parse/execution failed [\_SB_.C12F] (Node c1aeecfc), AE_AML_MUTEX_NOT_ACQUIRED
> ACPI Error (psparse-0537): Method parse/execution failed [\_SB_.C137._BST] (Node c1aeec84), AE_AML_MUTEX_NOT_ACQUIRED
> ACPI Exception (acpi_battery-0206): AE_AML_MUTEX_NOT_ACQUIRED, Evaluating _BST [20060623]
>
I've queued the below for sending via Len. Whether it is
correct/sufficient I do not know. The comment from Alexey:
It seems that there is an error in our code, that either releases ASL
mutex twice, etc. In either case the correct behavior seems to not abort
execution. Error seems to vanish in 2.6.17, so we should just apply this
patch.
is discouraging.
From: Alexey Starikovskiy <alexey.y.starikovskiy@intel.com>
See http://bugme.osdl.org/show_bug.cgi?id=6687
Closes: #50088
patch location:
http://www.kernel.org/git/?p=linux/kernel/git/bcollins/ubuntu-dapper.git;a=commitdiff;h=202ddb5b6498af53e110f78edd41a217587c1ffb
Signed-off-by: Chuck Short <zulcss@gmail.com>
Signed-off-by: Ben Collins <bcollins@ubuntu.com>
Cc: "Brown, Len" <len.brown@intel.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
---
drivers/acpi/executer/exmutex.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff -puN drivers/acpi/executer/exmutex.c~acpi-do-not-abort-method-execution-if-asked-to-release drivers/acpi/executer/exmutex.c
--- a/drivers/acpi/executer/exmutex.c~acpi-do-not-abort-method-execution-if-asked-to-release
+++ a/drivers/acpi/executer/exmutex.c
@@ -246,7 +246,7 @@ acpi_ex_release_mutex(union acpi_operand
ACPI_ERROR((AE_INFO,
"Cannot release Mutex [%4.4s], not acquired",
acpi_ut_get_node_name(obj_desc->mutex.node)));
- return_ACPI_STATUS(AE_AML_MUTEX_NOT_ACQUIRED);
+ return_ACPI_STATUS(AE_OK);
}
/* Sanity check -- we must have a valid thread ID */
_
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 19:44 ` David R
2006-07-06 22:17 ` Greg KH
@ 2006-07-07 1:05 ` Hervé Fache
1 sibling, 0 replies; 36+ messages in thread
From: Hervé Fache @ 2006-07-07 1:05 UTC (permalink / raw)
To: linux-kernel
Why is your prompt different? # first, then >. Could it be that your
system thought you were root in the second case? Just a stupid
thought...
On 7/6/06, David R <david@unsolicited.net> wrote:
> Linus Torvalds wrote:
> > Ok,
> > the merge window for 2.6.18 is closed, and -rc1 is out there (git trees
>
> Most things seem fine here with rc1, but I do see a permissions issue with my
> USB scanner.
>
> In 2.6.17
>
> david@davidux:/dev/bus/usb/001 # l
> total 0
> drwxr-xr-x 2 root root 100 2006-07-06 20:19 ./
> drwxr-xr-x 4 root root 80 2006-07-06 20:19 ../
> crw-r--r-- 1 root root 189, 0 2006-07-06 20:19 001
> crw-r--r-- 1 david root 189, 1 2006-07-06 20:19 002
> crw-r--r-- 1 root root 189, 4 2006-07-06 20:19 005
>
> but with 2.6.18
>
> david@davidux:/dev/bus/usb/001> l
> total 0
> drwxr-xr-x 2 root root 100 2006-07-06 20:24 ./
> drwxr-xr-x 4 root root 80 2006-07-06 20:24 ../
> crw-r--r-- 1 root root 189, 0 2006-07-06 20:24 001
> crw-r--r-- 1 root root 189, 1 2006-07-06 20:24 002
> crw-r--r-- 1 root root 189, 4 2006-07-06 20:24 005
>
> Does something need tweaking with udev scripts maybe? This is a SuSE 10.1 system.
>
> Cheers
> David
>
>
>
>
--
In a world without walls and fences, who needs Windows and Gates?
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 4:26 Linux v2.6.18-rc1 Linus Torvalds
` (3 preceding siblings ...)
2006-07-06 19:44 ` David R
@ 2006-07-07 15:41 ` Steve Fox
2006-07-09 10:34 ` Benjamin Herrenschmidt
2006-07-07 17:52 ` lost cpufreq (Re: Linux v2.6.18-rc1) Tomasz Torcz
2006-07-08 14:09 ` [patchset 0/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors Jim Cromie
6 siblings, 1 reply; 36+ messages in thread
From: Steve Fox @ 2006-07-07 15:41 UTC (permalink / raw)
To: linux-kernel; +Cc: linuxppc-dev
We've got a ppc64 machine that won't boot with this due to an IDE error.
[snip]
Freeing unused kernel memory: 256k freed
running (1:2) /init autobench_args: ABAT:1152213829
creating device nodes .hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
--
Steve Fox
IBM Linux Technology Center
^ permalink raw reply [flat|nested] 36+ messages in thread
* lost cpufreq (Re: Linux v2.6.18-rc1)
2006-07-06 4:26 Linux v2.6.18-rc1 Linus Torvalds
` (4 preceding siblings ...)
2006-07-07 15:41 ` Steve Fox
@ 2006-07-07 17:52 ` Tomasz Torcz
2006-07-07 19:07 ` Dave Jones
2006-07-10 10:03 ` Takashi Iwai
2006-07-08 14:09 ` [patchset 0/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors Jim Cromie
6 siblings, 2 replies; 36+ messages in thread
From: Tomasz Torcz @ 2006-07-07 17:52 UTC (permalink / raw)
To: linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1174 bytes --]
On Wed, Jul 05, 2006 at 09:26:35PM -0700, Linus Torvalds wrote:
>
> Ok,
> the merge window for 2.6.18 is closed, and -rc1 is out there
... and cpufreq-nforce2.ko fails to work. Module can't be loaded:
FATAL: Error inserting cpufreq_nforce2
(/lib/modules/2.6.18-rc1/kernel/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.ko):
Device or resource busy
Here's relevant difference between dmesg of 2.6.17 and 2.6.18-rc1:
@@ -244,7 +240,6 @@
lp: driver loaded but no devices found
cpufreq: Detected nForce2 chipset revision C1
cpufreq: FSB changing is maybe unstable and can lead to crashes and data loss.
-cpufreq: FSB currently at 165 MHz, FID 10.5
usbcore: registered new driver usbfs
usbcore: registered new driver hub
Motherboard is Abit, NF7,NF7-V (nVidia-nForce2), Version: 2.X,1.0,
(according to dmidecode).
Also alsactl restore didn't restore PCM volume and mute state. Some
mixer elements changed name? Module is snd_intel8x0, chip is integrated
in above mentioned motherboard.
--
Tomasz Torcz "Funeral in the morning, IDE hacking
zdzichu@irc.-nie.spam-.pl in the afternoon and evening." - Alan Cox
[-- Attachment #2: Type: application/pgp-signature, Size: 229 bytes --]
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: lost cpufreq (Re: Linux v2.6.18-rc1)
2006-07-07 17:52 ` lost cpufreq (Re: Linux v2.6.18-rc1) Tomasz Torcz
@ 2006-07-07 19:07 ` Dave Jones
2006-07-07 20:27 ` Tomasz Torcz
2006-07-10 10:03 ` Takashi Iwai
1 sibling, 1 reply; 36+ messages in thread
From: Dave Jones @ 2006-07-07 19:07 UTC (permalink / raw)
To: linux-kernel; +Cc: se.witt
On Fri, Jul 07, 2006 at 07:52:39PM +0200, Tomasz Torcz wrote:
> On Wed, Jul 05, 2006 at 09:26:35PM -0700, Linus Torvalds wrote:
> >
> > Ok,
> > the merge window for 2.6.18 is closed, and -rc1 is out there
>
> ... and cpufreq-nforce2.ko fails to work. Module can't be loaded:
> FATAL: Error inserting cpufreq_nforce2
> (/lib/modules/2.6.18-rc1/kernel/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.ko):
> Device or resource busy
>
> Here's relevant difference between dmesg of 2.6.17 and 2.6.18-rc1:
>
> @@ -244,7 +240,6 @@
> lp: driver loaded but no devices found
> cpufreq: Detected nForce2 chipset revision C1
> cpufreq: FSB changing is maybe unstable and can lead to crashes and data loss.
> -cpufreq: FSB currently at 165 MHz, FID 10.5
> usbcore: registered new driver usbfs
> usbcore: registered new driver hub
Does it work again if you apply this patch with -R ?
Dave
diff --git a/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.c b/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.c
index f275e0d..0d49d73 100644
--- a/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.c
+++ b/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.c
@@ -90,7 +90,7 @@ static int nforce2_calc_pll(unsigned int
/* Try to calculate multiplier and divider up to 4 times */
while (((mul == 0) || (div == 0)) && (tried <= 3)) {
- for (xdiv = 1; xdiv <= 0x80; xdiv++)
+ for (xdiv = 2; xdiv <= 0x80; xdiv++)
for (xmul = 1; xmul <= 0xfe; xmul++)
if (nforce2_calc_fsb(NFORCE2_PLL(xmul, xdiv)) ==
fsb + tried) {
--
http://www.codemonkey.org.uk
^ permalink raw reply related [flat|nested] 36+ messages in thread
* Re: lost cpufreq (Re: Linux v2.6.18-rc1)
2006-07-07 19:07 ` Dave Jones
@ 2006-07-07 20:27 ` Tomasz Torcz
2006-07-07 20:35 ` Dave Jones
0 siblings, 1 reply; 36+ messages in thread
From: Tomasz Torcz @ 2006-07-07 20:27 UTC (permalink / raw)
To: Dave Jones, linux-kernel, se.witt
[-- Attachment #1: Type: text/plain, Size: 2175 bytes --]
On Fri, Jul 07, 2006 at 03:07:39PM -0400, Dave Jones wrote:
> On Fri, Jul 07, 2006 at 07:52:39PM +0200, Tomasz Torcz wrote:
> > On Wed, Jul 05, 2006 at 09:26:35PM -0700, Linus Torvalds wrote:
> > >
> > > Ok,
> > > the merge window for 2.6.18 is closed, and -rc1 is out there
> >
> > ... and cpufreq-nforce2.ko fails to work. Module can't be loaded:
> > FATAL: Error inserting cpufreq_nforce2
> > (/lib/modules/2.6.18-rc1/kernel/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.ko):
> > Device or resource busy
> >
> > Here's relevant difference between dmesg of 2.6.17 and 2.6.18-rc1:
> >
> > @@ -244,7 +240,6 @@
> > lp: driver loaded but no devices found
> > cpufreq: Detected nForce2 chipset revision C1
> > cpufreq: FSB changing is maybe unstable and can lead to crashes and data loss.
> > -cpufreq: FSB currently at 165 MHz, FID 10.5
> > usbcore: registered new driver usbfs
> > usbcore: registered new driver hub
>
> Does it work again if you apply this patch with -R ?
No.
>
>
> diff --git a/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.c b/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.c
> index f275e0d..0d49d73 100644
> --- a/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.c
> +++ b/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.c
> @@ -90,7 +90,7 @@ static int nforce2_calc_pll(unsigned int
>
> /* Try to calculate multiplier and divider up to 4 times */
> while (((mul == 0) || (div == 0)) && (tried <= 3)) {
> - for (xdiv = 1; xdiv <= 0x80; xdiv++)
> + for (xdiv = 2; xdiv <= 0x80; xdiv++)
> for (xmul = 1; xmul <= 0xfe; xmul++)
> if (nforce2_calc_fsb(NFORCE2_PLL(xmul, xdiv)) ==
> fsb + tried) {
>
>
> --
> http://www.codemonkey.org.uk
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
--
Tomasz Torcz Morality must always be based on practicality.
zdzichu@irc.-nie.spam-.pl -- Baron Vladimir Harkonnen
[-- Attachment #2: Type: application/pgp-signature, Size: 229 bytes --]
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: lost cpufreq (Re: Linux v2.6.18-rc1)
2006-07-07 20:27 ` Tomasz Torcz
@ 2006-07-07 20:35 ` Dave Jones
0 siblings, 0 replies; 36+ messages in thread
From: Dave Jones @ 2006-07-07 20:35 UTC (permalink / raw)
To: linux-kernel, se.witt
On Fri, Jul 07, 2006 at 10:27:06PM +0200, Tomasz Torcz wrote:
> On Fri, Jul 07, 2006 at 03:07:39PM -0400, Dave Jones wrote:
> > On Fri, Jul 07, 2006 at 07:52:39PM +0200, Tomasz Torcz wrote:
> > > On Wed, Jul 05, 2006 at 09:26:35PM -0700, Linus Torvalds wrote:
> > > >
> > > > Ok,
> > > > the merge window for 2.6.18 is closed, and -rc1 is out there
> > >
> > > ... and cpufreq-nforce2.ko fails to work. Module can't be loaded:
> > > FATAL: Error inserting cpufreq_nforce2
> > > (/lib/modules/2.6.18-rc1/kernel/arch/i386/kernel/cpu/cpufreq/cpufreq-nforce2.ko):
> > > Device or resource busy
> > >
> > > Here's relevant difference between dmesg of 2.6.17 and 2.6.18-rc1:
> > >
> > > @@ -244,7 +240,6 @@
> > > lp: driver loaded but no devices found
> > > cpufreq: Detected nForce2 chipset revision C1
> > > cpufreq: FSB changing is maybe unstable and can lead to crashes and data loss.
> > > -cpufreq: FSB currently at 165 MHz, FID 10.5
> > > usbcore: registered new driver usbfs
> > > usbcore: registered new driver hub
> >
> > Does it work again if you apply this patch with -R ?
>
> No.
That's puzzling, as the only other changes to cpufreq-nforce are completely cosmetic
(whitespace and the like).
Dave
--
http://www.codemonkey.org.uk
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 22:17 ` Greg KH
@ 2006-07-07 21:11 ` David R
2006-07-08 8:14 ` Borislav Petkov
2006-07-08 14:19 ` Kay Sievers
2006-07-08 14:44 ` Andi Kleen
1 sibling, 2 replies; 36+ messages in thread
From: David R @ 2006-07-07 21:11 UTC (permalink / raw)
To: Greg KH; +Cc: Linus Torvalds, Linux Kernel Mailing List
[-- Attachment #1: Type: text/plain, Size: 307 bytes --]
Greg KH wrote:
> right? Please file a bug at bugzilla.novell.com and the SuSE people can
Done. I may also try to chase down any divergence in the udev/hal scripts the
weekend. Not a massive deal anyhow, I can always chown the device if I need
the scanner. It all works just fine.
Thanks
David
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 252 bytes --]
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-07 21:11 ` David R
@ 2006-07-08 8:14 ` Borislav Petkov
2006-07-08 14:19 ` Kay Sievers
1 sibling, 0 replies; 36+ messages in thread
From: Borislav Petkov @ 2006-07-08 8:14 UTC (permalink / raw)
To: David R; +Cc: Greg KH, Linus Torvalds, Linux Kernel Mailing List
On Fri, Jul 07, 2006 at 10:11:20PM +0100, David R wrote:
> Greg KH wrote:
> > right? Please file a bug at bugzilla.novell.com and the SuSE people can
>
> Done. I may also try to chase down any divergence in the udev/hal scripts the
> weekend. Not a massive deal anyhow, I can always chown the device if I need
> the scanner. It all works just fine.
We had the same problem with openGL applications here at the university. Suse 10
uses a daemon called resmgr that changes the /dev files ownership to the current user
who logs in. check /etc/resmgr.conf.d/*, /etc/resmgr.conf and the docs for
resmgr on how to add yourself to the, I think is the "desktop" group in resmgr
terms, in order to change ownership.
Regards,
Boris.
___________________________________________________________
Telefonate ohne weitere Kosten vom PC zum PC: http://messenger.yahoo.de
^ permalink raw reply [flat|nested] 36+ messages in thread
* [patchset 0/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors
2006-07-06 4:26 Linux v2.6.18-rc1 Linus Torvalds
` (5 preceding siblings ...)
2006-07-07 17:52 ` lost cpufreq (Re: Linux v2.6.18-rc1) Tomasz Torcz
@ 2006-07-08 14:09 ` Jim Cromie
2006-07-08 14:16 ` [patchset 1/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - define and use constants Jim Cromie
` (2 more replies)
6 siblings, 3 replies; 36+ messages in thread
From: Jim Cromie @ 2006-07-08 14:09 UTC (permalink / raw)
To: Linus Torvalds; +Cc: Linux Kernel Mailing List, Andrew Morton
Linus Torvalds wrote:
> Ok,
> the merge window for 2.6.18 is closed, and -rc1 is out there (git trees
>
The new pc8736x_gpio driver has at least 1 bug, which manifests when
module is rmmod'd then re-modprobe'd. This 3 part patchset addresses it
as follows:
1 - add constant defines - preparatory patch
- adds #define CONSTs for max-pin, gpio-addr-range (for reserving region)
- fix wrong max-pin check in gpio_open()
- add 'Winbond' to module description: NSC sold the product, Winbond
has us / lm-sensors
pc-const-decl - define several symbolic constants
pc-open-const - use reduced max-pin to what hardware supports
pc-req-region-const - can wait, but why
pc-mod-desc-add-winbond -
since this is just preparatory work, it still exhibits the bug:
soekris:~# rmmod pc8736x_gpio
soekris:~# modprobe pc8736x_gpio
[ 498.970735] kobject_add failed for pc8736x_gpio.0 with -EEXIST, don't
try to register things with the same name in the same directory.
[ 498.985568] [<c0102fbf>] show_trace_log_lvl+0x54/0xfd
[ 498.991398] [<c0103fba>] show_trace+0xd/0x10
[ 498.996629] [<c0103fd4>] dump_stack+0x17/0x1b
[ 499.001759] [<c01d1c6a>] kobject_add+0x17d/0x19c
[ 499.014898] [<c0203362>] device_add+0x7a/0x2cc
[ 499.029917] [<c020616d>] platform_device_add+0xdc/0x10c
[ 499.045902] [<c8820028>] pc8736x_gpio_init+0x28/0x258 [pc8736x_gpio]
[ 499.053212] [<c012a67a>] sys_init_module+0x1381/0x1463
[ 499.061327] [<c0102657>] syscall_call+0x7/0xb
FATAL: Error inserting pc8736x_gpio
(/lib/modules/2.6.18-rc1-jimc-sk/kernel/drivers/char/pc8736x_gpio.ko):
No such device
2 - pc-init-fix-undo-region-rollup
this patch fixes module-init-func by repairing usage of
platform_device_del/put in module-exit-func,
ie it imitates Ingo's 'mishaps' patch, which fixed the
module-init-func's undo handling.
Also fixes lack of release_region to undo the earlier registration.
Also included:
- add include <linux/cdev.h> - needed by next patches
and cuz theyre within the diff-context-window:
- remove include <linux/config.h> everyone's doing it
- copyright updates - current date is 'wrong'
the patch fixes the above bug, reducing it to a different one; (which is
fixed by 3rd patch)
soekris:~# rmmod pc8736x_gpio
soekris:~# modprobe pc8736x_gpio
[ 283.618997] platform pc8736x_gpio.0: NatSemi pc8736x GPIO Driver
Initializing
[ 283.626534] platform pc8736x_gpio.0: GPIO ioport 6600 reserved
[ 283.633319] platform pc8736x_gpio.0: register-chrdev failed: -16
FATAL: Error inserting pc8736x_gpio
(/lib/modules/2.6.18-rc1-jimc-sk/kernel/drivers/char/pc8736x_gpio.ko):
Device or resource busy
3 - pc-init-fix-chrdev-region-rollup
- Switch from register_chrdev() to (register|alloc)_chrdev_region().
- use a cdev. This was intended for original patchset, but was overlooked.
We use a single cdev for all pins (minor device-numbers), as gleaned
from cs5535_gpio,
and in contrast to whats currently done in scx200_gpio (which I'll fix soon)
By some reasoning, this patch should have been combined with patch 2;
the fixes
are to the same 2 mod-init/exit functions, and both touch cdevs,
regions, etc.
Despite this, I separated them (incompletely). I'll combine them if
thats better..
Given that this is a brand new driver, can I assume a bit more latitude to
clean up and normalize the code ? For example, s/DEVNAME/DRVNAME/
would improve consistency with other drivers. The scx200_gpio driver
could also
stand some improvements; a single cdev, and same DRVNAME normalization.
Whats the right way to mark this as (NEW/EXPERIMENTAL) ?
Is either appropriate for a reworked / existing driver (scx200_gpio)
cc'g AKPM since (I think) he pushed this driver to mainline.
thanks
Jim Cromie
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [patchset 1/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - define and use constants
2006-07-08 14:09 ` [patchset 0/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors Jim Cromie
@ 2006-07-08 14:16 ` Jim Cromie
2006-07-08 14:29 ` [patchset 2/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - undo region reservation Jim Cromie
2006-07-08 14:34 ` [patchset 3/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - fix/finish cdev-init Jim Cromie
2 siblings, 0 replies; 36+ messages in thread
From: Jim Cromie @ 2006-07-08 14:16 UTC (permalink / raw)
To: Linus Torvalds; +Cc: Linux Kernel Mailing List, Andrew Morton
1 - add constant defines - preparatory patch
- adds #define CONSTs for max-pin, gpio-addr-range (for reserving region)
- fix wrong max-pin check in gpio_open()
- add 'Winbond' to module description. NSC sold the product, Winbond
has supported us / lm-sensors
Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
$ diffstat fxd1/pc-const-rollup
pc8736x_gpio.c | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
diff -ruNp -X dontdiff -X exclude-diffs x4h-0/drivers/char/pc8736x_gpio.c x4h-1/drivers/char/pc8736x_gpio.c
--- x4h-0/drivers/char/pc8736x_gpio.c 2006-07-06 13:19:16.000000000 -0600
+++ x4h-1/drivers/char/pc8736x_gpio.c 2006-07-07 16:59:14.000000000 -0600
@@ -25,7 +25,7 @@
#define DEVNAME "pc8736x_gpio"
MODULE_AUTHOR("Jim Cromie <jim.cromie@gmail.com>");
-MODULE_DESCRIPTION("NatSemi PC-8736x GPIO Pin Driver");
+MODULE_DESCRIPTION("NatSemi/Winbond PC-8736x GPIO Pin Driver");
MODULE_LICENSE("GPL");
static int major; /* default to dynamic major */
@@ -38,14 +38,14 @@ static u8 pc8736x_gpio_shadow[4];
#define SIO_BASE1 0x2E /* 1st command-reg to check */
#define SIO_BASE2 0x4E /* alt command-reg to check */
-#define SIO_BASE_OFFSET 0x20
#define SIO_SID 0x20 /* SuperI/O ID Register */
#define SIO_SID_VALUE 0xe9 /* Expected value in SuperI/O ID Register */
#define SIO_CF1 0x21 /* chip config, bit0 is chip enable */
-#define PC8736X_GPIO_SIZE 16
+#define PC8736X_GPIO_RANGE 16 /* ioaddr range */
+#define PC8736X_GPIO_CT 32 /* minors matching 4 8 bit ports */
#define SIO_UNIT_SEL 0x7 /* unit select reg */
#define SIO_UNIT_ACT 0x30 /* unit enable */
@@ -231,7 +231,7 @@ static int pc8736x_gpio_open(struct inod
dev_dbg(&pdev->dev, "open %d\n", m);
- if (m > 63)
+ if (m >= PC8736X_GPIO_CT)
return -EINVAL;
return nonseekable_open(inode, file);
}
@@ -297,7 +297,7 @@ static int __init pc8736x_gpio_init(void
pc8736x_gpio_base = (superio_inb(SIO_BASE_HADDR) << 8
| superio_inb(SIO_BASE_LADDR));
- if (!request_region(pc8736x_gpio_base, 16, DEVNAME)) {
+ if (!request_region(pc8736x_gpio_base, PC8736X_GPIO_RANGE, DEVNAME)) {
rc = -ENODEV;
dev_err(&pdev->dev, "GPIO ioport %x busy\n",
pc8736x_gpio_base);
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-07 21:11 ` David R
2006-07-08 8:14 ` Borislav Petkov
@ 2006-07-08 14:19 ` Kay Sievers
2006-07-08 14:44 ` Andi Kleen
1 sibling, 1 reply; 36+ messages in thread
From: Kay Sievers @ 2006-07-08 14:19 UTC (permalink / raw)
To: David R; +Cc: Greg KH, Linus Torvalds, Linux Kernel Mailing List
On Fri, 2006-07-07 at 22:11 +0100, David R wrote:
> Greg KH wrote:
> > right? Please file a bug at bugzilla.novell.com and the SuSE people can
>
> Done. I may also try to chase down any divergence in the udev/hal scripts the
> weekend. Not a massive deal anyhow, I can always chown the device if I need
> the scanner. It all works just fine.
It's a bug in HAL, that feeds resmgr to grand access to the local user.
It should be fixed soon.
Thanks,
Kay
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [patchset 2/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - undo region reservation
2006-07-08 14:09 ` [patchset 0/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors Jim Cromie
2006-07-08 14:16 ` [patchset 1/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - define and use constants Jim Cromie
@ 2006-07-08 14:29 ` Jim Cromie
2006-07-08 14:34 ` [patchset 3/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - fix/finish cdev-init Jim Cromie
2 siblings, 0 replies; 36+ messages in thread
From: Jim Cromie @ 2006-07-08 14:29 UTC (permalink / raw)
To: Linus Torvalds; +Cc: Linux Kernel Mailing List, Andrew Morton
2 - pc-init-fix-undo-region-rollup
this patch fixes module-init-func by repairing usage of
platform_device_del/put in module-exit-func.
IOW, it imitates Ingo's 'mishaps' patch, which fixed the
module-init-func's undo handling.
Also fixes lack of release_region to undo the earlier registration.
Also starts to 'use a cdev' which was originally intended (its present
in scx200_gpio).
Code compiles and runs, exhibits a lesser error than previously.
(re-register-chrdev fails)
Since I had to add "include <linux/cdev.h>", I went ahead and made 2
tweaks that fell into diff-context-window:
- remove include <linux/config.h> everyone's doing it
- copyright updates - current date is 'wrong'
Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
---
$ diffstat fxd1/pc-init-fix-undo-region-rollup
pc8736x_gpio.c | 19 +++++++++++++------
1 files changed, 13 insertions(+), 6 deletions(-)
[jimc@harpo gpio-stuff]$
---
diff -ruNp -X dontdiff -X exclude-diffs x4h-1/drivers/char/pc8736x_gpio.c x4h-2/drivers/char/pc8736x_gpio.c
--- x4h-1/drivers/char/pc8736x_gpio.c 2006-07-07 16:59:14.000000000 -0600
+++ x4h-2/drivers/char/pc8736x_gpio.c 2006-07-07 17:02:17.000000000 -0600
@@ -3,18 +3,18 @@
National Semiconductor PC8736x GPIO driver. Allows a user space
process to play with the GPIO pins.
- Copyright (c) 2005 Jim Cromie <jim.cromie@gmail.com>
+ Copyright (c) 2005,2006 Jim Cromie <jim.cromie@gmail.com>
adapted from linux/drivers/char/scx200_gpio.c
Copyright (c) 2001,2002 Christer Weinigel <wingel@nano-system.com>,
*/
-#include <linux/config.h>
#include <linux/fs.h>
#include <linux/module.h>
#include <linux/errno.h>
#include <linux/kernel.h>
#include <linux/init.h>
+#include <linux/cdev.h>
#include <linux/io.h>
#include <linux/ioport.h>
#include <linux/mutex.h>
@@ -255,6 +255,8 @@ static void __init pc8736x_init_shadow(v
}
+static struct cdev pc8736x_gpio_cdev;
+
static int __init pc8736x_gpio_init(void)
{
int rc = 0;
@@ -308,7 +310,7 @@ static int __init pc8736x_gpio_init(void
rc = register_chrdev(major, DEVNAME, &pc8736x_gpio_fops);
if (rc < 0) {
dev_err(&pdev->dev, "register-chrdev failed: %d\n", rc);
- goto undo_platform_dev_add;
+ goto undo_request_region;
}
if (!major) {
major = rc;
@@ -318,6 +320,8 @@ static int __init pc8736x_gpio_init(void
pc8736x_init_shadow();
return 0;
+undo_request_region:
+ release_region(pc8736x_gpio_base, PC8736X_GPIO_RANGE);
undo_platform_dev_add:
platform_device_del(pdev);
undo_platform_dev_alloc:
@@ -328,11 +332,14 @@ undo_platform_dev_alloc:
static void __exit pc8736x_gpio_cleanup(void)
{
- dev_dbg(&pdev->dev, " cleanup\n");
+ dev_dbg(&pdev->dev, "cleanup\n");
- release_region(pc8736x_gpio_base, 16);
+ cdev_del(&pc8736x_gpio_cdev);
+ unregister_chrdev_region(MKDEV(major,0), PC8736X_GPIO_CT);
+ release_region(pc8736x_gpio_base, PC8736X_GPIO_RANGE);
- unregister_chrdev(major, DEVNAME);
+ platform_device_del(pdev);
+ platform_device_put(pdev);
}
EXPORT_SYMBOL(pc8736x_access);
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: [patchset 3/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - fix/finish cdev-init
2006-07-08 14:09 ` [patchset 0/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors Jim Cromie
2006-07-08 14:16 ` [patchset 1/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - define and use constants Jim Cromie
2006-07-08 14:29 ` [patchset 2/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - undo region reservation Jim Cromie
@ 2006-07-08 14:34 ` Jim Cromie
2 siblings, 0 replies; 36+ messages in thread
From: Jim Cromie @ 2006-07-08 14:34 UTC (permalink / raw)
To: Linus Torvalds; +Cc: Linux Kernel Mailing List, Andrew Morton
3 - pc-init-fix-chrdev-region-rollup
- Switch from register_chrdev() to (register|alloc)_chrdev_region().
- use a cdev. This was intended for original patchset, but was
overlooked.
We use a single cdev for all pins (minor device-numbers), as gleaned
from cs5535_gpio,
and in contrast to whats currently done in scx200_gpio (which I'll fix
soon)
Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
$ diffstat pc-init-fix-chrdev-region-rollup
pc8736x_gpio.c | 15 ++++++++++++++-
1 files changed, 14 insertions(+), 1 deletion(-)
---
diff -ruNp -X dontdiff -X exclude-diffs x4k-2/drivers/char/pc8736x_gpio.c x4k-3/drivers/char/pc8736x_gpio.c
--- x4k-2/drivers/char/pc8736x_gpio.c 2006-07-07 20:09:38.000000000 -0600
+++ x4k-3/drivers/char/pc8736x_gpio.c 2006-07-07 20:12:05.000000000 -0600
@@ -260,6 +260,7 @@ static struct cdev pc8736x_gpio_cdev;
static int __init pc8736x_gpio_init(void)
{
int rc = 0;
+ dev_t devid;
pdev = platform_device_alloc(DEVNAME, 0);
if (!pdev)
@@ -307,7 +308,14 @@ static int __init pc8736x_gpio_init(void
}
dev_info(&pdev->dev, "GPIO ioport %x reserved\n", pc8736x_gpio_base);
- rc = register_chrdev(major, DEVNAME, &pc8736x_gpio_fops);
+ if (major) {
+ devid = MKDEV(major, 0);
+ rc = register_chrdev_region(devid, PC8736X_GPIO_CT, DEVNAME);
+ } else {
+ rc = alloc_chrdev_region(&devid, 0, PC8736X_GPIO_CT, DEVNAME);
+ major = MAJOR(devid);
+ }
+
if (rc < 0) {
dev_err(&pdev->dev, "register-chrdev failed: %d\n", rc);
goto undo_request_region;
@@ -318,6 +326,11 @@ static int __init pc8736x_gpio_init(void
}
pc8736x_init_shadow();
+
+ /* ignore minor errs, and succeed */
+ cdev_init(&pc8736x_gpio_cdev, &pc8736x_gpio_fops);
+ cdev_add(&pc8736x_gpio_cdev, devid, PC8736X_GPIO_CT);
+
return 0;
undo_request_region:
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-06 22:17 ` Greg KH
2006-07-07 21:11 ` David R
@ 2006-07-08 14:44 ` Andi Kleen
2006-07-08 16:02 ` Greg KH
1 sibling, 1 reply; 36+ messages in thread
From: Andi Kleen @ 2006-07-08 14:44 UTC (permalink / raw)
To: Greg KH; +Cc: Linus Torvalds, Linux Kernel Mailing List
Greg KH <greg@kroah.com> writes:
>
> Perhaps, that is odd. The scanner should default to the logged in user,
> right? Please file a bug at bugzilla.novell.com and the SuSE people can
> work on it there.
I have a similar problem with my printer. But /dev/usblp0,
/dev/usb/lp0 don't even appear, no matter what the permissions are.
-Andi
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-08 14:19 ` Kay Sievers
@ 2006-07-08 14:44 ` Andi Kleen
0 siblings, 0 replies; 36+ messages in thread
From: Andi Kleen @ 2006-07-08 14:44 UTC (permalink / raw)
To: Kay Sievers; +Cc: Greg KH, Linus Torvalds, Linux Kernel Mailing List
Kay Sievers <kay.sievers@vrfy.org> writes:
> On Fri, 2006-07-07 at 22:11 +0100, David R wrote:
> > Greg KH wrote:
> > > right? Please file a bug at bugzilla.novell.com and the SuSE people can
> >
> > Done. I may also try to chase down any divergence in the udev/hal scripts the
> > weekend. Not a massive deal anyhow, I can always chown the device if I need
> > the scanner. It all works just fine.
>
> It's a bug in HAL, that feeds resmgr to grand access to the local user.
> It should be fixed soon.
I hope you mean in the kernel or did user space compatibility get
broken again?
-Andi
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-08 14:44 ` Andi Kleen
@ 2006-07-08 16:02 ` Greg KH
2006-07-09 23:17 ` Andi Kleen
0 siblings, 1 reply; 36+ messages in thread
From: Greg KH @ 2006-07-08 16:02 UTC (permalink / raw)
To: Andi Kleen; +Cc: Linus Torvalds, Linux Kernel Mailing List
On Sat, Jul 08, 2006 at 04:44:08PM +0200, Andi Kleen wrote:
> Greg KH <greg@kroah.com> writes:
> >
> > Perhaps, that is odd. The scanner should default to the logged in user,
> > right? Please file a bug at bugzilla.novell.com and the SuSE people can
> > work on it there.
>
> I have a similar problem with my printer. But /dev/usblp0,
> /dev/usb/lp0 don't even appear, no matter what the permissions are.
What version of udev are you using? It works fine for me here with a
USB printer (that's what I tested the changes with.)
thanks,
greg k-h
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-07 15:41 ` Steve Fox
@ 2006-07-09 10:34 ` Benjamin Herrenschmidt
2006-07-10 13:21 ` Will Schmidt
2006-07-10 16:38 ` Steve Fox
0 siblings, 2 replies; 36+ messages in thread
From: Benjamin Herrenschmidt @ 2006-07-09 10:34 UTC (permalink / raw)
To: Steve Fox; +Cc: linuxppc-dev, linux-kernel
On Fri, 2006-07-07 at 10:41 -0500, Steve Fox wrote:
> We've got a ppc64 machine that won't boot with this due to an IDE error.
What machine precisely ?
> [snip]
> Freeing unused kernel memory: 256k freed
> running (1:2) /init autobench_args: ABAT:1152213829
>
> creating device nodes .hda: lost interrupt
> hda: lost interrupt
> hda: lost interrupt
> hda: lost interrupt
> hda: lost interrupt
> hda: lost interrupt
> hda: lost interrupt
> hda: lost interrupt
> hda: lost interrupt
>
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-08 16:02 ` Greg KH
@ 2006-07-09 23:17 ` Andi Kleen
2006-07-09 23:36 ` Greg KH
0 siblings, 1 reply; 36+ messages in thread
From: Andi Kleen @ 2006-07-09 23:17 UTC (permalink / raw)
To: Greg KH; +Cc: Linus Torvalds, Linux Kernel Mailing List
On Saturday 08 July 2006 18:02, Greg KH wrote:
> On Sat, Jul 08, 2006 at 04:44:08PM +0200, Andi Kleen wrote:
> > Greg KH <greg@kroah.com> writes:
> > >
> > > Perhaps, that is odd. The scanner should default to the logged in user,
> > > right? Please file a bug at bugzilla.novell.com and the SuSE people can
> > > work on it there.
> >
> > I have a similar problem with my printer. But /dev/usblp0,
> > /dev/usb/lp0 don't even appear, no matter what the permissions are.
>
> What version of udev are you using? It works fine for me here with a
> USB printer (that's what I tested the changes with.)
udev-068git20050831-9 (from SUSE 10.0 I think)
-Andi
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-09 23:17 ` Andi Kleen
@ 2006-07-09 23:36 ` Greg KH
0 siblings, 0 replies; 36+ messages in thread
From: Greg KH @ 2006-07-09 23:36 UTC (permalink / raw)
To: Andi Kleen, Kay Sievers; +Cc: Linus Torvalds, Linux Kernel Mailing List
On Mon, Jul 10, 2006 at 01:17:30AM +0200, Andi Kleen wrote:
> On Saturday 08 July 2006 18:02, Greg KH wrote:
> > On Sat, Jul 08, 2006 at 04:44:08PM +0200, Andi Kleen wrote:
> > > Greg KH <greg@kroah.com> writes:
> > > >
> > > > Perhaps, that is odd. The scanner should default to the logged in user,
> > > > right? Please file a bug at bugzilla.novell.com and the SuSE people can
> > > > work on it there.
> > >
> > > I have a similar problem with my printer. But /dev/usblp0,
> > > /dev/usb/lp0 don't even appear, no matter what the permissions are.
> >
> > What version of udev are you using? It works fine for me here with a
> > USB printer (that's what I tested the changes with.)
>
> udev-068git20050831-9 (from SUSE 10.0 I think)
That's the issue, look in Documentation/Changes for the proper version
of udev for the past few kernel versions to handle this kind of issue
(again, it was a stupid libsysfs problem, which caused us to finally
drop that piece of crud from udev...) This has been documented as such
since last October, see commit ad7e14a55ed7648d02a4df8e460e291d80a18c98.
It was necessary to do this for the input drivers to get them to all
work properly, please see the old archives for details as to the
necessity of it.
Also, that change caused the creation of Documentation/ABI/ which
describes the rules that programs that look at sysfs class stuff, must
be able to handle a symlink there.
That older version of udev in 10.0 can't handle the symlink, but I think
that Kay had a version somewhere that was patched to handle it. I'd
just recommend upgrading to 10.1 :)
thanks,
greg k-h
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: lost cpufreq (Re: Linux v2.6.18-rc1)
2006-07-07 17:52 ` lost cpufreq (Re: Linux v2.6.18-rc1) Tomasz Torcz
2006-07-07 19:07 ` Dave Jones
@ 2006-07-10 10:03 ` Takashi Iwai
1 sibling, 0 replies; 36+ messages in thread
From: Takashi Iwai @ 2006-07-10 10:03 UTC (permalink / raw)
To: Tomasz Torcz; +Cc: linux-kernel
At Fri, 7 Jul 2006 19:52:39 +0200,
Tomasz Torcz wrote:
>
> Also alsactl restore didn't restore PCM volume and mute state. Some
> mixer elements changed name? Module is snd_intel8x0, chip is integrated
> in above mentioned motherboard.
Hm, I don't remember any relevant changes.
Could you compare the mixer states from "alsactl -f somefile store"
with old and new kernels?
Takashi
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-09 10:34 ` Benjamin Herrenschmidt
@ 2006-07-10 13:21 ` Will Schmidt
2006-07-10 14:40 ` Alan Cox
2006-07-10 21:16 ` Benjamin Herrenschmidt
2006-07-10 16:38 ` Steve Fox
1 sibling, 2 replies; 36+ messages in thread
From: Will Schmidt @ 2006-07-10 13:21 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: Steve Fox, linuxppc-dev, linux-kernel
On Sun, 2006-09-07 at 20:34 +1000, Benjamin Herrenschmidt wrote:
> On Fri, 2006-07-07 at 10:41 -0500, Steve Fox wrote:
> > We've got a ppc64 machine that won't boot with this due to an IDE error.
>
> What machine precisely ?
I see a slightly more verbose version on a JS20 blade.
hda: dma_timer_expiry: dma status == 0x24
hda: DMA interrupt recovery
hda: lost interrupt
>
> > [snip]
> > Freeing unused kernel memory: 256k freed
> > running (1:2) /init autobench_args: ABAT:1152213829
> >
> > creating device nodes .hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> >
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@ozlabs.org
> https://ozlabs.org/mailman/listinfo/linuxppc-dev
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-10 13:21 ` Will Schmidt
@ 2006-07-10 14:40 ` Alan Cox
2006-07-10 21:17 ` Benjamin Herrenschmidt
2006-07-10 21:16 ` Benjamin Herrenschmidt
1 sibling, 1 reply; 36+ messages in thread
From: Alan Cox @ 2006-07-10 14:40 UTC (permalink / raw)
To: will_schmidt
Cc: Benjamin Herrenschmidt, Steve Fox, linuxppc-dev, linux-kernel
Ar Llu, 2006-07-10 am 08:21 -0500, ysgrifennodd Will Schmidt:
> On Sun, 2006-09-07 at 20:34 +1000, Benjamin Herrenschmidt wrote:
> > On Fri, 2006-07-07 at 10:41 -0500, Steve Fox wrote:
> > > We've got a ppc64 machine that won't boot with this due to an IDE error.
> >
> > What machine precisely ?
>
> I see a slightly more verbose version on a JS20 blade.
>
> hda: dma_timer_expiry: dma status == 0x24
> hda: DMA interrupt recovery
> hda: lost interrupt
That in repeat generally means the IRQ logic on the platform has been
broken. If we don't get interrupts we don't work very well.
Also check if booting with "nodma" set on the relevant ide interface
makes a difference. Just to be sure. If it does then submit patches to
fix the bug.
Alan
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-09 10:34 ` Benjamin Herrenschmidt
2006-07-10 13:21 ` Will Schmidt
@ 2006-07-10 16:38 ` Steve Fox
2006-07-10 18:30 ` Steve Fox
1 sibling, 1 reply; 36+ messages in thread
From: Steve Fox @ 2006-07-10 16:38 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: linuxppc-dev, linux-kernel
On Sun, 2006-07-09 at 20:34 +1000, Benjamin Herrenschmidt wrote:
> On Fri, 2006-07-07 at 10:41 -0500, Steve Fox wrote:
> > We've got a ppc64 machine that won't boot with this due to an IDE error.
>
> What machine precisely ?
It's called bl5-6 in ABAT and lists itself as a 970 (8842-P1Z).
Also, booting with ide=nodma, as Alan suggested to Will, did not help.
> > [snip]
> > Freeing unused kernel memory: 256k freed
> > running (1:2) /init autobench_args: ABAT:1152213829
> >
> > creating device nodes .hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> > hda: lost interrupt
> >
>
--
Steve Fox
IBM Linux Technology Center
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-10 16:38 ` Steve Fox
@ 2006-07-10 18:30 ` Steve Fox
0 siblings, 0 replies; 36+ messages in thread
From: Steve Fox @ 2006-07-10 18:30 UTC (permalink / raw)
To: linux-kernel; +Cc: linuxppc-dev
On Mon, 10 Jul 2006 11:38:02 -0500, Steve Fox wrote:
> Also, booting with ide=nodma, as Alan suggested to Will, did not help.
I'm not sure if it was due to using the nodma parameter or not, but I did
get a few more details during this boot. No idea if they're useful or not.
Uniform Multi-Platform E-IDE driver Revision: 7.00alpha2 ide:
Assuming 33MHz system bus speed for PIO modes; override with idebus=xx
AMD8111: IDE controller at PCI slot 0000:00:04.1 AMD8111: chipset revision
3 AMD8111: 0000:00:04.1 (rev 03) UDMA133 controller AMD8111: 100% native
mode on irq 17
ide0: BM-DMA at 0x7c00-0x7c07, BIOS settings: hda:pio, hdb:pio ide1:
BM-DMA at 0x7c08-0x7c0f, BIOS settings: hdc:pio, hdd:pio
hda: TOSHIBA MK4019GAXB, ATA DISK drive ide0 at 0x7400-0x7407,0x6c02 on
irq 17 hda: max request size: 128KiB
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: 78140160 sectors (40007 MB), CHS=65535/16/63 hda: lost interrupt hda:
cache flushes supported
hda:<4>hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda1 hda2 hda3 hda4 <<4>hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda5<4>hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
hda: lost interrupt
--
Steve Fox
IBM Linux Technology Center
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-10 13:21 ` Will Schmidt
2006-07-10 14:40 ` Alan Cox
@ 2006-07-10 21:16 ` Benjamin Herrenschmidt
1 sibling, 0 replies; 36+ messages in thread
From: Benjamin Herrenschmidt @ 2006-07-10 21:16 UTC (permalink / raw)
To: will_schmidt; +Cc: Steve Fox, linuxppc-dev, linux-kernel
On Mon, 2006-07-10 at 08:21 -0500, Will Schmidt wrote:
> On Sun, 2006-09-07 at 20:34 +1000, Benjamin Herrenschmidt wrote:
> > On Fri, 2006-07-07 at 10:41 -0500, Steve Fox wrote:
> > > We've got a ppc64 machine that won't boot with this due to an IDE error.
> >
> > What machine precisely ?
>
> I see a slightly more verbose version on a JS20 blade.
Can you try with the new patches I posted yesterday ?
Ben.
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-10 14:40 ` Alan Cox
@ 2006-07-10 21:17 ` Benjamin Herrenschmidt
2006-07-11 17:00 ` Steve Fox
0 siblings, 1 reply; 36+ messages in thread
From: Benjamin Herrenschmidt @ 2006-07-10 21:17 UTC (permalink / raw)
To: Alan Cox; +Cc: will_schmidt, Steve Fox, linuxppc-dev, linux-kernel
> That in repeat generally means the IRQ logic on the platform has been
> broken. If we don't get interrupts we don't work very well.
>
> Also check if booting with "nodma" set on the relevant ide interface
> makes a difference. Just to be sure. If it does then submit patches to
> fix the bug.
it's most certainly an irq problem as I just rewrote the irq logic of
powerpc :) There have been some issues and I've just sent some new
patches fixing them, let's see if that's enough. I'll give a js20 a try
today at work.
Ben.
^ permalink raw reply [flat|nested] 36+ messages in thread
* Re: Linux v2.6.18-rc1
2006-07-10 21:17 ` Benjamin Herrenschmidt
@ 2006-07-11 17:00 ` Steve Fox
0 siblings, 0 replies; 36+ messages in thread
From: Steve Fox @ 2006-07-11 17:00 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: linuxppc-dev, linux-kernel
On Tue, 2006-07-11 at 07:17 +1000, Benjamin Herrenschmidt wrote:
> it's most certainly an irq problem as I just rewrote the irq logic of
> powerpc :) There have been some issues and I've just sent some new
> patches fixing them, let's see if that's enough. I'll give a js20 a try
> today at work.
FWIW, I tried applying the 3 non-Apple-specific patches posted on 7/10
but this didn't help.
--
Steve Fox
IBM Linux Technology Center
^ permalink raw reply [flat|nested] 36+ messages in thread
end of thread, other threads:[~2006-07-11 17:00 UTC | newest]
Thread overview: 36+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-07-06 4:26 Linux v2.6.18-rc1 Linus Torvalds
2006-07-06 9:44 ` Matt Keenan
2006-07-06 12:49 ` Jonathan Corbet
2006-07-06 16:22 ` Linus Torvalds
2006-07-06 12:34 ` Alistair John Strachan
2006-07-07 0:11 ` Andrew Morton
2006-07-06 12:44 ` Nigel Cunningham
2006-07-06 19:44 ` David R
2006-07-06 22:17 ` Greg KH
2006-07-07 21:11 ` David R
2006-07-08 8:14 ` Borislav Petkov
2006-07-08 14:19 ` Kay Sievers
2006-07-08 14:44 ` Andi Kleen
2006-07-08 14:44 ` Andi Kleen
2006-07-08 16:02 ` Greg KH
2006-07-09 23:17 ` Andi Kleen
2006-07-09 23:36 ` Greg KH
2006-07-07 1:05 ` Hervé Fache
2006-07-07 15:41 ` Steve Fox
2006-07-09 10:34 ` Benjamin Herrenschmidt
2006-07-10 13:21 ` Will Schmidt
2006-07-10 14:40 ` Alan Cox
2006-07-10 21:17 ` Benjamin Herrenschmidt
2006-07-11 17:00 ` Steve Fox
2006-07-10 21:16 ` Benjamin Herrenschmidt
2006-07-10 16:38 ` Steve Fox
2006-07-10 18:30 ` Steve Fox
2006-07-07 17:52 ` lost cpufreq (Re: Linux v2.6.18-rc1) Tomasz Torcz
2006-07-07 19:07 ` Dave Jones
2006-07-07 20:27 ` Tomasz Torcz
2006-07-07 20:35 ` Dave Jones
2006-07-10 10:03 ` Takashi Iwai
2006-07-08 14:09 ` [patchset 0/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors Jim Cromie
2006-07-08 14:16 ` [patchset 1/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - define and use constants Jim Cromie
2006-07-08 14:29 ` [patchset 2/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - undo region reservation Jim Cromie
2006-07-08 14:34 ` [patchset 3/3 -2.6.18-rc1] pc8736x_gpio: fix re-modprobe errors - fix/finish cdev-init Jim Cromie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox