All of lore.kernel.org
 help / color / mirror / Atom feed
From: "\"Martin Krüger\"" <m_a_krueger@gmx.de>
To: Pavel Hofman <pavel.hofman@insite.cz>
Cc: Takashi Iwai <tiwai@suse.de>, alsa-devel@alsa-project.org
Subject: Re: PATCH - MIDI on ice1724 - real-time kernel problem SOLVED(?)
Date: Thu, 29 May 2008 16:03:49 +0200	[thread overview]
Message-ID: <20080529140349.115520@gmx.net> (raw)
In-Reply-To: <483E78E7.4020106@insite.cz>

Pavel Hofman schrieb:
> Hi Martin,
>
> I will try a newer kernel on my home Ubuntu 7.10.
>
> We really need to find out what freezes your system. Is it a complete 
> freeze, or just heavy overload?
>
> Originally, I experienced a complete freeze caused by infinite loop in 
> IRQ handler ice1724.c:snd_vt1724_interrupt. This loop is already avoided 
> by the timeout variable. Do you run with  CONFIG_SND_DEBUG? Does dmesg 
> list any of the "ice1724: Too long irq loop..." messages?
>
> Later on, I experienced an IRQ flood, caused by intermittent throwing 
> ICE1724 IRQ (MPU transmit). I could see the IRQxx "process" hogging the 
> CPU in top. If that is the case, please add some debug statements to 
> ice1724.c:snd_vt1724_interrupt and look which IRQ gets fired (variable 
> status).
>
> In any case, please make sure you are using the latest GIT version with 
> all the patches, so that we all work on the same code.
>
> Disabling MIDI OUT - I have not tested the following patch, perhaps it 
> needs some changes:
>
>
> diff --git a/pci/ice1712/ice1724.c b/pci/ice1712/ice1724.c
> index e596d77..13695de 100644
> --- a/pci/ice1712/ice1724.c
> +++ b/pci/ice1712/ice1724.c
> @@ -2546,7 +2546,7 @@ static int __devinit snd_vt1724_probe(struct 
> pci_dev *pci,
>                  if (ice->eeprom.data[ICE_EEP2_SYSCONF] & 
> VT1724_CFG_MPU401) {
>                          struct snd_rawmidi *rmidi;
>
> -                       err = snd_rawmidi_new(card, "MIDI", 0, 1, 1, 
> &rmidi);
> +                       err = snd_rawmidi_new(card, "MIDI", 0, 0, 1, 
> &rmidi);
>                          if (err < 0) {
>                                  snd_card_free(card);
>                                  return err;
> @@ -2554,11 +2554,7 @@ static int __devinit snd_vt1724_probe(struct 
> pci_dev *pci
>                          ice->rmidi[0] = rmidi;
>                          rmidi->private_data = ice;
>                          strcpy(rmidi->name, "ICE1724 MIDI");
> -                       rmidi->info_flags = SNDRV_RAWMIDI_INFO_OUTPUT |
> -                                           SNDRV_RAWMIDI_INFO_INPUT |
> -                                           SNDRV_RAWMIDI_INFO_DUPLEX;
> -                       snd_rawmidi_set_ops(rmidi, 
> SNDRV_RAWMIDI_STREAM_OUTPUT,
> -                                           &vt1724_midi_output_ops);
> +                       rmidi->info_flags = SNDRV_RAWMIDI_INFO_INPUT;
>                          snd_rawmidi_set_ops(rmidi, 
> SNDRV_RAWMIDI_STREAM_INPUT,
>                                              &vt1724_midi_input_ops);
>
>
>
>
>
>
> Regards,
>
> Pavel.
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>   

Hi Pavel,

now i'm confused. I just compiled the current daily snapshot with
"./configure --with-isapnp=no --with-sequencer=yes --with-oss=no 
--with-debug=verbose --with-cards=ice1724".

That should give as much debug messages as i need, as i understood. I 
get only one Message, telling me that ice1724 is using the defined 
eeprom. After using the midi output port, there is no more message in 
any log at all.

Due to this, it is really hard to understand the problem, for me it 
looks like a complete freeze. If i would get "just" a heavy overload, 
the numlock should still react within 10 minutes, and the cpu cooler 
should be louder. Both of this should also happen during an irq flood, 
if i understood that right.

Do you have any idea how to find out whats happening there? I will try 
your patch this evening.

Thanks a lot for your help,
regards,

Martin

  reply	other threads:[~2008-05-29 14:03 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20080520112539.81C9D2484D@alsa0.perex.cz>
2008-05-23 10:26 ` [alsa-cvslog] alsa-kmirror: ALSA kernel mirror repository branch, master now at 9d46f4a919532c3f29a4ca1df3a4ce4686b11f37 Thierry Vignaud
2008-05-24 16:53   ` PATCH - MIDI on ice1724 - real-time kernel problem SOLVED(?) Martin Krüger
2008-05-26  7:57     ` Pavel Hofman
2008-05-26 10:54       ` Martin Krüger
2008-05-26 14:53         ` Takashi Iwai
2008-05-26 16:29           ` Martin Krüger
2008-05-28 11:58             ` Martin Krüger
2008-05-29  9:35               ` Pavel Hofman
2008-05-29 14:03                 ` "Martin Krüger" [this message]
2008-05-29 16:14                   ` Pavel Hofman
2008-06-07 21:36                     ` Pavel Hofman
2008-06-24 11:07                       ` Martin Krüger
2008-06-26 21:01                         ` Martin Krüger
2008-06-27 13:54                           ` Pavel Hofman
2008-06-27 14:24                             ` Takashi Iwai
2008-06-18  9:58   ` can we make the log appears in subject instead of useless sha1 ID? Thierry Vignaud
2008-06-19  9:54     ` 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=20080529140349.115520@gmx.net \
    --to=m_a_krueger@gmx.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=pavel.hofman@insite.cz \
    --cc=tiwai@suse.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.