[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Fri, 7 Jun 2019 07:42:15 +0200
From: Petr Vorel <petr.vorel@...il.com>
To: musl@...ts.openwall.com
Subject: Re: [PATCH 2/2] define mt_fileno and mt_fileno struct mtget
members as mt_blkno
Hi Rich,
...
> > +++ b/include/sys/mtio.h
> > @@ -54,8 +54,8 @@ struct mtget {
> > long mt_dsreg;
> > long mt_gstat;
> > long mt_erreg;
> > - int mt_fileno;
> > - int mt_blkno;
> > + daddr_t mt_fileno;
> > + daddr_t mt_blkno;
...
> Can you explain what problem this is supposed to fix? It definitely
> needs further discussion to determine what the right way is, but
> that's impossible without knowing the problem you're trying to solve.
Thanks for review. Not a problem for me actually. I've noticed, that glibc (and
thus uclibc-ng) and bionic follow kernel sources, which defines it long for mips
(and for sparc, but musl doesn't support it). Default is int [2]. Drop this
patch if this is not an issue.
> Rich
Kind regards,
Petr
[1] https://elixir.bootlin.com/linux/latest/source/arch/mips/include/uapi/asm/posix_types.h#L21
[2] https://elixir.bootlin.com/linux/latest/source/include/uapi/asm-generic/posix_types.h#L45
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
