| Message ID | 20191127001023.63271-1-yzaikin@google.com (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | 39101b2265440dfe83646a4bfc33015c86f36d71 |
| Headers | show |
| Series | fs/ext4/inode-test: Fix inode test on 32 bit platforms. | expand |
On 11/26/19 5:10 PM, Iurii Zaikin wrote: > Fixes the issue caused by the fact that in C in the expression > of the form -1234L only 1234L is the actual literal, the unary > minus is an operation applied to the literal. Which means that > to express the lower bound for the type one has to negate the > upper bound and subtract 1. > > Signed-off-by: Iurii Zaikin <yzaikin@google.com> Thanks for sending the patch quickly. Please add Reported-by for Geert and also include the error he is seeing in the commit log. > --- > fs/ext4/inode-test.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/ext4/inode-test.c b/fs/ext4/inode-test.c > index 92a9da1774aa..bbce1c328d85 100644 > --- a/fs/ext4/inode-test.c > +++ b/fs/ext4/inode-test.c > @@ -25,7 +25,7 @@ > * For constructing the negative timestamp lower bound value. > * binary: 10000000 00000000 00000000 00000000 > */ > -#define LOWER_MSB_1 (-0x80000000L) > +#define LOWER_MSB_1 (-(UPPER_MSB_0) - 1L) /* avoid overflow */ > /* > * For constructing the negative timestamp upper bound value. > * binary: 11111111 11111111 11111111 11111111 > -- > 2.24.0.432.g9d3f5f5b63-goog > thanks, -- Shuah
> Please add Reported-by for Geert and also include the error he is > seeing in the commit log. Done
diff --git a/fs/ext4/inode-test.c b/fs/ext4/inode-test.c index 92a9da1774aa..bbce1c328d85 100644 --- a/fs/ext4/inode-test.c +++ b/fs/ext4/inode-test.c @@ -25,7 +25,7 @@ * For constructing the negative timestamp lower bound value. * binary: 10000000 00000000 00000000 00000000 */ -#define LOWER_MSB_1 (-0x80000000L) +#define LOWER_MSB_1 (-(UPPER_MSB_0) - 1L) /* avoid overflow */ /* * For constructing the negative timestamp upper bound value. * binary: 11111111 11111111 11111111 11111111
Fixes the issue caused by the fact that in C in the expression of the form -1234L only 1234L is the actual literal, the unary minus is an operation applied to the literal. Which means that to express the lower bound for the type one has to negate the upper bound and subtract 1. Signed-off-by: Iurii Zaikin <yzaikin@google.com> --- fs/ext4/inode-test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.24.0.432.g9d3f5f5b63-goog