* Patch fs/xfs/xfs_alloc.c
@ 2010-09-01 19:27 Poyo VL
2010-09-02 5:21 ` Dave Chinner
0 siblings, 1 reply; 4+ messages in thread
From: Poyo VL @ 2010-09-01 19:27 UTC (permalink / raw)
To: aelder; +Cc: xfs
[-- Attachment #1: Type: text/plain, Size: 631 bytes --]
File: fs/xfs/xfs_alloc.c
Kernel: 2.6.35.4
It is not an important patch, but I got some warnings:
fs/xfs/xfs_alloc.c: In function ‘xfs_alloc_ag_vextent_near’:
fs/xfs/xfs_alloc.c:694:15: warning: ‘ltlena’ may be used uninitialized in this
function
fs/xfs/xfs_alloc.c:683:15: warning: ‘gtlena’ may be used uninitialized in this
function
And I hate warnings.
So I initialized that structures with 0. I attached the "patch".
Sorry, I am not a C or a kernel expert. But I try do something usefull :)
I am sorry If I did something wrong and thanks for your time.
Ionut Gabriel Popescu
[-- Attachment #2: patch.diff --]
[-- Type: application/octet-stream, Size: 905 bytes --]
--- xfs_alloc.c 2010-09-01 22:00:50.044004253 +0300
+++ xfs_alloc_2.c 2010-09-01 22:01:32.400007466 +0300
@@ -680,7 +680,7 @@
xfs_agblock_t gtbnoa; /* aligned ... */
xfs_extlen_t gtdiff; /* difference to right side entry */
xfs_extlen_t gtlen; /* length of right side entry */
- xfs_extlen_t gtlena; /* aligned ... */
+ xfs_extlen_t gtlena = {0}; /* aligned ... */
xfs_agblock_t gtnew; /* useful start bno of right side */
int error; /* error code */
int i; /* result code, temporary */
@@ -691,7 +691,7 @@
/*REFERENCED*/
xfs_agblock_t ltend; /* end bno of left side entry */
xfs_extlen_t ltlen; /* length of left side entry */
- xfs_extlen_t ltlena; /* aligned ... */
+ xfs_extlen_t ltlena = {0}; /* aligned ... */
xfs_agblock_t ltnew; /* useful start bno of left side */
xfs_extlen_t rlen; /* length of returned extent */
#if defined(DEBUG) && defined(__KERNEL__)
[-- Attachment #3: Type: text/plain, Size: 121 bytes --]
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Patch fs/xfs/xfs_alloc.c
2010-09-01 19:27 Patch fs/xfs/xfs_alloc.c Poyo VL
@ 2010-09-02 5:21 ` Dave Chinner
2010-09-02 7:41 ` Poyo VL
0 siblings, 1 reply; 4+ messages in thread
From: Dave Chinner @ 2010-09-02 5:21 UTC (permalink / raw)
To: Poyo VL; +Cc: xfs, aelder
On Wed, Sep 01, 2010 at 12:27:39PM -0700, Poyo VL wrote:
> File: fs/xfs/xfs_alloc.c
> Kernel: 2.6.35.4
>
> It is not an important patch, but I got some warnings:
>
> fs/xfs/xfs_alloc.c: In function ‘xfs_alloc_ag_vextent_near’:
> fs/xfs/xfs_alloc.c:694:15: warning: ‘ltlena’ may be used uninitialized in this
> function
> fs/xfs/xfs_alloc.c:683:15: warning: ‘gtlena’ may be used uninitialized in this
> function
I don't see this. What compiler version are you using?
Also, for posting patches, you might want to read and follow
Documentation/SubmittingPatches.txt. At minimum, you need to include
a Signed-off-by tag (we can't accept patches without that) and
attach your patches as plain text.
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 4+ messages in thread
* Patch fs/xfs/xfs_alloc.c
2010-09-02 5:21 ` Dave Chinner
@ 2010-09-02 7:41 ` Poyo VL
2010-09-16 15:38 ` Alex Elder
0 siblings, 1 reply; 4+ messages in thread
From: Poyo VL @ 2010-09-02 7:41 UTC (permalink / raw)
To: Dave Chinner; +Cc: xfs
Im using "gcc version 4.5.0 20100604 [gcc-4_5-branch revision 160292] (SUSE
Linux)"
Before compiling kernel (2.6.35.4) I selected from menuconfig all XFS "options":
Quota supprt, POSIX ACL support, Realtime subvolume support and Debugging
support. I saved the .config and I typed "make".
I got the following warnings:
fs/xfs/xfs_alloc.c: In function ‘xfs_alloc_ag_vextent_near’:
fs/xfs/xfs_alloc.c:694:15: warning: ‘ltlena’ may be used uninitialized in this
function
fs/xfs/xfs_alloc.c:683:15: warning: ‘gtlena’ may be used uninitialized in this
function
So I tried to initialize those variables (structures) with 0.
Sorry, I am not neither a C, nor a kernel expert.
"Patch":
--- xfs_alloc.c 2010-09-01 22:00:50.044004253 +0300
+++ xfs_alloc_2.c 2010-09-01 22:01:32.400007466 +0300
@@ -680,7 +680,7 @@
xfs_agblock_t gtbnoa; /* aligned ... */
xfs_extlen_t gtdiff; /* difference to right side entry */
xfs_extlen_t gtlen; /* length of right side entry */
- xfs_extlen_t gtlena; /* aligned ... */
+ xfs_extlen_t gtlena = {0}; /* aligned ... */
xfs_agblock_t gtnew; /* useful start bno of right side */
int error; /* error code */
int i; /* result code, temporary */
@@ -691,7 +691,7 @@
/*REFERENCED*/
xfs_agblock_t ltend; /* end bno of left side entry */
xfs_extlen_t ltlen; /* length of left side entry */
- xfs_extlen_t ltlena; /* aligned ... */
+ xfs_extlen_t ltlena = {0}; /* aligned ... */
xfs_agblock_t ltnew; /* useful start bno of left side */
xfs_extlen_t rlen; /* length of returned extent */
#if defined(DEBUG) && defined(__KERNEL__)
Signed-off-by: Ionut Gabriel Popescu <poyo_vl@yahoo.com>
Thank you for your time.
----- Original Message ----
From: Dave Chinner <david@fromorbit.com>
To: Poyo VL <poyo_vl@yahoo.com>
Cc: aelder@sgi.com; xfs@oss.sgi.com
Sent: Thu, September 2, 2010 8:21:06 AM
Subject: Re: Patch fs/xfs/xfs_alloc.c
On Wed, Sep 01, 2010 at 12:27:39PM -0700, Poyo VL wrote:
> File: fs/xfs/xfs_alloc.c
> Kernel: 2.6.35.4
>
> It is not an important patch, but I got some warnings:
>
> fs/xfs/xfs_alloc.c: In function ‘xfs_alloc_ag_vextent_near’:
> fs/xfs/xfs_alloc.c:694:15: warning: ‘ltlena’ may be used uninitialized in this
> function
> fs/xfs/xfs_alloc.c:683:15: warning: ‘gtlena’ may be used uninitialized in this
> function
I don't see this. What compiler version are you using?
Also, for posting patches, you might want to read and follow
Documentation/SubmittingPatches.txt. At minimum, you need to include
a Signed-off-by tag (we can't accept patches without that) and
attach your patches as plain text.
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Patch fs/xfs/xfs_alloc.c
2010-09-02 7:41 ` Poyo VL
@ 2010-09-16 15:38 ` Alex Elder
0 siblings, 0 replies; 4+ messages in thread
From: Alex Elder @ 2010-09-16 15:38 UTC (permalink / raw)
To: Poyo VL; +Cc: xfs
On Thu, 2010-09-02 at 00:41 -0700, Poyo VL wrote:
> Im using "gcc version 4.5.0 20100604 [gcc-4_5-branch revision 160292] (SUSE
> Linux)"
>
> Before compiling kernel (2.6.35.4) I selected from menuconfig all XFS "options":
> Quota supprt, POSIX ACL support, Realtime subvolume support and Debugging
> support. I saved the .config and I typed "make".
>
>
> I got the following warnings:
>
> fs/xfs/xfs_alloc.c: In function ‘xfs_alloc_ag_vextent_near’:
> fs/xfs/xfs_alloc.c:694:15: warning: ‘ltlena’ may be used uninitialized in this
> function
> fs/xfs/xfs_alloc.c:683:15: warning: ‘gtlena’ may be used uninitialized in this
> function
>
> So I tried to initialize those variables (structures) with 0.
>
> Sorry, I am not neither a C, nor a kernel expert.
I'm going to take this in (to top-of-tree), but
I'll be modifying it slightly (xfs_extlen_t is
not a struct, so I'll initialize with a simple 0).
It may well be that these variables never get used
without being set, but it's not at all clear from
a quick look so this is a simple fix.
Thanks for reporting the warning.
Reviewed-by: Alex Elder <aelder@sgi.com>
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-09-16 15:37 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-01 19:27 Patch fs/xfs/xfs_alloc.c Poyo VL
2010-09-02 5:21 ` Dave Chinner
2010-09-02 7:41 ` Poyo VL
2010-09-16 15:38 ` Alex Elder
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.