[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] crc: Add PCLMUL implementation
From: |
Jim Meyering |
Subject: |
Re: [PATCH] crc: Add PCLMUL implementation |
Date: |
Fri, 13 Dec 2024 11:03:17 -0800 |
That's not the way it usually works.
Typically, the author and/or person most familiar with the code is
listed as the Maintainer.
If you don't want to list yourself and no one else steps up, just mark
it as "all".
On Fri, Dec 13, 2024 at 10:52 AM Sam Russell <sam.h.russell@gmail.com> wrote:
>
> Jim, would you be willing to be the maintainer for this in gnulib? If so then
> I can fix the autoconf this week and get it ready to go
>
> On Fri, Dec 13, 2024, 19:29 Jim Meyering <jim@meyering.net> wrote:
>>
>> As soon as this is added to gnulib, I'll be happy to prepare for a new
>> gzip release that includes it.
>>
>> On Fri, Dec 13, 2024 at 9:45 AM Sam Russell <sam.h.russell@gmail.com> wrote:
>> >
>> > I should also add that I'm planning long term to try and get the major
>> > open source tools (e.g. gzip) using the most efficient algorithms for
>> > CRC32, so there will be more patches coming in future to match the ones
>> > I've submitted to coreutils. Are there any volunteers to be maintainer for
>> > these?
>> >
>> > On Fri, Dec 13, 2024, 17:52 Sam Russell <sam.h.russell@gmail.com> wrote:
>> >>
>> >> > I'd prefer of the crc PCLMUL feature is "opt-in" from a package
>> >> > maintainer point of view.
>> >>
>> >> fine by me, this just requires a change to the build script then? the
>> >> binding in crc.c has #ifdefs around it already
>> >>
>> >> > So this would be removed.
>> >>
>> >> ok, so we just need crc pclmul to be explicitly flagged on by a
>> >> downstream maintainer to make it work? I'll look into how to make that
>> >> work
>> >>
>> >> > Add yourself as maintainer of the new module please :)
>> >>
>> >> I can't commit to being a maintainer, but I can commit to being available
>> >> for a few months in the case of any bugs arising. My goal here is
>> >> improving the speed of gzip, would it be a better bet to work directly
>> >> with them to get it included there?
>> >>
- Re: [PATCH] crc: Add PCLMUL implementation, Sam Russell, 2024/12/12
- Re: [PATCH] crc: Add PCLMUL implementation, Simon Josefsson, 2024/12/13
- Re: [PATCH] crc: Add PCLMUL implementation, Sam Russell, 2024/12/13
- Re: [PATCH] crc: Add PCLMUL implementation, Sam Russell, 2024/12/13
- Re: [PATCH] crc: Add PCLMUL implementation, Jim Meyering, 2024/12/13
- Re: [PATCH] crc: Add PCLMUL implementation, Sam Russell, 2024/12/13
- Re: [PATCH] crc: Add PCLMUL implementation,
Jim Meyering <=
- Re: [PATCH] crc: Add PCLMUL implementation, Sam Russell, 2024/12/13
- Re: [PATCH] crc: Add PCLMUL implementation, Sam Russell, 2024/12/16
- Re: [PATCH] crc: Add PCLMUL implementation, Bruno Haible, 2024/12/16
- Re: [PATCH] crc: Add PCLMUL implementation, Sam Russell, 2024/12/16
- Re: [PATCH] crc: Add PCLMUL implementation, Simon Josefsson, 2024/12/17
- Re: [PATCH] crc: Add PCLMUL implementation, Sam Russell, 2024/12/17
- Re: [PATCH] crc: Add PCLMUL implementation, Simon Josefsson, 2024/12/17
- Re: [PATCH] crc: Add PCLMUL implementation, Sam Russell, 2024/12/17
- Re: [PATCH] crc: Add PCLMUL implementation, Bruno Haible, 2024/12/17
- Re: [PATCH] crc: Add PCLMUL implementation, Simon Josefsson, 2024/12/17