[PATCH v3] Fix POSIX compliance: Make blkcnt_t and blksize_t signed on alpha

Osama Abdelkader osama.abdelkader@gmail.com
Tue Nov 25 21:03:44 GMT 2025


On Mon, Nov 24, 2025 at 08:09:21PM +0000, Joseph Myers wrote:
> On Sun, 23 Nov 2025, Osama Abdelkader wrote:
> 
> > POSIX requires that blkcnt_t and blksize_t are signed integer types.
> > On alpha, these were incorrectly defined as unsigned types.
> > This change makes them signed to match:
> > - POSIX requirements
> > - All other architectures in glibc
> > 
> > Fixes #33355
> 
> How has this been tested?  I'd have expected c++-types-check to fail 
> because it verified the mangling of various types, including these ones, 
> hasn't accidentally changed.

Thanks for the review, yes we need to update the expected data
sysdeps/unix/sysv/linux/alpha/c++-types.data:

-blkcnt_t:j
-blksize_t:j
+blkcnt_t:l
+blksize_t:l

-fsblkcnt_t:i
+fsblkcnt_t:m

I prepared that for v4, but waiting for the feedback on first patch thread
to send it afterwards.

BR,
Osama

> 
> -- 
> Joseph S. Myers
> josmyers@redhat.com
> 


More information about the Libc-alpha mailing list