Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1285096 > unrolled thread

radeon -Wmaybe-uninitialized crap

Started byBorislav Petkov <bp@alien8.de>
First post2015-12-07 10:00 +0100
Last post2015-12-11 11:40 +0100
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

Contents

  radeon -Wmaybe-uninitialized crap Borislav Petkov <bp@alien8.de> - 2015-12-07 10:00 +0100
    RE: radeon -Wmaybe-uninitialized crap "Deucher, Alexander" <Alexander.Deucher@amd.com> - 2015-12-07 17:50 +0100
      Re: radeon -Wmaybe-uninitialized crap Borislav Petkov <bp@alien8.de> - 2015-12-07 18:20 +0100
        Re: radeon -Wmaybe-uninitialized crap Borislav Petkov <bp@alien8.de> - 2015-12-11 11:40 +0100

#1285096 — radeon -Wmaybe-uninitialized crap

FromBorislav Petkov <bp@alien8.de>
Date2015-12-07 10:00 +0100
Subjectradeon -Wmaybe-uninitialized crap
Message-ID<qCZOi-2cf-11@gated-at.bofh.it>
Hi guys,

this just started appearing when building -rc4. Got fixes yet? :-)

In file included from drivers/gpu/drm/radeon/radeon_mode.h:37:0,
                 from drivers/gpu/drm/radeon/radeon.h:80,
                 from drivers/gpu/drm/radeon/r100.c:33:
drivers/gpu/drm/radeon/r100.c: In function ‘r100_bandwidth_update’:
include/drm/drm_fixed.h:64:13: warning: ‘crit_point_ff.full’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  u64 tmp = ((u64)A.full << 13);
             ^
drivers/gpu/drm/radeon/r100.c:3153:63: note: ‘crit_point_ff.full’ was declared here
  fixed20_12 peak_disp_bw, mem_bw, pix_clk, pix_clk2, temp_ff, crit_point_ff;
                                                               ^
drivers/gpu/drm/radeon/r100.c:3583:42: warning: ‘disp_drain_rate.full’ may be used uninitialized in this function [-Wmaybe-uninitialized]
     temp_ff.full = read_return_rate.full - disp_drain_rate.full;
                                          ^
-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1285797

From"Deucher, Alexander" <Alexander.Deucher@amd.com>
Date2015-12-07 17:50 +0100
Message-ID<qD799-75n-27@gated-at.bofh.it>
In reply to#1285096
DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogQm9yaXNsYXYgUGV0a292
IFttYWlsdG86YnBAYWxpZW44LmRlXQ0KPiBTZW50OiBNb25kYXksIERlY2VtYmVyIDA3LCAyMDE1
IDM6NTYgQU0NCj4gVG86IERldWNoZXIsIEFsZXhhbmRlcjsgS29lbmlnLCBDaHJpc3RpYW4NCj4g
Q2M6IGxrbWwNCj4gU3ViamVjdDogcmFkZW9uIC1XbWF5YmUtdW5pbml0aWFsaXplZCBjcmFwDQo+
IA0KPiBIaSBndXlzLA0KPiANCj4gdGhpcyBqdXN0IHN0YXJ0ZWQgYXBwZWFyaW5nIHdoZW4gYnVp
bGRpbmcgLXJjNC4gR290IGZpeGVzIHlldD8gOi0pDQoNCk9kZC4gIE5vdGhpbmcgcmVsYXRlZCB0
byB0aGVzZSB2YXJpYWJsZXMgaGFzIGNoYW5nZWQgaW4geWVhcnMuDQoNCkFsZXgNCg0KPiANCj4g
SW4gZmlsZSBpbmNsdWRlZCBmcm9tIGRyaXZlcnMvZ3B1L2RybS9yYWRlb24vcmFkZW9uX21vZGUu
aDozNzowLA0KPiAgICAgICAgICAgICAgICAgIGZyb20gZHJpdmVycy9ncHUvZHJtL3JhZGVvbi9y
YWRlb24uaDo4MCwNCj4gICAgICAgICAgICAgICAgICBmcm9tIGRyaXZlcnMvZ3B1L2RybS9yYWRl
b24vcjEwMC5jOjMzOg0KPiBkcml2ZXJzL2dwdS9kcm0vcmFkZW9uL3IxMDAuYzogSW4gZnVuY3Rp
b24g4oCYcjEwMF9iYW5kd2lkdGhfdXBkYXRl4oCZOg0KPiBpbmNsdWRlL2RybS9kcm1fZml4ZWQu
aDo2NDoxMzogd2FybmluZzog4oCYY3JpdF9wb2ludF9mZi5mdWxs4oCZIG1heSBiZSB1c2VkDQo+
IHVuaW5pdGlhbGl6ZWQgaW4gdGhpcyBmdW5jdGlvbiBbLVdtYXliZS11bmluaXRpYWxpemVkXQ0K
PiAgIHU2NCB0bXAgPSAoKHU2NClBLmZ1bGwgPDwgMTMpOw0KPiAgICAgICAgICAgICAgXg0KPiBk
cml2ZXJzL2dwdS9kcm0vcmFkZW9uL3IxMDAuYzozMTUzOjYzOiBub3RlOiDigJhjcml0X3BvaW50
X2ZmLmZ1bGzigJkgd2FzDQo+IGRlY2xhcmVkIGhlcmUNCj4gICBmaXhlZDIwXzEyIHBlYWtfZGlz
cF9idywgbWVtX2J3LCBwaXhfY2xrLCBwaXhfY2xrMiwgdGVtcF9mZiwNCj4gY3JpdF9wb2ludF9m
ZjsNCj4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgXg0KPiBkcml2ZXJzL2dwdS9kcm0vcmFkZW9uL3IxMDAuYzozNTgzOjQyOiB3
YXJuaW5nOiDigJhkaXNwX2RyYWluX3JhdGUuZnVsbOKAmSBtYXkNCj4gYmUgdXNlZCB1bmluaXRp
YWxpemVkIGluIHRoaXMgZnVuY3Rpb24gWy1XbWF5YmUtdW5pbml0aWFsaXplZF0NCj4gICAgICB0
ZW1wX2ZmLmZ1bGwgPSByZWFkX3JldHVybl9yYXRlLmZ1bGwgLSBkaXNwX2RyYWluX3JhdGUuZnVs
bDsNCj4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgXg0KPiAtLQ0K
PiBSZWdhcmRzL0dydXNzLA0KPiAgICAgQm9yaXMuDQo+IA0KPiBFQ08gdGlwICMxMDE6IFRyaW0g
eW91ciBtYWlscyB3aGVuIHlvdSByZXBseS4NCg==
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1285813

FromBorislav Petkov <bp@alien8.de>
Date2015-12-07 18:20 +0100
Message-ID<qD7Ca-7uL-15@gated-at.bofh.it>
In reply to#1285797
On Mon, Dec 07, 2015 at 04:15:22PM +0000, Deucher, Alexander wrote:
> Odd.  Nothing related to these variables has changed in years.

Odd indeed. Building that same config on two other boxes doesn't trip
those up so it must be something gcc-related on that particular box.

Hmmm.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1289404

FromBorislav Petkov <bp@alien8.de>
Date2015-12-11 11:40 +0100
Message-ID<qEthf-311-17@gated-at.bofh.it>
In reply to#1285813
On Mon, Dec 07, 2015 at 06:10:02PM +0100, Borislav Petkov wrote:
> On Mon, Dec 07, 2015 at 04:15:22PM +0000, Deucher, Alexander wrote:
> > Odd.  Nothing related to these variables has changed in years.
> 
> Odd indeed. Building that same config on two other boxes doesn't trip
> those up so it must be something gcc-related on that particular box.

Just appeared on another box (ph) too:

In file included from drivers/gpu/drm/radeon/radeon_mode.h:37:0,
                 from drivers/gpu/drm/radeon/radeon.h:80,
                 from drivers/gpu/drm/radeon/r100.c:33:
drivers/gpu/drm/radeon/r100.c: In function ‘r100_bandwidth_update’:
include/drm/drm_fixed.h:64:13: warning: ‘crit_point_ff.full’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  u64 tmp = ((u64)A.full << 13);
             ^
drivers/gpu/drm/radeon/r100.c:3153:63: note: ‘crit_point_ff.full’ was declared here
  fixed20_12 peak_disp_bw, mem_bw, pix_clk, pix_clk2, temp_ff, crit_point_ff;
                                                               ^
drivers/gpu/drm/radeon/r100.c:3583:42: warning: ‘disp_drain_rate.full’ may be used uninitialized in this function [-Wmaybe-uninitialized]
     temp_ff.full = read_return_rate.full - disp_drain_rate.full;
                                          ^
-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web