* [PATCH] jsflash: fix compilation
@ 2018-05-15 18:32 ` Christoph Hellwig
0 siblings, 0 replies; 12+ messages in thread
From: Christoph Hellwig @ 2018-05-15 18:32 UTC (permalink / raw)
To: axboe; +Cc: linux-block, sparclinux
No bio in this whole function, use req->bio instead.
Fixes: 37a5b5c6 ("jsflash: handle highmem pages");
Reported-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Christoph Hellwig <hch@lst.de>
---
Looks like no one except for Guenters build bot cared. I wonder if we
should just get rid of the driver given that it doesn't look in a good
shape at all based on his build logs even with the fix..
drivers/sbus/char/jsflash.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/sbus/char/jsflash.c b/drivers/sbus/char/jsflash.c
index 8520587b8d09..821367ef9dd3 100644
--- a/drivers/sbus/char/jsflash.c
+++ b/drivers/sbus/char/jsflash.c
@@ -230,7 +230,7 @@ static void jsfd_request(void)
goto end;
}
- p = kmap_atomic(bio_page(bio)) + bio_offset(bio);
+ p = kmap_atomic(bio_page(req->bio)) + bio_offset(req->bio);
jsfd_read(p, jdp->dbase + offset, len);
kunmap_atomic(p);
err = BLK_STS_OK;
--
2.17.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH] jsflash: fix compilation
@ 2018-05-15 18:32 ` Christoph Hellwig
0 siblings, 0 replies; 12+ messages in thread
From: Christoph Hellwig @ 2018-05-15 18:32 UTC (permalink / raw)
To: axboe; +Cc: linux-block, sparclinux
No bio in this whole function, use req->bio instead.
Fixes: 37a5b5c6 ("jsflash: handle highmem pages");
Reported-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Christoph Hellwig <hch@lst.de>
---
Looks like no one except for Guenters build bot cared. I wonder if we
should just get rid of the driver given that it doesn't look in a good
shape at all based on his build logs even with the fix..
drivers/sbus/char/jsflash.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/sbus/char/jsflash.c b/drivers/sbus/char/jsflash.c
index 8520587b8d09..821367ef9dd3 100644
--- a/drivers/sbus/char/jsflash.c
+++ b/drivers/sbus/char/jsflash.c
@@ -230,7 +230,7 @@ static void jsfd_request(void)
goto end;
}
- p = kmap_atomic(bio_page(bio)) + bio_offset(bio);
+ p = kmap_atomic(bio_page(req->bio)) + bio_offset(req->bio);
jsfd_read(p, jdp->dbase + offset, len);
kunmap_atomic(p);
err = BLK_STS_OK;
--
2.17.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
2018-05-15 18:32 ` Christoph Hellwig
@ 2018-05-15 18:51 ` Jens Axboe
-1 siblings, 0 replies; 12+ messages in thread
From: Jens Axboe @ 2018-05-15 18:51 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: linux-block, sparclinux
On 5/15/18 12:32 PM, Christoph Hellwig wrote:
> No bio in this whole function, use req->bio instead.
Applied.
> Looks like no one except for Guenters build bot cared. I wonder if we
> should just get rid of the driver given that it doesn't look in a good
> shape at all based on his build logs even with the fix..
Needs input from the folks that actually might use it. The driver looks
abandoned, it's only getting updates through API changes etc.
--
Jens Axboe
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
@ 2018-05-15 18:51 ` Jens Axboe
0 siblings, 0 replies; 12+ messages in thread
From: Jens Axboe @ 2018-05-15 18:51 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: linux-block, sparclinux
On 5/15/18 12:32 PM, Christoph Hellwig wrote:
> No bio in this whole function, use req->bio instead.
Applied.
> Looks like no one except for Guenters build bot cared. I wonder if we
> should just get rid of the driver given that it doesn't look in a good
> shape at all based on his build logs even with the fix..
Needs input from the folks that actually might use it. The driver looks
abandoned, it's only getting updates through API changes etc.
--
Jens Axboe
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
2018-05-15 18:51 ` Jens Axboe
@ 2018-05-15 18:58 ` David Miller
-1 siblings, 0 replies; 12+ messages in thread
From: David Miller @ 2018-05-15 18:58 UTC (permalink / raw)
To: axboe; +Cc: hch, linux-block, sparclinux
From: Jens Axboe <axboe@kernel.dk>
Date: Tue, 15 May 2018 12:51:20 -0600
> On 5/15/18 12:32 PM, Christoph Hellwig wrote:
>> No bio in this whole function, use req->bio instead.
>
> Applied.
>
>> Looks like no one except for Guenters build bot cared. I wonder if we
>> should just get rid of the driver given that it doesn't look in a good
>> shape at all based on his build logs even with the fix..
>
> Needs input from the folks that actually might use it. The driver looks
> abandoned, it's only getting updates through API changes etc.
I don't think anyone is using it.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
@ 2018-05-15 18:58 ` David Miller
0 siblings, 0 replies; 12+ messages in thread
From: David Miller @ 2018-05-15 18:58 UTC (permalink / raw)
To: axboe; +Cc: hch, linux-block, sparclinux
From: Jens Axboe <axboe@kernel.dk>
Date: Tue, 15 May 2018 12:51:20 -0600
> On 5/15/18 12:32 PM, Christoph Hellwig wrote:
>> No bio in this whole function, use req->bio instead.
>
> Applied.
>
>> Looks like no one except for Guenters build bot cared. I wonder if we
>> should just get rid of the driver given that it doesn't look in a good
>> shape at all based on his build logs even with the fix..
>
> Needs input from the folks that actually might use it. The driver looks
> abandoned, it's only getting updates through API changes etc.
I don't think anyone is using it.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
2018-05-15 18:58 ` David Miller
@ 2018-05-15 19:00 ` Jens Axboe
-1 siblings, 0 replies; 12+ messages in thread
From: Jens Axboe @ 2018-05-15 19:00 UTC (permalink / raw)
To: David Miller; +Cc: hch, linux-block, sparclinux
On 5/15/18 12:58 PM, David Miller wrote:
> From: Jens Axboe <axboe@kernel.dk>
> Date: Tue, 15 May 2018 12:51:20 -0600
>
>> On 5/15/18 12:32 PM, Christoph Hellwig wrote:
>>> No bio in this whole function, use req->bio instead.
>>
>> Applied.
>>
>>> Looks like no one except for Guenters build bot cared. I wonder if we
>>> should just get rid of the driver given that it doesn't look in a good
>>> shape at all based on his build logs even with the fix..
>>
>> Needs input from the folks that actually might use it. The driver looks
>> abandoned, it's only getting updates through API changes etc.
>
> I don't think anyone is using it.
Are you fine with removing it then?
--
Jens Axboe
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
@ 2018-05-15 19:00 ` Jens Axboe
0 siblings, 0 replies; 12+ messages in thread
From: Jens Axboe @ 2018-05-15 19:00 UTC (permalink / raw)
To: David Miller; +Cc: hch, linux-block, sparclinux
On 5/15/18 12:58 PM, David Miller wrote:
> From: Jens Axboe <axboe@kernel.dk>
> Date: Tue, 15 May 2018 12:51:20 -0600
>
>> On 5/15/18 12:32 PM, Christoph Hellwig wrote:
>>> No bio in this whole function, use req->bio instead.
>>
>> Applied.
>>
>>> Looks like no one except for Guenters build bot cared. I wonder if we
>>> should just get rid of the driver given that it doesn't look in a good
>>> shape at all based on his build logs even with the fix..
>>
>> Needs input from the folks that actually might use it. The driver looks
>> abandoned, it's only getting updates through API changes etc.
>
> I don't think anyone is using it.
Are you fine with removing it then?
--
Jens Axboe
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
2018-05-15 19:00 ` Jens Axboe
@ 2018-05-15 19:51 ` David Miller
-1 siblings, 0 replies; 12+ messages in thread
From: David Miller @ 2018-05-15 19:51 UTC (permalink / raw)
To: axboe; +Cc: hch, linux-block, sparclinux
From: Jens Axboe <axboe@kernel.dk>
Date: Tue, 15 May 2018 13:00:36 -0600
> On 5/15/18 12:58 PM, David Miller wrote:
>> From: Jens Axboe <axboe@kernel.dk>
>> Date: Tue, 15 May 2018 12:51:20 -0600
>>
>>> On 5/15/18 12:32 PM, Christoph Hellwig wrote:
>>>> No bio in this whole function, use req->bio instead.
>>>
>>> Applied.
>>>
>>>> Looks like no one except for Guenters build bot cared. I wonder if we
>>>> should just get rid of the driver given that it doesn't look in a good
>>>> shape at all based on his build logs even with the fix..
>>>
>>> Needs input from the folks that actually might use it. The driver looks
>>> abandoned, it's only getting updates through API changes etc.
>>
>> I don't think anyone is using it.
>
> Are you fine with removing it then?
Yes, I am:
Acked-by: David S. Miller <davem@davemloft.net>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
@ 2018-05-15 19:51 ` David Miller
0 siblings, 0 replies; 12+ messages in thread
From: David Miller @ 2018-05-15 19:51 UTC (permalink / raw)
To: axboe; +Cc: hch, linux-block, sparclinux
From: Jens Axboe <axboe@kernel.dk>
Date: Tue, 15 May 2018 13:00:36 -0600
> On 5/15/18 12:58 PM, David Miller wrote:
>> From: Jens Axboe <axboe@kernel.dk>
>> Date: Tue, 15 May 2018 12:51:20 -0600
>>
>>> On 5/15/18 12:32 PM, Christoph Hellwig wrote:
>>>> No bio in this whole function, use req->bio instead.
>>>
>>> Applied.
>>>
>>>> Looks like no one except for Guenters build bot cared. I wonder if we
>>>> should just get rid of the driver given that it doesn't look in a good
>>>> shape at all based on his build logs even with the fix..
>>>
>>> Needs input from the folks that actually might use it. The driver looks
>>> abandoned, it's only getting updates through API changes etc.
>>
>> I don't think anyone is using it.
>
> Are you fine with removing it then?
Yes, I am:
Acked-by: David S. Miller <davem@davemloft.net>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
2018-05-15 19:51 ` David Miller
@ 2018-05-15 19:57 ` Jens Axboe
-1 siblings, 0 replies; 12+ messages in thread
From: Jens Axboe @ 2018-05-15 19:57 UTC (permalink / raw)
To: David Miller; +Cc: hch, linux-block, sparclinux
On 5/15/18 1:51 PM, David Miller wrote:
> From: Jens Axboe <axboe@kernel.dk>
> Date: Tue, 15 May 2018 13:00:36 -0600
>
>> On 5/15/18 12:58 PM, David Miller wrote:
>>> From: Jens Axboe <axboe@kernel.dk>
>>> Date: Tue, 15 May 2018 12:51:20 -0600
>>>
>>>> On 5/15/18 12:32 PM, Christoph Hellwig wrote:
>>>>> No bio in this whole function, use req->bio instead.
>>>>
>>>> Applied.
>>>>
>>>>> Looks like no one except for Guenters build bot cared. I wonder if we
>>>>> should just get rid of the driver given that it doesn't look in a good
>>>>> shape at all based on his build logs even with the fix..
>>>>
>>>> Needs input from the folks that actually might use it. The driver looks
>>>> abandoned, it's only getting updates through API changes etc.
>>>
>>> I don't think anyone is using it.
>>
>> Are you fine with removing it then?
>
> Yes, I am:
>
> Acked-by: David S. Miller <davem@davemloft.net>
Great, done!
--
Jens Axboe
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] jsflash: fix compilation
@ 2018-05-15 19:57 ` Jens Axboe
0 siblings, 0 replies; 12+ messages in thread
From: Jens Axboe @ 2018-05-15 19:57 UTC (permalink / raw)
To: David Miller; +Cc: hch, linux-block, sparclinux
On 5/15/18 1:51 PM, David Miller wrote:
> From: Jens Axboe <axboe@kernel.dk>
> Date: Tue, 15 May 2018 13:00:36 -0600
>
>> On 5/15/18 12:58 PM, David Miller wrote:
>>> From: Jens Axboe <axboe@kernel.dk>
>>> Date: Tue, 15 May 2018 12:51:20 -0600
>>>
>>>> On 5/15/18 12:32 PM, Christoph Hellwig wrote:
>>>>> No bio in this whole function, use req->bio instead.
>>>>
>>>> Applied.
>>>>
>>>>> Looks like no one except for Guenters build bot cared. I wonder if we
>>>>> should just get rid of the driver given that it doesn't look in a good
>>>>> shape at all based on his build logs even with the fix..
>>>>
>>>> Needs input from the folks that actually might use it. The driver looks
>>>> abandoned, it's only getting updates through API changes etc.
>>>
>>> I don't think anyone is using it.
>>
>> Are you fine with removing it then?
>
> Yes, I am:
>
> Acked-by: David S. Miller <davem@davemloft.net>
Great, done!
--
Jens Axboe
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2018-05-15 19:57 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-15 18:32 [PATCH] jsflash: fix compilation Christoph Hellwig
2018-05-15 18:32 ` Christoph Hellwig
2018-05-15 18:51 ` Jens Axboe
2018-05-15 18:51 ` Jens Axboe
2018-05-15 18:58 ` David Miller
2018-05-15 18:58 ` David Miller
2018-05-15 19:00 ` Jens Axboe
2018-05-15 19:00 ` Jens Axboe
2018-05-15 19:51 ` David Miller
2018-05-15 19:51 ` David Miller
2018-05-15 19:57 ` Jens Axboe
2018-05-15 19:57 ` Jens Axboe
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.