]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
ALSA: emu10k1: Rate-limit error messages about page errors
authorTakashi Iwai <tiwai@suse.de>
Thu, 17 May 2018 18:02:23 +0000 (20:02 +0200)
committerTakashi Iwai <tiwai@suse.de>
Thu, 17 May 2018 18:02:23 +0000 (20:02 +0200)
The error messages at sanity checks of memory pages tend to repeat too
many times once when it hits, and without the rate limit, it may flood
and become unreadable.  Replace such messages with the *_ratelimited()
variant.

Bugzilla: http://bugzilla.opensuse.org/show_bug.cgi?id=1093027
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/emu10k1/memory.c

index 5865f3b90b34ba36f96f6cb29eaeb1ff1aff8f0f..dbc7d8d0e1c434d61e279478b97ed29c7ad52229 100644 (file)
@@ -248,13 +248,13 @@ search_empty(struct snd_emu10k1 *emu, int size)
 static int is_valid_page(struct snd_emu10k1 *emu, dma_addr_t addr)
 {
        if (addr & ~emu->dma_mask) {
-               dev_err(emu->card->dev,
+               dev_err_ratelimited(emu->card->dev,
                        "max memory size is 0x%lx (addr = 0x%lx)!!\n",
                        emu->dma_mask, (unsigned long)addr);
                return 0;
        }
        if (addr & (EMUPAGESIZE-1)) {
-               dev_err(emu->card->dev, "page is not aligned\n");
+               dev_err_ratelimited(emu->card->dev, "page is not aligned\n");
                return 0;
        }
        return 1;
@@ -345,7 +345,7 @@ snd_emu10k1_alloc_pages(struct snd_emu10k1 *emu, struct snd_pcm_substream *subst
                else
                        addr = snd_pcm_sgbuf_get_addr(substream, ofs);
                if (! is_valid_page(emu, addr)) {
-                       dev_err(emu->card->dev,
+                       dev_err_ratelimited(emu->card->dev,
                                "emu: failure page = %d\n", idx);
                        mutex_unlock(&hdr->block_mutex);
                        return NULL;