linux-kernel - Re: [PATCH] h8300: move definition of __kernel_size_t etc. to posix

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Tue, 21 Apr 2020 15:24:04 +0900
From:   Masahiro Yamada <masahiroy@...nel.org>
To:     Yoshinori Sato <ysato@...rs.sourceforge.jp>,
        "moderated list:H8/300 ARCHITECTURE" 
        <uclinux-h8-devel@...ts.sourceforge.jp>
Cc:     Arnd Bergmann <arnd@...db.de>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] h8300: move definition of __kernel_size_t etc. to posix_types.h

Sato-san,

Your h8300 repository is strange now.


See the following two commits in h8300-next.


commit de98871736825349b367275b0790e42ea4157350
Author:     Yoshinori Sato <ysato@...rs.sourceforge.jp>
AuthorDate: Thu Apr 16 13:43:01 2020 +0900
Commit:     Yoshinori Sato <ysato@...rs.sourceforge.jp>
CommitDate: Thu Apr 16 13:43:01 2020 +0900

    h8300: move definition of __kernel_size_t etc. to posix_types.h

    These types should be defined in posix_types.h, not in bitsperlong.h .

    With these defines moved, h8300-specific bitsperlong.h is no longer
    needed since Kbuild will automatically create a wrapper of
    include/uapi/asm-generic/bitsperlong.h

    Signed-off-by: Masahiro Yamada <yamada.masahiro@...ionext.com>
    Signed-off-by: Yoshinori Sato <ysato@...rs.sourceforge.jp>

commit ece7be2c964c2aa5015e74628ca33c03a5ff4e47
Author:     Masahiro Yamada <yamada.masahiro@...ionext.com>
AuthorDate: Tue Jul 23 19:21:06 2019 +0900
Commit:     Yoshinori Sato <ysato@...rs.sourceforge.jp>
CommitDate: Thu Apr 16 13:40:31 2020 +0900

    h8300: move definition of __kernel_size_t etc. to posix_types.h

    These types should be defined in posix_types.h, not in bitsperlong.h .

    With these defines moved, h8300-specific bitsperlong.h is no longer
    needed since Kbuild will automatically create a wrapper of
    include/uapi/asm-generic/bitsperlong.h

    Signed-off-by: Masahiro Yamada <yamada.masahiro@...ionext.com>
    Signed-off-by: Yoshinori Sato <ysato@...rs.sourceforge.jp>




Since you were silent for a long time,
I pushed the patch to Linus via my tree.
(c9ee4bf9e0f5e353ae1c81ee30a373b6653329d7)


Now, your recent rebase is wrong.
Please drop both commits above.


BTW, are you still maintaining h8300 (and sh) ?


Masahiro Yamada








On Sat, Jul 27, 2019 at 10:46 PM Yoshinori Sato
<ysato@...rs.sourceforge.jp> wrote:
>
> On Tue, 23 Jul 2019 19:21:06 +0900,
> Masahiro Yamada wrote:
> >
> > These types should be defined in posix_types.h, not in bitsperlong.h .
> >
> > With these defines moved, h8300-specific bitsperlong.h is no longer
> > needed since Kbuild will automatically create a wrapper of
> > include/uapi/asm-generic/bitsperlong.h
> >
> > Signed-off-by: Masahiro Yamada <yamada.masahiro@...ionext.com>
> > ---
> >
> >  arch/h8300/include/uapi/asm/bitsperlong.h | 15 ---------------
> >  arch/h8300/include/uapi/asm/posix_types.h | 13 +++++++++++++
> >  2 files changed, 13 insertions(+), 15 deletions(-)
> >  delete mode 100644 arch/h8300/include/uapi/asm/bitsperlong.h
> >  create mode 100644 arch/h8300/include/uapi/asm/posix_types.h
> >
> > diff --git a/arch/h8300/include/uapi/asm/bitsperlong.h b/arch/h8300/include/uapi/asm/bitsperlong.h
> > deleted file mode 100644
> > index a33e358f1c1b..000000000000
> > --- a/arch/h8300/include/uapi/asm/bitsperlong.h
> > +++ /dev/null
> > @@ -1,15 +0,0 @@
> > -/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
> > -#ifndef _UAPI__ASM_H8300_BITS_PER_LONG
> > -#define _UAPI__ASM_H8300_BITS_PER_LONG
> > -
> > -#include <asm-generic/bitsperlong.h>
> > -
> > -#if !defined(__ASSEMBLY__)
> > -/* h8300-unknown-linux required long */
> > -#define __kernel_size_t __kernel_size_t
> > -typedef unsigned long        __kernel_size_t;
> > -typedef long         __kernel_ssize_t;
> > -typedef long         __kernel_ptrdiff_t;
> > -#endif
> > -
> > -#endif /* _UAPI__ASM_H8300_BITS_PER_LONG */
> > diff --git a/arch/h8300/include/uapi/asm/posix_types.h b/arch/h8300/include/uapi/asm/posix_types.h
> > new file mode 100644
> > index 000000000000..3efc9dd59476
> > --- /dev/null
> > +++ b/arch/h8300/include/uapi/asm/posix_types.h
> > @@ -0,0 +1,13 @@
> > +/* SPDX-License-Identifier: GPL-2.0-only WITH Linux-syscall-note */
> > +#ifndef _UAPI_ASM_POSIX_TYPES_H
> > +#define _UAPI_ASM_POSIX_TYPES_H
> > +
> > +/* h8300-unknown-linux required long */
> > +#define __kernel_size_t __kernel_size_t
> > +typedef unsigned long        __kernel_size_t;
> > +typedef long         __kernel_ssize_t;
> > +typedef long         __kernel_ptrdiff_t;
> > +
> > +#include <asm-generic/posix_types.h>
> > +
> > +#endif /* _UAPI_ASM_POSIX_TYPES_H */
> > --
> > 2.17.1
> >
>
> Applied h8300-next.
> Thanks.
>
> --
> Yosinori Sato



-- 
Best Regards
Masahiro Yamada