Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-19 Thread Arnd Bergmann
On Tue, Oct 19, 2021 at 4:16 PM Rich Felker wrote: > On Mon, Oct 18, 2021 at 04:42:04PM -0400, Rich Felker wrote: > > > > Well for little endian either would work (because adj is 0 :) but yes > > there are 3 such paddings before the second member on big endian, so > > it should be 3. > > How about

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-19 Thread Rich Felker
On Mon, Oct 18, 2021 at 04:42:04PM -0400, Rich Felker wrote: > On Mon, Oct 18, 2021 at 05:26:35PM +0200, Arnd Bergmann wrote: > > On Mon, Oct 18, 2021 at 5:08 PM Rich Felker wrote: > > > On Mon, Oct 18, 2021 at 04:58:03PM +0200, Takashi Iwai wrote: > > > > On Mon, 18 Oct 2021 16:43:00 +0200, Rich

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-18 Thread Rich Felker
On Mon, Oct 18, 2021 at 05:26:35PM +0200, Arnd Bergmann wrote: > On Mon, Oct 18, 2021 at 5:08 PM Rich Felker wrote: > > On Mon, Oct 18, 2021 at 04:58:03PM +0200, Takashi Iwai wrote: > > > On Mon, 18 Oct 2021 16:43:00 +0200, Rich Felker wrote: > > > > No, I don't think so. The musl translator is to

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-18 Thread Arnd Bergmann
On Mon, Oct 18, 2021 at 5:08 PM Rich Felker wrote: > On Mon, Oct 18, 2021 at 04:58:03PM +0200, Takashi Iwai wrote: > > On Mon, 18 Oct 2021 16:43:00 +0200, Rich Felker wrote: > > No, I don't think so. The musl translator is to translate between the > time64 ioctl structures and the old time32 ones

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-18 Thread Rich Felker
On Mon, Oct 18, 2021 at 04:58:03PM +0200, Takashi Iwai wrote: > On Mon, 18 Oct 2021 16:43:00 +0200, > Rich Felker wrote: > > > > On Sun, Oct 10, 2021 at 09:53:38AM +0200, Takashi Iwai wrote: > > > On Fri, 08 Oct 2021 14:07:39 +0200, > > > Rich Felker wrote: > > > > > > > > On Fri, Oct 08, 2021 at

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-18 Thread Takashi Iwai
On Mon, 18 Oct 2021 16:43:00 +0200, Rich Felker wrote: > > On Sun, Oct 10, 2021 at 09:53:38AM +0200, Takashi Iwai wrote: > > On Fri, 08 Oct 2021 14:07:39 +0200, > > Rich Felker wrote: > > > > > > On Fri, Oct 08, 2021 at 01:11:34PM +0200, Takashi Iwai wrote: > > > > On Fri, 08 Oct 2021 11:24:39 +0

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-18 Thread Rich Felker
On Sun, Oct 10, 2021 at 09:53:38AM +0200, Takashi Iwai wrote: > On Fri, 08 Oct 2021 14:07:39 +0200, > Rich Felker wrote: > > > > On Fri, Oct 08, 2021 at 01:11:34PM +0200, Takashi Iwai wrote: > > > On Fri, 08 Oct 2021 11:24:39 +0200, > > > Arnd Bergmann wrote: > > > > > > > > On Fri, Oct 8, 2021 a

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-10 Thread Takashi Iwai
On Fri, 08 Oct 2021 14:07:39 +0200, Rich Felker wrote: > > On Fri, Oct 08, 2021 at 01:11:34PM +0200, Takashi Iwai wrote: > > On Fri, 08 Oct 2021 11:24:39 +0200, > > Arnd Bergmann wrote: > > > > > > On Fri, Oct 8, 2021 at 10:43 AM Takashi Iwai wrote: > > > > On Thu, 07 Oct 2021 18:51:58 +0200, Ri

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Rich Felker
On Fri, Oct 08, 2021 at 02:37:12PM +0200, Arnd Bergmann wrote: > On Fri, Oct 8, 2021 at 2:06 PM Rich Felker wrote: > > On Fri, Oct 08, 2021 at 11:24:39AM +0200, Arnd Bergmann wrote: > > > > > > I've tried to understand this part of musl's convert_ioctl_struct(), but > > > I just > > > can't figur

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Arnd Bergmann
On Fri, Oct 8, 2021 at 2:06 PM Rich Felker wrote: > On Fri, Oct 08, 2021 at 11:24:39AM +0200, Arnd Bergmann wrote: > > > > I've tried to understand this part of musl's convert_ioctl_struct(), but I > > just > > can't figure out whether it does the conversion based the on the layout that > > is cu

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Arnd Bergmann
On Fri, Oct 8, 2021 at 1:53 PM Takashi Iwai wrote: > On Fri, 08 Oct 2021 13:45:45 +0200, Arnd Bergmann wrote: > > On Fri, Oct 8, 2021 at 1:11 PM Takashi Iwai wrote: > > > On Fri, 08 Oct 2021 11:24:39 +0200, Arnd Bergmann wrote: > > > The below is a revised kernel patch (again untested), just corr

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Rich Felker
On Fri, Oct 08, 2021 at 01:11:34PM +0200, Takashi Iwai wrote: > On Fri, 08 Oct 2021 11:24:39 +0200, > Arnd Bergmann wrote: > > > > On Fri, Oct 8, 2021 at 10:43 AM Takashi Iwai wrote: > > > On Thu, 07 Oct 2021 18:51:58 +0200, Rich Felker wrote: > > > > On Thu, Oct 07, 2021 at 06:18:52PM +0200, Tak

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Rich Felker
On Fri, Oct 08, 2021 at 11:24:39AM +0200, Arnd Bergmann wrote: > On Fri, Oct 8, 2021 at 10:43 AM Takashi Iwai wrote: > > On Thu, 07 Oct 2021 18:51:58 +0200, Rich Felker wrote: > > > On Thu, Oct 07, 2021 at 06:18:52PM +0200, Takashi Iwai wrote: > > > > @@ -557,11 +558,15 @@ struct __snd_pcm_sync_pt

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Takashi Iwai
On Fri, 08 Oct 2021 13:45:45 +0200, Arnd Bergmann wrote: > > On Fri, Oct 8, 2021 at 1:11 PM Takashi Iwai wrote: > > On Fri, 08 Oct 2021 11:24:39 +0200, Arnd Bergmann wrote: > > > > > > > I've tried to understand this part of musl's convert_ioctl_struct(), but > > > I just > > > can't figure out

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Arnd Bergmann
On Fri, Oct 8, 2021 at 1:11 PM Takashi Iwai wrote: > On Fri, 08 Oct 2021 11:24:39 +0200, Arnd Bergmann wrote: > > > > I've tried to understand this part of musl's convert_ioctl_struct(), but I > > just > > can't figure out whether it does the conversion based the on the layout that > > is curren

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Takashi Iwai
On Fri, 08 Oct 2021 11:24:39 +0200, Arnd Bergmann wrote: > > On Fri, Oct 8, 2021 at 10:43 AM Takashi Iwai wrote: > > On Thu, 07 Oct 2021 18:51:58 +0200, Rich Felker wrote: > > > On Thu, Oct 07, 2021 at 06:18:52PM +0200, Takashi Iwai wrote: > > > > @@ -557,11 +558,15 @@ struct __snd_pcm_sync_ptr {

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Arnd Bergmann
On Fri, Oct 8, 2021 at 10:43 AM Takashi Iwai wrote: > On Thu, 07 Oct 2021 18:51:58 +0200, Rich Felker wrote: > > On Thu, Oct 07, 2021 at 06:18:52PM +0200, Takashi Iwai wrote: > > @@ -557,11 +558,15 @@ struct __snd_pcm_sync_ptr { > #if defined(__BYTE_ORDER) ? __BYTE_ORDER == __BIG_ENDIAN : > defi

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Takashi Iwai
On Fri, 08 Oct 2021 10:43:24 +0200, Takashi Iwai wrote: > > On Thu, 07 Oct 2021 18:51:58 +0200, > Rich Felker wrote: > > > > On Thu, Oct 07, 2021 at 06:18:52PM +0200, Takashi Iwai wrote: > > > On Thu, 07 Oct 2021 18:06:36 +0200, > > > Rich Felker wrote: > > > > > > > > On Thu, Oct 07, 2021 at 05

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-08 Thread Takashi Iwai
On Thu, 07 Oct 2021 18:51:58 +0200, Rich Felker wrote: > > On Thu, Oct 07, 2021 at 06:18:52PM +0200, Takashi Iwai wrote: > > On Thu, 07 Oct 2021 18:06:36 +0200, > > Rich Felker wrote: > > > > > > On Thu, Oct 07, 2021 at 05:33:19PM +0200, Takashi Iwai wrote: > > > > On Thu, 07 Oct 2021 15:11:00 +0

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-07 Thread Rich Felker
On Thu, Oct 07, 2021 at 06:18:52PM +0200, Takashi Iwai wrote: > On Thu, 07 Oct 2021 18:06:36 +0200, > Rich Felker wrote: > > > > On Thu, Oct 07, 2021 at 05:33:19PM +0200, Takashi Iwai wrote: > > > On Thu, 07 Oct 2021 15:11:00 +0200, > > > Arnd Bergmann wrote: > > > > > > > > On Thu, Oct 7, 2021

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-07 Thread Takashi Iwai
On Thu, 07 Oct 2021 18:06:36 +0200, Rich Felker wrote: > > On Thu, Oct 07, 2021 at 05:33:19PM +0200, Takashi Iwai wrote: > > On Thu, 07 Oct 2021 15:11:00 +0200, > > Arnd Bergmann wrote: > > > > > > On Thu, Oct 7, 2021 at 2:43 PM Takashi Iwai wrote: > > > > On Thu, 07 Oct 2021 13:48:44 +0200, Ar

Re: [Y2038] [musl] Re: [alsa-devel] [PATCH v7 8/9] ALSA: add new 32-bit layout for snd_pcm_mmap_status/control

2021-10-07 Thread Rich Felker
On Thu, Oct 07, 2021 at 05:33:19PM +0200, Takashi Iwai wrote: > On Thu, 07 Oct 2021 15:11:00 +0200, > Arnd Bergmann wrote: > > > > On Thu, Oct 7, 2021 at 2:43 PM Takashi Iwai wrote: > > > On Thu, 07 Oct 2021 13:48:44 +0200, Arnd Bergmann wrote: > > > > On Thu, Oct 7, 2021 at 12:53 PM Takashi Iwa