All of lore.kernel.org
 help / color / mirror / Atom feed
From: Linda <lindaj@jma3.com>
To: Lars Kurth <lars.kurth.xen@gmail.com>
Cc: Julien Grall <julien.grall@citrix.com>,
	Wei Liu <wei.liu2@citrix.com>,
	"<xen-devel@lists.xen.org>" <xen-devel@lists.xen.org>
Subject: Xen Outreachy application
Date: Thu, 02 Apr 2015 07:47:08 -0600	[thread overview]
Message-ID: <551D485C.3050403@jma3.com> (raw)
In-Reply-To: <8911DF20-E76B-4A0E-BF73-944FC87F7130@gmail.com>

Hi Lars,
     I am aware of it.  Thank you.  I've been having trouble with builds 
and git send-emails.  The code is written and minimally unit tested.  I 
would love to have it reviewed.
     I'm waiting on answers/suggestions from Julien and Wei, who have 
been very responsive.  They deserve gold stars for their patience with me.
     BTW, I hope you don't mind I changed the subject line, so the 
developers on the list understand the circumstances.

Thanks,

Linda Jacobson


On 4/2/2015 7:15 AM, Lars Kurth wrote:
> Linda,
> I do want to remind you that you need to start communicating on the public list and submit some patches. Also some of us are away before the application deadline on the 7th. So things may get tight
> Regards
> Lars
>
>> On 30 Mar 2015, at 20:26, Linda Jacobson <lindaj@jma3.com> wrote:
>>
>> Will do.   Thanks for all the info.   L
>>
>> Sent from my iPhone
>>
>>> On Mar 30, 2015, at 12:52 PM, Wei Liu <wei.liu2@citrix.com> wrote:
>>>
>>>> On Mon, Mar 30, 2015 at 12:29:41PM -0600, Linda wrote:
>>>> Hi Wei,
>>>>    I have a few questions before we chat Tuesday.  Per your instructions, I
>>>> read the coding standards, which referred me to libxl.h for memory
>>>> management.  They are contradictory, in that the coding standards imply that
>>>> I should be using internal libxl_*alloc functions, but the memory management
>>>> section says to do a malloc. I thought someone should know.
>>> They are not contradictory. libxl_*alloc are wrappers to libc's *alloc
>>> functions.
>>>
>>> I think that section is mostly material for external callers (i.e. from
>>> the libxl user's point of view). You're going to work on the library
>>> itself so your point of view is a bit different.
>>>
>>>>    I assume I should look for and use the appropriate libxl_*alloc
>>>> function.
>>> Yes.
>>>
>>>>    Also, you mentioned that all libxl functions should take a ctx
>>>> (context?) as the first argument.  I had looked at several libxl_bitmap*
>>>> functions (init, test, set, and dispose) which don't, but looking at the
>>>> alloc code you sent me, I'm not clear on the purpose of the ctx.  Perhaps we
>>>> can go over that in our chat.
>>> I think I said "usually". There are exceptions. :-)
>>>
>>> Yes, ctx means context. The application (libxl user) is going to
>>> initialise that context and pass that to libxl public API. Then your
>>> public API (the functions that you're going to implement are public
>>> APIs) can use that context to do various stuff.
>>>
>>> There are many things wrapped in context, but I don't think you really
>>> need to care about what they are unless you really need them.
>>>
>>> For the time being, you can use the code snippet (libxl_bitmap_alloc) I
>>> sent you as skeleton to implement your own functions.
>>>
>>> Wei.
>>>
>>>> Thanks.
>>>>
>>>> Linda
>

      reply	other threads:[~2015-04-02 13:47 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-30 18:29 coding standards Linda
2015-03-30 18:52 ` Wei Liu
2015-03-30 19:26   ` Linda Jacobson
2015-04-02 13:15     ` Lars Kurth
2015-04-02 13:47       ` Linda [this message]

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=551D485C.3050403@jma3.com \
    --to=lindaj@jma3.com \
    --cc=julien.grall@citrix.com \
    --cc=lars.kurth.xen@gmail.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.org \
    /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.