Re: [PATCH 2/2] xfs: cleanup XFS_BB_FSB

To:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 2/2] xfs: cleanup XFS_BB_FSB_OFFSET
From:	Amit Sahrawat <amit.sahrawat83@xxxxxxxxx>
Date:	Mon, 16 Jan 2012 17:49:00 +0530
Cc:	Alex Elder <aelder@xxxxxxx>, Ben Myers <bpm@xxxxxxx>, xfs masters <xfs-masters@xxxxxxxxxxx>, xfs <xfs@xxxxxxxxxxx>, linux kernel <linux-kernel@xxxxxxxxxxxxxxx>
Dkim-signature:	v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=7AspCeQdjmCCI4FL93vaNom+166sGx2XMWyOL78EDGU=; b=JGhGQVpC47jP1NjP7Z3o+b3RBW9Oq+2LTur5txpxM84knFKVwCMXpluuMm5WO5xBzZ c1sSEPAprHlSzrhiSU8gCa1PJ2AbiG3ZbrF9kWxNM425JoSqdK1DrEKGPt093VfKXyEO tr6KbY9l+nIjw8cCmLl5s1APAFHu9QoALsShk=
In-reply-to:	<20120111113507.GB14423@xxxxxxxxxxxxx>
References:	<1325848338-6063-1-git-send-email-amit.sahrawat83@xxxxxxxxx> <20120111113507.GB14423@xxxxxxxxxxxxx>

thanks Christoph,

Resent the patch after the subject change.

Regards,
Amit Sahrawat

On Wed, Jan 11, 2012 at 5:05 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> On Fri, Jan 06, 2012 at 04:42:18PM +0530, Amit Sahrawat wrote:
>> Removing the macro, as this is no more needed in the code.
>> Tried to find the reference when it was last used - but the usage
>> for this seemed to have been dropped long time ago.
>>
>> Signed-off-by: Amit Sahrawat <amit.sahrawat83@xxxxxxxxx>
>
> This looks good, but the subject should be something like:
>
> "xfs: kill the unused XFS_BB_FSB_OFFSET macro"
>
>
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>

<Prev in Thread]	Current Thread	[Next in Thread>
Re: [PATCH 2/2] xfs: cleanup XFS_BB_FSB_OFFSET, Amit Sahrawat <=

Previous by Date:	[PATCH 1/1] xfs: kill the unused XFS_BB_FSB_OFFSET macro, Amit Sahrawat
Next by Date:	Welcome to the eftrmf group, eftrmf Moderator
Previous by Thread:	[PATCH 1/1] xfs: kill the unused XFS_BB_FSB_OFFSET macro, Amit Sahrawat
Next by Thread:	Welcome to the eftrmf group, eftrmf Moderator
Indexes:	[Date] [Thread] [Top] [All Lists]

Re: [PATCH 2/2] xfs: cleanup XFS_BB_FSB_OFFSET