From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: [PATCH for-4.5 1/2] xen/vsprintf: Introduce %*ph
 extended format specifier for hex buffers
Date: Fri, 26 Sep 2014 13:32:34 +0100
Message-ID: <54255CE2.8060802@citrix.com>
References: <1411726207-2689-1-git-send-email-andrew.cooper3@citrix.com>
	<1411726207-2689-2-git-send-email-andrew.cooper3@citrix.com>
	<54256AF90200007800039974@mail.emea.novell.com>
	<54255906.8010405@citrix.com>
	<542578B30200007800039A3E@mail.emea.novell.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <542578B30200007800039A3E@mail.emea.novell.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Jan Beulich <JBeulich@suse.com>
Cc: Tim Deegan <tim@xen.org>, Keir Fraser <keir@xen.org>, Xen-devel <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On 26/09/14 13:31, Jan Beulich wrote:
>>>> On 26.09.14 at 14:16, <andrew.cooper3@citrix.com> wrote:
>> On 26/09/14 12:32, Jan Beulich wrote:
>>>>>> On 26.09.14 at 12:10, <andrew.cooper3@citrix.com> wrote:
>>>> --- a/docs/misc/printk-formats.txt
>>>> +++ b/docs/misc/printk-formats.txt
>>>> @@ -18,3 +18,9 @@ Symbol/Function pointers:
>>>>  
>>>>         %pv     Domain and vCPU ID from a 'struct vcpu *' (printed as
>>>>                 "d<domid>v<vcpuid>")
>>>> +
>>>> +
>>>> +Raw buffer as hex string:
>>>> +
>>>> +       %*ph    Up to 64 characters, printed as "00 01 02 ... ff".  Buffer 
>> length
>>>> +               expected via the field_width paramter. i.e. printk("%*ph", 
>> 8, buffer);
>>> Let's keep this list sorted alphabetically please.
>> Ok, but then the "Symbol/Function pointers:" paragraph marker should be
>> dropped.
>>
>> I am happy with doing either.
> Actually it looks like I should have added a header when adding %pv,
> so maybe that's what wants to be corrected? Sorting by formatting
> character still would see desirable to me, as would keeping the
> headings.

I will introduce the heading for %pv

>
>>>> --- a/xen/common/vsprintf.c
>>>> +++ b/xen/common/vsprintf.c
>>>> @@ -272,6 +272,31 @@ static char *pointer(char *str, char *end, const char **fmt_ptr,
>>>>      /* Custom %p suffixes. See XEN_ROOT/docs/misc/printk-formats.txt */
>>>>      switch ( fmt[1] )
>>>>      {
>>>> +    case 'h': /* Raw buffer as hex string. */
>>>> +    {
>>>> +        /*
>>>> +         * User expected to provide an explicit count using %*.  Bound between
>>>> +         * 0 and 64 bytes, defaulting to 0.
>>>> +         */
>>>> +        unsigned i, nr_bytes =
>>>> +            ((field_width < 1) || (field_width > 64)) ? 0 : field_width;
>>> Producing no output for too small a field width makes sense, but why
>>> not print 64 bytes if more were requested?
>> 64 is arbitrary (taken from the Linux statement to the same effect). 
>> Even with an upper bound of 64, the caller should be using something
>> shorter and putting in newlines.
> I'd be fine with you limiting it to a lower value; I just find it odd to
> zap a value exceeding the boundary to zero rather than to the
> upper bound.
>
> Jan
>

Ok

~Andrew