From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42907) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bhv1X-0001jB-W0 for qemu-devel@nongnu.org; Thu, 08 Sep 2016 04:49:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bhv1T-0000pL-HX for qemu-devel@nongnu.org; Thu, 08 Sep 2016 04:49:31 -0400 Received: from mx1.redhat.com ([209.132.183.28]:33940) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bhv1T-0000pB-C7 for qemu-devel@nongnu.org; Thu, 08 Sep 2016 04:49:27 -0400 Date: Thu, 8 Sep 2016 09:49:22 +0100 From: "Daniel P. Berrange" Message-ID: <20160908084922.GC30602@redhat.com> Reply-To: "Daniel P. Berrange" References: <1471935390-126476-1-git-send-email-ashish.mittal@veritas.com> <20160830173507.GA1501@localhost.localdomain> <20160908084319.GA4206@noname.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20160908084319.GA4206@noname.redhat.com> Subject: Re: [Qemu-devel] [PATCH v4 RFC] block/vxhs: Initial commit to add Veritas HyperScale VxHS block device support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf Cc: ashish mittal , Jeff Cody , qemu-devel@nongnu.org, Paolo Bonzini , Markus Armbruster , Ashish Mittal , Stefan Hajnoczi , Ketan.Nilangekar@veritas.com, Abhijit.Dey@veritas.com On Thu, Sep 08, 2016 at 10:43:19AM +0200, Kevin Wolf wrote: > Am 08.09.2016 um 00:32 hat ashish mittal geschrieben: > > >> + > > >> +void vxhs_set_acb_buffer(void *ptr, void *buffer) > > >> +{ > > >> + VXHSAIOCB *acb = ptr; > > >> + > > >> + acb->buffer = buffer; > > >> +} > > >> + > > > > > > Unused function? > > > > This is called from within libqnio. > > Wait, you mean the library references a symbol in the qemu binary? This > sounds completely wrong to me. I wouldn't even do something like this if > it were an internal qemu library because I think libraries should be > self-contained, but it's a much larger problem in something that is > supposed to be an independent library. I'd be surprised if that actually worked. If an external library wants to refrence symbols in the QEMU binary, then QEMU would ned to be using the -export-dynamic / -rdynamic linker flags to export all its symbols, and AFAIK, we're not doing that. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|