xfs
[Top] [All Lists]

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

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 <=