Re: [PATCHv3 3/4] phyp: add storage management driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/25/2010 08:32 AM, Eric Blake wrote:
From: Eduardo Otubo<otubo@xxxxxxxxxxxxxxxxxx>

Add the storage management driver to the Power Hypervisor driver.
This is a big but simple patch, it's just a new set of functions.

This patch includes:
  * Storage driver: The set of pool-* and vol-* functions.
  * attach-disk function.
  * Support for IVM on the new functions.

Signed-off-by: Eric Blake<eblake@xxxxxxxxxx>
<snip>

Hi guys,

This commit is causing a segfault here. virConnectNumOfStoragePools() calls phypNumOfStoragePools(), which calls phypExec(). In this case with session equaling NULL.

That's flowing through to libssh2_channel_open_ex(), which tries to access data in the (NULL) session structure and segfaults.

Unsure if this mailing list accepts attachments. I'll try sending 4 screenshots, that illustrates the problem session variable at each point in the backtrace.

(Note the "Target request failed: Cannot access memory at address 0x4700." in the bottom right of the attached screenshot0001.png file, when it attempts to access the session->open_state value.)

Rolling git back 2 commits to this entry, and the above segfault doesn't occur:

  8914453 phyp: reduce scope of driver functions

Hope that helps.

Regards and best wishes,

Justin Clift

--
Salasaga  -  Open Source eLearning IDE
              http://www.salasaga.org

Attachment: screenshot0001.png
Description: PNG image

Attachment: screenshot0002.png
Description: PNG image

Attachment: screenshot0003.png
Description: PNG image

Attachment: screenshot0004.png
Description: PNG image

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]