Skip to contentSkip to conversation

xfs: error: called object 'panic' is not a function

next In review Geert Uytterhoeven 3 participants 3 messages Mar 11, 2011 – Mar 13, 2011

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

http://kisskb.ellerman.id.au/kisskb/buildresult/3977742/:

| fs/xfs/linux-2.6/xfs_message.c:116: error: called object 'panic' is not a function
| make[3]: *** [fs/xfs/linux-2.6/xfs_message.o] Error 1

Lemme guess, the problem fixed by commit cf78859f520f8275318f47d7864f4459d940cb6b (xfs: Do not name variables "panic") has reincarnated somewhere else?

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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

#1 lore raw

PATCH 1/1 xfs: don't name variables "panic"

The new xfs_alert_tag() used a variable named "panic", and that is to be avoided. Rename it.

---
 fs/xfs/linux-2.6/xfs_message.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
fs/xfs/linux-2.6/xfs_message.c +3 -3
Index: b/fs/xfs/linux-2.6/xfs_message.c
===================================================================
@@ -96,13 +96,13 @@ xfs_alert_tag(
 {
        struct va_format        vaf;
        va_list                 args;
-       int                     panic = 0;
+       int                     do_panic = 0;
        int                      r;
 
        if (xfs_panic_mask && (xfs_panic_mask & panic_tag)) {
                xfs_printk(KERN_ALERT, mp,
                        "XFS: Transforming an alert into a BUG.");
-               panic = 1;
+               do_panic = 1;
        }
 
        va_start(args, fmt);
@@ -113,7 +113,7 @@ xfs_alert_tag(
        r = __xfs_printk(KERN_ALERT, mp, &vaf);
        va_end(args);
 
-       BUG_ON(panic);
+       BUG_ON(do_panic);
 
        return r;
 }

_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs

Reviewed-by on v1 1/1
#2 lore raw

Re: [PATCH] xfs: don't name variables "panic"

Quoted 4 lines · Alex Elder

The new xfs_alert_tag() used a variable named "panic", and that is to be avoided. Rename it.

My fault. Fix looks good.

Cheers,

Dave

Signature · 7 lines
Dave Chinner
[email protected]

_______________________________________________
xfs mailing list
[email protected]
http://oss.sgi.com/mailman/listinfo/xfs