move swblk_t from sys/types.h to sys/blist.h and use it in blist (was: change its type and us Re: patch: change swblk_t type and use it in blist)

2022-08-06 Thread Jeremie Courreges-Anglas
On Sat, Aug 06 2022, Sebastien Marie wrote: > On Sat, Aug 06, 2022 at 02:19:31AM +0200, Jeremie Courreges-Anglas wrote: >> On Fri, Aug 05 2022, Sebastien Marie wrote: >> > Hi, >> > >> > When initially ported blist from DragonFlyBSD, we used custom type bsblk_t >> > and >> > bsbmp_t instead of t

Re: patch: change swblk_t type and use it in blist

2022-08-05 Thread Sebastien Marie
On Sat, Aug 06, 2022 at 02:19:31AM +0200, Jeremie Courreges-Anglas wrote: > On Fri, Aug 05 2022, Sebastien Marie wrote: > > Hi, > > > > When initially ported blist from DragonFlyBSD, we used custom type bsblk_t > > and > > bsbmp_t instead of the one used by DragonFlyBSD (swblk_t and u_swblk_t).

Re: patch: change swblk_t type and use it in blist

2022-08-05 Thread Theo de Raadt
Todd C. Miller wrote: > On Sat, 06 Aug 2022 02:19:31 +0200, Jeremie Courreges-Anglas wrote: > > > This seems fair, but maybe we should just zap the type from sys/types.h and > > define it only in sys/blist.h, as done in DragonflyBSD? > > Fine with me. It is a non-standard type not used by base

Re: patch: change swblk_t type and use it in blist

2022-08-05 Thread Todd C . Miller
On Sat, 06 Aug 2022 02:19:31 +0200, Jeremie Courreges-Anglas wrote: > This seems fair, but maybe we should just zap the type from sys/types.h and > define it only in sys/blist.h, as done in DragonflyBSD? Fine with me. It is a non-standard type not used by base. - todd

Re: patch: change swblk_t type and use it in blist

2022-08-05 Thread Jeremie Courreges-Anglas
On Fri, Aug 05 2022, Sebastien Marie wrote: > Hi, > > When initially ported blist from DragonFlyBSD, we used custom type bsblk_t > and > bsbmp_t instead of the one used by DragonFlyBSD (swblk_t and u_swblk_t). > > The reason was swblk_t is already defined on OpenBSD, and was incompatible > with

Re: patch: change swblk_t type and use it in blist

2022-08-05 Thread Martin Pieuchot
On 05/08/22(Fri) 18:10, Sebastien Marie wrote: > Hi, > > When initially ported blist from DragonFlyBSD, we used custom type bsblk_t > and > bsbmp_t instead of the one used by DragonFlyBSD (swblk_t and u_swblk_t). > > The reason was swblk_t is already defined on OpenBSD, and was incompatible >

patch: change swblk_t type and use it in blist

2022-08-05 Thread Sebastien Marie
Hi, When initially ported blist from DragonFlyBSD, we used custom type bsblk_t and bsbmp_t instead of the one used by DragonFlyBSD (swblk_t and u_swblk_t). The reason was swblk_t is already defined on OpenBSD, and was incompatible with blist (int32_t). It is defined, but not used (outside some