* Re: [PATCH] Add StorCenter DTS first draft.
From: Segher Boessenkool @ 2007-07-20 7:14 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: linuxppc-dev, Jon Loeliger
In-Reply-To: <1184881198.25235.347.camel@localhost.localdomain>
>> So your plan is to prefer the device tree over the cputable, and
>> maybe even deprecate the cputable?
>
> Nah, you are fine not putting anything, but if it's in the DT it
> should
> be correct, not "0". Else, just don't put it in the DT.
Ah okay. So we should recommend to not put any of those
properties into DTS files at all? If so, I'll create a
patch to remove them.
> We use the DT to override cputable but that's mostly useful for things
> like pSeries where you can get some processors in "compatibility" mode
> over another one, or some virtual PVRs, and other fancy things like
> that
> where the cputable isn't very useful.
Yeah, and it's a good idea to have the device tree override
the cputable always, with real OF, anyway. (Well there are
buggy device trees out there, of course ;-) )
Segher
^ permalink raw reply
* Re: [PATCH] [updated] PHY fixed driver: rework release path and update phy_id notation
From: Andrew Morton @ 2007-07-20 6:23 UTC (permalink / raw)
To: Vitaly Bordug; +Cc: linuxppc-dev, linux-kernel, Jeff Garzik, netdev
In-Reply-To: <20070718233804.24451.22543.stgit@localhost.localdomain>
On Thu, 19 Jul 2007 03:38:04 +0400 Vitaly Bordug <vitb@kernel.crashing.org> wrote:
>
> device_bind_driver() error code returning has been fixed.
> release() function has been written, so that to free resources
> in correct way; the release path is now clean.
>
> Before the rework, it used to cause
> Device 'fixed@100:1' does not have a release() function, it is broken
> and must be fixed.
> BUG: at drivers/base/core.c:104 device_release()
>
> Call Trace:
> [<ffffffff802ec380>] kobject_cleanup+0x53/0x7e
> [<ffffffff802ec3ab>] kobject_release+0x0/0x9
> [<ffffffff802ecf3f>] kref_put+0x74/0x81
> [<ffffffff8035493b>] fixed_mdio_register_device+0x230/0x265
> [<ffffffff80564d31>] fixed_init+0x1f/0x35
> [<ffffffff802071a4>] init+0x147/0x2fb
> [<ffffffff80223b6e>] schedule_tail+0x36/0x92
> [<ffffffff8020a678>] child_rip+0xa/0x12
> [<ffffffff80311714>] acpi_ds_init_one_object+0x0/0x83
> [<ffffffff8020705d>] init+0x0/0x2fb
> [<ffffffff8020a66e>] child_rip+0x0/0x12
>
>
> Also changed the notation of the fixed phy definition on
> mdio bus to the form of <speed>+<duplex> to make it able to be used by
> gianfar and ucc_geth that define phy_id strictly as "%d:%d" and cleaned up
> the whitespace issues.
>
Confused. Does the above refer to the difference between this patch and
the previous version, or does it just describe this patch? Hopefully the
latter, because the former isn't interesting, long-term.
If is _is_ a full standalone description of this patch then it's a bit hard
to follow ;)
> +config FIXED_MII_1000_FDX
> + bool "Emulation for 1000M Fdx fixed PHY behavior"
> + depends on FIXED_PHY
> +
> +config FIXED_MII_AMNT
> + int "Number of emulated PHYs to allocate "
> + depends on FIXED_PHY
> + default "1"
> + ---help---
> + Sometimes it is required to have several independent emulated
> + PHYs on the bus (in case of multi-eth but phy-less HW for instance).
> + This control will have specified number allocated for each fixed
> + PHY type enabled.
Shouldn't these be runtime options (ie: module parameters)?
>
> ...
>
> + * Private information hoder for mii_bus
tpyo.
^ permalink raw reply
* Re: [PATCH] of_detach_node()'s device node argument cannot be const
From: Stephen Rothwell @ 2007-07-20 6:11 UTC (permalink / raw)
To: Segher Boessenkool; +Cc: linuxppc-dev
In-Reply-To: <11849111183072-git-send-email-segher@kernel.crashing.org>
[-- Attachment #1: Type: text/plain, Size: 343 bytes --]
On Fri, 20 Jul 2007 07:58:38 +0200 Segher Boessenkool <segher@kernel.crashing.org> wrote:
>
> ...since it modifies it (when it sets the OF_DETACHED flag).
http://patchwork.ozlabs.org/linuxppc/patch?q=Stephen%20Rothwell&id=12212
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply
* Please pull Open Firmware consolidation patches
From: Stephen Rothwell @ 2007-07-20 6:00 UTC (permalink / raw)
To: Linus; +Cc: LKML, wli, ppc-dev, paulus, sparclinux, David S. Miller
[-- Attachment #1: Type: text/plain, Size: 3845 bytes --]
Hi Linus,
Please pull the Open Firmware device tree consolidation patches from the
master branch of
git://git.kernel.org/pub/scm/linux/kernel/git/sfr/ofcons.git master
The intention here is that there should be no behavioural changes and
minimal code changes (apart from movement of the code to common places).
There are, however a couple of unavoidable changes:
- some of the accessor functions now take a read lock on Sparc[64]
where they didn't before.
- the bus structures are initialised at runtime.
These changes have been reviewed on the PowerPC and Sparc Linux mailing
lists and acked by both maintainers (Dave Miller and Paul Mackerras).
Kernels with tese changes have been built and booted on a SunBlade1500
(thanks Dave), a Power5+ pSeries machine and a small iSeries machine.
Dave is rather keen for these to go in ("Did you send him a pull request
yet? Sorry, I'm quite anxious :)").
Stephen Rothwell (12):
Split out common parts of prom.h
Start split out of common open firmware code
Consolidate of_device_is_compatible
Consolidate of_find_property
Consolidate of_get_parent
Consolidate of_get_next_child
Consolidate of_find_node_by routines
Begin to consolidate of_device.c
Begin consolidation of of_device.h
[SPARC/64] Rename some functions like PowerPC
Create linux/of_platorm.h
Create drivers/of/platform.c
arch/powerpc/Kconfig | 3 +
arch/powerpc/kernel/of_device.c | 122 +----------------
arch/powerpc/kernel/of_platform.c | 82 +-----------
arch/powerpc/kernel/prom.c | 250 +---------------------------------
arch/sparc/Kconfig | 3 +
arch/sparc/kernel/of_device.c | 222 ++----------------------------
arch/sparc/kernel/prom.c | 173 +-----------------------
arch/sparc/kernel/time.c | 2 +-
arch/sparc64/Kconfig | 3 +
arch/sparc64/kernel/auxio.c | 2 +-
arch/sparc64/kernel/of_device.c | 238 +++-----------------------------
arch/sparc64/kernel/power.c | 2 +-
arch/sparc64/kernel/prom.c | 173 +-----------------------
arch/sparc64/kernel/time.c | 2 +-
drivers/Kconfig | 2 +
drivers/Makefile | 1 +
drivers/of/Kconfig | 3 +
drivers/of/Makefile | 2 +
drivers/of/base.c | 275 +++++++++++++++++++++++++++++++++++++
drivers/of/device.c | 131 ++++++++++++++++++
drivers/of/platform.c | 96 +++++++++++++
include/asm-powerpc/of_device.h | 22 +---
include/asm-powerpc/of_platform.h | 38 +-----
include/asm-powerpc/prom.h | 50 ++-----
include/asm-sparc/of_device.h | 49 +------
include/asm-sparc/of_platform.h | 32 +++++
include/asm-sparc/prom.h | 62 +++------
include/asm-sparc64/of_device.h | 50 +------
include/asm-sparc64/of_platform.h | 33 +++++
include/asm-sparc64/prom.h | 62 +++------
include/linux/of.h | 61 ++++++++
include/linux/of_device.h | 26 ++++
include/linux/of_platform.h | 57 ++++++++
33 files changed, 846 insertions(+), 1483 deletions(-)
create mode 100644 drivers/of/Kconfig
create mode 100644 drivers/of/Makefile
create mode 100644 drivers/of/base.c
create mode 100644 drivers/of/device.c
create mode 100644 drivers/of/platform.c
create mode 100644 include/asm-sparc/of_platform.h
create mode 100644 include/asm-sparc64/of_platform.h
create mode 100644 include/linux/of.h
create mode 100644 include/linux/of_device.h
create mode 100644 include/linux/of_platform.h
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply
* [PATCH] of_detach_node()'s device node argument cannot be const
From: Segher Boessenkool @ 2007-07-20 5:58 UTC (permalink / raw)
To: linuxppc-dev
...since it modifies it (when it sets the OF_DETACHED flag).
Signed-off-by: Segher Boessenkool <segher@kernel.crashing.org>
---
arch/powerpc/kernel/prom.c | 2 +-
include/asm-powerpc/prom.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/kernel/prom.c b/arch/powerpc/kernel/prom.c
index 37ff99b..67a1bda 100644
--- a/arch/powerpc/kernel/prom.c
+++ b/arch/powerpc/kernel/prom.c
@@ -1433,7 +1433,7 @@ void of_attach_node(struct device_node *np)
* a reference to the node. The memory associated with the node
* is not freed until its refcount goes to zero.
*/
-void of_detach_node(const struct device_node *np)
+void of_detach_node(struct device_node *np)
{
struct device_node *parent;
diff --git a/include/asm-powerpc/prom.h b/include/asm-powerpc/prom.h
index 1632baa..1bba741 100644
--- a/include/asm-powerpc/prom.h
+++ b/include/asm-powerpc/prom.h
@@ -160,7 +160,7 @@ extern unsigned long __init of_get_flat_dt_root(void);
/* For updating the device tree at runtime */
extern void of_attach_node(struct device_node *);
-extern void of_detach_node(const struct device_node *);
+extern void of_detach_node(struct device_node *);
/* Other Prototypes */
extern void finish_device_tree(void);
--
1.5.2.1.144.gabc40-dirty
^ permalink raw reply related
* Re: [PATCH 12/12] Create drivers/of/platform.c
From: David Miller @ 2007-07-20 4:20 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720012032.ad4419a1.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:20:32 +1000
> and populate it with the common parts from PowerPC and Sparc[64].
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
All tested on SunBlade1500.
Please push this to Linus Stephen, thanks!
^ permalink raw reply
* Re: [PATCH 11/12] Create linux/of_platorm.h
From: David Miller @ 2007-07-20 4:01 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720011925.f51f998a.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:19:25 +1000
> Move common stuff from asm-powerpc/of_platform.h to here and
> move the common bits from asm-sparc*/of_device.h here as well.
>
> Create asm-sparc*/of_platform.h and move appropriate parts of
> of_device.h to them.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
Tested on SunBlade1500.
^ permalink raw reply
* Re: [PATCH 09/12] Begin consolidation of of_device.h
From: David Miller @ 2007-07-20 3:33 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720011708.7919947c.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:17:08 +1000
> This just moves the common stuff from the arch of_device.h files to
> linux/of_device.h.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
Tested on SunBlade1500.
^ permalink raw reply
* Re: [PATCH 08/12] Begin to consolidate of_device.c
From: David Miller @ 2007-07-20 3:21 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720011612.cc101b92.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:16:12 +1000
> This moves all the common parts for the Sparc, Sparc64 and PowerPC
> of_device.c files into drivers/of/device.c.
>
> Apart from the simple move, Sparc gains of_match_node() and a call to
> of_node_put in of_release_dev(). PowerPC gains better recovery if
> device_create_file() fails in of_device_register().
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
Tested on SunBlade1500.
^ permalink raw reply
* Re: [PATCH 07/12] Consolidate of_find_node_by routines
From: David Miller @ 2007-07-20 3:12 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720011450.d0b97300.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:14:50 +1000
> This consolidates the routines of_find_node_by_path, of_find_node_by_name,
> of_find_node_by_type and of_find_compatible_device. Again, the comparison
> of strings are done differently by Sparc and PowerPC and also these add
> read_locks around the iterations.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
Tested on SunBlade1500.
^ permalink raw reply
* Re: Fix ARCH=ppc builds
From: Kumar Gala @ 2007-07-20 3:10 UTC (permalink / raw)
To: Josh Boyer; +Cc: linuxppc-dev, paulus
In-Reply-To: <20070720031110.GP3925@crusty.rchland.ibm.com>
On Jul 19, 2007, at 10:11 PM, Josh Boyer wrote:
> The recent signal rework broke ARCH=3Dppc builds with the following =20=
> error:
>
> CC arch/powerpc/kernel/signal.o
> arch/powerpc/kernel/signal.c: In function =91do_signal=92:
> arch/powerpc/kernel/signal.c:142: error: implicit declaration of =20
> function =91set_dabr=92
> make[1]: *** [arch/powerpc/kernel/signal.o] Error 1
>
> The patch below fixes it by including a function prototype in
> asm-ppc/system.h
>
> Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com>
Acked-by: Kumar Gala <galak@kernel.crashing.org>
(Bet me to posting this :)
> diff --git a/include/asm-ppc/system.h b/include/asm-ppc/system.h
> index f1311a8..cc45780 100644
> --- a/include/asm-ppc/system.h
> +++ b/include/asm-ppc/system.h
> @@ -54,6 +54,7 @@ extern void show_regs(struct pt_regs * regs);
> extern void flush_instruction_cache(void);
> extern void hard_reset_now(void);
> extern void poweroff_now(void);
> +extern int set_dabr(unsigned long dabr);
> #ifdef CONFIG_6xx
> extern long _get_L2CR(void);
> extern long _get_L3CR(void);
>
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@ozlabs.org
> https://ozlabs.org/mailman/listinfo/linuxppc-dev
^ permalink raw reply
* Possible eHEA performance issue
From: Michael Neuling @ 2007-07-20 3:02 UTC (permalink / raw)
To: Thomas Klein, Jan-Bernd Themann, netdev, Christoph Raisch,
Stefan Roscher, linux-ppc
Cc: anton
>From ehea_start_xmit in ehea_main.c we have:
if (unlikely(atomic_read(&pr->swqe_avail) <= 1)) {
spin_lock_irqsave(&pr->netif_queue, flags);
if (unlikely(atomic_read(&pr->swqe_avail) <= 1)) {
pr->p_stats.queue_stopped++;
netif_stop_queue(dev);
pr->queue_stopped = 1;
}
spin_unlock_irqrestore(&pr->netif_queue, flags);
}
Since the conditions are the same, isn't it likely that the second 'if'
is going to be taken. Hence, shouldn't the second 'unlikely' hint be
removed or even changed to likely?
Either way, some documentation here as to why it's done this way would
be useful. I assume the atomic_read is cheap compared to the
spin_unlock_irqsave, so we quickly check swqe_avail before we check it
again properly with the lock on so we can change some stuff.
Mikey
^ permalink raw reply
* Fix ARCH=ppc builds
From: Josh Boyer @ 2007-07-20 3:11 UTC (permalink / raw)
To: paulus; +Cc: linuxppc-dev
The recent signal rework broke ARCH=ppc builds with the following error:
CC arch/powerpc/kernel/signal.o
arch/powerpc/kernel/signal.c: In function ‘do_signal’:
arch/powerpc/kernel/signal.c:142: error: implicit declaration of function ‘set_dabr’
make[1]: *** [arch/powerpc/kernel/signal.o] Error 1
The patch below fixes it by including a function prototype in
asm-ppc/system.h
Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com>
diff --git a/include/asm-ppc/system.h b/include/asm-ppc/system.h
index f1311a8..cc45780 100644
--- a/include/asm-ppc/system.h
+++ b/include/asm-ppc/system.h
@@ -54,6 +54,7 @@ extern void show_regs(struct pt_regs * regs);
extern void flush_instruction_cache(void);
extern void hard_reset_now(void);
extern void poweroff_now(void);
+extern int set_dabr(unsigned long dabr);
#ifdef CONFIG_6xx
extern long _get_L2CR(void);
extern long _get_L3CR(void);
^ permalink raw reply related
* Re: [PATCH 3/4] mpc8349emitx: Add chosen node for default stdout path
From: Jerry Van Baren @ 2007-07-20 2:00 UTC (permalink / raw)
To: Scott Wood; +Cc: linuxppc-dev
In-Reply-To: <469FDB3B.5000901@freescale.com>
Scott Wood wrote:
> Kim Phillips wrote:
>> the old FLAT_TREE u-boot fdt fixup code renames any existing chosen
>> node out of the way, and adds its fixed up version as /chosen.
Not in my experience. My experience is that it blindly created a second
/chosen node.
>> The LIBFDT implementation replaces any existing /chosen with its fixed
>> up version.
Sort of. If /chosen doesn't exist, it creates it.
If /chosen exists and "force" parameter is false, it doesn't touch it.
If "force" is true, it creates or fixes up properties. The "bootm"
command passes in force == false. The "fdt" command passes in force ==
true.
The "force" parameter was added to sort of emulate the previous bootm
command behavior (but behave better in the case where /chosen already
existed).
Such is the price for not totally breaking user expectations. Hopefully
we recalibrate user expectations in the future and improve this.
> Could you point out the code that does this? I don't see it in either
> the old code or the new.
>
> -Scott
Old code - I don't think so.
New code...
u-boot-fdt repo:
<http://www.denx.de/cgi-bin/gitweb.cgi?p=u-boot/u-boot-fdt.git;a=blob;f=common/fdt_support.c;h=259bd42cc62c55d11370579a7af0d6519fc34c8d;hb=01f771763ed822145b54819abb9c4516c8216d48#l96>
u-boot repo:
<http://www.denx.de/cgi-bin/gitweb.cgi?p=u-boot.git;a=blob;f=common/fdt_support.c;h=69099c4275ccb925cce7b7c69afc5ab4f76cfe6f;hb=HEAD#l92>
Hope this makes sense,
gvb
^ permalink raw reply
* Re: [PATCH 06/12] Consolidate of_get_next_child
From: David Miller @ 2007-07-20 1:07 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720011348.3071f52a.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:13:48 +1000
> This adds a read_lock around the child/next accesses on Sparc.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
Tested on SunBlade1500.
^ permalink raw reply
* Re: [PATCH 05/12] Consolidate of_get_parent
From: David Miller @ 2007-07-20 1:01 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720011235.8199eca2.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:12:35 +1000
> This requires creating dummy of_node_{get,put} routines for sparc and
> sparc64. It also adds a read_lock around the parent accesses.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
Tested on SunBlade1500.
^ permalink raw reply
* Re: [PATCH 04/12] Consolidate of_find_property
From: David Miller @ 2007-07-20 0:45 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720011127.8c700e59.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:11:27 +1000
> The only change here is that a readlock is taken while the property list
> is being traversed on Sparc where it was not taken previously.
>
> Also, Sparc uses strcasecmp to compare property names while PowerPC
> uses strcmp.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
Tested on SunBlade1500.
^ permalink raw reply
* Re: [PATCH 03/12] Consolidate of_device_is_compatible
From: David Miller @ 2007-07-20 0:24 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720011012.4eca281d.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:10:12 +1000
> The only difference here is that Sparc uses strncmp to match compatibility
> names while PowerPC uses strncasecmp.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
Tested on SunBlade1500.
^ permalink raw reply
* Re: [PATCH 02/12] Start split out of common open firmware code
From: David Miller @ 2007-07-19 23:48 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720010911.01402add.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:09:11 +1000
> This creates drivers/of/base.c (depending on CONFIG_OF) and puts
> the first trivially common bits from the prom.c files into it.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
Tested on SunBlade1500.
^ permalink raw reply
* Re: [PATCH] Treat ISI faults as read faults on classic 32-bit PowerPC
From: Paul Mackerras @ 2007-07-19 23:40 UTC (permalink / raw)
To: Segher Boessenkool; +Cc: linuxppc-dev, Johannes Berg
In-Reply-To: <FB9F7C22-8598-4E72-B663-130A7989925D@kernel.crashing.org>
Segher Boessenkool writes:
> Should you really be testing VM_READ|VM_WRITE, or should it just
> be VM_READ?
We test VM_READ | VM_WRITE | VM_EXEC in the read case below, and that
is because we have no HPTE encoding to say "writable but not readable"
or "executable but not readable". Similarly we have no encoding to
say "writable but not executable" on classic processors, so if you
have just VM_WRITE set, you get a page that is readable, writable and
executable.
Paul.
^ permalink raw reply
* Re: [PATCH 01/12] Split out common parts of prom.h
From: Paul Mackerras @ 2007-07-19 23:27 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: sparclinux, ppc-dev, David S. Miller, wli
In-Reply-To: <20070720010742.e60c67a7.sfr@canb.auug.org.au>
Stephen Rothwell writes:
> This creates linux/of.h and includes asm/prom.h from it.
>
> We also include linux/of.h from asm/prom.h while we transition.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: Paul Mackerras <paulus@samba.org>
for all 12 of the series.
^ permalink raw reply
* Re: 2.6.22-git* regression: Kconfig prompts without help text
From: Geert Uytterhoeven @ 2007-07-19 21:05 UTC (permalink / raw)
To: Stefan Richter
Cc: Randy Dunlap, Linux/m68k, Michal Piotrowski,
Linux Kernel Mailing List, Linux/PPC Development, Jan Engelhardt,
Andrew Morton
In-Reply-To: <469FCE5E.80803@s5r6.in-berlin.de>
On Thu, 19 Jul 2007, Stefan Richter wrote:
> Geert Uytterhoeven wrote:
> > On Wed, 18 Jul 2007, Jan Engelhardt wrote:
> >> Add some help texts to recently-introduced kconfig items
> >>
> >> Index: linux-2.6.23/drivers/macintosh/Kconfig
> >> ===================================================================
> >> --- linux-2.6.23.orig/drivers/macintosh/Kconfig
> >> +++ linux-2.6.23/drivers/macintosh/Kconfig
> >> @@ -3,6 +3,12 @@ menuconfig MACINTOSH_DRIVERS
> >> bool "Macintosh device drivers"
> >> depends on PPC || MAC || X86
> >> default y if (PPC_PMAC || MAC)
> >> + ---help---
> >> + Say Y here to get to see options for devices used with Macintosh
> >> + computers, both PPC and Intel based. This option alone does not add
> > ^^^^^
> >> + any kernel code.
> >> +
> >> + If you say N, all options in this submenu will be skipped and disabled.
> >
> > `MAC' is the symbol for m68k Macs.
>
> OK, then this should read
>
> Say Y here to get to see options for devices used with Macintosh
> computers. This option alone does not add any kernel code.
>
> If you say N, all options in this submenu will be skipped and disabled.
>
> > From a quick look at drivers/macintosh/Kconfig, all of the entries are for m68k
> > and PPC Macs only. The only thing that depends on X86 is the main menu...
>
> There is one entry for all sorts of Macs, MAC_EMUMOUSEBTN.
You're right, I missed that one, as I used `grep depends' ;-)
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply
* Re: [PATCH 01/12] Split out common parts of prom.h
From: David Miller @ 2007-07-19 22:55 UTC (permalink / raw)
To: sfr; +Cc: sparclinux, linuxppc-dev, paulus, wli
In-Reply-To: <20070720010742.e60c67a7.sfr@canb.auug.org.au>
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Jul 2007 01:07:42 +1000
> This creates linux/of.h and includes asm/prom.h from it.
>
> We also include linux/of.h from asm/prom.h while we transition.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
Acked-by: David S. Miller <davem@davemloft.net>
^ permalink raw reply
* Re: Powerpc - Include pagemap.h in asm/powerpc/tlb.h
From: Kumar Gala @ 2007-07-19 22:46 UTC (permalink / raw)
To: Andrew Morton
Cc: linuxppc-dev, Mathieu Desnoyers, linux-kernel, Paul Mackerras
In-Reply-To: <20070719143020.5e0b7474.akpm@linux-foundation.org>
On Jul 19, 2007, at 4:30 PM, Andrew Morton wrote:
> On Thu, 19 Jul 2007 16:04:38 -0500
> Kumar Gala <galak@kernel.crashing.org> wrote:
>
>>>> Andrew,
>>>>
>>>> Are you sending this to linus directly or should this go via
>>>> paul and
>>>> me?
>>>>
>>>
>>> I queued it up for Paul. I can send it over to Linus today if we
>>> have a
>>> reason for that. It's just that I dont understand the patch: does
>>> it fix
>>> -mm-only breakage? Does it fix something which already got fixed,
>>> or what?
>>
>> I'm seeing the same breakage in my tree.
>
> Oh, OK. Let's just merge it.
Hmm, depending on the config it goes away.
Lets hold off on this for a little while so understand exactly what's
going on.
>>> One might think "gee, it's trivial, just slam it in", but these
>>> nested
>>> includes are often not-trivial and something else can explode as a
>>> result.
>>
>> I'll do a git-bisect and see if I can find the patch that causes this
>> breakage so we know what's going on.
>
> eh, don't bother.
I think I'll poke around a little since it clearly has some config
dependancy.
- k
^ permalink raw reply
* Re: PS3: Fix build with 32-bit toolchains
From: Geoff Levand @ 2007-07-19 22:36 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev, Paul Mackerras, Arnd Bergmann
In-Reply-To: <A127B558-BA1C-4F18-890B-7052F2DF7488@kernel.crashing.org>
Kumar Gala wrote:
> On Jul 19, 2007, at 4:47 PM, Geoff Levand wrote:
>
>> The PS3 bootwrapper files use instructions only available on
>> 64-bit CPUs. Add the code generation directive '.machine "ppc64"'
>> for toolchains configured for 32-bit CPUs.
>>
>> Signed-off-by: Geoff Levand <geoffrey.levand@am.sony.com>
>
> while this is a fix, why are we even building it Grant's case of
> building 834x itx.
I originally had the makefile setup to conditionally build these
files on CONFIG_PPC_PS3, but during the review I was requested to
change it to build them unconditionally. See:
http://ozlabs.org/pipermail/linuxppc-dev/2007-June/038072.html
-Geoff
^ permalink raw reply
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox