On Thu, Aug 13, 2015 at 08:17:17AM -0500, Mark Tinguely wrote: > On 08/13/15 04:26, Jan Kara wrote: > >struct xfs_attr_leafblock contains 'entries' array which is declared > >with size 1 altough it can in fact contain much more entries. Since this > >array is followed by further struct members, gcc (at least in version > >4.8.3) thinks that the array has the fixed size of 1 element and thus > >may optimize away all accesses beyond the end of array resulting in > >non-working code. This problem was only observed with userspace code in > >xfsprogs, however it's better to be safe in kernel as well and have > >matching kernel and xfsprogs definitions. > > > >Signed-off-by: Jan Kara<jack@xxxxxxxx> > >--- > > I hit this bug in SLES12 and RHEL7 XFS kernel code. It is the gcc > 4.8's new loop optimization routine. A person can bypass it with the > option: > -fno-aggressive-loop-optimizations > but this fixes the source of the problem. > > Acked-by: Mark Tinguely <tinguely@xxxxxxx> Hi Mark - it's great that you had found and identified the exact compiler optimisation that was causing miscompilation of the XFS code. Troy, what's not great about this is that SGI hasn't seen fit to report this bug upstream or to their distro partners in a prompt manner. Can you please make sure that any other XFS bug fixes that SGI are sitting on get communicated to the appropriate venue so they can be addressed without undue delay? Thanks, Dave. -- Dave Chinner david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
