From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] pxa: patches for next merge window
Date: Mon, 1 Mar 2010 17:07:22 +0000 [thread overview]
Message-ID: <20100301170722.GB3002@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <alpine.LFD.2.00.1003010856200.3616@localhost.localdomain>
On Mon, Mar 01, 2010 at 08:58:12AM -0800, Linus Torvalds wrote:
> On Mon, 1 Mar 2010, Linus Torvalds wrote:
> >
> > rmk: I think it's ok if you end up having to rebase something this time in
> > order to fix up a mistake that has already happened - no rule should be
> > _so_ set in stone that it can't be violated when bad things happen.
>
> Btw, since I have a pending pull from you already, let me know if I should
> pull it or not. I was just about to pull it, and then I started wondering
> whether the pax patches are involved, so I kept off.
That pull request contains just local commits, so it's safe to be pulled
in respect of this issue. The only concern raised on it is from Geert
Uytterhoeven over a bit of missing documentation - basically:
5) void update_mmu_cache(struct vm_area_struct *vma,
- unsigned long address, pte_t pte)
+ unsigned long address, pte_t *ptep)
At the end of every page fault, this routine is invoked to
tell the architecture specific code that a translation
- described by "pte" now exists at virtual address "address"
- for address space "vma->vm_mm", in the software page tables.
+ now exists at virtual address "address" for address space
+ "vma->vm_mm", in the software page tables.
the complaint is that 'ptep' is not documented. "ptep is a pointer to
the pte" wouldn't (at least to me) provide any additional information
to the reader.
> But if you're just going to ask the pxa people to fix things up (which is
> my preference), just send me an email saying "go ahead and pull". I'm
> checking, just in case.
I hope that the PXA situation does get straightened out soon, at which
point this will be part of a pull request next week.
next prev parent reply other threads:[~2010-03-01 17:07 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-25 2:49 [GIT PULL] pxa: patches for next merge window Eric Miao
2010-02-25 20:51 ` Russell King - ARM Linux
2010-02-25 21:29 ` Russell King - ARM Linux
2010-02-26 2:50 ` Eric Miao
2010-02-26 9:05 ` Eric Miao
2010-02-28 16:14 ` Russell King - ARM Linux
2010-03-01 0:42 ` Nicolas Pitre
2010-03-01 13:23 ` Eric Miao
2010-03-01 15:16 ` Russell King - ARM Linux
2010-03-02 4:48 ` Eric Miao
2010-03-01 9:39 ` Uwe Kleine-König
2010-03-01 9:48 ` Russell King - ARM Linux
2010-03-01 10:11 ` Paul Mundt
2010-03-01 10:27 ` Uwe Kleine-König
2010-03-02 0:33 ` Stephen Rothwell
2010-03-01 10:12 ` Uwe Kleine-König
2010-03-01 16:40 ` Linus Torvalds
2010-03-01 16:58 ` Linus Torvalds
2010-03-01 17:07 ` Russell King - ARM Linux [this message]
-- strict thread matches above, loose matches on Subject: below --
2010-12-22 9:09 Eric Miao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100301170722.GB3002@n2100.arm.linux.org.uk \
--to=linux@arm.linux.org.uk \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).