<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - [BAT][BRW] NULL pointer dereference in snd_hda_codec_generic"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=101246#c8">Comment # 8</a>
              on <a class="bz_bug_link 
          bz_status_NEEDINFO "
   title="NEEDINFO - [BAT][BRW] NULL pointer dereference in snd_hda_codec_generic"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=101246">bug 101246</a>
              from <span class="vcard"><a class="email" href="mailto:libin.yang@intel.com" title="Libin Yang <libin.yang@intel.com>"> <span class="fn">Libin Yang</span></a>
</span></b>
        <pre>(In reply to Martin Peres from <a href="show_bug.cgi?id=101246#c7">comment #7</a>)
<span class="quote">> (In reply to Libin Yang from <a href="show_bug.cgi?id=101246#c6">comment #6</a>)
> > My suggestion test is to try to find why it is using generic driver.

> Hmm, I get your point, but whatever the reason why, it should still not make
> any null pointer de-reference.</span >

Yes, there seems 2 issues: 1) why using generic; 2) the null pointer.

For the second issue, it seems you've already find a solution. I recommend you
submit a patch.

<span class="quote">> 
> However, did you see that this is for the i965G platform, which is 10 years
> old? Is the non-generic driver supporting this platform?</span >

We did not do any test on i965G platform. It's not our scope.

<span class="quote">> 
> > However, if you can fix it in generic driver, please make a patch and send
> > it to community.

> Well, I already wrote a patch that papers around the issue and will still
> report it in the logs[1], but this is not a proper patch.

> I do not have any background in the sound subsystem, so I do not know what
> are the correct assumptions to have, especially around the binding and
> unbinding of the driver. I am merely doing some QA/CI job here, and
> reporting the issue to you, who own this code.

> I am however willing to help, but not by configuring the system differently
> in order to get rid of the problem. This would be a workaround, not a fix.

> [1] <a href="https://patchwork.freedesktop.org/patch/160525/">https://patchwork.freedesktop.org/patch/160525/</a></span >

Please feel free to make the patch and submit it. Takashi is very professional
and nice. I did think we need check the null pointer. But maybe it is not as
your patch. Maybe it is in another place.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the QA Contact for the bug.</li>
          <li>You are on the CC list for the bug.</li>
      </ul>
    </body>
</html>