From: Baolin Wang <baolin.wang@linaro.org>
To: perex@perex.cz, tiwai@suse.com, arnd@arndb.de
Cc: lgirdwood@gmail.com, broonie@kernel.org, o-takashi@sakamocchi.jp,
mingo@kernel.org, elfring@users.sourceforge.net,
dan.carpenter@oracle.com, jeeja.kp@intel.com,
vinod.koul@intel.com, guneshwor.o.singh@intel.com,
subhransu.s.prusty@intel.com, bhumirks@gmail.com,
gudishax.kranthikumar@intel.com, naveen.m@intel.com,
hardik.t.shah@intel.com, arvind.yadav.cs@gmail.com,
fabf@skynet.be, alsa-devel@alsa-project.org,
linux-kernel@vger.kernel.org, baolin.wang@linaro.org
Subject: [RFC PATCH v2 0/7] Fix year 2038 issue for sound subsystem
Date: Thu, 2 Nov 2017 19:06:50 +0800 [thread overview]
Message-ID: <cover.1509612175.git.baolin.wang@linaro.org> (raw)
Since many structures will use timespec type variables to record time stamp
in uapi/asound.h, which are not year 2038 safe on 32bit system. This patchset
tries to introduce new structures removing timespec type to compatible native
mode and compat mode.
Moreover this patchset also converts the internal structrures to use timespec64
type and related APIs.
Changes since v1:
- Flat all structues to make them more clear.
- Re-modify the convertion for struct snd_timer_tread.
- Use same coding style when converting struct snd_rawmidi_status.
- Use struct snd_timer_user_status64 in compat mode directly when converting
struct snd_timer_status.
- Add #ifdef __KERNEL__ when converting struct snd_ctl_elem_value.
- Re-modify the convertion for struct snd_pcm_sync_ptr.
- Fix some building errors.
Appreciated for any comments, especially for patch6 which I am not sure if it
is correct.
Baolin Wang (7):
sound: Replace timespec with timespec64
sound: core: Avoid using timespec for struct snd_pcm_status
sound: core: Avoid using timespec for struct snd_rawmidi_status
sound: core: Avoid using timespec for struct snd_timer_status
uapi: sound: Avoid using timespec for struct snd_ctl_elem_value
sound: core: Avoid using timespec for struct snd_pcm_sync_ptr
sound: core: Avoid using timespec for struct snd_timer_tread
include/sound/pcm.h | 126 +++++++++-
include/sound/timer.h | 4 +-
include/uapi/sound/asound.h | 19 +-
sound/core/pcm.c | 20 +-
sound/core/pcm_compat.c | 469 ++++++++++++++++++++++++++++---------
sound/core/pcm_lib.c | 33 ++-
sound/core/pcm_native.c | 220 +++++++++++++----
sound/core/rawmidi.c | 81 ++++++-
sound/core/rawmidi_compat.c | 121 +++++++---
sound/core/timer.c | 253 ++++++++++++++++----
sound/core/timer_compat.c | 67 ++----
sound/pci/hda/hda_controller.c | 10 +-
sound/soc/intel/skylake/skl-pcm.c | 4 +-
13 files changed, 1090 insertions(+), 337 deletions(-)
--
1.7.9.5
next reply other threads:[~2017-11-02 11:08 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-02 11:06 Baolin Wang [this message]
2017-11-02 11:06 ` [RFC PATCH v2 1/7] sound: Replace timespec with timespec64 Baolin Wang
2017-11-02 11:06 ` [RFC PATCH v2 2/7] sound: core: Avoid using timespec for struct snd_pcm_status Baolin Wang
2017-11-05 10:23 ` [alsa-devel] " Takashi Iwai
2017-11-05 13:48 ` Arnd Bergmann
2017-11-02 11:06 ` [RFC PATCH v2 3/7] sound: core: Avoid using timespec for struct snd_rawmidi_status Baolin Wang
2017-11-02 11:06 ` [RFC PATCH v2 4/7] sound: core: Avoid using timespec for struct snd_timer_status Baolin Wang
2017-11-02 11:06 ` [RFC PATCH v2 5/7] uapi: sound: Avoid using timespec for struct snd_ctl_elem_value Baolin Wang
2017-11-08 13:44 ` Takashi Sakamoto
2017-11-02 11:06 ` [RFC PATCH v2 6/7] sound: core: Avoid using timespec for struct snd_pcm_sync_ptr Baolin Wang
2017-11-02 11:06 ` [RFC PATCH v2 7/7] sound: core: Avoid using timespec for struct snd_timer_tread Baolin Wang
2017-11-05 10:29 ` [alsa-devel] " Takashi Iwai
2017-11-05 13:16 ` Arnd Bergmann
2017-11-05 16:59 ` Takashi Iwai
2017-11-06 16:33 ` Arnd Bergmann
2017-11-09 16:52 ` Takashi Iwai
2017-11-09 17:01 ` Arnd Bergmann
2017-11-09 18:11 ` Takashi Iwai
2017-11-09 23:20 ` Arnd Bergmann
2017-11-10 7:19 ` Takashi Iwai
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1509612175.git.baolin.wang@linaro.org \
--to=baolin.wang@linaro.org \
--cc=alsa-devel@alsa-project.org \
--cc=arnd@arndb.de \
--cc=arvind.yadav.cs@gmail.com \
--cc=bhumirks@gmail.com \
--cc=broonie@kernel.org \
--cc=dan.carpenter@oracle.com \
--cc=elfring@users.sourceforge.net \
--cc=fabf@skynet.be \
--cc=gudishax.kranthikumar@intel.com \
--cc=guneshwor.o.singh@intel.com \
--cc=hardik.t.shah@intel.com \
--cc=jeeja.kp@intel.com \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=naveen.m@intel.com \
--cc=o-takashi@sakamocchi.jp \
--cc=perex@perex.cz \
--cc=subhransu.s.prusty@intel.com \
--cc=tiwai@suse.com \
--cc=vinod.koul@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).