All of lore.kernel.org
 help / color / mirror / Atom feed
From: Boaz Harrosh <bharrosh-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
To: Jens Axboe <jens.axboe-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
Cc: James Bottomley
	<James.Bottomley-d9PhHud1JfjCXq6kfMZ53/egYHeGw8Jk@public.gmane.org>,
	Matthew Dharm
	<mdharm-kernel-JGfshJpz5UybPZpvUQj5UqxOck334EZe@public.gmane.org>,
	Oliver Neukum <oliver-GvhC2dPhHPQdnm+yROfE0A@public.gmane.org>,
	Greg KH <greg-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-scsi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [BUG] 2.6.24-git usb reset problems
Date: Tue, 29 Jan 2008 22:26:48 +0200	[thread overview]
Message-ID: <479F8C08.2060207@panasas.com> (raw)
In-Reply-To: <20080129201328.GZ15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>

On Tue, Jan 29 2008 at 22:13 +0200, Jens Axboe <jens.axboe-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org> wrote:
> On Tue, Jan 29 2008, Boaz Harrosh wrote:
>>> Ok this is not in Linus tree is it? Hence I did not have that failure.
>>>
>>> Boaz
>>>
>>>
>> actually James bidi tree has a fix for this in the scsi_data_buffer patch.
>>
>> what you sent is not enough there are other places. look at this patch I
>> sent to the list.
>>
>> http://www.spinics.net/lists/linux-scsi/msg21938.html
> 
> Hard to compare, since its on different bases.
Yes in this patchset I have taken your sg branch at the time, and
rebased it ontop of scsi_data_buffer patch. Because I felt that
it is more natural for this patch to come after the scsi total
cleanup that is scsi_data_buffer. Then the extraction to sg_table
is simple and trivial.

What I meant to point out with this patch is that all the exact same
places that are touched there should be fixed when moving to sg_table.

Look at it. It is a revised version of your patch.

> 
>> Could we take the 2 SG patches and submit them through the scsi
>> bidi tree? It is much more natural to have them in one tree as one
>> patchset then try coordinate with git-merge. Actually if you look at it,
>> the biggest change is to SCSI. So I think it is more natural this way
> 
> What 2 sg patches do you mean?
> 

I mean the patches that where in sg branch of the linux-block tree, But
I see that it is now to late, and that they are in Linus already

James the most simple is to submit the scsi_data_buff patch that fixes
all these places. If not do you want that I send in fixes?

Boaz

WARNING: multiple messages have this Message-ID (diff)
From: Boaz Harrosh <bharrosh@panasas.com>
To: Jens Axboe <jens.axboe@oracle.com>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>,
	Matthew Dharm <mdharm-kernel@one-eyed-alien.net>,
	Oliver Neukum <oliver@neukum.org>, Greg KH <greg@kroah.com>,
	linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org,
	linux-scsi@vger.kernel.org
Subject: Re: [BUG] 2.6.24-git usb reset problems
Date: Tue, 29 Jan 2008 22:26:48 +0200	[thread overview]
Message-ID: <479F8C08.2060207@panasas.com> (raw)
In-Reply-To: <20080129201328.GZ15220@kernel.dk>

On Tue, Jan 29 2008 at 22:13 +0200, Jens Axboe <jens.axboe@oracle.com> wrote:
> On Tue, Jan 29 2008, Boaz Harrosh wrote:
>>> Ok this is not in Linus tree is it? Hence I did not have that failure.
>>>
>>> Boaz
>>>
>>>
>> actually James bidi tree has a fix for this in the scsi_data_buffer patch.
>>
>> what you sent is not enough there are other places. look at this patch I
>> sent to the list.
>>
>> http://www.spinics.net/lists/linux-scsi/msg21938.html
> 
> Hard to compare, since its on different bases.
Yes in this patchset I have taken your sg branch at the time, and
rebased it ontop of scsi_data_buffer patch. Because I felt that
it is more natural for this patch to come after the scsi total
cleanup that is scsi_data_buffer. Then the extraction to sg_table
is simple and trivial.

What I meant to point out with this patch is that all the exact same
places that are touched there should be fixed when moving to sg_table.

Look at it. It is a revised version of your patch.

> 
>> Could we take the 2 SG patches and submit them through the scsi
>> bidi tree? It is much more natural to have them in one tree as one
>> patchset then try coordinate with git-merge. Actually if you look at it,
>> the biggest change is to SCSI. So I think it is more natural this way
> 
> What 2 sg patches do you mean?
> 

I mean the patches that where in sg branch of the linux-block tree, But
I see that it is now to late, and that they are in Linus already

James the most simple is to submit the scsi_data_buff patch that fixes
all these places. If not do you want that I send in fixes?

Boaz

  parent reply	other threads:[~2008-01-29 20:26 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-28 20:49 [BUG] 2.6.24-git usb reset problems Jens Axboe
2008-01-28 21:21 ` Greg KH
2008-01-29  7:48   ` Jens Axboe
2008-01-29 12:15   ` Boaz Harrosh
     [not found]     ` <479F18D3.9030709-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
2008-01-29 13:54       ` Jens Axboe
2008-01-29 13:54         ` Jens Axboe
     [not found]         ` <20080129135443.GU15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>
2008-01-29 14:06           ` Boaz Harrosh
2008-01-29 14:06             ` Boaz Harrosh
     [not found]             ` <479F32E9.10609-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
2008-01-29 14:11               ` Jens Axboe
2008-01-29 14:11                 ` Jens Axboe
     [not found]                 ` <20080129141108.GV15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>
2008-01-29 14:14                   ` Boaz Harrosh
2008-01-29 14:14                     ` Boaz Harrosh
2008-01-29 14:31                 ` Oliver Neukum
     [not found]                   ` <200801291531.39825.oliver-GvhC2dPhHPQdnm+yROfE0A@public.gmane.org>
2008-01-29 14:31                     ` Jens Axboe
2008-01-29 14:31                       ` Jens Axboe
2008-01-29 18:39                       ` Jens Axboe
     [not found]                         ` <20080129183910.GI15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>
2008-01-29 19:09                           ` Boaz Harrosh
2008-01-29 19:09                             ` Boaz Harrosh
2008-01-29 19:10                           ` Matthew Dharm
2008-01-29 19:10                             ` Matthew Dharm
     [not found]                             ` <20080129191024.GO14375-JGfshJpz5UybPZpvUQj5UqxOck334EZe@public.gmane.org>
2008-01-29 19:15                               ` Jens Axboe
2008-01-29 19:15                                 ` Jens Axboe
     [not found]                                 ` <20080129191504.GO15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>
2008-01-29 19:26                                   ` Jens Axboe
2008-01-29 19:26                                     ` Jens Axboe
2008-01-29 19:37                                 ` Matthew Dharm
2008-01-29 19:33                             ` James Bottomley
2008-01-29 19:35                               ` Jens Axboe
2008-01-29 19:45                                 ` Jens Axboe
     [not found]                                   ` <20080129194543.GR15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>
2008-01-29 19:58                                     ` Boaz Harrosh
2008-01-29 19:58                                       ` Boaz Harrosh
     [not found]                                       ` <479F856C.4090900-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
2008-01-29 20:03                                         ` Jens Axboe
2008-01-29 20:03                                           ` Jens Axboe
     [not found]                                           ` <20080129200311.GV15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>
2008-01-29 20:04                                             ` James Bottomley
2008-01-29 20:04                                               ` James Bottomley
2008-01-29 20:06                                               ` Jens Axboe
     [not found]                                                 ` <20080129200651.GW15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>
2008-01-29 20:24                                                   ` James Bottomley
2008-01-29 20:24                                                     ` James Bottomley
2008-01-29 20:53                                                     ` Boaz Harrosh
2008-01-29 20:09                                         ` Boaz Harrosh
2008-01-29 20:09                                           ` Boaz Harrosh
     [not found]                                           ` <479F880F.20709-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
2008-01-29 20:13                                             ` Jens Axboe
2008-01-29 20:13                                               ` Jens Axboe
     [not found]                                               ` <20080129201328.GZ15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>
2008-01-29 20:26                                                 ` Boaz Harrosh [this message]
2008-01-29 20:26                                                   ` Boaz Harrosh
2008-01-30 10:27                                     ` Geert Uytterhoeven
2008-01-30 10:27                                       ` Geert Uytterhoeven
     [not found]                                       ` <Pine.LNX.4.64.0801301125350.26562-DVqXPGhgXSn9uFGNBm7GzQ@public.gmane.org>
2008-01-30 10:38                                         ` Jens Axboe
2008-01-30 10:38                                           ` Jens Axboe
2008-01-30 14:38                                           ` James Bottomley
2008-01-30 18:06                                             ` Jens Axboe
     [not found]                                               ` <20080130180613.GU15220-tSWWG44O7X1aa/9Udqfwiw@public.gmane.org>
2008-01-30 19:07                                                 ` Jens Axboe
2008-01-30 19:07                                                   ` Jens Axboe
2008-01-29 15:50                   ` Boaz Harrosh
     [not found]                     ` <479F4B4B.2020000-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
2008-01-29 17:42                       ` Oliver Neukum
2008-01-29 17:42                         ` Oliver Neukum
2008-01-29 14:13             ` Boaz Harrosh
2008-01-29 15:36       ` Alan Stern
2008-01-29 15:36         ` Alan Stern
     [not found]         ` <Pine.LNX.4.44L0.0801291032210.4229-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2008-01-29 15:54           ` Boaz Harrosh
2008-01-29 15:54             ` Boaz Harrosh
2008-01-29 16:34         ` James Bottomley
2008-01-29 18:27           ` Boaz Harrosh
     [not found]             ` <479F7009.3020306-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
2008-01-29 18:48               ` James Bottomley
2008-01-29 18:48                 ` James Bottomley
2008-01-29 18:58                 ` Boaz Harrosh
     [not found]                   ` <479F774B.60303-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
2008-01-29 19:17                     ` James Bottomley
2008-01-29 19:17                       ` James Bottomley
     [not found]                       ` <1201634240.3069.33.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2008-01-29 19:28                         ` Boaz Harrosh
2008-01-29 19:28                           ` Boaz Harrosh
2008-01-29 15:00     ` Matthew Dharm

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=479F8C08.2060207@panasas.com \
    --to=bharrosh-c4p08nqkorlbdgjk7y7tuq@public.gmane.org \
    --cc=James.Bottomley-d9PhHud1JfjCXq6kfMZ53/egYHeGw8Jk@public.gmane.org \
    --cc=greg-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org \
    --cc=jens.axboe-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-scsi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mdharm-kernel-JGfshJpz5UybPZpvUQj5UqxOck334EZe@public.gmane.org \
    --cc=oliver-GvhC2dPhHPQdnm+yROfE0A@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.