netdev - Re: [PATCH net-next 5/7] net:fec: add support for dumping transmit ring on timeout

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAHrpEqQL7LviXA4_NLNwSBoW84W_vbnzWkc9At5QBqUkqu63yw@mail.gmail.com>
Date:	Tue, 29 Apr 2014 09:54:09 -0500
From:	Frank Li <lznuaa@...il.com>
To:	Russell King - ARM Linux <linux@....linux.org.uk>
Cc:	David Laight <David.Laight@...lab.com>,
	Frank Li <Frank.Li@...escale.com>,
	"shawn.guo@...aro.org" <shawn.guo@...aro.org>,
	"B38611@...escale.com" <B38611@...escale.com>,
	"davem@...emloft.net" <davem@...emloft.net>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: Re: [PATCH net-next 5/7] net:fec: add support for dumping transmit
 ring on timeout

On Tue, Apr 29, 2014 at 9:32 AM, Russell King - ARM Linux
<linux@....linux.org.uk> wrote:
> On Tue, Apr 29, 2014 at 09:22:13AM -0500, Frank Li wrote:
>> On Tue, Apr 29, 2014 at 9:15 AM, Russell King - ARM Linux
>> <linux@....linux.org.uk> wrote:
>> > On Tue, Apr 29, 2014 at 02:01:54PM +0000, David Laight wrote:
>> >> From: Frank ...
>> >> > > You probably want the read and write indexes as well.
>> >> >
>> >> >                      bdp == fep->cur_tx ? 'S' : ' ',
>> >> >                      bdp == fep->dirty_tx ? 'H' : ' ',
>> >> >
>> >> > Above code already print read and write index. 'S', 'H'
>> >>
>> >> Gah I must be asleep!
>> >> Something made be think that was to do with the ring ownership bit!
>> >
>> > Err, what's going on... this is my patch.  If it's been submitted by
>> > others, why wasn't its submission at least Cc'd to me?
>>
>> I added " --to rmk+kernel@....linux.org.uk" when I send patch.
>> I don't know what's wrong.
>
> Yes... I guess you don't know what went wrong because you'll never know
> if it failed to be delivered, because you sent them with an envelope
> address of "b20596@...inux1.ap.freescale.net" which doesn't actually
> exist.  Therefore, DSNs can't be returned to you.
>
>> I ask you if need someone help send to david millar before.
>
> You did, but I didn't agree to it.  I did point out that they need a
> certain amount of rework first:

Sorry, I miss understood your means.
I will stop working on this.

Actually, we want to upstream imx6 SoloX enet.
Solox have not more features, like AVB.

If we upstream base on David's tree, there will be big conflict what
you already done.

best regards
Frank Li

>
>   I know they need to re-worked to convert unsigned -> unsigned int before
>   David sees them, otherwise it'd be an instant review failure.  Doing the
>   conversion is the easy bit, doing it without hitting the 80 column limit
>   is much harder...
>
> Some of them need a little more work in addition to that before being
> sent to David.
>
> I've also subsequently received in private email one issue which indicates
> a problem with the patch set - as yet I've not been able to look at it,
> but it sounds like the RX ring is full, but NAPI is in hard-irq mode,
> and the hardware won't deliver an IRQ without free space in the RX ring.
>
> Since people have got wind that I've been looking at this driver, everyone
> has started sending their various FEC issues directly to me...
>
> --
> FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
> improving, and getting towards what was expected from it.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html