linux-nvme.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: eliezer.tamir@linux.intel.com (Eliezer Tamir)
Subject: [RFC PATCH] nvme: add HW API tags
Date: Fri, 20 Nov 2015 15:04:42 +0200	[thread overview]
Message-ID: <564F1A6A.6020401@linux.intel.com> (raw)
In-Reply-To: <20151118135530.GA9473@infradead.org>


On 18/11/2015 15:55, Christoph Hellwig wrote:
> On Tue, Nov 17, 2015@07:08:37PM +0200, Eliezer Tamir wrote:
>> HW validation would like to use parts of the real Linux driver to
>> exercise the HW, instead of writing their own driver.
>>
>> This is a very custom environment (not a full OS) and the code needs to
>> be preprocessed for it. The hwapi tag tells our preprocessor which
>> functions must be left in.
>>
>> We thought that this would be less intrusive than moving the code to a
>> separate file, which would force some unneeded changes to the code.
> 
> Just mark them manually when you generate your validation workload.
> These functions will regularly change, so your code calling it will
> change as well anyway.

That is a valid point. However, I hope to find a better solution than
making a manual list whenever the tool is used.

Marking manually will not work well, because the people that use the
tools are HW experts, not Linux guys.

Even tough the functions themselves change regularly, the list of
functions to include would change much less thank that.
The code calling the imported code would sometimes break and need
fixing, but this would mostly happen only when you change the
interface, not the implementation.

The rules for maintaining the tags would be very simple:
If you replace a tagged function, the new function needs the tag.
If you split a tagged function into two both should have it.

      reply	other threads:[~2015-11-20 13:04 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-17 14:48 [RFC PATCH] nvme: add HW API tags Eliezer Tamir
2015-11-17 15:05 ` Christoph Hellwig
2015-11-17 17:08   ` Eliezer Tamir
2015-11-18 13:55     ` Christoph Hellwig
2015-11-20 13:04       ` Eliezer Tamir [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=564F1A6A.6020401@linux.intel.com \
    --to=eliezer.tamir@linux.intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).