Messages in this thread | | | From | David Howells <> | Subject | Re: [RFC PATCH 14/43] ptrace, frv: change signature of arch_ptrace() | Date | Fri, 27 Aug 2010 13:12:03 +0100 |
| |
Namhyung Kim <namhyung@gmail.com> wrote:
> > That description means nothing. Commit > > f76671df26ef06321480e702770f88f61272be29 is not upstream. > > Hi, > Thank you for noticing. My bad.
The problem with using a non-upstream commit ID like this is that it likely won't be the same once that commit is committed by Linus.
> I just wanted to let you know it depends on that.
The patch being part of the series is probably sufficient, though a note of the subject line of the previous patch would be useful.
> What is the proper way to handle this?
A summary of the changes being made is good:
ptrace: Fix up the arguments arch_ptrace() in arch FRV
Fix up the arguments to arch_ptrace() to take account of the fact that addr and data are now unsigned long rather than long as of a preceding patch in this series.
Signed-off-by: ...
Note, however, that if the earlier patch breaks the compilation and then this patch fixes it up, you should roll this patch into the earlier patch, and the earlier patch is not complete without it.
Think what happens if patch 3/43 breaks an arch, and then patch 43/43, say, mends that arch, and then bisection lands on patch 3 during its progress. You may end up having to 'git bisect skip' all the patches between 3 and 43 one at a time.
David
| |