All of lore.kernel.org
 help / color / mirror / Atom feed
From: Frederic Barrat <fbarrat@linux.vnet.ibm.com>
To: Michael Neuling <michael.neuling@au1.ibm.com>,
	ian@ozlabs.au.ibm.com, linuxppc-dev@ozlabs.org
Subject: Re: [PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API
Date: Thu, 21 Jan 2016 19:48:52 +0100	[thread overview]
Message-ID: <56A12814.5080607@linux.vnet.ibm.com> (raw)
In-Reply-To: <1453256449.28504.56.camel@au.ibm.com>



Le 20/01/2016 03:20, Michael Neuling a écrit :
> The only thing I'm a bit concerned about is are we going to end up
> duplicating a lot of the linux PCI API, but I guess we are only going
> to do this for things the papr HCALL interface mimics.

There are actually very few operations we can do on the adapter with 
hcalls. papr defines 'reset', 'read the VPD' and flashing a new image on 
the card. So we'll soon run out of APIs to mimic.

I guess it means the usage of cxl_get_phys_dev() should be discouraged, 
since it's going to lead to different behaviors between bare-metal and 
powerVM guest. Was there another expected use case for a kernel driver 
other than accessing the VPD?

   Fred

  reply	other threads:[~2016-01-21 18:49 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-19 17:14 [PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API Frederic Barrat
2016-01-20  0:24 ` Andrew Donnellan
2016-01-20  2:20 ` Michael Neuling
2016-01-21 18:48   ` Frederic Barrat [this message]
2016-01-22  0:38     ` Michael Neuling
2016-01-22 10:56       ` Frederic Barrat
2016-01-20  2:21 ` Ian Munsie
2016-03-04 11:22 ` Frederic Barrat

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=56A12814.5080607@linux.vnet.ibm.com \
    --to=fbarrat@linux.vnet.ibm.com \
    --cc=ian@ozlabs.au.ibm.com \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=michael.neuling@au1.ibm.com \
    /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.