Home » Mailing list archives » linux-kernel » 2008 » August » 12

Re: [PATCH 2/2] CRED: Introduce credential access wrappers

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Ingo Oeser
Subject: Re: [PATCH 2/2] CRED: Introduce credential access wrappers
Date: Tuesday, August 12, 2008 - 12:02 pm

Hi David,

On Tuesday 12 August 2008, David Howells wrote:

Why macros? When introducing APIs using trivial inlines makes sure that 
the conversion is correct, type correct and side effect free for the callers.
Macros cannot ensure this without pain.

Please respin this patch with inlines. If that needs header changes,
your CRED subsytem might need that, too.


Best Regards

Ingo Oeser
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH 0/2] Introduce credentials API, David Howells, (Tue Aug 12, 6:28 am)
[PATCH 1/2] CRED: Alter XFS so as to avoid namespace colli ..., David Howells, (Tue Aug 12, 6:28 am)
[PATCH 2/2] CRED: Introduce credential access wrappers, David Howells, (Tue Aug 12, 6:28 am)
Re: [PATCH 2/2] CRED: Introduce credential access wrappers, Ingo Oeser, (Tue Aug 12, 12:02 pm)
Re: [PATCH 2/2] CRED: Introduce credential access wrappers, David Howells, (Tue Aug 12, 1:27 pm)
Re: [PATCH 1/2] CRED: Alter XFS so as to avoid namespace c ..., James Morris, (Tue Aug 12, 3:55 pm)
Re: [PATCH 2/2] CRED: Introduce credential access wrappers, Ingo Oeser, (Tue Aug 12, 3:59 pm)
Re: [PATCH 1/2] CRED: Alter XFS so as to avoid namespace c ..., Christoph Hellwig, (Tue Aug 12, 4:01 pm)