[PATCH] powerpc: Remove double free

2019-05-15 Thread Tobin C. Harding
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

[PATCH] powerpc: Fix kobject memleak

2019-04-29 Thread Tobin 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

[PATCH 0/1] Start conversion of PowerPC docs

2019-02-06 Thread Tobin C. Harding
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.

[PATCH 1/1] docs: powerpc: Convert to RST format

2019-02-06 Thread Tobin C. Harding
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

Re: [PATCH] kdb: use correct pointer when 'btc' calls 'btt'

2018-09-16 Thread Tobin C. Harding
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

[no subject]

2017-07-18 Thread Tobin C. Harding
Reply-To: X-Mailer: Mutt 1.5.24 (2015-08-30) unsubscribe

Re: [PATCH] powerpc: fix sparse warning, include kernel header

2017-03-11 Thread Tobin C. Harding
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

Re: [PATCH] powerpc: asm: convert directive .llong to .8byte

2017-03-10 Thread Tobin C. Harding
> >> .8byte. > >> > >> Convert directives .llong -> .8byte > >> > >> Signed-off-by: Tobin C. Harding > >> --- > >> > >> Fixes: issue #33 (github) > > > > Thanks for tackling these! > > > >

Re: [PATCH] powerpc: asm: convert directive .llong to .8byte

2017-03-10 Thread Tobin C. Harding
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/

[PATCH] powerpc: asm: convert directive .llong to .8byte

2017-03-08 Thread Tobin C. Harding
.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

kernel versions

2017-03-08 Thread Tobin C. Harding
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

Re: open list?

2017-03-07 Thread Tobin C. Harding
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 > >

open list?

2017-03-07 Thread Tobin C. Harding
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

[RESEND PATCH] powerpc/pseries: move struct hcall_stats to c file

2017-03-07 Thread Tobin C. Harding
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

[PATCH] powerpc/pseries: move struct hcall_stats to c file

2017-03-06 Thread Tobin C. Harding
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

[PATCH] powerpc/ftrace: add ftrace.h, fix sparse warning

2017-03-06 Thread Tobin C. Harding
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

[PATCH] powerpc: fix sparse warning, include kernel header

2017-03-06 Thread Tobin C. Harding
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

Re: PowerPC build fail

2017-02-27 Thread Tobin C. Harding
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

PowerPC build fail

2017-02-21 Thread Tobin C. Harding
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