| 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
| ||
|
Message-ID: <s5hppdc8iy7.wl-tiwai@suse.de>
Date: Tue, 28 Oct 2014 13:41:52 +0100
From: Takashi Iwai <tiwai@...e.de>
To: Aya Mahfouz <mahfouz.saif.elyazal@...il.com>
Cc: perex@...ex.cz, bhelgaas@...gle.com, benoit.taine@...6.fr,
arnd@...db.de, opw-kernel@...glegroups.com,
alsa-devel@...a-project.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] sound: pci: pcxhr: convert timeval to ktime_t
At Tue, 28 Oct 2014 14:35:11 +0200,
Aya Mahfouz wrote:
>
> On Tue, Oct 28, 2014 at 12:59:59PM +0100, Takashi Iwai wrote:
> > At Tue, 28 Oct 2014 13:29:56 +0200,
> > Aya Mahfouz wrote:
> > >
> > > This patch is concerned with migrating the time variables in the pcxhr
> > > module found in the sound driver. The changes are concerend with the
> > > y2038 problem where timeval will overflow in the year 2038. ktime_t
> > > was used instead of timeval to get the wall time. The difference
> > > is displayed now in nanoseconds instead of microseconds.
> > >
> > > Signed-off-by: Aya Mahfouz <mahfouz.saif.elyazal@...il.com>
> > > Reviewed-by: Arnd Bergmann <arnd@...db.de>
> >
> > It gets compile errors like below. Please submit a patch at least
> > after build testing.
> >
>
> I do actually compile all my patches before submitting them. Since
> the change is minor I use make M=sound/pci. My second patch is based
> on your compile errors. Kindly let me know how do you compile the
> patch to avoid breaking your system in the future.
Did you set CONFIG_SND_DEBUG_VERBOSE=y? Otherwise it makes no sense,
obviously.
Takashi
>
> > thanks,
> >
> > Takashi
> >
>
> Kind Regards,
> Aya Saif El-yazal Mahfouz
>
> > ===
> >
> > sound/pci/pcxhr/pcxhr_core.c: In function ‘pcxhr_set_pipe_state’:
> > sound/pci/pcxhr/pcxhr_core.c:913:10: error: ‘start’ redeclared as different kind of symbol
> > ktime_t start, stop, diff;
> > ^
> > sound/pci/pcxhr/pcxhr_core.c:907:27: note: previous definition of ‘start’ was here
> > int capture_mask, int start)
> > ^
> > In file included from include/linux/printk.h:260:0,
> > from include/linux/kernel.h:13,
> > from include/linux/delay.h:10,
> > from sound/pci/pcxhr/pcxhr_core.c:23:
> > sound/pci/pcxhr/pcxhr_core.c:923:13: error: used union type value where scalar is required
> > start ? "START" : "STOP", audio_mask, state);
> > ^
> > include/linux/dynamic_debug.h:87:9: note: in definition of macro ‘dynamic_dev_dbg’
> > ##__VA_ARGS__); \
> > ^
> > sound/pci/pcxhr/pcxhr_core.c:921:2: note: in expansion of macro ‘dev_dbg’
> > dev_dbg(&mgr->pci->dev,
> > ^
> > sound/pci/pcxhr/pcxhr_core.c:924:6: error: used union type value where scalar is required
> > if (start) {
> > ^
> > sound/pci/pcxhr/pcxhr_core.c:950:38: error: used union type value where scalar is required
> > if ((state & audio_mask) == (start ? audio_mask : 0))
> > ^
> > sound/pci/pcxhr/pcxhr_core.c:958:6: error: wrong type argument to unary exclamation mark
> > if (!start) {
> > ^
> > scripts/Makefile.build:257: recipe for target 'sound/pci/pcxhr/pcxhr_core.o' failed
> >
> >
> > > ---
> > > sound/pci/pcxhr/pcxhr.c | 10 ++++++----
> > > sound/pci/pcxhr/pcxhr_core.c | 10 ++++++----
> > > 2 files changed, 12 insertions(+), 8 deletions(-)
> > >
> > > diff --git a/sound/pci/pcxhr/pcxhr.c b/sound/pci/pcxhr/pcxhr.c
> > > index b854fc5..3f4607c 100644
> > > --- a/sound/pci/pcxhr/pcxhr.c
> > > +++ b/sound/pci/pcxhr/pcxhr.c
> > > @@ -711,8 +711,9 @@ static void pcxhr_start_linked_stream(struct pcxhr_mgr *mgr)
> > > int playback_mask = 0;
> > >
> > > #ifdef CONFIG_SND_DEBUG_VERBOSE
> > > - struct timeval my_tv1, my_tv2;
> > > - do_gettimeofday(&my_tv1);
> > > + ktime_t start, stop, diff;
> > > +
> > > + start = ktime_get();
> > > #endif
> > > mutex_lock(&mgr->setup_mutex);
> > >
> > > @@ -823,9 +824,10 @@ static void pcxhr_start_linked_stream(struct pcxhr_mgr *mgr)
> > > mutex_unlock(&mgr->setup_mutex);
> > >
> > > #ifdef CONFIG_SND_DEBUG_VERBOSE
> > > - do_gettimeofday(&my_tv2);
> > > + stop = ktime_get();
> > > + diff = ktime_sub(stop, start);
> > > dev_dbg(&mgr->pci->dev, "***TRIGGER START*** TIME = %ld (err = %x)\n",
> > > - (long)(my_tv2.tv_usec - my_tv1.tv_usec), err);
> > > + (long)(ktime_to_ns(diff)), err);
> > > #endif
> > > }
> > >
> > > diff --git a/sound/pci/pcxhr/pcxhr_core.c b/sound/pci/pcxhr/pcxhr_core.c
> > > index a584acb..be3c003 100644
> > > --- a/sound/pci/pcxhr/pcxhr_core.c
> > > +++ b/sound/pci/pcxhr/pcxhr_core.c
> > > @@ -910,8 +910,9 @@ int pcxhr_set_pipe_state(struct pcxhr_mgr *mgr, int playback_mask,
> > > int audio_mask;
> > >
> > > #ifdef CONFIG_SND_DEBUG_VERBOSE
> > > - struct timeval my_tv1, my_tv2;
> > > - do_gettimeofday(&my_tv1);
> > > + ktime_t start, stop, diff;
> > > +
> > > + start = ktime_get();
> > > #endif
> > > audio_mask = (playback_mask |
> > > (capture_mask << PCXHR_PIPE_STATE_CAPTURE_OFFSET));
> > > @@ -960,9 +961,10 @@ int pcxhr_set_pipe_state(struct pcxhr_mgr *mgr, int playback_mask,
> > > return err;
> > > }
> > > #ifdef CONFIG_SND_DEBUG_VERBOSE
> > > - do_gettimeofday(&my_tv2);
> > > + stop = ktime_get();
> > > + diff = ktime_sub(stop, start);
> > > dev_dbg(&mgr->pci->dev, "***SET PIPE STATE*** TIME = %ld (err = %x)\n",
> > > - (long)(my_tv2.tv_usec - my_tv1.tv_usec), err);
> > > + (long)(ktime_to_ns(diff)), err);
> > > #endif
> > > return 0;
> > > }
> > > --
> > > 1.9.3
> > >
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists