kfree() after kobject_put(). Who ever wrote this was on crack.
Fixes: 7e8039795a80 ("powerpc/cacheinfo: Fix kobject memleak")
Signed-off-by: Tobin C. Harding
---
FTR
git log --pretty=format:"%h%x09%an%x09%ad%x09%s" | grep 7e8039795a80
7e8039795a80Tobin C. Harding
Currently error return from kobject_init_and_add() is not followed by a
call to kobject_put(). This means there is a memory leak.
Add call to kobject_put() in error path of kobject_init_and_add().
Signed-off-by: Tobin C. Harding
---
Untested, did not even build. Bad Tobin, no biscuit.
No
bvious typos (lean towards not making changes so that
we don't introduce errors).
Edited as text files (obviously) and formatted as HTML to verify
rendering, no other formats verified.
thanks,
Tobin.
Tobin C. Harding (1):
docs: powerpc: Convert docs to RST format.
Documentation/index.
ited as text files (obviously) and formatted as HTML to verify
rendering, no other formats verified.
Convert docs to RST format, adding license.
Signed-off-by: Tobin C. Harding
---
Documentation/index.rst | 1 +
Documentation/powerpc/DAWR-POWER9.rst | 60
Doc
On Sun, Sep 16, 2018 at 12:06:10PM -0700, Daniel Thompson wrote:
> On Fri, Sep 14, 2018 at 12:35:44PM +, Christophe Leroy wrote:
> > On a powerpc 8xx, 'btc' fails as follows:
> >
> > Entering kdb (current=0x(ptrval), pid 282) due to Keyboard Entry
> > kdb> btc
> > btc: cpu status: Currently on
Reply-To:
X-Mailer: Mutt 1.5.24 (2015-08-30)
unsubscribe
such, this patch is:
>
> Reviewed-by: Daniel Axtens
Awesome, thanks Daniel
>
> Thanks for fixing this!
>
> Regards,
> Daniel
>
> [1] https://github.com/daxtens/smart-sparse-diff
>
> "Tobin C. Harding" writes:
>
> > Spares emits two sy
> >> .8byte.
> >>
> >> Convert directives .llong -> .8byte
> >>
> >> Signed-off-by: Tobin C. Harding
> >> ---
> >>
> >> Fixes: issue #33 (github)
> >
> > Thanks for tackling these!
> >
> >
es .llong -> .8byte
> >
> > Signed-off-by: Tobin C. Harding
> > ---
> >
> > Fixes: issue #33 (github)
>
> Thanks for tackling these!
>
> I have applied your patch to my local tree. I ran `git grep '\.llong'`,
> and found:
>
> tools/
.llong is an undocumented PPC specific directive. The generic
equivalent is .quad, but even better (because it's self describing) is
.8byte.
Convert directives .llong -> .8byte
Signed-off-by: Tobin C. Harding
---
Fixes: issue #33 (github)
Patch is not tested. Has been built on Power8
Recently the kernel version on the github
repository did not match up with the master branch of
https://kernel.googlesource.com/pub/scm/linux/kernel/git/powerpc/linux
Specifically, github was hosting v4.10-rc5 while master had
v4.10-rc8. All the while Linus' mainline was at v4.11-rc1.
Was this a
On Tue, Mar 07, 2017 at 01:16:34PM +0100, Christophe LEROY wrote:
>
>
> Le 07/03/2017 à 11:02, Tobin C. Harding a écrit :
> >scripts/get_maintainers.pl says this is an open list;
> >
> >linuxppc-dev@lists.ozlabs.org (open list:LINUX FOR POWERPC (32-BIT AND
> >
scripts/get_maintainers.pl says this is an open list;
linuxppc-dev@lists.ozlabs.org (open list:LINUX FOR POWERPC (32-BIT AND
64-BIT))
Patches I've sent with this list cc'd have not been getting through. I
resent one to check if it was a user error at my end.
This email will obviously serve as an
struct hcall_stats is only used in hvCall_inst.c.
Move struct hcall_stats to hvCall_inst.c
Resolves: #54
Signed-off-by: Tobin C. Harding
---
Is this correct, adding 'Resolves: #XX' when fixing
github.com/linuxppc/linux issues?
arch/powerpc/include/asm/hvcall.h
struct hcall_stats is only used in hvCall_inst.c.
Move struct hcall_stats to hvCall_inst.c
Resolves: #54
Signed-off-by: Tobin C. Harding
---
Is this correct, adding 'Resolves: #XX' when fixing
github.com/linuxppc/linux issues?
arch/powerpc/include/asm/hvcall.h
eader file ftrace.h with single function declaration. Protect
declaration with preprocessor guard so it may be included in
assembly. Include new header in all files that call
prepare_ftrace_return() and in ftrace.c where function is defined.
Signed-off-by: Tobin C. Harding
---
Github issue: #37
Spares emits two symbol not declared warnings. The two functions in
question are declared already in a kernel header.
Add include directive to include kernel header.
Signed-off-by: Tobin C. Harding
---
github issue: #37 Fix sparse errors
arch/powerpc/kernel/swsusp.c | 1 +
1 file changed, 1
On Mon, Feb 27, 2017 at 09:38:13PM +1100, Michael Ellerman wrote:
> Hi Tobin,
>
> "Tobin C. Harding" writes:
> > The current (2bfe01e) torvalds git tree fails to build on powerpc64. Build
> > machine
> > is virtualized.
> >
> > - Build
The current (2bfe01e) torvalds git tree fails to build on powerpc64. Build
machine
is virtualized.
- Build error
arch/powerpc/kernel/time.c: In function ‘running_clock’:
arch/powerpc/kernel/time.c:712:25: error: implicit declaration of function
‘cputime_to_nsecs’
return local_clock() -
cputi
19 matches
Mail list logo